Alert when page linked to menu item is deleted.

Author: sten

README.md

@@ -50,16 +50,28 @@ You can publish and run the migrations with:
 
 ```bash
 php artisan vendor:publish --tag="filament-flexible-content-block-pages-migrations"
+php artisan vendor:publish --provider="Spatie\MediaLibrary\MediaLibraryServiceProvider" --tag="medialibrary-migrations"
 php artisan migrate
 ```
 
-Configure the Filament tailwind styling by adding these view paths to `tailwind.config.js`:
+Configure the Filament tailwind styling of the frontend by adding these view paths to `content` array of `tailwind.config.js`:
 
-```php
+```javascript
 content: [
     ...
     './vendor/solution-forest/filament-tree/resources/**/*.blade.php',
     './vendor/statikbe/laravel-filament-flexible-content-block-pages/**/*.blade.php',
+    './vendor/statikbe/laravel-filament-flexible-content-blocks/**/*.blade.php',
+    './config/filament-flexible-content-blocks.php',
+]
+```
+
+In the tailwind config of your filament back-end, add the following lines to the `content` array:
+
+```javascript
+content: [
+    ...
+    './config/filament-flexible-content-blocks.php',
 ]
 ```
 
@@ -164,13 +176,16 @@ To make your models available in the menu builder, add them to the configuration
 ],
 ```
 
