Fix Lint

PortableStudios · Apr 11, 2024 · 51878d3 · 51878d3
1 parent b1e1c18
commit 51878d3
Show file tree

Hide file tree

Showing 7 changed files with 72 additions and 9 deletions.
diff --git a/src/Filament/Resources/RoleResource.php b/src/Filament/Resources/RoleResource.php
@@ -11,6 +11,7 @@
 use Portable\FilaCms\Filament\Resources\RoleResource\Pages;
 use Portable\FilaCms\Filament\Traits\IsProtectedResource;
 use Spatie\Permission\Models\Role;
+use Portable\FilaCms\Filament\Resources\RoleResource\RelationManagers;
 
 class RoleResource extends AbstractResource
 {
@@ -55,19 +56,19 @@ public static function table(Table $table): Table
             ]);
     }
 
-    public static function getRelations(): array
+    public static function getPages(): array
     {
         return [
-            //
+            'index' => Pages\ListRoles::route('/'),
+            'create' => Pages\CreateRole::route('/create'),
+            'edit' => Pages\EditRole::route('/{record}/edit'),
         ];
     }
 
-    public static function getPages(): array
+    public static function getRelations(): array
     {
         return [
-            'index' => Pages\ListRoles::route('/'),
-            'create' => Pages\CreateRole::route('/create'),
-            'edit' => Pages\EditRole::route('/{record}/edit'),
+            RelationManagers\UsersRelationManager::class,
         ];
     }
 }
diff --git a/src/Filament/Resources/RoleResource/Pages/EditRole.php b/src/Filament/Resources/RoleResource/Pages/EditRole.php
@@ -5,6 +5,8 @@
 use Filament\Actions;
 use Filament\Resources\Pages\EditRecord;
 use Portable\FilaCms\Filament\Resources\RoleResource;
+use Filament\Notifications\Notification;
+use Spatie\Permission\Models\Role;
 
 class EditRole extends EditRecord
 {
@@ -13,7 +15,17 @@ class EditRole extends EditRecord
     protected function getHeaderActions(): array
     {
         return [
-            Actions\DeleteAction::make(),
+            Actions\DeleteAction::make()
+                ->before(function (Actions\DeleteAction $action, Role $role) {
+                    if ($role->users->count() > 0) {
+                        Notification::make()
+                            ->title('Unable to perform action')
+                            ->body('You cannot delete a role that are assigned to user(s)')
+                            ->status('warning')
+                            ->send();
+                        $action->cancel();
+                    }
+                }),
         ];
     }
 }
diff --git a/src/Filament/Resources/RoleResource/RelationManagers/UsersRelationManager.php b/src/Filament/Resources/RoleResource/RelationManagers/UsersRelationManager.php
@@ -0,0 +1,32 @@
+<?php
+
+namespace Portable\FilaCms\Filament\Resources\RoleResource\RelationManagers;
+
+use Filament\Forms\Form;
+use Filament\Resources\RelationManagers\RelationManager;
+use Filament\Tables\Columns\ViewColumn;
+use Filament\Tables\Table;
+
+class UsersRelationManager extends RelationManager
+{
+    protected static string $relationship = 'users';
+
+    public function form(Form $form): Form
+    {
+        return $form
+            ->schema([]);
+    }
+
+    public function table(Table $table): Table
+    {
+        return $table
+            ->recordTitleAttribute('Users')
+            ->columns([
+                ViewColumn::make('name')
+                    ->view('fila-cms::tables.columns.roles-user'),
+                ViewColumn::make('created_at')
+                    ->label('Creation Date')
+                    ->view('fila-cms::tables.columns.created_at'),
+            ]);
+    }
+}
diff --git a/tests/Filament/PageResourceTest.php b/tests/Filament/PageResourceTest.php
@@ -155,7 +155,7 @@ public function test_can_save_form(): void
         $this->assertEquals($data->title, $new->title);
         $this->assertEquals($data->author_id, $new->author_id);
         $this->assertEquals($data->is_draft, $new->is_draft);
-        $this->assertGreaterThanOrEqual($data->updated_at->format('U'), $updatedTime->format('U'));
+        $this->assertGreaterThanOrEqual($updatedTime->format('U'), $data->updated_at->format('U'));
     }
 
     public function test_can_create_page_with_taxonomies(): void

diff --git a/tests/Filament/PermissionResourceTest.php b/tests/Filament/PermissionResourceTest.php
@@ -100,6 +100,6 @@ public function test_can_save_form(): void
 
         $data->refresh();
         $this->assertEquals($data->name, $new->name);
-        $this->assertGreaterThanOrEqual($data->updated_at->format('U'), $updatedTime->format('U'));
+        $this->assertGreaterThanOrEqual($updatedTime->format('U'), $data->updated_at->format('U'));
     }
 }
diff --git a/views/tables/columns/created_at.blade.php b/views/tables/columns/created_at.blade.php
@@ -0,0 +1,9 @@
+<div class="fi-ta-text grid w-full gap-y-1 px-3 py-4">
+  <div class="flex">
+    <div class="fi-ta-text-item inline-flex items-center gap-1.5 ">
+      <span class="fi-ta-text-item-label text-sm leading-6 text-gray-950 dark:text-white">
+        {{ $getRecord()->created_at }}
+      </span>
+    </div>
+  </div>
+</div>
diff --git a/views/tables/columns/roles-user.blade.php b/views/tables/columns/roles-user.blade.php
@@ -0,0 +1,9 @@
+<div class="fi-ta-text grid w-full gap-y-1 px-3 py-4">
+  <div class="flex">
+    <div class="fi-ta-text-item inline-flex items-center gap-1.5 ">
+      <span class="fi-ta-text-item-label text-sm leading-6 text-gray-950 dark:text-white">
+        {{ $getRecord()->name ?? $getRecord()->first_name }}
+      </span>
+    </div>
+  </div>
+</div>