UPD: cleanup org-form-page, make the form look different when editing.

diff --git a/src/org/cacert/gigi/pages/orga/AffiliationForm.java b/src/org/cacert/gigi/pages/orga/AffiliationForm.java
index defd3baf7755b0ada176249e2f49ee6d44c1de5b..a74bf50f8533b13a84cddbe365ffabb034cb3604 100644 (file)
--- a/src/org/cacert/gigi/pages/orga/AffiliationForm.java
+++ b/src/org/cacert/gigi/pages/orga/AffiliationForm.java
@@ -30,14 +30,17 @@ public class AffiliationForm extends Form {
 
     @Override
     public boolean submit(PrintWriter out, HttpServletRequest req) throws GigiApiException {
-        User toRemove = User.getByEmail(req.getParameter("del"));
-        if (toRemove != null) {
-            o.removeAdmin(toRemove, LoginPage.getUser(req));
+        if (req.getParameter("del") != null) {
+            User toRemove = User.getByEmail(req.getParameter("del"));
+            if (toRemove != null) {
+                o.removeAdmin(toRemove, LoginPage.getUser(req));
+            }
         }
-
-        User byEmail = User.getByEmail(req.getParameter("email"));
-        if (byEmail != null) {
-            o.addAdmin(byEmail, LoginPage.getUser(req), req.getParameter("master") != null);
+        if (req.getParameter("do_affiliate") != null) {
+            User byEmail = User.getByEmail(req.getParameter("email"));
+            if (byEmail != null) {
+                o.addAdmin(byEmail, LoginPage.getUser(req), req.getParameter("master") != null);
+            }
         }
         return true;
     }

diff --git a/src/org/cacert/gigi/pages/orga/AffiliationForm.templ b/src/org/cacert/gigi/pages/orga/AffiliationForm.templ
index 7fbc513caa1dc1fe33ea15c3860db6bd566fa370..7a1c84fcdbec412ef245a16fcdff3304335b55ef 100644 (file)
--- a/src/org/cacert/gigi/pages/orga/AffiliationForm.templ
+++ b/src/org/cacert/gigi/pages/orga/AffiliationForm.templ
@@ -1,20 +1,22 @@
 <table align="center" valign="middle" border="0" cellspacing="0" cellpadding="0" class="wrapper dataTable">
   <tr>
     <th><?=_Name?></th>
+    <th><?=_Email?></th>
     <th><?=_Master?></th>
     <th> </th>
   </tr>
 <? foreach($admins) { ?>
   <tr>
     <td><?=$name?></td>
+    <td><?=$e-mail?></td>
     <td><?=$master?></td>
     <td><button type="submit" name="del" value="<?=$e-mail?>">X</button> </td>
   </tr>
 <? } ?>
+  <tr>
+    <td></td>
+    <td><input type="text" name="email"></td>
+    <td><input type="checkbox" name="master" value="y"></td>
+    <td><input type="submit" name="do_affiliate" value="<?=_Add?>"><input type='hidden' name='affiliate' value='y'></td>
+  </tr>
 </table>
-<?=_Add new admin?>:
-<input type="text" name="email">
-<?=_Master?>?
-<input type="hidden" name="affiliate" value="y">
-<input type="checkbox" name="master" value="y">
-<input type="submit" value="<?=_Add?>">

diff --git a/src/org/cacert/gigi/pages/orga/CreateOrgForm.java b/src/org/cacert/gigi/pages/orga/CreateOrgForm.java
index 4cd37b9a7705e0dd755ab5ba9424b4bd1a4d0ea0..27f7dbcfef1cab0c2303934c9b81bd3df4ecdbb8 100644 (file)
--- a/src/org/cacert/gigi/pages/orga/CreateOrgForm.java
+++ b/src/org/cacert/gigi/pages/orga/CreateOrgForm.java
@@ -26,12 +26,15 @@ public class CreateOrgForm extends Form {
 
     private String l = "";
 
+    private boolean isEdit = false;
+
     public CreateOrgForm(HttpServletRequest hsr) {
         super(hsr);
     }
 
     public CreateOrgForm(HttpServletRequest hsr, Organisation t) {
         super(hsr);
+        isEdit = true;
         result = t;
         o = t.getName();
         c = t.getState();
@@ -64,6 +67,9 @@ public class CreateOrgForm extends Form {
         vars.put("C", c);
         vars.put("ST", st);
         vars.put("L", this.l);
+        if (isEdit) {
+            vars.put("edit", true);
+        }
         t.output(out, l, vars);
     }
 }

diff --git a/src/org/cacert/gigi/pages/orga/CreateOrgForm.templ b/src/org/cacert/gigi/pages/orga/CreateOrgForm.templ
index 3d16d0bee31fbbe490b44857c40a0417582c8ac3..787dec39146e12b474a9f8709bdcd2e7d957d106 100644 (file)
--- a/src/org/cacert/gigi/pages/orga/CreateOrgForm.templ
+++ b/src/org/cacert/gigi/pages/orga/CreateOrgForm.templ
@@ -1,6 +1,11 @@
 <table align="center" valign="middle" border="0" cellspacing="0" cellpadding="0" class="wrapper dataTable">
   <tr>
-    <th colspan="2"><?=_New Organisation?></th>
+    <th colspan="2">
+    <? if($edit) { ?>
+    <?=_Edit Organisation?>
+    <? } else { ?>
+    <?=_New Organisation?>
+    <? } ?></th>
   </tr>
   <tr>
     <td><?=_Organisation Name?>:</td>
@@ -28,6 +33,12 @@
     <td><?=_Comments?>:</td>
     <td><textarea name="comments" cols="60" rows="10"></textarea></td>
   </tr>
+  <? if($edit) { ?>
+  <tr>
+    <td></td>
+    <td><?=_WARNING: updating the data will revoke all issued certificates.?></td>
+  </tr>
+  <? } ?>
   <tr>
     <td colspan="2"><input type="submit" value="<?=_Submit?>"></td>
   </tr>

diff --git a/src/org/cacert/gigi/pages/orga/EditOrg.templ b/src/org/cacert/gigi/pages/orga/EditOrg.templ
index fa3fbb646231edf8f2c4affff3147e1687780a4d..bd415d7f561784f8313346ab42483a50ec440af0 100644 (file)
--- a/src/org/cacert/gigi/pages/orga/EditOrg.templ
+++ b/src/org/cacert/gigi/pages/orga/EditOrg.templ
@@ -1,3 +1,3 @@
 <?=$editForm?>
-<?=_WARNING: updating the data will revoke all issued certificates.?>
+<br/>
 <?=$affForm?>