improve delete department

add cascade on delete and update department for letters show soft deleted departments and allow force delete or restor them
lara-zeus · Jul 2, 2023 · 01c5a22 · 01c5a22
1 parent 585b80d
commit 01c5a22
Show file tree

Hide file tree

Showing 4 changed files with 101 additions and 5 deletions.
diff --git a/database/migrations/alter_letters_constraints.php.stub b/database/migrations/alter_letters_constraints.php.stub
@@ -0,0 +1,39 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        Schema::table('letters', function (Blueprint $table) {
+            Schema::table('letters', function (Blueprint $table) {
+            Schema::disableForeignKeyConstraints();
+            $table->dropForeign(['department_id']);
+            Schema::enableForeignKeyConstraints();
+
+            $table->foreign('department_id')
+                ->onUpdate('cascade')
+                ->onDelete('cascade')
+                ->references('id')
+                ->on('departments');
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+
+    }
+};
diff --git a/src/Filament/Resources/DepartmentResource.php b/src/Filament/Resources/DepartmentResource.php
@@ -13,15 +13,23 @@
 use Filament\Tables\Actions\Action;
 use Filament\Tables\Actions\ActionGroup;
 use Filament\Tables\Actions\DeleteAction;
+use Filament\Tables\Actions\DeleteBulkAction;
 use Filament\Tables\Actions\EditAction;
+use Filament\Tables\Actions\ForceDeleteAction;
+use Filament\Tables\Actions\ForceDeleteBulkAction;
+use Filament\Tables\Actions\RestoreAction;
+use Filament\Tables\Actions\RestoreBulkAction;
 use Filament\Tables\Actions\ViewAction;
 use Filament\Tables\Columns\IconColumn;
 use Filament\Tables\Columns\ImageColumn;
 use Filament\Tables\Columns\TextColumn;
 use Filament\Tables\Filters\Filter;
+use Filament\Tables\Filters\TrashedFilter;
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\SoftDeletingScope;
 use LaraZeus\Wind\Filament\Resources\DepartmentResource\Pages;
+use LaraZeus\Wind\Models\Department;
 
 class DepartmentResource extends Resource
 {
@@ -74,6 +82,17 @@ public static function form(Form $form): Form
             ]);
     }
 
+    /**
+     * @return Builder<Department>
+     */
+    public static function getEloquentQuery(): Builder
+    {
+        return parent::getEloquentQuery()
+            ->withoutGlobalScopes([
+                SoftDeletingScope::class,
+            ]);
+    }
+
     public static function table(Table $table): Table
     {
         return $table
@@ -105,6 +124,7 @@ public static function table(Table $table): Table
             ])
             ->defaultSort('id', 'desc')
             ->filters([
+                TrashedFilter::make(),
                 Filter::make('is_active')
                     ->label(__('is active'))
                     ->toggle()
@@ -114,6 +134,11 @@ public static function table(Table $table): Table
                     ->toggle()
                     ->query(fn (Builder $query): Builder => $query->where('is_active', false)),
             ])
+            ->bulkActions([
+                DeleteBulkAction::make(),
+                ForceDeleteBulkAction::make(),
+                RestoreBulkAction::make(),
+            ])
             ->actions([
                 ActionGroup::make([
                     EditAction::make('edit')->label(__('Edit')),
@@ -126,8 +151,9 @@ public static function table(Table $table): Table
                         ->label(__('Open'))
                         ->url(fn (Model $record): string => route('contact', ['departmentSlug' => $record]))
                         ->openUrlInNewTab(),
-                    DeleteAction::make('delete')
-                        ->label(__('Delete')),
+                    DeleteAction::make('delete'),
+                    ForceDeleteAction::make(),
+                    RestoreAction::make(),
                 ]),
             ]);
     }

diff --git a/src/Filament/Resources/LetterResource.php b/src/Filament/Resources/LetterResource.php
@@ -11,13 +11,22 @@
 use Filament\Tables\Actions\Action;
 use Filament\Tables\Actions\ActionGroup;
 use Filament\Tables\Actions\DeleteAction;
+use Filament\Tables\Actions\DeleteBulkAction;
 use Filament\Tables\Actions\EditAction;
+use Filament\Tables\Actions\ForceDeleteAction;
+use Filament\Tables\Actions\ForceDeleteBulkAction;
+use Filament\Tables\Actions\RestoreAction;
+use Filament\Tables\Actions\RestoreBulkAction;
 use Filament\Tables\Actions\ViewAction;
 use Filament\Tables\Columns\TextColumn;
 use Filament\Tables\Columns\ViewColumn;
 use Filament\Tables\Filters\SelectFilter;
+use Filament\Tables\Filters\TrashedFilter;
+use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\SoftDeletingScope;
 use LaraZeus\Wind\Filament\Resources\LetterResource\Pages;
+use LaraZeus\Wind\Models\Letter;
 
 class LetterResource extends Resource
 {
@@ -35,6 +44,17 @@ protected static function getNavigationBadge(): ?string
         return static::getModel()::where('status', config('zeus-wind.default_status'))->count();
     }
 
+    /**
+     * @return Builder<Letter>
+     */
+    public static function getEloquentQuery(): Builder
+    {
+        return parent::getEloquentQuery()
+            ->withoutGlobalScopes([
+                SoftDeletingScope::class,
+            ]);
+    }
+
     public static function form(Form $form): Form
     {
         return $form
@@ -112,7 +132,13 @@ public static function table(Table $table): Table
                     ->formatStateUsing(fn (string $state): string => __("status_{$state}")),
             ])
             ->defaultSort('id', 'desc')
+            ->bulkActions([
+                DeleteBulkAction::make(),
+                ForceDeleteBulkAction::make(),
+                RestoreBulkAction::make(),
+            ])
             ->filters([
+                TrashedFilter::make(),
                 SelectFilter::make('status')
                     ->options([
                         'NEW' => __('NEW'),
@@ -137,8 +163,9 @@ public static function table(Table $table): Table
                         ->label(__('Open'))
                         ->url(fn (Model $record): string => route('contact', ['departmentSlug' => $record]))
                         ->openUrlInNewTab(),
-                    DeleteAction::make('delete')
-                        ->label(__('Delete')),
+                    DeleteAction::make('delete'),
+                    ForceDeleteAction::make(),
+                    RestoreAction::make(),
                 ]),
             ]);
     }

diff --git a/src/WindServiceProvider.php b/src/WindServiceProvider.php
@@ -54,7 +54,11 @@ protected function getCommands(): array
     public function packageConfiguring(Package $package): void
     {
         $package
-            ->hasMigrations(['create_department_table', 'create_letters_table'])
+            ->hasMigrations([
+                'create_department_table',
+                'create_letters_table',
+                'alter_letters_constraints',
+            ])
             ->hasRoute('web');
     }
 }