Fixes jackson-module-paranamer issue 5 problem with paranamer + PropertyNamingStrategy #379
Conversation
…er + PropertyNamingStrategy. Update the creator properties after renaming using PropertyNamingStrategy as per initial _renameProperties() call.
|
Looks good -- I assume you ran full unit test suite with the patch? If so, will merge and also backport in 2.3. |
Fixes jackson-module-paranamer issue 5 problem with paranamer + PropertyNamingStrategy
|
Full unit test was ran. I will issue a pull request for the paranamer / PropertyNamingStrategy issue on the module project shortly. I needed to update the jackson-paranamer project to remove the deprecated AnnotationIntrospector methods that have been removed recently and to update the reference to the HEAD version of jackson-databind. Shall I leave those changes out of the PR? |
|
Thanks again. On 2.4; yeah, if you could do another PR I can merge that in -- I haven't really synced up all 2.4 pieces yet after refactoring/cleanup of core pieces, so help there would be appreciated. Also: I merged the fix in 2.3 branch as well. There will probably eventually be 2.3.2 out before 2.4. I don't think I yet moved Paranamer module master to 2.4 yet. |
|
Glad to help. I've added a PR for the 2.4 fixes to jackson-module-paranamer here: FasterXML/jackson-module-paranamer#8 |
...yNamingStrategy. Update the creator properties after renaming using PropertyNamingStrategy as per initial _renameProperties() call.