You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by sc...@apache.org on 2007/05/25 04:41:27 UTC

svn commit: r541514 - /incubator/uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/ParameterSection.java

Author: schor
Date: Thu May 24 19:41:26 2007
New Revision: 541514

URL: http://svn.apache.org/viewvc?view=rev&rev=541514
Log:
[UIMA-414] [UIMA-415] This fixes the two problems of not marking the edit buffer as needing
to be saved when adding a 2nd override to an existing override, and making remove of a
2nd (or subsequent) override work correctly.

Modified:
    incubator/uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/ParameterSection.java

Modified: incubator/uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/ParameterSection.java
URL: http://svn.apache.org/viewvc/incubator/uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/ParameterSection.java?view=diff&rev=541514&r1=541513&r2=541514
==============================================================================
--- incubator/uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/ParameterSection.java (original)
+++ incubator/uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/ParameterSection.java Thu May 24 19:41:26 2007
@@ -673,8 +673,8 @@
   private String[] removeOverride(ConfigurationParameter cp, int i) {
     String[] oldOverrides = cp.getOverrides();
     String[] newOverrides = new String[oldOverrides.length - 1];
-    if (i - 1 > 0)
-      System.arraycopy(oldOverrides, 0, newOverrides, 0, i - 1);
+    if (i > 0)
+      System.arraycopy(oldOverrides, 0, newOverrides, 0, i);
     if (oldOverrides.length - 1 - i > 0)
       System.arraycopy(oldOverrides, i + 1, newOverrides, i, oldOverrides.length - 1 - i);
     return newOverrides;
@@ -862,6 +862,7 @@
     cp.setOverrides(addOverrideToArray(cp.getOverrides(), override));
     addOverrideToGUI(parent, override);
     parent.setExpanded(true);
+    commonActionFinish();
   }
 
   private void alterExistingConfigurationParameter(AddParameterDialog dialog,