Update FollowPipeline, improve follower/following count calculation

app/Jobs/FollowPipeline/FollowPipeline.php

@@ -17,91 +17,71 @@ use App\Services\FollowerService;
 
 class FollowPipeline implements ShouldQueue
 {
-	use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
+    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
 
-	protected $follower;
+    protected $follower;
 
-	/**
+    /**
-	 * Delete the job if its models no longer exist.
+     * Delete the job if its models no longer exist.
-	 *
+     *
-	 * @var bool
+     * @var bool
-	 */
+     */
-	public $deleteWhenMissingModels = true;
+    public $deleteWhenMissingModels = true;
-	
-	/**
-	 * Create a new job instance.
-	 *
-	 * @return void
-	 */
-	public function __construct($follower)
-	{
-		$this->follower = $follower;
-	}
 
-	/**
+    /**
-	 * Execute the job.
+     * Create a new job instance.
-	 *
+     *
-	 * @return void
+     * @return void
-	 */
+     */
-	public function handle()
+    public function __construct($follower)
-	{
+    {
-		$follower = $this->follower;
+        $this->follower = $follower;
-		$actor = $follower->actor;
+    }
-		$target = $follower->target;
 
-		if(!$actor || !$target) {
+    /**
-			return;
+     * Execute the job.
-		}
+     *
+     * @return void
+     */
+    public function handle()
+    {
+        $follower = $this->follower;
+        $actor = $follower->actor;
+        $target = $follower->target;
 
-		Cache::forget('profile:following:' . $actor->id);
+        if(!$actor || !$target) {
-		Cache::forget('profile:following:' . $target->id);
+            return;
+        }
 
-		FollowerService::add($actor->id, $target->id);
+        if($target->domain || !$target->private_key) {
+            return;
+        }
 
-		$actorProfileSync = Cache::get(FollowerService::FOLLOWING_SYNC_KEY . $actor->id);
+        Cache::forget('profile:following:' . $actor->id);
-		if(!$actorProfileSync) {
+        Cache::forget('profile:following:' . $target->id);
-			FollowServiceWarmCache::dispatch($actor->id)->onQueue('low');
-		} else {
-			if($actor->following_count) {
-				$actor->increment('following_count');
-			} else {
-				$count = Follower::whereProfileId($actor->id)->count();
-				$actor->following_count = $count;
-				$actor->save();
-			}
-			Cache::put(FollowerService::FOLLOWING_SYNC_KEY . $actor->id, 1, 604800);
-			AccountService::del($actor->id);
-		}
 
-		$targetProfileSync = Cache::get(FollowerService::FOLLOWERS_SYNC_KEY . $target->id);
+        FollowerService::add($actor->id, $target->id);
-		if(!$targetProfileSync) {
-			FollowServiceWarmCache::dispatch($target->id)->onQueue('low');
-		} else {
-			if($target->followers_count) {
-				$target->increment('followers_count');
-			} else {
-				$count = Follower::whereFollowingId($target->id)->count();
-				$target->followers_count = $count;
-				$target->save();
-			}
-			Cache::put(FollowerService::FOLLOWERS_SYNC_KEY . $target->id, 1, 604800);
-			AccountService::del($target->id);
-		}
 
-		if($target->domain || !$target->private_key) {
+        $count = Follower::whereProfileId($actor->id)->count();
-			return;
+        $actor->following_count = $count;
-		}
+        $actor->save();
+        AccountService::del($actor->id);
 
-		try {
+        $count = Follower::whereFollowingId($target->id)->count();
-			$notification = new Notification();
+        $target->followers_count = $count;
-			$notification->profile_id = $target->id;
+        $target->save();
-			$notification->actor_id = $actor->id;
+        AccountService::del($target->id);
-			$notification->action = 'follow';
+
-			$notification->item_id = $target->id;
+        try {
-			$notification->item_type = "App\Profile";
+            $notification = new Notification();
-			$notification->save();
+            $notification->profile_id = $target->id;
-		} catch (Exception $e) {
+            $notification->actor_id = $actor->id;
-			Log::error($e);
+            $notification->action = 'follow';
-		}
+            $notification->item_id = $target->id;
-	}
+            $notification->item_type = "App\Profile";
+            $notification->save();
+        } catch (Exception $e) {
+            Log::error($e);
+        }
+    }
 }