You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by rg...@apache.org on 2012/07/02 20:55:45 UTC

svn commit: r1356406 - /tika/trunk/tika-parsers/src/main/java/org/apache/tika/parser/xml/AbstractMetadataHandler.java

Author: rgauss
Date: Mon Jul  2 18:55:44 2012
New Revision: 1356406

URL: http://svn.apache.org/viewvc?rev=1356406&view=rev
Log:
TIKA-947: AbstractMetadataHandler addMetadata Does not Check Property.isMultiValuePermitted
   - Added check for isMultiValuePermitted, if false call metadata.set instead of metadata.add

Modified:
    tika/trunk/tika-parsers/src/main/java/org/apache/tika/parser/xml/AbstractMetadataHandler.java

Modified: tika/trunk/tika-parsers/src/main/java/org/apache/tika/parser/xml/AbstractMetadataHandler.java
URL: http://svn.apache.org/viewvc/tika/trunk/tika-parsers/src/main/java/org/apache/tika/parser/xml/AbstractMetadataHandler.java?rev=1356406&r1=1356405&r2=1356406&view=diff
==============================================================================
--- tika/trunk/tika-parsers/src/main/java/org/apache/tika/parser/xml/AbstractMetadataHandler.java (original)
+++ tika/trunk/tika-parsers/src/main/java/org/apache/tika/parser/xml/AbstractMetadataHandler.java Mon Jul  2 18:55:44 2012
@@ -70,7 +70,12 @@ class AbstractMetadataHandler extends De
                 if (previous != null && previous.length() > 0) {
                     if (!previous.equals(value)) {
                        if (property != null) {
-                          metadata.add(property, value);
+                          if (property.isMultiValuePermitted()) {
+                              metadata.add(property, value);
+                          } else {
+                              // Replace the existing value if isMultiValuePermitted is false
+                              metadata.set(property, value);
+                          }
                        } else {
                           metadata.add(name, value);
                        }