From db63ad1cf4668c31751140ee081e3b038efc217b Mon Sep 17 00:00:00 2001 From: spencerrlongg Date: Tue, 17 Jun 2025 18:13:37 -0500 Subject: [PATCH] add image check to supplier action --- app/Actions/Suppliers/DestroySupplierAction.php | 10 ++++++++++ app/Http/Controllers/BulkManufacturersController.php | 2 +- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/app/Actions/Suppliers/DestroySupplierAction.php b/app/Actions/Suppliers/DestroySupplierAction.php index 04ce09de90..1533e55f19 100644 --- a/app/Actions/Suppliers/DestroySupplierAction.php +++ b/app/Actions/Suppliers/DestroySupplierAction.php @@ -6,6 +6,8 @@ use App\Models\Supplier; use App\Exceptions\ModelStillHasAssets; use App\Exceptions\ModelStillHasAssetMaintenances; use App\Exceptions\ModelStillHasLicenses; +use Illuminate\Support\Facades\Log; +use Illuminate\Support\Facades\Storage; class DestroySupplierAction { @@ -33,6 +35,14 @@ class DestroySupplierAction throw new ModelStillHasLicenses($supplier); } + if ($supplier->image) { + try { + Storage::disk('public')->delete('suppliers/'.$supplier->image); + } catch (\Exception $e) { + Log::info($e->getMessage()); + } + } + $supplier->delete(); return true; diff --git a/app/Http/Controllers/BulkManufacturersController.php b/app/Http/Controllers/BulkManufacturersController.php index c1fa5528b2..4cccbf2433 100644 --- a/app/Http/Controllers/BulkManufacturersController.php +++ b/app/Http/Controllers/BulkManufacturersController.php @@ -21,7 +21,7 @@ class BulkManufacturersController extends Controller foreach ($ids as $id) { try { DestroyManufacturerAction::run(manufacturer: $id); - } catch (ModelStillHasAccessories|ModelStillHasAssetModels|ModelStillHasAssets|ModelStillHasComponents|ModelStillHasConsumables|ModelStillHasLicenses $e) { + } catch (ModelStillHasAccessories|ModelStillHasAssets|ModelStillHasComponents|ModelStillHasConsumables|ModelStillHasLicenses $e) { $errors[] = `{$id} still has {$id->thing}`; } catch (\Exception $e) { report($e);