Changeset 18199 for gli

Timestamp:

2008-12-15T09:24:03+13:00 (15 years ago)

Author:

kjdon

Message:

a fix provided by John from Dl Consulting. File->Save now saves the metadata in the Enrich Pane as well as the collection config file. Previously, metadata was only saved when you left the pane.

Location:

gli/trunk/src/org/greenstone/gatherer/gui

Files:

• EnrichPane.java (modified) (1 diff)
• GUIManager.java (modified) (2 diffs)

gli/trunk/src/org/greenstone/gatherer/gui/EnrichPane.java

@@ -259 +259 @@
 
 
-    /** Called whenever the collection tree selection changes. This causes the metadata value table to be rebuilt. */
-    public void valueChanged(TreeSelectionEvent event)
+  /** Allows other classes to request the enrich pane to prepare for metadata
+   *  saving.
+   *  @author John Thompson, DL Consulting Ltd
+   */
+  public void stopEditingAndRebuild()
+  {
+    // Update the metadata value table (and consequently, the metadata value tree)
+    metadata_value_table_pane.stopEditingAndRebuild(file_nodes);
+  }
+  /** stopEditingAndRebuild() **/
+  
+  /** Called whenever the collection tree selection changes. This causes the metadata value table to be rebuilt. */
+  public void valueChanged(TreeSelectionEvent event)
     {
     // If we haven't got focus then it must have been a selection in the Gather pane, so don't bother rebuilding

gli/trunk/src/org/greenstone/gatherer/gui/GUIManager.java

@@ -60 +60 @@
 import org.greenstone.gatherer.gui.tree.DragTree;
 import org.greenstone.gatherer.metadata.MetadataSet;
+import org.greenstone.gatherer.metadata.MetadataXMLFileManager;
 import org.greenstone.gatherer.remote.RemoteGreenstoneServer;
 import org.greenstone.gatherer.util.JarTools;
@@ -203 +204 @@
     }
     else if (esrc == menu_bar.file_save) {
-        Gatherer.c_man.saveCollection();
+      // Very important: make sure metadata values are saved too
+      enrich_pane.stopEditingAndRebuild();
+      // Make sure all the metadata has been saved to file
+      MetadataXMLFileManager.saveMetadataXMLFiles();
+      
+      Gatherer.c_man.saveCollection();
     }