Add support for Objenesis proxy creation. #327
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I already done it :) |
|
I think @olivergierke based his from your original pull request but with some modifications to allow Objenesis to be an optional dependency. |
Extended DefaultAopProxyFactory to create Objenesis based proxies if the library is on the classpath. This allows classes without a default constructor being CGLib proxied. We're now falling back to original CGLib based behavior in case the proxy creation using Objenesis fails. Objenesis 2.0 is now inlined into spring-core to avoid interfering with other Objenesis versions on the classpath. Issue: SPR-10594
philwebb
pushed a commit
that referenced
this pull request
Sep 12, 2013
# By Oliver Gierke * SPR-10594: Add support for Objenesis proxy creation.
|
@philwebb Why you merged this request not mine? |
|
@Fuud we went though several code iterations, the final result repackages objenesis as part of spring-core rather than it being an optional dependency. The net result should fix SPR-10594, let me know if you find problems. Cheers for your original contribution, and for raising the issue. It was incredibly helpful. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Extended
DefaultAopProxyFactoryto create Objenesis based proxies if the library is on the classpath. This allows classes without a default constructor being CGLib proxied.Open issues:
Issue: SPR-10594