Linting updates

Author: navneetrai

src/Filament/Concerns/HasSidebar.php

@@ -2,10 +2,10 @@
 
 namespace Kirschbaum\Commentions\Filament\Concerns;
 
+use Closure;
 use Illuminate\Database\Eloquent\Model;
 use Kirschbaum\Commentions\Config;
 use Kirschbaum\Commentions\Contracts\Commenter;
-use Closure;
 
 trait HasSidebar
 {
@@ -60,6 +60,48 @@ public function showSubscribers(): bool
         return $this->showSubscribers;
     }
 
+    public function subscribeLabel(Closure|string $label): static
+    {
+        $this->subscribeLabelOverride = $label;
+
+        return $this;
+    }
+
+    public function unsubscribeLabel(Closure|string $label): static
+    {
+        $this->unsubscribeLabelOverride = $label;
+
+        return $this;
+    }
+
+    public function subscribeIcon(Closure|string $icon): static
+    {
+        $this->subscribeIconOverride = $icon;
+
+        return $this;
+    }
+
+    public function unsubscribeIcon(Closure|string $icon): static
+    {
+        $this->unsubscribeIconOverride = $icon;
+
+        return $this;
+    }
+
+    public function subscribeColor(Closure|string $color, string $context = 'action'): static
+    {
+        $this->subscriptionColorOverrides[$context]['subscribe'] = $color;
+
+        return $this;
+    }
+
+    public function unsubscribeColor(Closure|string $color, string $context = 'action'): static
+    {
+        $this->subscriptionColorOverrides[$context]['unsubscribe'] = $color;
+
+        return $this;
+    }
+
     protected function resolveCurrentUser(): ?Commenter
     {
         return Config::resolveAuthenticatedUser();
@@ -110,50 +152,8 @@ protected function computeSubscriptionColor(Model $record, string $context = 'ac
         return $this->evaluateSubscriptionOverride($this->subscriptionColorOverrides[$context]['subscribe'] ?? null, $record) ?? ($context === 'table' ? 'primary' : 'gray');
     }
 
-    public function subscribeLabel(Closure|string $label): static
-    {
-        $this->subscribeLabelOverride = $label;
-
-        return $this;
-    }
-
-    public function unsubscribeLabel(Closure|string $label): static
-    {
-        $this->unsubscribeLabelOverride = $label;
-
-        return $this;
-    }
-
-    public function subscribeIcon(Closure|string $icon): static
-    {
-        $this->subscribeIconOverride = $icon;
-
-        return $this;
-    }
-
-    public function unsubscribeIcon(Closure|string $icon): static
-    {
-        $this->unsubscribeIconOverride = $icon;
-
-        return $this;
-    }
-
-    public function subscribeColor(Closure|string $color, string $context = 'action'): static
-    {
-        $this->subscriptionColorOverrides[$context]['subscribe'] = $color;
-
-        return $this;
-    }
-
-    public function unsubscribeColor(Closure|string $color, string $context = 'action'): static
-    {
-        $this->subscriptionColorOverrides[$context]['unsubscribe'] = $color;
-
-        return $this;
-    }
-
     /**
-     * @param (Closure(Model): string)|string|null $override
+     * @param  (Closure(Model): string)|string|null  $override
      */
     protected function evaluateSubscriptionOverride(Closure|string|null $override, Model $record): ?string
     {
@@ -164,10 +164,6 @@ protected function evaluateSubscriptionOverride(Closure|string|null $override, M
         return $override;
     }
 
-    /**
-     *
-     * @return bool|null
-     */
     protected function toggleSubscriptionForRecord(Model $record): ?bool
     {
         $user = $this->resolveCurrentUser();

src/Livewire/Concerns/HasSidebar.php

@@ -94,5 +94,3 @@ protected function getCurrentUser(): ?Commenter
         return Config::resolveAuthenticatedUser();
     }
 }
-
-

src/Livewire/SubscriptionSidebar.php

@@ -23,5 +23,3 @@ public function render()
         return view('commentions::subscription-sidebar');
     }
 }
-
-

tests/Livewire/CommentsSubscriptionTest.php

@@ -55,14 +55,14 @@
     livewire(Comments::class, [
         'record' => $post,
     ])->assertSet('isSubscribed', false)
-      ->assertSet('subscribers', fn ($subscribers) => $subscribers instanceof \Illuminate\Support\Collection && $subscribers->isEmpty());
+        ->assertSet('subscribers', fn ($subscribers) => $subscribers instanceof \Illuminate\Support\Collection && $subscribers->isEmpty());
 
     $post->subscribe($user);
 
     livewire(Comments::class, [
         'record' => $post,
     ])->assertSet('isSubscribed', true)
-      ->assertSet('subscribers', fn ($subscribers) => $subscribers->contains('id', $user->id));
+        ->assertSet('subscribers', fn ($subscribers) => $subscribers->contains('id', $user->id));
 });
 
 test('toggleSubscription subscribes and unsubscribes the current user', function () {
@@ -78,7 +78,7 @@
     livewire(Comments::class, [
         'record' => $post,
     ])->call('toggleSubscription')
-      ->assertSet('isSubscribed', true);
+        ->assertSet('isSubscribed', true);
 
     expect(CommentSubscription::query()->where([
         'subscribable_type' => $post->getMorphClass(),
@@ -90,7 +90,7 @@
     livewire(Comments::class, [
         'record' => $post,
     ])->call('toggleSubscription')
-      ->assertSet('isSubscribed', false);
+        ->assertSet('isSubscribed', false);
 
     expect(CommentSubscription::query()->where([
         'subscribable_type' => $post->getMorphClass(),
@@ -113,5 +113,3 @@
         'record' => $post,
     ])->assertSet('showSubscribers', false);
 });
-
-