diff --git a/app/Http/Controllers/CommentController.php b/app/Http/Controllers/CommentController.php index 0a8e3b66..20685da0 100644 --- a/app/Http/Controllers/CommentController.php +++ b/app/Http/Controllers/CommentController.php @@ -43,8 +43,8 @@ class CommentController extends Controller abort(403); } $this->validate($request, [ - 'item' => 'required|integer', - 'comment' => 'required|string|max:500', + 'item' => 'required|integer|min:1', + 'comment' => 'required|string|max:'.(int) config('pixelfed.max_caption_length'), ]); $comment = $request->input('comment'); $statusId = $request->item; @@ -57,8 +57,6 @@ class CommentController extends Controller return; } - Cache::forget('transform:status:'.$status->url()); - $reply = DB::transaction(function() use($comment, $status, $profile) { $autolink = Autolink::create()->autolink($comment); $reply = new Status(); diff --git a/app/Http/Controllers/LikeController.php b/app/Http/Controllers/LikeController.php index 9ee452d3..2b87353d 100644 --- a/app/Http/Controllers/LikeController.php +++ b/app/Http/Controllers/LikeController.php @@ -20,14 +20,12 @@ class LikeController extends Controller public function store(Request $request) { $this->validate($request, [ - 'item' => 'required|integer', + 'item' => 'required|integer|min:1', ]); $profile = Auth::user()->profile; $status = Status::withCount('likes')->findOrFail($request->input('item')); - Cache::forget('transform:status:'.$status->url()); - $count = $status->likes_count; if ($status->likes()->whereProfileId($profile->id)->count() !== 0) { @@ -48,8 +46,6 @@ class LikeController extends Controller ->take(1000) ->pluck('status_id'); - Cache::put('api:like-ids:user:'.$profile->id, $likes, now()->addMinutes(1440)); - if ($request->ajax()) { $response = ['code' => 200, 'msg' => 'Like saved', 'count' => $count]; } else { diff --git a/app/Http/Controllers/Settings/LabsSettings.php b/app/Http/Controllers/Settings/LabsSettings.php index 1d783e2f..20e2106d 100644 --- a/app/Http/Controllers/Settings/LabsSettings.php +++ b/app/Http/Controllers/Settings/LabsSettings.php @@ -53,7 +53,9 @@ trait LabsSettings { if($profile->is_suggestable == false) { $profile->is_suggestable = true; $changes = true; - SuggestionService::set($profile->id); + if($profile->statuses->count() > 0) { + SuggestionService::set($profile->id); + } } } else { $profile->is_suggestable = false; diff --git a/app/Http/Controllers/StatusController.php b/app/Http/Controllers/StatusController.php index 35d335e5..b2246440 100644 --- a/app/Http/Controllers/StatusController.php +++ b/app/Http/Controllers/StatusController.php @@ -229,14 +229,12 @@ class StatusController extends Controller $this->authCheck(); $this->validate($request, [ - 'item' => 'required|integer', + 'item' => 'required|integer|min:1', ]); $profile = Auth::user()->profile; $status = Status::withCount('shares')->findOrFail($request->input('item')); - Cache::forget('transform:status:'.$status->url()); - $count = $status->shares_count; $exists = Status::whereProfileId(Auth::user()->profile->id) diff --git a/app/Services/SuggestionService.php b/app/Services/SuggestionService.php index 9160b047..d34ac17a 100644 --- a/app/Services/SuggestionService.php +++ b/app/Services/SuggestionService.php @@ -45,6 +45,7 @@ class SuggestionService { $ids = Profile::whereNull('domain') ->whereIsSuggestable(true) ->whereIsPrivate(false) + ->whereHas('statuses') ->pluck('id'); foreach($ids as $id) { self::set($id);