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 2014/10/28 19:43:02 UTC

svn commit: r1634944 - /pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java

Author: tilman
Date: Tue Oct 28 18:43:01 2014
New Revision: 1634944

URL: http://svn.apache.org/r1634944
Log:
PDFBOX-2416: fix wrong array message, as suggested by Ralf Hauser

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
URL: http://svn.apache.org/viewvc/pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java?rev=1634944&r1=1634943&r2=1634944&view=diff
==============================================================================
--- pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java (original)
+++ pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java Tue Oct 28 18:43:01 2014
@@ -410,8 +410,15 @@ public class DomXmpParser
         if (bagOrSeq == null)
         {
             // not an array
+            String whatFound = "nothing";
+            if (property.getFirstChild() != null)
+            {
+                whatFound = property.getFirstChild().getClass().getName();
+            }
             throw new XmpParsingException(ErrorType.Format, "Invalid array definition, expecting " + type.card()
-                    + " and found nothing [prefix="+prefix+"; name="+name+"]");
+                    + " and found "
+                    + whatFound
+                    + " [prefix=" + prefix + "; name=" + name + "]");
         }
         if (!bagOrSeq.getLocalName().equals(type.card().name()))
         {