PDFBOX-6130: sonar fix

THausherr · THausherr · commit 6defadef11c6 · 2025-12-20T17:40:03.000Z
git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1930765 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java b/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java
@@ -991,13 +991,10 @@ private Element findDescriptionsParent(Element root) throws XmpParsingException
                 // empty description
                 throw new XmpParsingException(ErrorType.Format, "No rdf description found in xmp");
             }
-            else if (nl.getLength() > 1)
+            else if (nl.getLength() > 1 && strictParsing)
             {
                 // only expect one element
-                if (strictParsing)
-                {
-                    throw new XmpParsingException(ErrorType.Format, "More than one element found in x:xmpmeta");
-                }
+                throw new XmpParsingException(ErrorType.Format, "More than one element found in x:xmpmeta");
             }
             // find element (there may be a text before the element)
             for (int i = 0; i < nl.getLength(); ++i)

Original file line number	Diff line number	Diff line change
`@@ -991,13 +991,10 @@ private Element findDescriptionsParent(Element root) throws XmpParsingException`
`991`	`991`	`// empty description`
`992`	`992`	`throw new XmpParsingException(ErrorType.Format, "No rdf description found in xmp");`
`993`	`993`	`}`
`994`		`- else if (nl.getLength() > 1)`
	`994`	`+ else if (nl.getLength() > 1 && strictParsing)`
`995`	`995`	`{`
`996`	`996`	`// only expect one element`
`997`		`- if (strictParsing)`
`998`		`- {`
`999`		`- throw new XmpParsingException(ErrorType.Format, "More than one element found in x:xmpmeta");`
`1000`		`- }`
	`997`	`+ throw new XmpParsingException(ErrorType.Format, "More than one element found in x:xmpmeta");`
`1001`	`998`	`}`
`1002`	`999`	`// find element (there may be a text before the element)`
`1003`	`1000`	`for (int i = 0; i < nl.getLength(); ++i)`