Fix PHP code style issues

Author: AlizHarb

src/Commands/InstallCommand.php

@@ -34,7 +34,7 @@ public function handle(): int
             '--provider' => "Spatie\Activitylog\ActivitylogServiceProvider",
             '--tag' => 'activitylog-config',
         ]);
-        
+
         $this->info('Publishing Spatie Activity Log migration...');
         $this->call('vendor:publish', [
             '--provider' => "Spatie\Activitylog\ActivitylogServiceProvider",

src/Contracts/HasActivityLogTitle.php

@@ -4,8 +4,6 @@
 
 namespace AlizHarb\ActivityLog\Contracts;
 
-use Illuminate\Database\Eloquent\Model;
-
 interface HasActivityLogTitle
 {
     public function getActivityLogTitle(): string;

src/Enums/ActivityLogEvent.php

@@ -8,7 +8,7 @@
 use Filament\Support\Contracts\HasIcon;
 use Filament\Support\Contracts\HasLabel;
 
-enum ActivityLogEvent: string implements HasLabel, HasColor, HasIcon
+enum ActivityLogEvent: string implements HasColor, HasIcon, HasLabel
 {
     case Created = 'created';
     case Updated = 'updated';

src/Pages/UserActivitiesPage.php

@@ -11,7 +11,6 @@
 use Filament\Tables\Contracts\HasTable;
 use Filament\Tables\Filters\SelectFilter;
 use Filament\Tables\Table;
-use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Support\Facades\Auth;
 use Spatie\Activitylog\Models\Activity;
 
@@ -143,7 +142,7 @@ public function table(Table $table): Table
                     ->label(__('filament-activity-log::activity.filter.causer'))
                     ->options(function () {
                         $model = config('auth.providers.users.model');
-                        
+
                         if (! class_exists($model)) {
                             return [];
                         }

src/Resources/ActivityLogs/Tables/ActivityLogTable.php

@@ -217,7 +217,7 @@ public static function configure(Table $table): Table
                             );
                     })
                     ->visible(config('filament-activity-log.table.filters.created_at', true)),
-                
+
                 Filter::make('batch_uuid')
                     ->label('Batch UUID')
                     ->hidden()
@@ -244,13 +244,13 @@ public static function configure(Table $table): Table
                         ->visible(config('filament-activity-log.table.actions.timeline', true)),
                     ViewAction::make()
                         ->visible(config('filament-activity-log.table.actions.view', true)),
-                    
+
                     Action::make('view_batch')
                         ->label('Batch')
                         ->icon('heroicon-m-rectangle-stack')
                         ->color('gray')
                         ->visible(fn ($record) => $record->batch_uuid)
-                        ->url(fn ($record) => request()->url() . '?tableFilters[batch_uuid][value]=' . $record->batch_uuid),
+                        ->url(fn ($record) => request()->url().'?tableFilters[batch_uuid][value]='.$record->batch_uuid),
 
                     Action::make('revert')
                         ->icon('heroicon-m-arrow-uturn-left')

src/Support/ActivityLogTitle.php

@@ -30,11 +30,11 @@ public static function get(mixed $model): string
         if ($title = $model->getAttribute('email')) {
             return (string) $title;
         }
-        
+
         if ($title = $model->getAttribute('username')) {
             return (string) $title;
         }
 
-        return class_basename($model) . ' #' . $model->getKey();
+        return class_basename($model).' #'.$model->getKey();
     }
 }

tests/Feature/ImprovementsTest.php

@@ -23,38 +23,48 @@
 });
 
 it('can set and get cluster in plugin', function () {
-    $plugin = new ActivityLogPlugin();
+    $plugin = new ActivityLogPlugin;
     $plugin->cluster('System');
 
     expect($plugin->getCluster())->toBe('System');
 });
 
 it('resolves subject title using helper', function () {
-    $user = new class extends Model {
+    $user = new class extends Model
+    {
         protected $guarded = [];
     };
     $user->setAttribute('name', 'Test User');
     $user->setAttribute('id', 1);
-    
+
     // mimic accessors? No, just attributes.
     // getAttribute checks attributes array.
-    
+
     expect(ActivityLogTitle::get($user))->toBe('Test User');
 
-    $post = new class extends Model {
+    $post = new class extends Model
+    {
         protected $guarded = [];
     };
     $post->setAttribute('title', 'Test Post');
     $post->setAttribute('id', 2);
-    
+
     expect(ActivityLogTitle::get($post))->toBe('Test Post');
 
     expect(ActivityLogTitle::get($post))->toBe('Test Post');
 
-    $unknown = new class extends Model {
+    $unknown = new class extends Model
+    {
         // Mocking getKey explicitly as anonymous class table might issue
-        public function getKey() { return 3; }
-        public function getTable() { return 'unknowns'; }
+        public function getKey()
+        {
+            return 3;
+        }
+
+        public function getTable()
+        {
+            return 'unknowns';
+        }
     };
     $unknown->setAttribute('id', 3);
 
@@ -67,17 +77,17 @@ public function getTable() { return 'unknowns'; }
     // but we can verify the action exists on the table if we render it.
     // However, recreating table in test is complex.
     // We trust standard Filament testing for actions.
-    
+
     // We can test if the Filter query works
     $query = Activity::query();
     $filter = function ($data, $query) {
-         $query->when(
+        $query->when(
             $data['value'] ?? null,
             fn ($q, $uuid) => $q->where('batch_uuid', $uuid)
         );
     };
-    
+
     $filter(['value' => 'test-uuid'], $query);
-    
+
     expect($query->toSql())->toContain('batch_uuid');
 });

tests/Feature/UserActivitiesPageTest.php

@@ -3,6 +3,7 @@
 use AlizHarb\ActivityLog\Pages\UserActivitiesPage;
 use AlizHarb\ActivityLog\Tests\Fixtures\User;
 use Spatie\Activitylog\Models\Activity;
+
 use function Pest\Livewire\livewire;
 
 beforeEach(function () {
@@ -39,7 +40,7 @@
         'causer_id' => 1,
         'causer_type' => User::class,
     ]);
-    
+
     $activity2 = Activity::create([
         'log_name' => 'default',
         'description' => 'Activity 2',