Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Jump to bottom

Case-insensitive property, enum deserialization should be supported #80

Closed
MikeDombo opened this issue May 7, 2021 · 4 comments
Closed

Case-insensitive property, enum deserialization should be supported #80

MikeDombo opened this issue May 7, 2021 · 4 comments
Milestone
2.13.0

Comments

@MikeDombo
Copy link
Contributor

Full Jackson has the option ACCEPT_CASE_INSENSITIVE_PROPERTIES and ACCEPT_CASE_INSENSITIVE_ENUMS, these or similar options should be available for jackson-jr.
@MikeDombo MikeDombo mentioned this issue May 11, 2021
Merged
@cowtowncoder cowtowncoder changed the title Case insensitive property deserialization should be supported Case-insensitive property, enum deserialization should be supported May 14, 2021
@cowtowncoder
Copy link
Member

Another excellent idea!

@cowtowncoder cowtowncoder added this to the 2.13.0 milestone May 14, 2021
@MikeDombo
Copy link
Contributor Author

Closed by #82

cowtowncoder added a commit that referenced this issue May 14, 2021
@cowtowncoder
Update release notes wrt #80
d212e50
@cowtowncoder
Copy link
Member

Merged but does not work in 3.0 (master) due to implementation differences. Need to think of what to do there: 3.0 would actually have efficient mechanism for case-insensitive matching via PropertyNameMatcher... but needs to be wired. There's an existing comment that suggests caching might be challenging as well.

cowtowncoder added a commit that referenced this issue May 14, 2021
@cowtowncoder
Fix #80 handling for 3.0 as well (can/need to use PropertyNameMatcher)
48725ba
@cowtowncoder
Copy link
Member

Was able to make master/3.0 work as well with PropertyNameMatcher.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.13.0
Development

No branches or pull requests
2 participants
@cowtowncoder @MikeDombo