Browse Source

Merge pull request #1694 from pixelfed/staging

Update StatusHashtagService, fix count bug
daniel 5 years ago
parent
commit
19a78a46f9
3 changed files with 8 additions and 8 deletions
  1. 2 2
      CHANGELOG.md
  2. 5 5
      app/Services/StatusHashtagService.php
  3. 1 1
      package.json

+ 2 - 2
CHANGELOG.md

@@ -3,13 +3,13 @@
 ## [Unreleased](https://github.com/pixelfed/pixelfed/compare/v0.10.3...dev)
 
 ### Added
--
 
 ### Fixed
-- 
+- Fixed count bug in StatusHashtagService [#1694]((https://github.com/pixelfed/pixelfed/pull/1694)
 
 ### Changed
 - Updated EmailService, added new domains [#1690](https://github.com/pixelfed/pixelfed/pull/1690)
+- Updated quill.js to v1.3.7 [#1692](https://github.com/pixelfed/pixelfed/pull/1690)
     
     
 ## [v0.10.3 (2019-09-08)](https://github.com/pixelfed/pixelfed/compare/v0.10.2...v0.10.3)

+ 5 - 5
app/Services/StatusHashtagService.php

@@ -56,11 +56,11 @@ class StatusHashtagService {
 
 	public static function count($id)
 	{
-		$count = Redis::zcount(self::CACHE_KEY . $id, '-inf', '+inf');
-		if(empty($count)) {
-			$count = StatusHashtag::whereHashtagId($id)->count();
-		}
-		return $count;
+		$key = 'pf:services:status-hashtag:count:' . $id;
+		$ttl = now()->addHours(6);
+		return Cache::remember($key, $ttl, function() use($id) {
+			return StatusHashtag::whereHashtagId($id)->count();
+		});
 	}
 
 	public static function getStatus($statusId, $hashtagId)

+ 1 - 1
package.json

@@ -40,7 +40,7 @@
         "plyr": "^3.5.6",
         "promise-polyfill": "8.1.0",
         "pusher-js": "^4.4.0",
-        "quill": "^1.3.6",
+        "quill": "^1.3.7",
         "readmore-js": "^2.2.1",
         "socket.io-client": "^2.2.0",
         "sweetalert": "^2.1.2",