crnk-project / crnk-framework

JSON API library for Java
Apache License 2.0
285 stars 153 forks source link

Field level patchStrategy not always applied #838

Open cgendreau opened 2 years ago

cgendreau commented 2 years ago

The code in ResourcePatchController updateValues is not re-checking for every fields.

https://github.com/crnk-project/crnk-framework/blob/252b7822c5bf8436da72a96b8e8900f63b579abc/crnk-core/src/main/java/io/crnk/core/engine/internal/dispatcher/controller/ResourcePatchController.java#L196

Not sure about the intention or intended behaviour but I suspect it should check at every field of type Map to get the specific patchStrategy and if there is nothing default to the repo strategy ?