feat(filament): added custom slug field (#712)

AnimeThemes · Jul 6, 2024 · 36d7c3e · 36d7c3e
1 parent 50e0bab
commit 36d7c3e
Show file tree

Hide file tree

Showing 36 changed files with 86 additions and 437 deletions.
diff --git a/app/Concerns/Filament/Actions/HasActionLogs.php b/app/Concerns/Filament/Actions/HasActionLogs.php
@@ -84,6 +84,10 @@ public function updateLog(Model $relatedModel, Model $pivot): void
     public function failedLog(Throwable|string|null $exception): void
     {
         $this->actionLog->failed($exception);
+
+        // Filament notifications
+        $this->failureNotificationTitle(Str::limit($exception, 200));
+        $this->failure();
     }
 
     /**
@@ -95,6 +99,9 @@ public function finishedLog(): void
     {
         if (!$this->isFailedLog()) {
             $this->actionLog->finished();
+
+            // Filament notifications
+            $this->success();
         }
     }
 
@@ -117,4 +124,6 @@ public function isFailedLog(): bool
     {
         return $this->actionLog->isFailed();
     }
+
+
 }
diff --git a/app/Filament/Actions/BaseAction.php b/app/Filament/Actions/BaseAction.php
@@ -28,15 +28,14 @@ protected function setUp(): void
         parent::setUp();
 
         $this->requiresConfirmation();
+        $this->failure();
 
-        $this->afterFormValidated(function (BaseAction $action, $livewire) {
+        $this->before(function (BaseAction $action) {
             $this->createActionLog($action);
-            $livewire->dispatch('updateAllRelationManager');
         });
 
-        $this->after(function ($livewire) {
+        $this->after(function () {
             $this->finishedLog();
-            $livewire->dispatch('updateAllRelationManager');
         });
 
         $this->modalWidth(MaxWidth::FourExtraLarge);

diff --git a/app/Filament/Actions/Models/Wiki/Anime/BackfillAnimeAction.php b/app/Filament/Actions/Models/Wiki/Anime/BackfillAnimeAction.php
@@ -27,7 +27,6 @@
 use Filament\Forms\Form;
 use Filament\Notifications\Actions\Action as NotificationAction;
 use Filament\Notifications\Notification;
-use Filament\Support\Enums\MaxWidth;
 use Illuminate\Bus\Queueable;
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Queue\InteractsWithQueue;
@@ -66,8 +65,6 @@ protected function setUp(): void
         $this->label(__('filament.actions.anime.backfill.name'));
         $this->icon('heroicon-o-bars-4');
 
-        $this->modalWidth(MaxWidth::FourExtraLarge);
-
         $this->authorize('create', Anime::class);
 
         $this->action(fn (Anime $record, array $data) => $this->handle($record, $data));

diff --git a/app/Filament/BulkActions/BaseBulkAction.php b/app/Filament/BulkActions/BaseBulkAction.php
@@ -6,7 +6,6 @@
 
 use App\Concerns\Filament\Actions\HasActionLogs;
 use App\Models\BaseModel;
-use Filament\Forms\Form;
 use Filament\Support\Enums\MaxWidth;
 use Filament\Tables\Actions\BulkAction;
 use Illuminate\Database\Eloquent\Collection;

diff --git a/app/Filament/Components/Fields/Slug.php b/app/Filament/Components/Fields/Slug.php
@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Filament\Components\Fields;
+
+use Filament\Forms\Components\TextInput;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Validation\Rule;
+
+/**
+ * Class Slug.
+ */
+class Slug extends TextInput
+{
+    /**
+     * Initial setup for the field.
+     *
+     * @return void
+     */
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->required();
+        $this->unique(ignoreRecord: true);
+        $this->rules([
+            fn (?Model $record) => [
+                'required',
+                'max:192',
+                'alpha_dash',
+                $record !== null
+                    ? Rule::unique($record::class, $this->getName())
+                        ->ignore($record->getKey(), $record->getKeyName())
+                        ->__toString()
+                    : Rule::unique($this->getModel(), $this->getName())->__toString(),
+            ],
+        ]);
+    }
+}
diff --git a/app/Filament/HeaderActions/BaseHeaderAction.php b/app/Filament/HeaderActions/BaseHeaderAction.php
@@ -28,7 +28,7 @@ protected function setUp(): void
 
         $this->requiresConfirmation();
 
