-
-
Notifications
You must be signed in to change notification settings - Fork 222
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
Empty root tag into List
deserialization bug
#319
Comments
Yes, can reproduce this; adding a (failing) unit test. |
For some reason, seems to work for |
Is there any plan to back port a potential fix? |
@Reamer if someone can figure out how, sure. Likely would go in 2.11, unless it's a trivial change (in which could go in 2.10.x patch). I don't think active work is going on for this so help would be appreciated. |
Hi @cowtowncoder, |
@Reamer no problem, and thank you for the offer: we do have a failing (in 2.x) test already, so that's fine. Only need to have time to dig in, hope to find solution for 2.x (and maybe understand what changes to 3.0 are relevant). |
Have not been able to fix, but I have bit better idea on what is going on, and hope to fix this for 2.11. Challenge is that of coercing empty XML String into "empty" value of POJO, Map or List, as a special case, being accepted as empty/null when not being bound to String (or scalar) |
List
deserialization bug
# Conflicts: # release-notes/VERSION-2.x # src/test/java/com/fasterxml/jackson/dataformat/xml/failing/ListDeser319Test.java
/*copied from FasterXML/jackson-databind#2163 */
Hello,
I have a strange problem with jackson-dataformat 2.9.* (with 2.8.* everything work good):
Data class: (Lombok -- modify for test)
Simple test (Kotlin?):
throw an error:
When adding @JsonIgnoreProperties(ignoreUnknown = true) annotation to PackInstanceOrder, error was changed:
BUT! When I change xml to
test didn't throw any errors and pass well.
The text was updated successfully, but these errors were encountered: