Handle cascaded delete constraints #184 (#185)

cepdnaclk · Oct 20, 2024 · 610ad5e · 610ad5e
1 parent 9cf5654
commit 610ad5e
Show file tree

Hide file tree

Showing 5 changed files with 84 additions and 37 deletions.
diff --git a/app/Domains/Taxonomy/Models/TaxonomyTerm.php b/app/Domains/Taxonomy/Models/TaxonomyTerm.php
@@ -91,6 +91,15 @@ public function getMetadata($code)
         return null;
     }
 
+    protected static function boot()
+    {
+        parent::boot();
+
+        static::deleting(function ($taxonomyTerm) {
+            $taxonomyTerm->children()->delete();
+        });
+    }
+
     public static function getHierarchicalPath($id)
     {
         $term = TaxonomyTerm::find($id);
@@ -139,4 +148,4 @@ protected static function newFactory()
     {
         return TaxonomyTermFactory::new();
     }
-}
+}
diff --git a/app/Http/Controllers/Backend/TaxonomyController.php b/app/Http/Controllers/Backend/TaxonomyController.php
@@ -7,6 +7,7 @@
 use Illuminate\Support\Facades\Auth;
 use App\Http\Controllers\Controller;
 use App\Domains\Taxonomy\Models\Taxonomy;
+use App\Domains\Taxonomy\Models\TaxonomyTerm;
 
 class TaxonomyController extends Controller
 {
@@ -115,7 +116,8 @@ public function update(Request $request, Taxonomy $taxonomy)
      */
     public function delete(Taxonomy $taxonomy)
     {
-        return view('backend.taxonomy.delete', compact('taxonomy'));
+        $terms = TaxonomyTerm::where('taxonomy_id', $taxonomy->id)->get();
+        return view('backend.taxonomy.delete', compact('taxonomy', 'terms'));
     }
 
 
@@ -128,6 +130,12 @@ public function delete(Taxonomy $taxonomy)
     public function destroy(Taxonomy $taxonomy)
     {
         try {
+            $terms = TaxonomyTerm::where('taxonomy_id', $taxonomy->id)->get();
+            if ($terms->count() > 0) {
+                return redirect()->route('dashboard.taxonomy.index')
+                    ->withErrors('Can not delete the Taxonomy as it already has associated Taxonomy Terms. Please reassign or delete those first.');
+            }
+
             $taxonomy->delete();
             return redirect()->route('dashboard.taxonomy.index')->with('Success', 'Taxonomy was deleted !');
         } catch (\Exception $ex) {

diff --git a/composer.lock b/composer.lock
diff --git a/resources/views/backend/taxonomy/delete.blade.php b/resources/views/backend/taxonomy/delete.blade.php
@@ -13,18 +13,34 @@
                 <p>Are you sure you want to delete
                     <strong><i>"{{ $taxonomy->name }}"</i></strong> ?
                 </p>
-                <div class="d-flex">
-                    {!! Form::open([
-                        'url' => route('dashboard.taxonomy.destroy', compact('taxonomy')),
-                        'method' => 'delete',
-                        'class' => 'container',
-                    ]) !!}
 
+                @if ($terms->count() > 0)
+                    <p>The following terms are linked to this Taxonomy. Deletion is not permitted until these terms are
+                        reassigned or deleted.</p>
+                    <ul>
+                        @foreach ($terms as $term)
+                            <li>
+                                <a href="{{ route('dashboard.taxonomy.terms.edit', compact('taxonomy', 'term')) }}">
+                                    {{ $term->name }} ({{ $term->code }})
+                                </a>
+                            </li>
+                        @endforeach
+                    </ul>
                     <a href="{{ route('dashboard.taxonomy.index') }}" class="btn btn-light mr-2">Back</a>
-                    {!! Form::submit('Delete', ['class' => 'btn btn-danger']) !!}
+                @else
+                    <div class="d-flex">
+                        {!! Form::open([
+                            'url' => route('dashboard.taxonomy.destroy', compact('taxonomy')),
+                            'method' => 'delete',
+                            'class' => 'container',
+                        ]) !!}
+
+                        <a href="{{ route('dashboard.taxonomy.index') }}" class="btn btn-light mr-2">Back</a>
+                        {!! Form::submit('Delete', ['class' => 'btn btn-danger']) !!}
+                        {!! Form::close() !!}
+                    </div>
+                @endif
 
-                    {!! Form::close() !!}
-                </div>
             </x-slot>
         </x-backend.card>
     </div>

diff --git a/resources/views/backend/taxonomy/terms/delete.blade.php b/resources/views/backend/taxonomy/terms/delete.blade.php
@@ -13,6 +13,20 @@
                 <p>Are you sure you want to delete
                     <strong><i>"{{ $term->name }}"</i></strong> ?
                 </p>
+
+                @if ($term->children()->count() > 0)
+                    <p>The following terms are linked to this Taxonomy Term, and will be deleted with this. </p>
+                    <ul>
+                        @foreach ($term->children()->get() as $childTerm)
+                            <li>
+                                <a href="{{ route('dashboard.taxonomy.terms.edit', ['taxonomy', 'childTerm']) }}">
+                                    {{ $childTerm->name }} ({{ $childTerm->code }})
+                                </a>
+                            </li>
+                        @endforeach
+                    </ul>
+                @endif
+
                 <div class="d-flex">
                     {!! Form::open([
                         'url' => route('dashboard.taxonomy.terms.destroy', ['taxonomy' => $taxonomy, 'term' => $term]),