Backport PR #1711: Fix/schema validation in patch_asset (#1711)

* fix: patch_asset should raise a ValidationError instead of returning a tuple

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: stricter error handling

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: changelog entries

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: F.N. Claessen <felix@seita.nl>

(cherry picked from commit 9cc815a)
Signed-off-by: F.N. Claessen <felix@seita.nl>

Author: Flix6x

documentation/api/change_log.rst

@@ -5,6 +5,10 @@ API change log
 
 .. note:: The FlexMeasures API follows its own versioning scheme. This is also reflected in the URL (e.g. `/api/v3_0`), allowing developers to upgrade at their own pace.
 
+v3.0-27 | 2025-09-16
+""""""""""""""""""""
+- Fix schema validation in ``PATCH /assets/<id>``.
+
 v3.0-26 | 2025-09-10
 """"""""""""""""""""
 - Added endpoint `POST /users`.

documentation/changelog.rst

@@ -3,6 +3,15 @@
 FlexMeasures Changelog
 **********************
 
+
+v0.28.1 | September XX, 2025
+============================
+
+Bugfixes
+-----------
+* Fix schema validation in ``PATCH /assets/(id)`` [see `PR #1711 <https://www.github.com/FlexMeasures/flexmeasures/pull/1711>`_]
+
+
 v0.28.0 | September 10, 2025
 ============================
 

flexmeasures/api/v3_0/assets.py

@@ -573,7 +573,10 @@ def patch(self, asset_data: dict, id: int, db_asset: GenericAsset):
         :status 403: INVALID_SENDER
         :status 422: UNPROCESSABLE_ENTITY
         """
-        db_asset = patch_asset(db_asset, asset_data)
+        try:
+            db_asset = patch_asset(db_asset, asset_data)
+        except ValidationError as e:
+            return {"error": str(e)}, 422
         db.session.add(db_asset)
         db.session.commit()
         return asset_schema.dump(db_asset), 200

flexmeasures/data/services/generic_assets.py

@@ -130,11 +130,8 @@ def patch_asset(db_asset: GenericAsset, asset_data: dict) -> GenericAsset:
                     )
             continue
         if k == "flex_context":
-            try:
-                # Validate the flex context schema
-                DBFlexContextSchema().load(v)
-            except Exception as e:
-                return {"error": str(e)}, 422
+            # Validate the flex context schema
+            DBFlexContextSchema().load(v)
 
         if k.lower() in {"sensors_to_show", "flex_context", "flex_model"}:
             audit_log_data.append(format_json_field_change(k, getattr(db_asset, k), v))