♻️ refactoring code

Author: mckenziearts

app/Http/Livewire/Articles/Browse.php

@@ -6,6 +6,7 @@
 use App\Models\Tag;
 use App\Traits\WithInfiniteScroll;
 use App\Traits\WithTags;
+use Illuminate\Contracts\View\View;
 use Livewire\Component;
 
 class Browse extends Component
@@ -26,7 +27,7 @@ public function validSort($sort): bool
         ]);
     }
 
-    public function render()
+    public function render(): View
     {
         $articles = Article::with('tags')->published()
             ->notPinned()

app/Http/Livewire/Discussions/Browse.php

@@ -6,6 +6,7 @@
 use App\Models\Tag;
 use App\Traits\WithInfiniteScroll;
 use App\Traits\WithTags;
+use Illuminate\Contracts\View\View;
 use Livewire\Component;
 
 class Browse extends Component
@@ -17,7 +18,7 @@ class Browse extends Component
         'sortBy' => ['except' => 'recent'],
     ];
 
-    public function validSort($sort): bool
+    public function validSort(string $sort): bool
     {
         return in_array($sort, [
             'recent',
@@ -26,7 +27,7 @@ public function validSort($sort): bool
         ]);
     }
 
-    public function render()
+    public function render(): View
     {
         $discussions = Discussion::with('tags')
             ->withCount('replies')

app/Traits/WithInfiniteScroll.php

@@ -10,7 +10,7 @@ trait WithInfiniteScroll
 
     public int $perPage = 10;
 
-    public function loadMore()
+    public function loadMore(): void
     {
         $this->perPage += 10;
     }

app/Traits/WithTags.php

@@ -10,12 +10,12 @@ trait WithTags
 
     public string $sortBy = 'recent';
 
-    public function toggleTag($tag): void
+    public function toggleTag(string $tag): void
     {
         $this->tag = $this->tag !== $tag && $this->tagExists($tag) ? $tag : null;
     }
 
-    public function sortBy($sort): void
+    public function sortBy(string $sort): void
     {
         $this->sortBy = $this->validSort($sort) ? $sort : 'recent';
     }

database/factories/DiscussionFactory.php

@@ -7,7 +7,12 @@
 
 class DiscussionFactory extends Factory
 {
-    public function definition()
+    /**
+     * Define the model's default state.
+     *
+     * @return array<string, mixed>
+     */
+    public function definition(): array
     {
         return [
             'user_id' => $attributes['user_id'] ?? User::factory(),