From fa45ca14531e5a08c464e74815a0bb88ee1827ce Mon Sep 17 00:00:00 2001 From: snipe Date: Wed, 23 Apr 2025 21:56:43 +0100 Subject: [PATCH] Stupid fix for when people use `id` instead of an actual ID @todo - use RMB for these Signed-off-by: snipe --- app/Http/Controllers/Api/AssetModelFilesController.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/app/Http/Controllers/Api/AssetModelFilesController.php b/app/Http/Controllers/Api/AssetModelFilesController.php index 7f0f06c63..d0a4747bf 100644 --- a/app/Http/Controllers/Api/AssetModelFilesController.php +++ b/app/Http/Controllers/Api/AssetModelFilesController.php @@ -75,6 +75,11 @@ class AssetModelFilesController extends Controller */ public function list($assetmodel_id) : JsonResponse | array { + // Start by checking if the asset being acted upon exists + if (! $assetModel = AssetModel::find($assetmodel_id)) { + return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/models/message.does_not_exist')), 404); + } + $assetmodel = AssetModel::with('uploads')->find($assetmodel_id); $this->authorize('view', $assetmodel); return (new AssetModelsTransformer)->transformAssetModelFiles($assetmodel, $assetmodel->uploads()->count());