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

[michi at wyona.com]

Author: michi
Date: 2010-01-28 13:53:31 +0100 (Thu, 28 Jan 2010)
New Revision: 47313

Modified:
   public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java
   public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java
Log:
select by default

Modified: public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java
===================================================================
--- public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java	2010-01-28 12:34:30 UTC (rev 47312)
+++ public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java	2010-01-28 12:53:31 UTC (rev 47313)
@@ -68,9 +68,11 @@
             for (int i = identitiesLB.getItemCount() - 1; i >= 0; i--) { // INFO: One needs to step backwards, because the size of the list decreases, because items are being removed if selected
                 if (identitiesLB.isItemSelected(i)) {
                     String selectedIdentity = identitiesLB.getValue(i);
-                    //Window.alert("Add selected identity " + selectedIdentity + " to policy");
                     identitiesLB.removeItem(i);
-                    policyLBW.addItem(selectedIdentity.substring(0, 1), selectedIdentity.substring(2).trim());
+                    String type = selectedIdentity.substring(0, 1); // e.g. 'g' or 'u'
+                    String name = selectedIdentity.substring(2).trim(); // e.g. 'lenya'
+                    //Window.alert("Add selected identity " + selectedIdentity + " (" + item + ", " + value + ") to policy");
+                    policyLBW.addItem(type, name, true);
                     noItemSelected = false;
                 }
             }

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	2010-01-28 12:34:30 UTC (rev 47312)
+++ public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java	2010-01-28 12:53:31 UTC (rev 47313)
@@ -432,13 +432,16 @@
     /*
      * @param type User or Group
      * @param name Name of user or group
+     * @param selected Sets wheter list item is selected. True to select the item.
      */
-    public void addItem(String type, String name) {
+    public void addItem(String type, String name, boolean selected) {
         StringBuffer emptyRights = new StringBuffer("(-");
         for (int i = 1; i < availableRightsCB.length; i++) {
             emptyRights.append(",-");
         }
         emptyRights.append(")");
+
         lb.addItem(type + ": " + emptyRights + " " + name, type + ": " + name);
+        lb.setItemSelected(lb.getItemCount() - 1, selected);
     }
 }