Changeset 13706 for trunk/gli/src/org/greenstone/gatherer/metadata/MetadataXMLFile.java

Timestamp:

2007-01-19T16:16:22+13:00 (17 years ago)

Author:

mdewsnip

Message:

The second half of speeding up metadata access and editing substantially (particularly when metadata.xml files are large) by caching one metadata.xml file in memory. The loaded metadata.xml file is no longer written to disk every time a piece of metadata is added/edited, but only when a new metadata.xml file is loaded.

File:

• trunk/gli/src/org/greenstone/gatherer/metadata/MetadataXMLFile.java (modified) (8 diffs)

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

@@ -48 +48 @@
     static private File loaded_file = null;
     static private Document loaded_file_document = null;
+    static private boolean loaded_file_changed = false;
 
 
@@ -60 +61 @@
     // If this metadata.xml file isn't the one currently loaded, load it now
     if (loaded_file != this) {
+        // First we must save out the currently loaded file
+        saveLoadedFile();
+
         // Parse the metadata.xml file
         Document document = XMLTools.parseXMLFile(this);
@@ -177 +181 @@
     }
 
-    // Rewrite the metadata.xml file
-    XMLTools.writeXMLFile(this, loaded_file_document);
+    // Remember that we've changed the file so it gets saved when a new one is loaded
+    loaded_file_changed = true;
     }
 
@@ -186 +190 @@
     // If this metadata.xml file isn't the one currently loaded, load it now
     if (loaded_file != this) {
+        // First we must save out the currently loaded file
+        saveLoadedFile();
+
         // Parse the metadata.xml file
         Document document = XMLTools.parseXMLFile(this);
@@ -319 +326 @@
     // If this metadata.xml file isn't the one currently loaded, load it now
     if (loaded_file != this) {
+        // First we must save out the currently loaded file
+        saveLoadedFile();
+
         // Parse the metadata.xml file
         Document document = XMLTools.parseXMLFile(this);
@@ -413 +423 @@
     }
 
-    // Rewrite the metadata.xml file
-    XMLTools.writeXMLFile(this, loaded_file_document);
+    // Remember that we've changed the file so it gets saved when a new one is loaded
+    loaded_file_changed = true;
     }
 
@@ -422 +432 @@
     // If this metadata.xml file isn't the one currently loaded, load it now
     if (loaded_file != this) {
+        // First we must save out the currently loaded file
+        saveLoadedFile();
+
         // Parse the metadata.xml file
         Document document = XMLTools.parseXMLFile(this);
@@ -524 +537 @@
     }
 
-    // Rewrite the metadata.xml file
-    XMLTools.writeXMLFile(this, loaded_file_document);
+    // Remember that we've changed the file so it gets saved when a new one is loaded
+    loaded_file_changed = true;
+    }
+
+
+    static public void saveLoadedFile()
+    {
+    // If we have a file loaded into memory and it has been modified, save it now
+    if (loaded_file != null && loaded_file_changed == true) {
+        XMLTools.writeXMLFile(loaded_file, loaded_file_document);
+        loaded_file_changed = false;
+    }
     }