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);