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

Problem with @JsonCreator with 1-arg factory-method, implicit param names #1383

Closed
cowtowncoder opened this issue Sep 24, 2016 · 1 comment
Closed

Problem with @JsonCreator with 1-arg factory-method, implicit param names #1383

cowtowncoder opened this issue Sep 24, 2016 · 1 comment
Milestone
2.7.8

Comments

@cowtowncoder
Copy link
Member

It appear that auto-detection of @JsonCreator mode may not be working as desired for the case where a single-arg constructor has implicit name. This should NOT result in properties-creator, at least in case where there is no matching getter available, but rather should default to delegating-creator; same as if no name was available. Only explicit name, or, possibly, existence of getter with name same as implicit name, should result in properties-creator.
@cowtowncoder cowtowncoder mentioned this issue Sep 24, 2016
Closed
@cowtowncoder cowtowncoder changed the title Problem with auto-detection of properties-based creator with implicit constructor names Problem with @JsonCreator with 1-arg factory-method, implicit param names Sep 24, 2016
cowtowncoder added a commit that referenced this issue Sep 24, 2016
@cowtowncoder
backport #1383 for 2.6, just in case
6d5887d
@cowtowncoder cowtowncoder added this to the 2.7.8 milestone Sep 24, 2016
@cowtowncoder
Copy link
Member Author

Fixed for 2.7.8 (to be released soon), 2.8.4 (will take bit longer since just released 2.8.3).

@cowtowncoder cowtowncoder mentioned this issue Sep 24, 2016
Closed
This was referenced Sep 28, 2016
Closed
Closed
@cowtowncoder cowtowncoder mentioned this issue Sep 30, 2016
Closed
@jmoney jmoney mentioned this issue Sep 30, 2016
Merged
@cowtowncoder cowtowncoder mentioned this issue Oct 5, 2016
Closed
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.7.8
Development

No branches or pull requests
1 participant
@cowtowncoder