quick HTML cleanup

6 files changed, 234 insertions, 197 deletions

diff --git a/root/admin/account.tt b/root/admin/account.tt
index b00b8a4..aae4888 100644
--- a/root/admin/account.tt
+++ b/root/admin/account.tt
@@ -1,47 +1,49 @@
-<form method='POST'>
-<table>
-<tr>
-<td>
-Search by
-</td>
-<td>
-<select name="attribute">
-<option value="uid">Username</option>
-<option value="mail">Email</option>
-<option value="cn">Full Name</option>
-<option value="sn">Surname</option>
-</select>
-</td>
-<!-- td>
-<select name="matchtype">
-<option value="substring">contains</option>
-<option value="exact">is exactly</option>
-<option value="gte">greater than or equal to</option>
-<option value="lt">less than</option>
-</select>
-</td -->
-<td><input name='value'></td>
-</table>
+<form method="post" action="">
+    <table>
+        <tr>
+            <td>Search by</td>
+            <td>
+                <select name="attribute">
+                    <option value="uid">Username</option>
+                    <option value="mail">Email</option>
+                    <option value="cn">Full Name</option>
+                    <option value="sn">Surname</option>
+                </select>
+            </td>
+            <!-- td>
+            <select name="matchtype">
+            <option value="substring">contains</option>
+            <option value="exact">is exactly</option>
+            <option value="gte">greater than or equal to</option>
+            <option value="lt">less than</option>
+            </select>
+            </td -->
+            <td><input name="value" value="" /></td>
+        </tr>
+    </table>
 </form>
+
+<hr />
+
 [% IF entries %]
-<table border=0>
-<tr>
-<th>Username</th>
-<th>Email</th>
-<th>First Name</th>
-<th>Surname</td>
-<th>Full Name</td>
-</tr>
-[% FOREACH entry IN entries %]
-<tr>
-<td><a href="[% c.uri_for('/admin/account_modify') %]/[% entry.uid %]">[% entry.uid %]</a></td>
-<td>[% entry.mail %]</td>
-<td>[% entry.givenName %]</td>
-<td>[% entry.sn %]</td>
-<td>[% entry.cn %]</td>
-</tr>
-[% END %]
-</table>
+    <table border=0>
+        <tr>
+            <th>Username</th>
+            <th>Email</th>
+            <th>First Name</th>
+            <th>Surname</td>
+            <th>Full Name</td>
+        </tr>
+        [% FOREACH entry IN entries %]
+            <tr>
+                <td><a href="[% c.uri_for('/admin/account_modify') %]/[% entry.uid %]">[% entry.uid %]</a></td>
+                <td>[% entry.mail %]</td>
+                <td>[% entry.givenName %]</td>
+                <td>[% entry.sn %]</td>
+                <td>[% entry.cn %]</td>
+            </tr>
+        [% END %]
+    </table>
 [% END %]
 
 
diff --git a/root/admin/account_addoc.tt b/root/admin/account_addoc.tt
index 771ac8d..75c6898 100644
--- a/root/admin/account_addoc.tt
+++ b/root/admin/account_addoc.tt
@@ -1,27 +1,30 @@
 Adding objectclass [% oc %] to dn [% dn %]
-<form method=post>
-<input type=hidden name='dn' value='[% dn %]'>
-<input type=hidden name='uid' value='[% uid %]'>
-<input type='hidden' name='objectclass' value='[% oc %]'>
-<table>
-<tr>
-<th>Attribute</th>
-<th>Value</th>
-</tr>
-[% FOREACH attr IN must %]
-[% IF attr != "objectClass" %]
-<tr>
-<td>[% attr %]</td>
-<td><input name='[% attr %]'><span color='red'>*</span></td>
-</tr>
-[% END %]
-[% END %]
-[% FOREACH attr IN may %]
-<tr>
-<td>[% attr %]</td>
-<td><input name='[% attr %]'></td>
-</tr>
-[% END %]
-</table>
-<input type='submit' value='Add'>
 
