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/28 18:15:16 UTC
svn commit: r1930939 - pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/type
Author: tilman
Date: Sun Dec 28 18:15:16 2025
New Revision: 1930939
Log:
PDFBOX-6133: refactor
Modified:
pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/type/TypeMapping.java
Modified: pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/type/TypeMapping.java
==============================================================================
--- pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/type/TypeMapping.java Sun Dec 28 18:15:11 2025 (r1930938)
+++ pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/type/TypeMapping.java Sun Dec 28 18:15:16 2025 (r1930939)
@@ -341,22 +341,19 @@ public final class TypeMapping
}
return null;
}
- else
+ // try in defined
+ if (!definedStructuredNamespaces2.containsKey(qName.getNamespaceURI()))
{
- // try in defined
- if (!definedStructuredNamespaces2.containsKey(qName.getNamespaceURI()))
- {
- // not found
- if (factory != null)
- {
- return null; // pre PDFBOX-6133 behavior
- }
- throw new BadFieldValueException("No descriptor found for " + qName);
- }
- else
+ // not found
+ if (factory != null)
{
- return createPropertyType(Types.DefinedType, Cardinality.Simple);
+ return null; // pre PDFBOX-6133 behavior
}
+ throw new BadFieldValueException("No descriptor found for " + qName);
+ }
+ else
+ {
+ return createPropertyType(Types.DefinedType, Cardinality.Simple);
}
}