[Yanel-commits] rev 32801 - public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client

[michi at wyona.com]

Author: michi
Date: 2008-03-03 12:31:44 +0100 (Mon, 03 Mar 2008)
New Revision: 32801

Modified:
   public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java
Log:
bug fixed re adding Write

Modified: public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java
===================================================================
--- public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java	2008-03-03 11:24:05 UTC (rev 32800)
+++ public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java	2008-03-03 11:31:44 UTC (rev 32801)
@@ -122,22 +122,24 @@
             String selectedIdentity = getSelectedItemText();
             if (selectedIdentity != null) {
                 if (sender == readCB) {
-                    Window.alert("Add/Remove Read right from selected identity " + selectedIdentity + " from policy");
                     String[] currentRights = getRights(selectedIdentity);
                     String[] newRights;
                     if (readCB.isChecked()) {
+                        Window.alert("Add Read right from selected identity " + selectedIdentity + " from policy");
                         newRights = addRight(currentRights, "Read");
                     } else {
+                        Window.alert("Remove Read right from selected identity " + selectedIdentity + " from policy");
                         newRights = removeRight(currentRights, "Read");
                     }
                     setSelectedListItem(newRights);
                 } else if (sender == writeCB) {
-                    Window.alert("Add/Remove Write right from selected identity " + selectedIdentity + " from policy");
                     String[] currentRights = getRights(selectedIdentity);
                     String[] newRights;
-                    if (readCB.isChecked()) {
+                    if (writeCB.isChecked()) {
+                        Window.alert("Add Write right from selected identity " + selectedIdentity + " from policy");
                         newRights = addRight(currentRights, "Write");
                     } else {
+                        Window.alert("Remove Write right from selected identity " + selectedIdentity + " from policy");
                         newRights = removeRight(currentRights, "Write");
                     }
                     setSelectedListItem(newRights);