[Obo-svn-commit] SF.net SVN: obo:[2170] phenex/trunk/src/org/phenoscape

[<ba...@us...>]

Revision: 2170
          http://obo.svn.sourceforge.net/obo/?rev=2170&view=rev
Author:   balhoff
Date:     2008-10-17 19:40:21 +0000 (Fri, 17 Oct 2008)

Log Message:
-----------
Removed Apply Changes button from Data Set interface.  Now continuously saves edits.

Modified Paths:
--------------
    phenex/trunk/src/org/phenoscape/io/NeXMLReader.java
    phenex/trunk/src/org/phenoscape/view/DataSetComponent.java

Modified: phenex/trunk/src/org/phenoscape/io/NeXMLReader.java
===================================================================
--- phenex/trunk/src/org/phenoscape/io/NeXMLReader.java	2008-10-17 19:12:43 UTC (rev 2169)
+++ phenex/trunk/src/org/phenoscape/io/NeXMLReader.java	2008-10-17 19:40:21 UTC (rev 2170)
@@ -77,11 +77,17 @@
     public String getCharactersBlockID() {
         return this.charactersBlockID;
     }
-    
+
+    /**
+     * Returns true if the reader had to create dangling terms for referenced IDs not found in the OBOSession.
+     */
     public boolean didCreateDanglers() {
         return !this.danglers.isEmpty();
     }
-    
+
+    /**
+     * Returns the list of IDs referenced in the file that were not found in the OBOSession.
+     */
     public List<String> getDanglersList() {
         return this.danglers;
     }
@@ -228,7 +234,7 @@
         }
         return null;
     }
-    
+
     private OBOClass getTerm(String id) {
         final IdentifiedObject term = this.session.getObject(id);
         if (term instanceof OBOClass) {
@@ -240,7 +246,7 @@
             return dangler;
         }
     }  
-  
+
     private Logger log() {
         return Logger.getLogger(this.getClass());
     }

Modified: phenex/trunk/src/org/phenoscape/view/DataSetComponent.java
===================================================================
--- phenex/trunk/src/org/phenoscape/view/DataSetComponent.java	2008-10-17 19:12:43 UTC (rev 2169)
+++ phenex/trunk/src/org/phenoscape/view/DataSetComponent.java	2008-10-17 19:40:21 UTC (rev 2170)
@@ -5,7 +5,6 @@
 import java.awt.event.ActionEvent;
 
 import javax.swing.AbstractAction;
-import javax.swing.JButton;
 import javax.swing.JLabel;
 import javax.swing.JScrollPane;
 import javax.swing.JTextArea;
@@ -24,7 +23,6 @@
   private JTextField curatorsField;
   private JTextField publicationField;
   private JTextArea pubNotesField;
-  private JButton applyButton;
 
   public DataSetComponent(String id, PhenoscapeController controller) {
     super(id, controller);
@@ -56,14 +54,6 @@
     this.pubNotesField.setLineWrap(true);
     this.pubNotesField.setWrapStyleWord(true);
     this.pubNotesField.getDocument().addDocumentListener(new PubNotesListener());
-    this.applyButton = new JButton();
-    this.applyButton.setEnabled(false);
-    this.applyButton.setAction(new AbstractAction("Apply Changes") {
-      public void actionPerformed(ActionEvent e) {
-        getController().getDataSet().setPublicationNotes(pubNotesField.getText());
-        applyButton.setEnabled(false);
-      }
-    });
     
     final GridBagConstraints curatorsLabelConstraints = new GridBagConstraints();
     curatorsLabelConstraints.anchor = GridBagConstraints.EAST;
@@ -98,12 +88,6 @@
     pubNotesFieldConstraints.weighty = 1.0;
     this.add(new JScrollPane(this.pubNotesField), pubNotesFieldConstraints);
     
-    final GridBagConstraints applyButtonConstraints = new GridBagConstraints();
-    applyButtonConstraints.gridy = 4;
-    applyButtonConstraints.gridwidth = 2;
-    applyButtonConstraints.anchor = GridBagConstraints.EAST;
-    this.add(this.applyButton, applyButtonConstraints);
-    
     this.getController().addNewDataListener(new DataListener());
     this.updateInterface();
   }
@@ -129,7 +113,7 @@
     }
     
     private void documentChanged() {
-      applyButton.setEnabled(true);
+      getController().getDataSet().setPublicationNotes(pubNotesField.getText());
     }
     
   }


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.