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

michi at wyona.com michi at wyona.com
Fri Feb 8 10:11:07 CET 2008


Author: michi
Date: 2008-02-08 10:11:06 +0100 (Fri, 08 Feb 2008)
New Revision: 31480

Modified:
   public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/yanel/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java
Log:
alerts added if no item selected yet

Modified: public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/yanel/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java
===================================================================
--- public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/yanel/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java	2008-02-08 08:39:16 UTC (rev 31479)
+++ public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/yanel/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java	2008-02-08 09:11:06 UTC (rev 31480)
@@ -62,16 +62,24 @@
     public void onClick(Widget sender) {
         if (sender == addButton) {
             int i = identitiesLB.getSelectedIndex();
-            String selectedIdentity = identitiesLB.getValue(i);
-            Window.alert("Add selected identity " + selectedIdentity + " to policy");
-            identitiesLB.removeItem(i);
-            policyLB.addItem(selectedIdentity);
+            if (i >= 0) {
+                String selectedIdentity = identitiesLB.getValue(i);
+                Window.alert("Add selected identity " + selectedIdentity + " to policy");
+                identitiesLB.removeItem(i);
+                policyLB.addItem(selectedIdentity);
+            } else {
+                Window.alert("No identity selected yet! Please select an identity.");
+            }
         } else if (sender == removeButton) {
             int i = policyLB.getSelectedIndex();
-            String selectedIdentity = policyLB.getValue(i);
-            Window.alert("Remove selected identity " + selectedIdentity + " from policy");
-            policyLB.removeItem(i);
-            identitiesLB.addItem(removeRights(selectedIdentity));
+            if (i >= 0) {
+                String selectedIdentity = policyLB.getValue(i);
+                Window.alert("Remove selected identity " + selectedIdentity + " from policy");
+                policyLB.removeItem(i);
+                identitiesLB.addItem(removeRights(selectedIdentity));
+            } else {
+                Window.alert("No identity selected yet! Please select an identity.");
+            }
         }
     }
 



More information about the Yanel-commits mailing list