diff options
| author | Michael Williamson <mike@zwobble.org> | 2022-03-09 21:05:53 +0000 |
|---|---|---|
| committer | Michael Williamson <mike@zwobble.org> | 2022-03-09 21:05:53 +0000 |
| commit | 85194879a8848f01a8db3f42d8ee6d4f24e6cfdb (patch) | |
| tree | a7ee4e40ea037664a6e31bd09b5254137a969f97 /src | |
| parent | deff5d7c556f3cf3bab3309cc21548a1dcac4f0a (diff) | |
Ignore w:u elements when w:val is missing
Diffstat (limited to 'src')
| -rw-r--r-- | src/main/java/org/zwobble/mammoth/internal/docx/StatefulBodyXmlReader.java | 2 | ||||
| -rw-r--r-- | src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/main/java/org/zwobble/mammoth/internal/docx/StatefulBodyXmlReader.java b/src/main/java/org/zwobble/mammoth/internal/docx/StatefulBodyXmlReader.java index de4a73b..326319c 100644 --- a/src/main/java/org/zwobble/mammoth/internal/docx/StatefulBodyXmlReader.java +++ b/src/main/java/org/zwobble/mammoth/internal/docx/StatefulBodyXmlReader.java @@ -209,7 +209,7 @@ class StatefulBodyXmlReader { return properties.findChild("w:u") .map(child -> child.getAttributeOrNone("w:val") .map(value -> !value.equals("false") && !value.equals("0") && !value.equals("none")) - .orElse(true)) + .orElse(false)) .orElse(false); } diff --git a/src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java b/src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java index cff4338..1898897 100644 --- a/src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java +++ b/src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java @@ -579,12 +579,12 @@ public class BodyXmlTests { } @Test - public void runIsUnderlinedIfUnderlineElementIsPresentWithoutValAttribute() { + public void runIsNotUnderlinedIfUnderlineElementIsPresentWithoutValAttribute() { XmlElement element = runXmlWithProperties(element("w:u")); assertThat( readSuccess(bodyReader(), element), - hasProperty("underline", equalTo(true)) + hasProperty("underline", equalTo(false)) ); } |
