diff --git a/app/Http/Controllers/Api/AssetsController.php b/app/Http/Controllers/Api/AssetsController.php index e49edc4db4..4d146d361f 100644 --- a/app/Http/Controllers/Api/AssetsController.php +++ b/app/Http/Controllers/Api/AssetsController.php @@ -530,7 +530,6 @@ class AssetsController extends Controller * @author [A. Gianotto] [] * @param \App\Http\Requests\ImageUploadRequest $request * @since [v4.0] - * @return JsonResponse */ public function store(ImageUploadRequest $request) { @@ -579,6 +578,10 @@ class AssetsController extends Controller if (($model) && ($model->fieldset)) { foreach ($model->fieldset->fields as $field) { + //reduce "array to string conversion" exceptions - ideally we'd handle this in a form request, but this works for now + if(is_array($request->input($field->db_column, null))) { + return response()->json(Helper::formatStandardApiResponse('error', null, 'This custom field can not be an array', 200)); + } // Set the field value based on what was sent in the request $field_val = $request->input($field->db_column, null); diff --git a/app/Models/Asset.php b/app/Models/Asset.php index a3e00ee941..7bce0df242 100644 --- a/app/Models/Asset.php +++ b/app/Models/Asset.php @@ -220,10 +220,6 @@ class Asset extends Depreciable } } - if (!is_array($params)){ - return false; - } - return parent::save($params); }