Changeset 5753

Timestamp:

2003-10-29T12:17:25+13:00 (21 years ago)

Author:

jmt12

Message:

Fixed the enabled/disabled state of some of the buttons depending on what is selected. Also you can now add the UnknownPlug multiple times, and you can no longer remove the Arc and Rec plugs

File:

• trunk/gli/src/org/greenstone/gatherer/cdm/PlugInManager.java (modified) (3 diffs)

trunk/gli/src/org/greenstone/gatherer/cdm/PlugInManager.java

@@ -755 +755 @@
             new_plugin = new PlugIn(element, null);
             }
-            if(!model.contains(new_plugin)) {
+            if(!model.contains(new_plugin) || new_plugin.getName().equals(StaticStrings.UNKNOWNPLUG_STR)) {
             // Automatically chain to configuration. This ensures required arguments are filled out.
             ArgumentConfiguration ac = new ArgumentConfiguration(new_plugin);
@@ -761 +761 @@
                 assignPlugIn(new_plugin);
                 plugin_list.setSelectedValue(new_plugin, true);
-                // Since we weren't cancelled, and if there was a base plugin, ensure it no longer is shown as available
-                if(base_plugin != null) {
+                // Since we weren't cancelled, and if there was a base plugin, ensure it no longer is shown as available, unless it is the UnknownPlugIn which can be added several times
+                if(base_plugin != null && !base_plugin.getName().equals(StaticStrings.UNKNOWNPLUG_STR)) {
                 plugin.removeItem(base_plugin);
                 }
@@ -839 +839 @@
             }
             else {
-            move_top_button.setEnabled(true);
-            move_up_button.setEnabled(true);
-            move_down_button.setEnabled(true);
-            move_bottom_button.setEnabled(true);
-            configure.setEnabled(true);
-            remove.setEnabled(true);
+               configure.setEnabled(true);
+               // Some buttons are only available for plugins other than ArcPlug and RecPlug
+               PlugIn selected_plugin = (PlugIn) plugin_list.getSelectedValue();
+               String plugin_name = selected_plugin.getName();
+               if(plugin_name.equals(StaticStrings.ARCPLUG_STR) || plugin_name.equals(StaticStrings.RECPLUG_STR)) {
+               move_top_button.setEnabled(false);
+               move_up_button.setEnabled(false);
+               move_down_button.setEnabled(false);
+               move_bottom_button.setEnabled(false);
+               remove.setEnabled(false);
+               }
+               else {
+              // Move ups are only enabled if the selected plugin isn't already at the top
+              PlugIn first_plugin = (PlugIn) getElementAt(0);
+              if(!first_plugin.equals(selected_plugin)) {
+                 move_top_button.setEnabled(true);
+                 move_up_button.setEnabled(true);
+              }
+              else {
+                 move_top_button.setEnabled(false);
+                 move_up_button.setEnabled(false);
+              }
+              // And move downs are only allowed when the selected plugin isn't at an index one less than the separator line.
+              int separator_index = findSeparatorIndex();
+              int selected_index = plugin_list.getSelectedIndex();
+              if(selected_index != separator_index - 1) {
+                 move_down_button.setEnabled(true);
+                 move_bottom_button.setEnabled(true);
+              }
+              else {
+                 move_down_button.setEnabled(false);
+                 move_bottom_button.setEnabled(false);
+              }
+              remove.setEnabled(true);
+               }
+               selected_plugin = null;
+               plugin_name = null;
             }
         }