Changeset 9152

Timestamp:

2005-02-23T14:08:33+13:00 (19 years ago)

Author:

kjdon

Message:

some of the functionality in the MyArgumentControl class is now in the base class, so use this instead of repeating code

File:

• trunk/gli/src/org/greenstone/gatherer/gui/OptionsPane.java (modified) (1 diff)

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

@@ -388 +388 @@
     }
 
-    /** Update the values stored in the collection so as to rememebr the current state of this argument. */
+    /** Update the values stored in the collection so as to remember the current state of this argument. */
     public void update() {
-        String name = argument.getName();
-        boolean enable = true;
-        if(enabled != null) {
-        enable = enabled.isSelected();
-        }
-        String value = null;
-        if(value_control == null) {
-        // Flag value, nothing to do.
-        }
-        else if(value_control instanceof JTextField) {
-        value = ((JTextField)value_control).getText();
-        }
-        else if(value_control instanceof JSpinner) {
-        value = ((JSpinner)value_control).getValue().toString();
-        }
-        else if(value_control instanceof JComboBox) {
-        value = (((JComboBox)value_control).getSelectedItem()).toString();
-        }
+        String name = getName();
+        boolean enable = isEnabled();
+        String value = getValue();
         // If this argument was a flag, but is now disabled, remove from the build options altogether
         if(!enable && value == null) {