fix: Prevent deletion of document types with associated content

Author: cklei-carly

src/Filament/Resources/DocumentTypeResource.php

@@ -13,6 +13,7 @@
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsToMany;
+use Illuminate\Database\Eloquent\SoftDeletingScope;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Str;
@@ -172,6 +173,16 @@ public static function table(Table $table): Table
                     Tables\Actions\DeleteBulkAction::make(),
                 ])->iconButton(),
             ])
+            ->checkIfRecordIsSelectableUsing(function (DocumentType|Model $record) {
+                $hasContent = $record->content()->withoutGlobalScopes([
+                    SoftDeletingScope::class,
+                ])->count() > 0;
+                if ($hasContent) {
+                    // Disallow delete this document type if have content
+                    return false;
+                }
+                return true;
+            })
             ->filters([
                 Tables\Filters\TernaryFilter::make('show_as_table')
                     ->label(__('inspirecms::resources/document-type.show_as_table.label')),

src/Observers/DocumentTypeObserver.php

@@ -32,12 +32,12 @@ public function saving($model)
      */
     public function deleting($model)
     {
-        $content = $model->content()->withoutGlobalScopes([
+        $hasContent = $model->content()->withoutGlobalScopes([
             SoftDeletingScope::class,
-        ])->get();
+        ])->count() > 0;
 
         // Guard: If there is any content, then prevent deleting.
-        if ($content->isNotEmpty()) {
+        if ($hasContent) {
             throw new \Exception('Cannot delete this document type because it has content.');
         }
     }