Prechádzať zdrojové kódy

Update StatusReplyPipeline, fix comment counts

Daniel Supernault 2 rokov pred
rodič
commit
164aa5779a

+ 13 - 0
app/Jobs/CommentPipeline/CommentPipeline.php

@@ -60,10 +60,23 @@ class CommentPipeline implements ShouldQueue
         $actor = $comment->profile;
 
         if(config('database.default') === 'mysql') {
+            $count = DB::select(DB::raw("select id, in_reply_to_id from statuses, (select @pv := :kid) initialisation where deleted_at IS NULL and reblog_of_id IS NULL and id > @pv and find_in_set(in_reply_to_id, @pv) > 0 and @pv := concat(@pv, ',', id)"), [ 'kid' => $status->id]);
+            $status->reply_count = count($count);
+            $status->save();
+
+            $count = DB::select(DB::raw("select id, in_reply_to_id from statuses, (select @pv := :kid) initialisation where deleted_at IS NULL and reblog_of_id IS NULL and id > @pv and find_in_set(in_reply_to_id, @pv) > 0 and @pv := concat(@pv, ',', id)"), [ 'kid' => $comment->id]);
+            $comment->reply_count = count($count);
+            $comment->save();
+        } else {
             $status->reply_count = $status->reply_count + 1;
             $status->save();
         }
 
+        StatusService::del($comment->id);
+        StatusService::del($status->id);
+        Cache::forget('status:replies:all:' . $comment->id);
+        Cache::forget('status:replies:all:' . $status->id);
+
         if ($actor->id === $target->id || $status->comments_disabled == true) {
             return true;
         }

+ 1 - 0
app/Jobs/StatusPipeline/StatusDelete.php

@@ -100,6 +100,7 @@ class StatusDelete implements ShouldQueue
 			$parent = Status::findOrFail($status->in_reply_to_id);
 			--$parent->reply_count;
 			$parent->save();
+			StatusService::del($parent->id);
 		}
 
         Bookmark::whereStatusId($status->id)->delete();

+ 14 - 0
app/Jobs/StatusPipeline/StatusReplyPipeline.php

@@ -13,6 +13,7 @@ use Illuminate\Queue\InteractsWithQueue;
 use Illuminate\Queue\SerializesModels;
 use Illuminate\Support\Facades\Redis;
 use App\Services\NotificationService;
+use App\Services\StatusService;
 
 class StatusReplyPipeline implements ShouldQueue
 {
@@ -69,10 +70,23 @@ class StatusReplyPipeline implements ShouldQueue
         }
 
         if(config('database.default') === 'mysql') {
+            $count = DB::select( DB::raw("select id, in_reply_to_id from statuses, (select @pv := :kid) initialisation where deleted_at IS NULL and reblog_of_id IS NULL and id > @pv and find_in_set(in_reply_to_id, @pv) > 0 and @pv := concat(@pv, ',', id)"), [ 'kid' => $reply->id]);
+            $reply->reply_count = count($count);
+            $reply->save();
+
+            $count = DB::select( DB::raw("select id, in_reply_to_id from statuses, (select @pv := :kid) initialisation where deleted_at IS NULL and reblog_of_id IS NULL and id > @pv and find_in_set(in_reply_to_id, @pv) > 0 and @pv := concat(@pv, ',', id)"), [ 'kid' => $status->id]);
+            $status->reply_count = count($count);
+            $status->save();
+        } else {
             $reply->reply_count = $reply->reply_count + 1;
             $reply->save();
         }
 
+        StatusService::del($reply->id);
+        StatusService::del($status->id);
+        Cache::forget('status:replies:all:' . $reply->id);
+        Cache::forget('status:replies:all:' . $status->id);
+
         DB::transaction(function() use($target, $actor, $status) {
             $notification = new Notification();
             $notification->profile_id = $target->id;