-Your models should implement the `HasMenuLabel` contract:
+Your models should implement the `[HasMenuLabel](src/Models/Contracts/HasMenuLabel.php)` contract and the [HasTitleMenuLabelTrait](src/Models/Concerns/HasTitleMenuLabelTrait.php) trait:
 
 ```php
 use Statikbe\FilamentFlexibleContentBlockPages\Models\Contracts\HasMenuLabel;
+use Statikbe\FilamentFlexibleContentBlockPages\Models\Concerns\HasMenuItemTrait;
 
 class Product extends Model implements HasMenuLabel
 {
+    use HasMenuItemTrait;
+    
     public function getMenuLabel(?string $locale = null): string
     {
         return $this->getTranslation('name', $locale ?? app()->getLocale());
@@ -382,6 +397,7 @@ menu:
 
 page:
 - make table searchable, columns orderable, test global search
+- laravel scout
 
 release:
 - policies:

resources/lang/nl/filament-flexible-content-block-pages.php

@@ -16,6 +16,9 @@
         'table' => [
             'created_at_col' => 'Aangemaakt op',
         ],
+        'notifications' => [
+            'used_in_menu' => 'Deze pagina wordt in het ":menu" menu gebruikt, in item :menu_item. Gelieve eerst het menu aan te passen en dan deze pagina opnieuw te verwijderen.',
+        ],
     ],
     'redirects' => [
         'redirects_lbl' => 'redirect',

src/Models/Concerns/HasMenuItemTrait.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace Statikbe\FilamentFlexibleContentBlockPages\Models\Concerns;
+
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Relations\MorphOne;
+use Statikbe\FilamentFlexibleContentBlockPages\Models\MenuItem;
+
+/**
+ * @property ?MenuItem $menuItem
+ *
+ * @mixin Model
+ */
+trait HasMenuItemTrait
+{
+    public function menuItem(): MorphOne
+    {
+        return $this->morphOne(MenuItem::class, 'linkable', 'linkable_type', 'linkable_id');
+    }
+}

src/Models/Concerns/HasTitleMenuLabelTrait.php

@@ -4,6 +4,8 @@
 
 trait HasTitleMenuLabelTrait
 {
+    use HasMenuItemTrait;
+
     /**
      * Get the display label for menu items.
      * This implementation uses the 'title' field with locale support.

src/Models/Contracts/HasMenuLabel.php

@@ -2,10 +2,27 @@
 
 namespace Statikbe\FilamentFlexibleContentBlockPages\Models\Contracts;
 
+use Illuminate\Database\Eloquent\Relations\MorphOne;
+use Statikbe\FilamentFlexibleContentBlockPages\Models\Concerns\HasMenuItemTrait;
+use Statikbe\FilamentFlexibleContentBlockPages\Models\Concerns\HasTitleMenuLabelTrait;
+use Statikbe\FilamentFlexibleContentBlockPages\Models\MenuItem;
 use Statikbe\FilamentFlexibleContentBlocks\Models\Contracts\Linkable;
 
+/**
+ * @see HasMenuItemTrait for implementation
+ * @see HasTitleMenuLabelTrait for implementation
+ *
+ * @property ?MenuItem $menuItem
+ */
 interface HasMenuLabel extends Linkable
 {
+    /**
+     * Sets the menu item morph relationship.
+     *
+     * @see HasMenuItemTrait for a convenient implementation.
+     */
+    public function menuItem(): MorphOne;
+
     /**
      * Get the display label for menu items.
      * This method should return a translatable field value or fallback text.

src/Models/Page.php

@@ -34,12 +34,13 @@
 /**
  * @property bool $is_undeletable
  */
-class Page extends Model implements HasCode, HasContentBlocks, HasHeroImageAttributes, HasIntroAttribute, HasMedia, HasMediaAttributes, HasMenuLabel, HasOverviewAttributes, HasPageAttributes, HasParent, HasSEOAttributes, HasHeroCallToActionsAttribute
+class Page extends Model implements HasCode, HasContentBlocks, HasHeroCallToActionsAttribute, HasHeroImageAttributes, HasIntroAttribute, HasMedia, HasMediaAttributes, HasMenuLabel, HasOverviewAttributes, HasPageAttributes, HasParent, HasSEOAttributes
 {
     use HasAuthorAttributeTrait;
     use HasCodeTrait;
     use HasDefaultContentBlocksTrait;
     use HasFactory;
+    use HasHeroCallToActionsTrait;
     use HasParentTrait;
     use HasTitleMenuLabelTrait;
     use HasTranslatedContentBlocksTrait;
@@ -49,7 +50,6 @@ class Page extends Model implements HasCode, HasContentBlocks, HasHeroImageAttri
     use HasTranslatedPageAttributesTrait;
     use HasTranslatedSEOAttributesTrait;
     use HasTranslatedSlugAttributeTrait;
-    use HasHeroCallToActionsTrait;
 
     const HOME_PAGE = 'HOME';
 

src/Models/Settings.php

@@ -90,7 +90,7 @@ public static function setting(string $settingField, ?string $locale = null): st
             static::CACHE_TAG_SETTINGS,
             $cacheKey,
             function () use ($settingField) {
-                $setting = static::getSettings()->getAttribute($settingField);
+                $setting = static::getSettings()?->getAttribute($settingField);
 
                 // replace text params in settings if it is a text field (based on $translatable fields):
                 if (in_array($settingField, app(static::class)->translatable)) {

src/Resources/PageResource.php

@@ -2,17 +2,22 @@
 
 namespace Statikbe\FilamentFlexibleContentBlockPages\Resources;
 
+use Filament\Actions\Action;
 use Filament\Forms\Components\Grid;
 use Filament\Forms\Components\Tabs;
 use Filament\Forms\Components\Tabs\Tab;
 use Filament\Forms\Form;
+use Filament\Notifications\Notification;
 use Filament\Resources\Concerns\Translatable;
 use Filament\Resources\Resource;
 use Filament\Tables\Actions\DeleteBulkAction;
 use Filament\Tables\Actions\EditAction;
 use Filament\Tables\Columns\TextColumn;
 use Filament\Tables\Table;
+use Illuminate\Database\Eloquent\Collection;
 use Statikbe\FilamentFlexibleContentBlockPages\Facades\FilamentFlexibleContentBlockPages;
+use Statikbe\FilamentFlexibleContentBlockPages\Models\MenuItem;
+use Statikbe\FilamentFlexibleContentBlockPages\Models\Page;
 use Statikbe\FilamentFlexibleContentBlockPages\Resources\PageResource\Pages\CreatePage;
 use Statikbe\FilamentFlexibleContentBlockPages\Resources\PageResource\Pages\EditPage;
 use Statikbe\FilamentFlexibleContentBlockPages\Resources\PageResource\Pages\ListPages;
@@ -194,7 +199,32 @@ public static function table(Table $table): Table
                 ViewAction::make(),
             ])
             ->bulkActions([
-                DeleteBulkAction::make(),
+                DeleteBulkAction::make()
+                    ->action(function (Collection $records, Action $action) {
+                        $usedInMenu = false;
+                        foreach ($records as $record) {
+                            // Prevent deletion if the page is referenced by a menu item
+                            /** @var ?MenuItem $menuItem */
+                            $menuItem = $record->menuItem();
+
+                            if ($menuItem) {
+                                $usedInMenu = true;
+
+                                Notification::make()
+                                    ->title(flexiblePagesTrans('pages.notifications.used_in_menu', [
+                                        'menu' => $menuItem->menu->name,
+                                        'menu_item' => $menuItem->getDisplayLabel(),
+                                    ]))
+                                    ->danger()
+                                    ->send();
+                            }
+                        }
+
+                        if (! $usedInMenu) {
+                            $this->process(static fn (Collection $records) => $records->each(fn (Page $record) => $record->delete()));
+                            $action->success();
+                        }
+                    }),
             ])
             ->recordUrl(
                 fn ($record): string => static::getUrl('edit', ['record' => $record])

src/Resources/PageResource/Pages/EditPage.php

@@ -3,9 +3,11 @@
 namespace Statikbe\FilamentFlexibleContentBlockPages\Resources\PageResource\Pages;
 
 use Filament\Actions\DeleteAction;
+use Filament\Notifications\Notification;
 use Filament\Resources\Pages\EditRecord;
 use Statikbe\FilamentFlexibleContentBlockPages\Facades\FilamentFlexibleContentBlockPages;
 use Statikbe\FilamentFlexibleContentBlockPages\FilamentFlexibleContentBlockPagesConfig;
+use Statikbe\FilamentFlexibleContentBlockPages\Models\MenuItem;
 use Statikbe\FilamentFlexibleContentBlockPages\Models\Page;
 use Statikbe\FilamentFlexibleContentBlocks\Filament\Actions\FlexibleLocaleSwitcher;
 use Statikbe\FilamentFlexibleContentBlocks\Filament\Pages\EditRecord\Concerns\TranslatableWithMedia;
@@ -26,7 +28,25 @@ protected function getActions(): array
             CopyContentBlocksToLocalesAction::make(),
             FlexibleLocaleSwitcher::make(),
             DeleteAction::make()
-                ->visible(fn (Page $record) => $record->isDeletable()),
+                ->visible(fn (Page $record) => $record->isDeletable())
+                ->action(function (Page $record, $action) {
+                    // Prevent deletion if the page is referenced by a menu item
+                    /** @var ?MenuItem $menuItem */
+                    $menuItem = $record->menuItem();
+
+                    if ($menuItem) {
+                        Notification::make()
+                            ->title(flexiblePagesTrans('pages.notifications.used_in_menu', [
+                                'menu' => $menuItem->menu->name,
+                                'menu_item' => $menuItem->getDisplayLabel(),
+                            ]))
+                            ->danger()
+                            ->send();
+                    } else {
+                        $record->delete();
+                        $action->success();
+                    }
+                }),
         ];
     }