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

UntypedObjectDeserializer mixes multiple unwrapped collections (related to #2733) #3025

Closed
Migwel opened this issue Jan 25, 2021 · 2 comments
Milestone

Comments

@Migwel
Copy link
Contributor

Migwel commented Jan 25, 2021

(needed for FasterXML/jackson-dataformat-xml#445)

There's an issue when deserializing XML nodes containing multiple unwrapped collections. For example:

<person>
    <name>a</name>
    <name>b</name>
    <surname>c</surname>
    <surname>d</surname>
</person>

gets deserialized into

{
    name = [ c, d ], 
    surname = d
}

Expected deserialization:

{
    name = [ a, b ], 
    surname = [c, d]
}
@Migwel Migwel added the to-evaluate Issue that has been received but not yet evaluated label Jan 25, 2021
Migwel added a commit to Migwel/jackson-databind that referenced this issue Jan 25, 2021
@cowtowncoder cowtowncoder added 2.12 and removed to-evaluate Issue that has been received but not yet evaluated labels Jan 27, 2021
cowtowncoder pushed a commit that referenced this issue Jan 27, 2021
@cowtowncoder cowtowncoder changed the title UntypedObjectDeserializer mixes multiple unwrapped collections UntypedObjectDeserializer mixes multiple unwrapped collections (related to #2733) Jan 27, 2021
@cowtowncoder
Copy link
Member

Note: relevant for #2733 that was added in 2.12.0.

cowtowncoder added a commit that referenced this issue Jan 27, 2021
@cowtowncoder cowtowncoder added this to the 2.12.2 milestone Jan 28, 2021
@cowtowncoder
Copy link
Member

Fixed for 2.12.2.

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