We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PropertyNamingStrategyWrapper
PropertyNamingStrategy
NamingBase
Currently this uses reflection that doesn't work well with later versions of Jackson (e.g. 2.12.3) which added new implementations:
2.12.3
public PropertyNamingStrategyWrapper(PropertyNamingStrategy delegate) { // Doesn't support `NamingBase` decendants! if (delegate instanceof PropertyNamingStrategyBase) { this.delegate = (PropertyNamingStrategyBase) delegate; } else if (delegate == PropertyNamingStrategy.LOWER_CAMEL_CASE) { this.delegate = NO_OP; } else { this.delegate = SNAKE_TO_CAMEL; } }
So if this is used:
var mapper = new ObjectMapper().registerModule(new ProtobufModule()).setPropertyNamingStrategy(PropertyNamingStrategies.SNAKE_CASE); var value = mapper.convertValue(...);
will incorrectly result in value to have SNAKE_TO_CAMEL being applied.
value
SNAKE_TO_CAMEL
The text was updated successfully, but these errors were encountered:
Should be fixed by #88
Sorry, something went wrong.
Could this please be released?
.setPropertyNamingStrategy(PropertyNamingStrategies.SNAKE_CASE)
Successfully merging a pull request may close this issue.
Currently this uses reflection that doesn't work well with later versions of Jackson (e.g.
2.12.3
) which added new implementations:So if this is used:
will incorrectly result in
value
to haveSNAKE_TO_CAMEL
being applied.The text was updated successfully, but these errors were encountered: