Changeset 8990

Timestamp:

2005-02-09T17:22:09+13:00 (19 years ago)

Author:

mdewsnip

Message:

Now caches the metadata mapping so subsequent reads don't require reading the profile.xml file each time. This makes a huge difference when opening collections containing a lot of metadata that must be mapped into a metadata set.

File:

• trunk/gli/src/org/greenstone/gatherer/metadata/ProfileXMLFile.java (modified) (5 diffs)

trunk/gli/src/org/greenstone/gatherer/metadata/ProfileXMLFile.java

@@ -38 +38 @@
     extends File
 {
+    private HashMap metadata_mapping = null;
+
+
     public ProfileXMLFile(String profile_xml_file_path)
     {
@@ -44 +47 @@
 
 
-    public String getMetadataElementFor(String source_metadata_element_name_full)
+    public String getMetadataElementFor(String metadata_element_name_full)
     {
-    // Parse the profile.xml file
-    Document document = XMLTools.parseXMLFile(this);
-    if (document == null) {
-        System.err.println("Error: Could not parse profile.xml file " + getAbsolutePath());
+    if (getMetadataMapping() == null) {
         return null;
     }
 
-    // Read all the Action elements in the file
-    NodeList action_elements_nodelist = document.getElementsByTagName("Action");
-    for (int i = 0; i < action_elements_nodelist.getLength(); i++) {
-        Element current_action_element = (Element) action_elements_nodelist.item(i);
-
-        // Does this action map the desired element?
-        if (current_action_element.getAttribute("source").equals(source_metadata_element_name_full)) {
-        return current_action_element.getAttribute("target");
-        }
-    }
-
-    // No import mapping for this element
-    return null;
+    return (String) metadata_mapping.get(metadata_element_name_full);
     }
 
@@ -71 +59 @@
     public HashMap getMetadataMapping()
     {
-    HashMap metadata_import_mapping = new HashMap();
+    // If we have already loaded the metadata mapping, use it
+    if (metadata_mapping != null) {
+        return metadata_mapping;
+    }
+
+    // Build the metadata mapping from the profile.xml file
+    metadata_mapping = new HashMap();
 
     // Parse the profile.xml file
@@ -86 +80 @@
         String source_metadata_element_name_full = current_action_element.getAttribute("source");
         String target_metadata_element_name_full = current_action_element.getAttribute("target");
-        metadata_import_mapping.put(source_metadata_element_name_full, target_metadata_element_name_full);
+        metadata_mapping.put(source_metadata_element_name_full, target_metadata_element_name_full);
     }
 
-    return metadata_import_mapping;
+    return metadata_mapping;
     }
 
@@ -110 +104 @@
     // Rewrite the profile.xml file
     XMLTools.writeXMLFile(this, document);
+
+    // Invalidate the metadata mapping
+    metadata_mapping = null;
     }
 }