>> There are no guarantees on the type, mutability, serializability, or thread-safety of the List returned
`Collector.toList()` doesn't guarantee mutability, then passing it to `Collections.reverse()` is not safe.
Signed-off-by: Yanming Zhou <zhouyanming@gmail.com>
Closes#3304
The ProxyHandlerMethodArgumentResolver now avoids matching multipart parameters annotated with @ModelAttribute. This allows multipart parameters to be handled by RequestParamMethodArgumentResolver which properly handles multipart arguments.
Also, the `@ProjectedPayload` annotation can now be used on parameters. This prepares for the upcoming removal of support for non-annotated projections.
Fixes#3258
Related tickets #2937
Original pull request: #3277
Signed-off-by: Chris Bono <chris.bono@broadcom.com>
The documentation now clarifies that entity might get loaded and therefore possibly OptimisticLockingFailureException might get thrown.
Closes#3280
Original pull request: #3281
This fixes javadoc links in a couple of adoc files.
Original pull request: #3260
Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>
(cherry picked from commit af10ad393a)
This cleans up a couple of classes by removing unused imports.
Original pull request: #3240
Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>
(cherry picked from commit 18f27e7e95)
Examples for limiting queries with and without a property argument are now better mixed.
This makes it more clear that limiting and filtering by property are orthogonal concerns.
Closes#3268
Original pull request #3269
We now use a refined strategy to cache RepositoryInformation and RepositoryComposition.
Previously, RepositoryComposition wasn't cached at all and store modules that e.g. contributed a Querydsl (or a different) fragment based on the interface declaration returned a new RepositoryComposition (and thus a different hashCode) each time RepositoryInformation was obtained leading to memory leaks caused by HashMap caching.
We now use Fragment's hashCode for the cache key resulting in RepositoryComposition being created only once for a given repository interface and input-fragments arrangement.
Closes#3252
We now skip unnamed DTO projection properties and issue a warning log to raise awareness.
Skipping unnamed (null) properties avoids identification as DTO and only selects properties stemming from named constructor arguments.
Add tests for Kotlin data classes using value classes for verification.
Closes#3225