+<form method="post" action="">
+    <input type="hidden" name="dn" value="[% dn %]" />
+    <input type="hidden" name="uid" value="[% uid %]" />
+    <input type="hidden" name="objectclass" value="[% oc %]" />
+    <table>
+        <tr>
+            <th>Attribute</th>
+            <th>Value</th>
+        </tr>
+        [% FOREACH attr IN must %]
+            [% IF attr != "objectClass" %]
+                <tr>
+                    <td>[% attr %]</td>
+                    <td><input name='[% attr %]'>
+                        <span color='red'>*</span></td>
+                </tr>
+            [% END %]
+        [% END %]
+        [% FOREACH attr IN may %]
+            <tr>
+                <td>[% attr %]</td>
+                <td><input name='[% attr %]'></td>
+            </tr>
+        [% END %]
+    </table>
+    <p><input type="submit" value="Add" /></p>
+
+</form>
\ No newline at end of file
diff --git a/root/admin/account_group.tt b/root/admin/account_group.tt
index 3b04992..d2355c7 100644
--- a/root/admin/account_group.tt
+++ b/root/admin/account_group.tt
@@ -1,24 +1,29 @@
-Add user [% uid %] to a new group:
+<h2>Add user [% uid %] to a new group</h2>
 
-<form method='post'>
-<input type='hidden' name='uid' value='[% uid %]'>
-<input type='hidden' name='op' value='add'>
-<select name='group'>
-[% FOREACH group IN newgroups %]
-<option value='[% group.cn %]'>[% group.cn %]</option>
-[% END %]
-</select>
-<input type='submit' value='Add'>
+<form method="post" action="">
+    <input type='hidden' name='uid' value='[% uid %]' />
+    <input type='hidden' name='op' value='add' />
+    <select name='group'>
+        [% FOREACH group IN newgroups %]
+            <option value='[% group.cn %]'>[% group.cn %]</option>
+        [% END %]
+    </select>
+    <input type='submit' value='Add' />
 </form>
 
-Delete user [% uid %] from an existing group:
-<form method='post'>
-<input type='hidden' name='uid' value='[% uid %]'>
-<input type='hidden' name='op' value='delete'>
-<select name='group'>
-[% FOREACH group IN groups %]
-<option value='[% group.cn %]'>[% group.cn %]</option>
-[% END %]
-</select>
-<input type='submit' value='Delete'>
+<hr />
+
+<h2>Delete user [% uid %] from an existing group:</h2>
+
+<form method="post" action="">
+    <input type='hidden' name='uid' value='[% uid %]' />
+    <input type='hidden' name='op' value='delete' />
+    <select name='group'>
+        [% FOREACH group IN groups %]
+            <option value='[% group.cn %]'>[% group.cn %]</option>
+        [% END %]
+    </select>
+    <input type='submit' value='Delete' />
 </form>
+
+<hr />
\ No newline at end of file
diff --git a/root/admin/account_modify.tt b/root/admin/account_modify.tt
index 6a1ecc8..8acd4b4 100644
--- a/root/admin/account_modify.tt
+++ b/root/admin/account_modify.tt
@@ -1,66 +1,88 @@
 <a href="[% c.uri_for('/admin/password') %]/[% uid %]">Reset password</a>
 <a href="[% c.uri_for('/admin/account_group') %]/[% uid %]">Groups</a>
 
