add: show assigned groups to an account in My Detail

fixes issue #114

Change-Id: I2072080316247b6fca283cf548c4418da63ab8fc

diff --git a/src/org/cacert/gigi/output/GroupIterator.java b/src/org/cacert/gigi/output/GroupIterator.java
new file mode 100644 (file)
index 0000000..4e35db4
--- /dev/null
+++ b/src/org/cacert/gigi/output/GroupIterator.java
@@ -0,0 +1,37 @@
+package org.cacert.gigi.output;
+
+import java.util.Iterator;
+import java.util.Map;
+
+import org.cacert.gigi.dbObjects.Group;
+import org.cacert.gigi.localisation.Language;
+import org.cacert.gigi.output.template.IterableDataset;
+
+public class GroupIterator implements IterableDataset {
+
+    private final boolean asSupport;
+
+    private final Iterator<Group> i;
+
+    public GroupIterator(Iterator<Group> i, boolean asSupport) {
+        this.asSupport = asSupport;
+        this.i = i;
+    }
+
+    private int j = 0;
+
+    @Override
+    public boolean next(Language l, Map<String, Object> vars) {
+        while (i.hasNext()) {
+            Group g = i.next();
+            if (g.isManagedBySupport() == asSupport) {
+                vars.put("group_concat", (j > 0 ? ", " : ""));
+                vars.put("group", g.getName());
+                j++;
+                return true;
+            }
+        }
+
+        return false;
+    }
+}

diff --git a/src/org/cacert/gigi/output/GroupSelector.java b/src/org/cacert/gigi/output/GroupSelector.java
index f65fab750f4f14ae8260e43fedd8b23d4c379be9..66236e1610bb64657381f049b846552eb5240157 100644 (file)
--- a/src/org/cacert/gigi/output/GroupSelector.java
+++ b/src/org/cacert/gigi/output/GroupSelector.java
@@ -39,11 +39,13 @@ public class GroupSelector implements Outputable {
         out.println("<select name='" + name + "'>");
         for (Group g : Group.values()) {
             if (supportFlag == g.isManagedBySupport()) {
         out.println("<select name='" + name + "'>");
         for (Group g : Group.values()) {
             if (supportFlag == g.isManagedBySupport()) {

-                out.print("<option name='" + g.getDatabaseName());
+                out.print("<option value='" + g.getDatabaseName());

                 if (g.equals(value)) {
                     out.print(" selected");
                 }
                 if (g.equals(value)) {
                     out.print(" selected");
                 }

-                out.println("'>" + g.getDatabaseName() + "</option>");
+                out.println("'>");
+                g.getName().output(out, l, vars);
+                out.println("</option>");

             }
         }
         out.println("</select>");
             }
         }
         out.println("</select>");

diff --git a/src/org/cacert/gigi/pages/account/MyDetailsForm.java b/src/org/cacert/gigi/pages/account/MyDetailsForm.java
index a8ca49fc5d471e377fc0224b989604f744302482..d6b65e01842877154e0fc1c1307881ff96b2152a 100644 (file)
--- a/src/org/cacert/gigi/pages/account/MyDetailsForm.java
+++ b/src/org/cacert/gigi/pages/account/MyDetailsForm.java
@@ -2,16 +2,19 @@ package org.cacert.gigi.pages.account;
 
 import java.io.PrintWriter;
 import java.util.Map;
 
 import java.io.PrintWriter;
 import java.util.Map;

+import java.util.Set;

 
 import javax.servlet.http.HttpServletRequest;
 
 import org.cacert.gigi.GigiApiException;
 
 import javax.servlet.http.HttpServletRequest;
 
 import org.cacert.gigi.GigiApiException;

+import org.cacert.gigi.dbObjects.Group;

 import org.cacert.gigi.dbObjects.Name;
 import org.cacert.gigi.dbObjects.User;
 import org.cacert.gigi.localisation.Language;
 import org.cacert.gigi.output.ArrayIterable;
 import org.cacert.gigi.output.CountrySelector;
 import org.cacert.gigi.output.DateSelector;
 import org.cacert.gigi.dbObjects.Name;
 import org.cacert.gigi.dbObjects.User;
 import org.cacert.gigi.localisation.Language;
 import org.cacert.gigi.output.ArrayIterable;
 import org.cacert.gigi.output.CountrySelector;
 import org.cacert.gigi.output.DateSelector;

+import org.cacert.gigi.output.GroupIterator;

 import org.cacert.gigi.output.NameInput;
 import org.cacert.gigi.output.template.Form;
 import org.cacert.gigi.output.template.Template;
 import org.cacert.gigi.output.NameInput;
 import org.cacert.gigi.output.template.Form;
 import org.cacert.gigi.output.template.Template;


@@ -25,6 +28,8 @@ public class MyDetailsForm extends Form {
 
     private static final Template names = new Template(MyDetailsForm.class.getResource("NamesForm.templ"));
 
 
     private static final Template names = new Template(MyDetailsForm.class.getResource("NamesForm.templ"));
 

+    private static final Template roles = new Template(MyDetailsForm.class.getResource("MyDetailsRoles.templ"));
+

     private User target;
 
     private DateSelector ds;
     private User target;
 
     private DateSelector ds;


@@ -131,7 +136,13 @@ public class MyDetailsForm extends Form {
 
         });
         vars.put("name", ni);
 
         });
         vars.put("name", ni);