-        $this->afterFormValidated(function (BaseHeaderAction $action, $livewire) {
+        $this->before(function (BaseHeaderAction $action, $livewire) {
             $this->createActionLog($action);
             $livewire->dispatch('updateAllRelationManager');
         });

diff --git a/app/Filament/Resources/Admin/ActionLog.php b/app/Filament/Resources/Admin/ActionLog.php
@@ -98,18 +98,6 @@ public static function getRecordSlug(): string
         return 'action-logs';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return ActionLogModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/Admin/Announcement.php b/app/Filament/Resources/Admin/Announcement.php
@@ -91,18 +91,6 @@ public static function getRecordSlug(): string
         return 'announcements';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return AnnouncementModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/Admin/Dump.php b/app/Filament/Resources/Admin/Dump.php
@@ -96,18 +96,6 @@ public static function getRecordSlug(): string
         return 'dumps';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return DumpModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/Admin/Feature.php b/app/Filament/Resources/Admin/Feature.php
@@ -93,18 +93,6 @@ public static function getRecordSlug(): string
         return 'features';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return FeatureModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/Admin/FeaturedTheme.php b/app/Filament/Resources/Admin/FeaturedTheme.php
@@ -105,18 +105,6 @@ public static function getRecordSlug(): string
         return 'featured-themes';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return FeaturedThemeModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/Auth/Permission.php b/app/Filament/Resources/Auth/Permission.php
@@ -96,18 +96,6 @@ public static function getRecordSlug(): string
         return 'permissions';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return PermissionModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/Auth/Role.php b/app/Filament/Resources/Auth/Role.php
@@ -104,18 +104,6 @@ public static function getRecordSlug(): string
         return 'roles';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return RoleModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/Auth/User.php b/app/Filament/Resources/Auth/User.php
@@ -101,18 +101,6 @@ public static function getRecordSlug(): string
         return 'users';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return UserModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/BaseResource.php b/app/Filament/Resources/BaseResource.php
@@ -43,6 +43,21 @@ public static function canGloballySearch(): bool
         return false;
     }
 
+    /**
+     * Get the route key for the resource.
+     *
+     * @return string
+     *
+     * @noinspection PhpMissingParentCallCommonInspection
+     */
+    public static function getRecordRouteKeyName(): string
+    {
+        /** @var Model */
+        $model = static::getModel();
+
+        return (new $model)->getKeyName();
+    }
+
     /**
      * The index page of the resource.
      *

diff --git a/app/Filament/Resources/Discord/DiscordThread.php b/app/Filament/Resources/Discord/DiscordThread.php
@@ -113,18 +113,6 @@ public static function getRecordSlug(): string
         return 'discord-thread';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return DiscordThreadModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *

diff --git a/app/Filament/Resources/Document/Page.php b/app/Filament/Resources/Document/Page.php
@@ -7,6 +7,7 @@
 use App\Enums\Http\Api\Filter\ComparisonOperator;
 use App\Filament\Components\Columns\TextColumn;
 use App\Filament\Components\Fields\Select;
+use App\Filament\Components\Fields\Slug;
 use App\Filament\Components\Infolist\TextEntry;
 use App\Filament\Resources\BaseResource;
 use App\Filament\Resources\Document\Page\Pages\CreatePage;
@@ -27,7 +28,6 @@
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Str;
-use Illuminate\Validation\Rule;
 
 /**
  * Class Page.
@@ -101,18 +101,6 @@ public static function getRecordSlug(): string
         return 'pages';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return PageModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *
@@ -141,25 +129,10 @@ public static function form(Form $form): Form
                     ->live(true)
                     ->afterStateUpdated(fn (Set $set, Get $get) => static::setSlug($set, $get)),
 
-                TextInput::make(PageModel::ATTRIBUTE_SLUG)
+                Slug::make(PageModel::ATTRIBUTE_SLUG)
                     ->label(__('filament.fields.page.slug.name'))
                     ->helperText(__('filament.fields.page.slug.help'))
-                    ->required()
-                    ->unique(PageModel::class, PageModel::ATTRIBUTE_SLUG, ignoreRecord: true)
-                    ->maxLength(192)
-                    ->regex('/^[\pL\pM\pN\/_-]+$/u')
-                    ->rules([
-                        fn ($record) => [
-                            'required',
-                            'max:192',
-                            'regex:/^[\pL\pM\pN\/_-]+$/u',
-                            $record !== null
-                                ? Rule::unique(PageModel::class)
-                                    ->ignore($record->getKey(), PageModel::ATTRIBUTE_ID)
-                                    ->__toString()
-                                : Rule::unique(PageModel::class)->__toString(),
-                        ]
-                    ]),
+                    ->regex('/^[\pL\pM\pN\/_-]+$/u'),
 
                 MarkdownEditor::make(PageModel::ATTRIBUTE_BODY)
                     ->label(__('filament.fields.page.body.name'))

diff --git a/app/Filament/Resources/List/External/ExternalEntry.php b/app/Filament/Resources/List/External/ExternalEntry.php
@@ -102,18 +102,6 @@ public static function getRecordSlug(): string
         return 'external-entries';
     }
 
-    /**
-     * Get the route key for the resource.
-     *
-     * @return string
-     *
-     * @noinspection PhpMissingParentCallCommonInspection
-     */
-    public static function getRecordRouteKeyName(): string
-    {
-        return ExternalEntryModel::ATTRIBUTE_ID;
-    }
-
     /**
      * The form to the actions.
      *