-<form method=post>
-<input type='hidden' name='operation' value='replace'>
-<table border=0>
-<tr><th>Attribute</th><th>Value</th></tr>
-[% FOREACH attr IN values %]
-  <tr>
-    <td>[% attr.name %]</td>
-    <td>[% FOREACH val IN attr.values %][% IF attr.editable %]<input type=hidden name="[% attr.name %]_old" value="[% val %]"><input name="[% attr.name %]_new" value="[% val %]">[% ELSE %][% val %]<br/>[% END %]
-[% IF attr.addable AND attr.editable %]<a href="[% c.uri_for('/admin/account_edit') %]/add/[% attr.name %]">Add</a>[% END %]
-[% IF attr.removable AND attr.editable %]<a href="[% c.uri_for('/admin/account_modifydel') %]/[% uid %]/[% attr.name %]/[% val %]">Delete</a>[% END %]
-    [% END %]</td>
-  </tr>
-[% END %]
-<tr><td colspan=2 align=center><input type='Submit' value='Update'></td></tr>
-</table>
+<form method="post" action="">
+    <input type='hidden' name='operation' value='replace' />
+    <table border=0>
+        <tr>
+            <th>Attribute</th>
+            <th>Value</th>
+        </tr>
+        [% FOREACH attr IN values %]
+        <tr>
+            <td>[% attr.name %]</td>
+            <td>
+                [% FOREACH val IN attr.values %]
+                    [% IF attr.editable %]
+                        <input type="hidden" name="[% attr.name %]_old" value="[% val %]" />
+                        <input name="[% attr.name %]_new" value="[% val %]" />
+                    [% ELSE %]
+                        [% val %]
+                        <br/>
+                    [% END %]
+                    [% IF attr.addable AND attr.editable %]
+                        <a href="[% c.uri_for('/admin/account_edit') %]/add/[% attr.name %]">Add</a>
+                    [% END %]
+                    [% IF attr.removable AND attr.editable %]
+                        <a href="[% c.uri_for('/admin/account_modifydel') %]/[% uid %]/[% attr.name %]/[% val %]">Delete</a>
+                    [% END %]
+                [% END %]
+            </td>
+        </tr>
+        [% END %]
+        <tr>
+            <td colspan=2 align=center>
+                <input type='Submit' value='Update'>
+            </td>
+        </tr>
+    </table>
 </form>
 
-<table border=0>
-<tr>
-<td>
-<form method=post action="[% c.uri_for('/admin/account_modify') %]/[% uid %]">
-<input type='hidden' name='operation' value='add'>
- Add attribute 
- <select name='attribute'>[% FOREACH attr IN may %]
-  <option value="[% attr %]">[% attr %]</option>[% END %]
- </select>
- with value
-<input name='value'>
- <input type=submit value='Add'>
+<hr />
+
+<form method="post" action="[% c.uri_for('/admin/account_modify') %]/[% uid %]">
+    <input type="hidden" name="operation" value="add" />
+
+    <p>
+    Add attribute 
+    <select name='attribute'>
+        [% FOREACH attr IN may %]
+        <option value="[% attr %]">[% attr %]</option>
+        [% END %]
+    </select>
+    with value
+    <input name="value" value="" />
+    <input type="submit" value="Add" />
+    </p>
 </form>
-</td>
-</tr>
+
+<hr />
+
 [% IF groups %]
-<tr>
-<td>
-Promote user to posixAccount with primary group: 
-<form method=post action="[% c.uri_for('/admin/account_promote') %]">
-<input type='hidden' name='dn' value='[% dn %]'>
-<select name='gid'>
-[% FOREACH group IN groups %]
-<option value='[% group.gidNumber %]'>[% group.name %]</option>
-[% END %]
-</select>
-<input type=submit value='Promote'>
+<p>Promote user to posixAccount with primary group: </p>
+
+<form method="post" action="[% c.uri_for('/admin/account_promote') %]">
+    <input type="hidden" name="dn" value="[% dn %]" />
+    <select name="gid">
+        [% FOREACH group IN groups %]
+            <option value="[% group.gidNumber %]">[% group.name %]</option>
+        [% END %]
+    </select>
+    <input type="submit" value="Promote" />
 </form>
-</td>
-</tr>
-[% END %]
-<tr>
-<td>
-<form method=post action="[% c.uri_for('/admin/account_addoc') %]">
-<input type='hidden' name='dn' value='[% dn %]'>
-<input type='hidden' name='uid' value='[% uid %]'>
-<select name='objectclass'>
-[% FOREACH oc IN offer_ocs %]
-<option value='[% oc %]'>[% oc %]</option>
 [% END %]
-</select>
-<input type='submit' value='Add ObjectClass'>
+
+<hr />
+
+<form method="post" action="[% c.uri_for('/admin/account_addoc') %]">
+    <input type='hidden' name='dn' value='[% dn %]'>
+    <input type='hidden' name='uid' value='[% uid %]'>
+    <select name='objectclass'>
+        [% FOREACH oc IN offer_ocs %]
+            <option value='[% oc %]'>[% oc %]</option>
+        [% END %]
+    </select>
+    <input type="submit" value="Add ObjectClass" />
 </form>
