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.

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

Include name of unsettable property in exception from SetterlessProperty.set() #1843

Closed
andreh7 opened this issue Nov 28, 2017 · 1 comment
Closed
Milestone

Comments

@andreh7
Copy link

andreh7 commented Nov 28, 2017

While using jackson to serialize classes which deviate quite a bit from JavaBeans requirements, I've been struggling figuring out which property SetterlessProperty.set() complains about when it is called. Setting a breakpoint in this method and inspecting this in the debugger clarified this (the class is already contained in the error message Cannot construct instance of <classname>).

Adding the name of the property to the exception message e.g. like in commit d427ba5 would greatly simplify hunting down such errors. (this commit can be turned into a pull request easily).

@cowtowncoder
Copy link
Member

cowtowncoder commented Nov 28, 2017

Improving error messages is a good idea where possible.
I'll have a look at suggested change, thank you!

@cowtowncoder cowtowncoder added this to the 2.9.3 milestone Nov 28, 2017
@cowtowncoder cowtowncoder changed the title FEATURE: add name of unsettable property in SetterlessProperty.set() Include name of unsettable property in exception from SetterlessProperty.set() Nov 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants