From f9139e0f101c06e5310b5b072329d96185ddc73b Mon Sep 17 00:00:00 2001 From: snipe Date: Tue, 19 Dec 2023 13:25:32 +0000 Subject: [PATCH] Changed to use instanceof Signed-off-by: snipe --- app/Http/Controllers/Api/AssetsController.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/app/Http/Controllers/Api/AssetsController.php b/app/Http/Controllers/Api/AssetsController.php index 6a737b267..0947aaba5 100644 --- a/app/Http/Controllers/Api/AssetsController.php +++ b/app/Http/Controllers/Api/AssetsController.php @@ -574,10 +574,11 @@ class AssetsController extends Controller $asset = $request->handleImages($asset); // Update custom fields in the database. - // Sometimes people send arrays to this. They shouldn't, but they do, so we use "first()" to get the first match - $model = AssetModel::where('id', '=', $request->get('model_id'))->first(); + $model = AssetModel::find($request->input('model_id')); - if (($model) && ($model->fieldset)) { + // Check that it's an object and not a collection + // (Sometimes people send arrays here and they shouldn't + if (($model) && ($model instanceof AssetModel) && ($model->fieldset)) { foreach ($model->fieldset->fields as $field) { // Set the field value based on what was sent in the request