Collection values overwritten if item elements are not contiguous (mixed content)

未关闭
  • 哪里不对？
#363 5 条评论 6 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
42/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
停滞
技术栈
java
领域
backend

调研方向

使用 XmlMapper.readValue 和 issue 中的 Data 示例重现数据丢失，重点关注由 bar 分隔的重复 foo 元素。跟踪 XML 反序列化经过 Java 解析器的过程，直到为每个连续组调用 setter。完成的标准是 foo 保留全部四个值，而 bar 仍为 bar1，并为包含混合内容的 XML 添加 regression test。

由索引模型根据 Issue 内容生成。

描述

mixed-content will-not-fix

Looks like we have a bug in the parser. Parsing next simple document leads to data loss.

<Data>
    <foo>foo1</foo>
    <foo>foo2</foo>
    <bar>bar1</bar>
    <foo>foo3</foo>
    <foo>foo4</foo>
</Data>

Expected result: foo java-property contains list of 4 values.
Actual result: foo java-property contains list of 2.

Setter for foo property is called two times for every tags group: foo1, foo2 and foo3, foo4. Second setter call overrides data gathered by the first call. As result parsing result contains foo list as foo3, foo4 instead of expected foo1, foo2, foo3, foo4.
Here is a java-code to reproduce the issue.

import com.fasterxml.jackson.core.JsonProcessingException;
import com.fasterxml.jackson.dataformat.xml.XmlMapper;
import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlCData;
import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlElementWrapper;
import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlProperty;
import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlRootElement;
import java.util.List;

@JacksonXmlRootElement(localName = "data")
class Data {
    @JacksonXmlCData
    @JacksonXmlElementWrapper(useWrapping = false)
    @JacksonXmlProperty
    private List<String> foo;

    @JacksonXmlCData
    @JacksonXmlElementWrapper(useWrapping = false)
    @JacksonXmlProperty
    private List<String> bar;

    public List<String> getFoo() {
        return foo;
    }

    public void setFoo(List<String> foo) {
        this.foo = foo;
    }

    public List<String> getBar() {
        return bar;
    }

    public void setBar(List<String> bar) {
        this.bar = bar;
    }
}

public class Foo {
    public static void main(String[] args) throws JsonProcessingException {
        String xml = ""
                + "<Data>"
                + "    <foo>foo1</foo>"
                + "    <foo>foo2</foo>"
                + "    <bar>bar1</bar>"
                + "    <foo>foo3</foo>"
                + "    <foo>foo4</foo>"
                + "</Data>";

        XmlMapper m = new XmlMapper();
        Data data = m.readValue(xml, Data.class);

        System.err.println(data.getFoo()); // Expected ["foo1", "foo2", "foo3", "foo4"] but ["foo3", "foo4"] given.
    }
}

jackson-dataformat-xml version: 2.10.0

主要语言
Java
星标
636
派生
250
平均合并
7 小时 10 分钟
30 天内合并 PR
8

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

FasterXML/jackson-dataformat-xml 的其他 Issue

查看 FasterXML/jackson-dataformat-xml 的全部 Issue

相似的 Issue

更多 Java Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。