-</td>
-</tr>
-</table>
+
+<hr />
\ No newline at end of file
diff --git a/root/admin/account_promote.tt b/root/admin/account_promote.tt
index fd6400e..39dba87 100644
--- a/root/admin/account_promote.tt
+++ b/root/admin/account_promote.tt
@@ -1,34 +1,38 @@
-<form method=post>
-<table border=0>
-<tr>
-<th>Select</th>
-<th>Username</th>
-<th>Email</th>
-<th>First Name</th>
-<th>Surname</td>
-<th>Full Name</td>
-</tr>
-[% FOREACH entry IN entries %]
-<tr>
-<td><input type='checkbox' name='username' value="[% entry.uid %]">
-<td>[% entry.uid %]</td>
-<td>[% entry.mail %]</td>
-<td>[% entry.givenName %]</td>
-<td>[% entry.sn %]</td>
-<td>[% entry.cn %]</td>
-</tr>
-[% END %]
-</table>
+<form method="post" action="">
+    <table border=0>
+        <tr>
+            <th>Select</th>
+            <th>Username</th>
+            <th>Email</th>
+            <th>First Name</th>
+            <th>Surname</td>
+            <th>Full Name</td>
+        </tr>
+        [% FOREACH entry IN entries %]
+        <tr>
+            <td><input type='checkbox' name='username' value="[% entry.uid %]">
+            <td>[% entry.uid %]</td>
+            <td>[% entry.mail %]</td>
+            <td>[% entry.givenName %]</td>
+            <td>[% entry.sn %]</td>
+            <td>[% entry.cn %]</td>
+        </tr>
+        [% END %]
+    </table>
 
-<table border=0>
-<tr>
-<td>Primary group</td>
-<td><select name='gid'>
-[% FOREACH group IN groups %]
-<option value=[% group.gidNumber %]>[% group.cn %]</option>
-[% END %]
-</td>
-<td>
-<input type='submit' value='Promote'>
-</table>
+    <table border=0>
+        <tr>
+            <td>Primary group</td>
+            <td>
+                <select name="gid">
+                [% FOREACH group IN groups %]
+                    <option value=[% group.gidNumber %]>[% group.cn %]</option>
+                [% END %]
+                </select>
+            </td>
+            <td>
+                <input type="submit" value="Promote" />
+            </td>
+        </tr>
+    </table>
 </form>
diff --git a/root/admin/group_modify.tt b/root/admin/group_modify.tt
index 01520d5..15357f6 100644
--- a/root/admin/group_modify.tt
+++ b/root/admin/group_modify.tt
@@ -1,22 +1,23 @@
-<form method=post>
-<input type='hidden' name='dn' value='[% group.dn %]'>
-<table>
-<tr>
-<th>Attribute</th>
-<th>Value</th>
-</tr>
-[% FOREACH attr IN group.attributes %]
-<tr>
-<td>
-[% attr %]
-</td>
-<td>
-[% FOREACH value IN group.get_value(attr) %]
-[% value %] <a href="[% c.uri_for('/admin/group_modify') %]/delete/[% group.dn %]/[% attr %]/[% value %]">delete</a><br/>
-[% END %]
-<input name='[% attr %]'>
-<input type='submit' value='Add'>
-</td>
-</tr>
-[% END %]
-</table>
+<form method="post" action="">
+    <input type='hidden' name='dn' value='[% group.dn %]'>
+    <table>
+        <tr>
+            <th>Attribute</th>
+            <th>Value</th>
+        </tr>
+        [% FOREACH attr IN group.attributes %]
+        <tr>
+            <td>[% attr %]</td>
+            <td>
+                [% FOREACH value IN group.get_value(attr) %]
+                    [% value %]
+                    <a href="[% c.uri_for('/admin/group_modify') %]/delete/[% group.dn %]/[% attr %]/[% value %]">delete</a>
+                    <br/>
+                [% END %]
+                <input name="[% attr %]" value="" />
+                <input type="submit" value="Add" />
+            </td>
+        </tr>
+        [% END %]
+    </table>
+</form>
\ No newline at end of file