Fix nested models still having default values in models that use `For…

…UpdateMetaclass` (#15202)

src/middlewared/middlewared/api/base/model.py

@@ -133,7 +133,16 @@ def __new__(mcls, name, bases, namespaces, **kwargs):
 class _ForUpdateSerializerMixin(PydanticBaseModel):
     @model_serializer(mode="wrap")
     def serialize_model(self, serializer):
-        return {k: v for k, v in serializer(self).items() if v != undefined}
+        aliases = {field.alias or name: name for name, field in self.model_fields.items()}
+
+        return {
+            k: v
+            for k, v in serializer(self).items()
+            if (
+                (getattr(self, aliases[k]) != undefined) if k in aliases and hasattr(self, aliases[k])
+                else v != undefined
+            )
+        }
 
 
 def _field_for_update(field):

src/middlewared/middlewared/pytest/unit/api/base/test_model.py

@@ -1,9 +1,11 @@
+from pydantic import Field
 import pytest
 
 from middlewared.api.base import (BaseModel, Excluded, excluded_field, ForUpdateMetaclass, single_argument_args,
                                   single_argument_result)
 from middlewared.api.base.handler.accept import accept_params
 from middlewared.api.base.handler.result import serialize_result
+from middlewared.api.v25_04_0.pool_snapshottask import PoolSnapshotTaskCron
 from middlewared.service_exception import ValidationErrors
 
 
@@ -64,3 +66,31 @@ class MethodResult(BaseModel):
         count: int
 
     assert serialize_result(MethodResult, {"name": "ivan", "count": 1}, True) == {"name": "ivan", "count": 1}
+
+
+def test_update_with_cron():
+    class CreateObjectWithCron(BaseModel):
+        schedule: PoolSnapshotTaskCron = Field(default_factory=PoolSnapshotTaskCron)
+
+    class UpdateObjectWithCron(CreateObjectWithCron, metaclass=ForUpdateMetaclass):
+        pass
+
+    class UpdateWithCronArgs(BaseModel):
+        id: int
+        data: UpdateObjectWithCron
+
+    assert accept_params(UpdateWithCronArgs, [1, {}]) == [1, {}]
+
+
+def test_update_with_alias():
+    class CreateObjectWithAlias(BaseModel):
+        pass_: str = Field(alias="pass")
+
+    class UpdateObjectWithAlias(CreateObjectWithAlias, metaclass=ForUpdateMetaclass):
+        pass
+
+    class UpdateWithAliasArgs(BaseModel):
+        id: int
+        data: UpdateObjectWithAlias
+
+    assert accept_params(UpdateWithAliasArgs, [1, {"pass": "1"}]) == [1, {"pass": "1"}]