+        final Set<Group> gr = target.getGroups();

         names.output(out, l, vars);
         names.output(out, l, vars);

+
+        vars.put("support-groups", new GroupIterator(gr.iterator(), true));
+        vars.put("groups", new GroupIterator(gr.iterator(), false));
+        roles.output(out, l, vars);
+

         vars.put("residenceCountry", cs);
         if (target.getReceivedAssurances().length == 0) {
             vars.put("DoB", ds);
         vars.put("residenceCountry", cs);
         if (target.getReceivedAssurances().length == 0) {
             vars.put("DoB", ds);


@@ -140,6 +151,7 @@ public class MyDetailsForm extends Form {
             vars.put("DoB", target.getDoB());
             assured.output(out, l, vars);
         }
             vars.put("DoB", target.getDoB());
             assured.output(out, l, vars);
         }

+

     }
 
 }
     }
 
 }

diff --git a/src/org/cacert/gigi/pages/account/MyDetailsRoles.templ b/src/org/cacert/gigi/pages/account/MyDetailsRoles.templ
new file mode 100644 (file)
index 0000000..d01dd3f
--- /dev/null
+++ b/src/org/cacert/gigi/pages/account/MyDetailsRoles.templ
@@ -0,0 +1,9 @@
+<div class="panel panel-default">
+  <div class="panel-heading"><?=_Permissions / Group Assignments?></div>
+  <div class="panel-body">
+  <h4><?=_Support Managed Permissions?>:</h4>
+  <? foreach($support-groups) { ?><?=$group_concat?><?=$group?><? } ?>
+  <h4><?=_User Managed Permissions?>:</h4>
+  <? foreach($groups) { ?><?=$group_concat?><?=$group?><? } ?>
+  </div>
+</div>
\ No newline at end of file

diff --git a/src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.java b/src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.java
index df7840587fe94ba6daa228b6a3d99296434ec576..28597ca2f8bcc9f25b9bb227f05895ae23ab66dc 100644 (file)
--- a/src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.java
+++ b/src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.java
@@ -1,7 +1,6 @@
 package org.cacert.gigi.pages.admin.support;
 
 import java.io.PrintWriter;
 package org.cacert.gigi.pages.admin.support;
 
 import java.io.PrintWriter;

-import java.util.Iterator;

 import java.util.Map;
 import java.util.Set;
 
 import java.util.Map;
 import java.util.Set;
 


