You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2025/12/25 07:46:52 UTC

svn commit: r1930836 - pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml

Author: tilman
Date: Thu Dec 25 07:46:52 2025
New Revision: 1930836

Log:
PDFBOX-5660: refactor

Modified:
   pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java

Modified: pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java
==============================================================================
--- pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java	Thu Dec 25 07:46:47 2025	(r1930835)
+++ pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java	Thu Dec 25 07:46:52 2025	(r1930836)
@@ -1107,17 +1107,17 @@ public class DomXmpParser
     {
         TypeMapping tm = xmp.getTypeMapping();
         // test if namespace is set in xml
-        if (!nsFinder.containsNamespace(prop.getNamespaceURI()))
+        String nsuri = prop.getNamespaceURI();
+        if (!nsFinder.containsNamespace(nsuri))
         {
             throw new XmpParsingException(ErrorType.NoSchema, "Schema is not set in this document : "
-                    + prop.getNamespaceURI() + ", property: " + prop.getPrefix() + ":" + prop.getLocalPart());
+                    + nsuri + ", property: " + prop.getPrefix() + ":" + prop.getLocalPart());
         }
         // test if namespace is defined
-        String nsuri = prop.getNamespaceURI();
         if (!tm.isDefinedNamespace(nsuri))
         {
             throw new XmpParsingException(ErrorType.NoSchema, "Cannot find a definition for the namespace "
-                    + prop.getNamespaceURI() + ", property: " + prop.getPrefix() + ":" + prop.getLocalPart());
+                    + nsuri + ", property: " + prop.getPrefix() + ":" + prop.getLocalPart());
         }
         try
         {