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 2013/06/13 21:42:22 UTC

svn commit: r1492830 - /uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/AbstractSection.java

Author: schor
Date: Thu Jun 13 19:42:22 2013
New Revision: 1492830

URL: http://svn.apache.org/r1492830
Log:
[UIMA-2511] fix reversed test that was treating deleting an existing value to be the same as no change.

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

Modified: uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/AbstractSection.java
URL: http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/AbstractSection.java?rev=1492830&r1=1492829&r2=1492830&view=diff
==============================================================================
--- uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/AbstractSection.java (original)
+++ uima/uimaj/trunk/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/AbstractSection.java Thu Jun 13 19:42:22 2013
@@ -780,7 +780,7 @@ implements Listener, StandardStrings {
 
   protected String setValueChanged(String newValue, String oldValue) {
     if (null == newValue)
-      valueChanged = valueChanged || (null == oldValue);
+      valueChanged = valueChanged || (null != oldValue);
     else if (!newValue.equals(oldValue))
       valueChanged = true;
     return newValue;