@@ -15,9 +14,9 @@ import org.cacert.gigi.dbObjects.User;
 import org.cacert.gigi.localisation.Language;
 import org.cacert.gigi.output.ArrayIterable;
 import org.cacert.gigi.output.DateSelector;
 import org.cacert.gigi.localisation.Language;
 import org.cacert.gigi.output.ArrayIterable;
 import org.cacert.gigi.output.DateSelector;

+import org.cacert.gigi.output.GroupIterator;

 import org.cacert.gigi.output.GroupSelector;
 import org.cacert.gigi.output.template.Form;
 import org.cacert.gigi.output.GroupSelector;
 import org.cacert.gigi.output.template.Form;

-import org.cacert.gigi.output.template.IterableDataset;

 import org.cacert.gigi.output.template.Outputable;
 import org.cacert.gigi.output.template.SprintfCommand;
 import org.cacert.gigi.output.template.Template;
 import org.cacert.gigi.output.template.Outputable;
 import org.cacert.gigi.output.template.SprintfCommand;
 import org.cacert.gigi.output.template.Template;


@@ -107,20 +106,8 @@ public class SupportUserDetailsForm extends Form {
         vars.put("assurancepoints", user.getAssurancePoints());
         vars.put("exppoints", user.getExperiencePoints());
         final Set<Group> gr = user.getGroups();
         vars.put("assurancepoints", user.getAssurancePoints());
         vars.put("exppoints", user.getExperiencePoints());
         final Set<Group> gr = user.getGroups();

-        vars.put("groups", new IterableDataset() {
-
-            Iterator<Group> i = gr.iterator();
-
-            @Override
-            public boolean next(Language l, Map<String, Object> vars) {
-                if ( !i.hasNext()) {
-                    return false;
-                }
-                Group g = i.next();
-                vars.put("group_name", g.getName());
-                return true;
-            }
-        });
+        vars.put("support-groups", new GroupIterator(gr.iterator(), true));
+        vars.put("groups", new GroupIterator(gr.iterator(), false));

         vars.put("groupSelector", value);
         t.output(out, l, vars);
     }
         vars.put("groupSelector", value);
         t.output(out, l, vars);
     }

diff --git a/src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.templ b/src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.templ
index e259cd26404698d0144275b79282942555bec0c7..ced40d83adf6b909906a02f34b5235163f5388d5 100644 (file)
--- a/src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.templ
+++ b/src/org/cacert/gigi/pages/admin/support/SupportUserDetailsForm.templ
@@ -42,11 +42,18 @@
         </tr>
         
         <tr>
         </tr>
         
         <tr>

-            <td><?=_Groups?>:</td>
-            <td><p><? foreach($groups) { ?><?=$group_name?>, <? } ?></p>
+            <td><?=_Support Managed Groups?>:</td>
+            <td><p><? foreach($support-groups) { ?><?=$group_concat?><?=$group?><? } ?></p>

             <p><?=$groupSelector?><input class="btn btn-info" type='submit' value='<?=_Grant Group?>' name='grant'> <input class="btn btn-info" type='submit' value='<?=_Deny Group?>' name='deny'></p>
             </td>
         </tr>
             <p><?=$groupSelector?><input class="btn btn-info" type='submit' value='<?=_Grant Group?>' name='grant'> <input class="btn btn-info" type='submit' value='<?=_Deny Group?>' name='deny'></p>
             </td>
         </tr>

+
+        <tr>
+            <td><?=_User Managed Groups?>:</td>
+            <td><p><? foreach($groups) { ?><?=$group_concat?><?=$group?><? } ?></p>
+            </td>
+        </tr>
+

         <tr>
             <td><?=_Reset Password?>:</td>
             <td><input type="text" name="aword"> <input class="btn btn-primary" type="submit" value="<?=_Reset Password?>" name="resetPass"></td>
         <tr>
             <td><?=_Reset Password?>:</td>
             <td><input type="text" name="aword"> <input class="btn btn-primary" type="submit" value="<?=_Reset Password?>" name="resetPass"></td>