breaking: fix serialization issues in BaseService#update and use JsonPatch
for updatePartial
#608
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes issues with properties being overridden by removing the serialization in
BaseService#update
.Extracted from #515:
❗ Breaking change - Migration notes: ❗
1.
BaseService#updatePartial
changed its method signature.Old:
New:
Projects which override
updatePartial
or use it directly need to update their code accordingly.2.
BaseController#updatePartial
changed its signatureProjects using the endpoint
PATCH /{entityClass}/{id}
should update their client code:3. Annotate your hibernate models with
@DynamicUpdate
For more information see https://www.baeldung.com/spring-rest-json-patch#http-patch-request
Related issues or pull requests
uuid
field for all entities #515BaseService.update
returns object that is not a hibernate proxy #450updatePartial
still uses serialization andPull request type
Do you introduce a breaking change?
Checklist
Apache Licence Version 2.0.
mvn test
locally).