uicontrol : setEnabled take care of previous status 92/14292/2
Antoine ELIAS [Wed, 2 Apr 2014 09:26:07 +0000 (11:26 +0200)]
Change-Id: Ia962f933e477d8cd83f71b49ccc083bd71e9125d

scilab/modules/gui/src/java/org/scilab/modules/gui/bridge/editbox/SwingScilabEditBox.java
scilab/modules/gui/src/java/org/scilab/modules/gui/bridge/listbox/SwingScilabListBox.java
scilab/modules/gui/src/java/org/scilab/modules/gui/bridge/menu/SwingScilabMenu.java
scilab/modules/gui/src/java/org/scilab/modules/gui/bridge/menuitem/SwingScilabMenuItem.java
scilab/modules/gui/src/java/org/scilab/modules/gui/bridge/tab/SwingScilabTabGroup.java

index d80325c..1532e3a 100644 (file)
@@ -290,6 +290,10 @@ public class SwingScilabEditBox extends JScrollPane implements SwingViewObject,
      * @param status true if the EditBox is enabled
      */
     public void setEnabled(boolean status) {
+        if (status == isEnabled()) {
+            return;
+        }
+
         super.setEnabled(status);
         textPane.setEnabled(status);
 
index 2f94b9b..c413be9 100644 (file)
@@ -263,17 +263,19 @@ public class SwingScilabListBox extends JScrollPane implements SwingViewObject,
      * (true if the UIElement is enabled, false if not)
      */
     public void setEnabled(boolean newEnableState) {
-        if (newEnableState != super.isEnabled()) {
-            super.setEnabled(newEnableState);
-            getList().setEnabled(newEnableState);
-            if (newEnableState) {
-                if (listListener != null) {
-                    getList().addListSelectionListener(listListener);
-                }
-            } else {
-                if (listListener != null) {
-                    getList().removeListSelectionListener(listListener);
-                }
+        if (newEnableState == isEnabled()) {
+            return;
+        }
+
+        super.setEnabled(newEnableState);
+        getList().setEnabled(newEnableState);
+        if (newEnableState) {
+            if (listListener != null) {
+                getList().addListSelectionListener(listListener);
+            }
+        } else {
+            if (listListener != null) {
+                getList().removeListSelectionListener(listListener);
             }
         }
     }
index 05292b0..38892d9 100644 (file)
@@ -393,6 +393,10 @@ public class SwingScilabMenu extends JMenu implements SwingViewObject, SimpleMen
      * @param status true if the menu is enabled
      */
     public void setEnabled(boolean status) {
+        if (status == isEnabled()) {
+            return;
+        }
+
         super.setEnabled(status);
         /* (Des)Activate the callback */
         if (callback != null) {
index 12776ba..2b1c1bc 100644 (file)
@@ -322,6 +322,10 @@ public class SwingScilabMenuItem extends JMenuItem implements SwingViewObject, S
      * @param status true if the menu item is enabled
      */
     public void setEnabled(boolean status) {
+        if (status == isEnabled()) {
+            return;
+        }
+
         super.setEnabled(status);
         /* (Des)Activate the callback */
         if (callback != null) {
index 7fd4388..c243cc0 100644 (file)
@@ -301,6 +301,10 @@ public class SwingScilabTabGroup extends JTabbedPane implements SwingViewObject,
     }
 
     public void setEnabled(boolean status) {
+        if (status == isEnabled()) {
+            return;
+        }
+
         if (status) {
             // Enable the frame
             super.setEnabled(status);