Changeset 29997

Timestamp:

2015-06-29T14:01:07+12:00 (9 years ago)

Author:

kjdon

Message:

when getting extra metadata names, lets put them in a set and check it so we don't add the same ones to the list over and over again

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/core/TransformingReceptionist.java (modified) (3 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/core/TransformingReceptionist.java

@@ -7 +7 @@
 import java.util.ArrayList;
 import java.util.HashMap;
+import java.util.HashSet;
 
 import javax.xml.transform.Transformer;
@@ -383 +384 @@
         Document doc = XMLConverter.newDOM();
         Element extraMetadataList = doc.createElement(GSXML.EXTRA_METADATA + GSXML.LIST_MODIFIER);
+        HashSet<String> name_set = new HashSet<String>();
         for (File stylesheet : stylesheets)
         {
@@ -391 +393 @@
                 for (String metadataString : requiredMetadata)
                 {
+                  if (!name_set.contains(metadataString)) {
+                      name_set.add(metadataString);
                     Element metadataElem = doc.createElement(GSXML.EXTRA_METADATA);
                     metadataElem.setAttribute(GSXML.NAME_ATT, metadataString);
                     extraMetadataList.appendChild(metadataElem);
+                    }
                 }
             }