SF.net SVN: postfixadmin:[1098] trunk

[<chr...@us...>]

Revision: 1098
          http://postfixadmin.svn.sourceforge.net/postfixadmin/?rev=1098&view=rev
Author:   christian_boltz
Date:     2011-07-20 22:39:31 +0000 (Wed, 20 Jul 2011)

Log Message:
-----------
edit-admin.php:
- use additional column for error messages
- renamed $pAdminEdit_admin_password_text to $pAdminEdit_admin_password_text_error
- removed $tMessage

variables.inc.php:
- remove now unused $pAdminEdit_admin_password_text
  (this change was not contained in Dale's patch)

templates/admin_edit-admin.tpl
- added a 4th column for error messages
- HTML cleanup
- changed <h3> to <th>
- marked labels as <label>
- removed tMessage

This commit is part of the huge cleanup patch by Dale Blount (lnxus@SF),
https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3370510&group_id=191583

Modified Paths:
--------------
    trunk/edit-admin.php
    trunk/templates/admin_edit-admin.tpl
    trunk/variables.inc.php

Modified: trunk/edit-admin.php
===================================================================
--- trunk/edit-admin.php	2011-07-19 22:22:23 UTC (rev 1097)
+++ trunk/edit-admin.php	2011-07-20 22:39:31 UTC (rev 1098)
@@ -38,6 +38,8 @@
 authentication_require_role('global-admin');
 
 $error = 1;
+$pAdminEdit_admin_password_text_error = "";
+
 if(isset($_GET['username'])) {
     $username = escape_string ($_GET['username']);
     $result = db_query("SELECT * FROM $table_admin WHERE username = '$username'");
@@ -81,12 +83,12 @@
                 }
                 else {
                     $error = 1;
-                    flash_error(sprintf($PALANG['pPasswordTooShort'], $CONF['min_password_length']));
+                    $pAdminEdit_admin_password_text_error = sprintf($PALANG['pPasswordTooShort'], $CONF['min_password_length']);
                 }
             }
             else {
                 $error = 1;
-                $pAdminEdit_admin_password_text = $PALANG['pAdminEdit_admin_password_text_error'];
+                $pAdminEdit_admin_password_text_error = $PALANG['pAdminEdit_admin_password_text_error'];
             }
         }
     }
@@ -151,11 +153,10 @@
 }
 
 $smarty->assign ('username', $username);
-$smarty->assign ('pAdminEdit_admin_password_text', $pAdminEdit_admin_password_text, false);
+$smarty->assign ('pAdminEdit_admin_password_text_error', $pAdminEdit_admin_password_text_error, false);
 $smarty->assign ('tActive_checked', $tActive_checked);
 $smarty->assign ('tSadmin_checked', $tSadmin_checked);
 $smarty->assign ('select_options', select_options ($tAllDomains, $tDomains), false);
-$smarty->assign ('tMessage', $tMessage, false);
 $smarty->assign ('smarty_template', 'admin_edit-admin');
 $smarty->display ('index.tpl');
 

Modified: trunk/templates/admin_edit-admin.tpl
===================================================================
--- trunk/templates/admin_edit-admin.tpl	2011-07-19 22:22:23 UTC (rev 1097)
+++ trunk/templates/admin_edit-admin.tpl	2011-07-20 22:39:31 UTC (rev 1098)
@@ -2,45 +2,44 @@
 <form name="alias" method="post" action="">
 <table>
 	<tr>
-		<td colspan="3"><h3>{$PALANG.pAdminEdit_admin_welcome}</h3></td></tr>
+		<th colspan="4">{$PALANG.pAdminEdit_admin_welcome}</th>
+	</tr>
 	<tr>
-		<td>{$PALANG.pAdminEdit_admin_username}:</td>
+		<td class="label"><label>{$PALANG.pAdminEdit_admin_username}:</label></td>
 		<td>{$username}</td>
-		<td>&nbsp;</td>
+		<td colspan="2">&nbsp;</td>
 	</tr>
 	<tr>
-		<td>{$PALANG.pAdminEdit_admin_password}:</td>
+		<td class="label"><label>{$PALANG.pAdminEdit_admin_password}:</label></td>
 		<td><input class="flat" type="password" name="fPassword" value=""/></td>
-		<td>{$pAdminEdit_admin_password_text}</td>
+		<td>&nbsp;</td>
+		<td><span class="error_msg">{$pAdminEdit_admin_password_text_error}</span></td>
 	</tr>
 	<tr>
-		<td>{$PALANG.pAdminEdit_admin_password2}:</td>
+		<td class="label"><label>{$PALANG.pAdminEdit_admin_password2}:</label></td>
 		<td><input class="flat" type="password" name="fPassword2" value="" /></td>
-		<td>&nbsp;</td>
+		<td colspan="2">&nbsp;</td>
 	</tr>
 	<tr>
-		<td>{$PALANG.pAdminEdit_admin_active}:</td>
+		<td class="label"><label>{$PALANG.pAdminEdit_admin_active}:</label></td>
 		<td><input class="flat" type="checkbox" name="fActive"{$tActive_checked}/></td>
-		<td>&nbsp;</td>
+		<td colspan="2">&nbsp;</td>
 	</tr>
 	<tr>
-		<td>{$PALANG.pAdminEdit_admin_super_admin}:</td>
+		<td class="label"><label>{$PALANG.pAdminEdit_admin_super_admin}:</label></td>
 		<td><input class="flat" type="checkbox" name="fSadmin"{$tSadmin_checked}/></td>
-                <td>&nbsp;</td>
+		<td colspan="2">&nbsp;</td>
 	</tr>
 	<tr>
-	       <td>{$PALANG.pAdminCreate_admin_address}:</td>
-		<td>
-		<select name="fDomains[]" size="10" multiple="multiple">{$select_options}</select>
+	    <td class="label"><label>{$PALANG.pAdminCreate_admin_address}:</label></td>
+		<td><select name="fDomains[]" size="10" multiple="multiple">{$select_options}</select></td>
 		<td>&nbsp;</td>
-		</td>
 	</tr>
 	<tr>
-		<td colspan="3" class="hlp_center"><input class="button" type="submit" name="submit" value="{$PALANG.save}" /></td>
+		<td>&nbsp;</td>
+		<td><input class="button" type="submit" name="submit" value="{$PALANG.save}" /></td>
+		<td colspan="2">&nbsp;</td>
 	</tr>
-	<tr>
-		<td colspan="3" class="standout">{$tMessage}</td>
-	</tr>
 </table>
 </form>
 </div>

Modified: trunk/variables.inc.php
===================================================================
--- trunk/variables.inc.php	2011-07-19 22:22:23 UTC (rev 1097)
+++ trunk/variables.inc.php	2011-07-20 22:39:31 UTC (rev 1098)
@@ -81,7 +81,6 @@
 $pAdminCreate_admin_username_text = "&nbsp;";
 $pAdminCreate_domain_defaultaliases_text = "&nbsp;";
 $pAdminCreate_domain_domain_text = "&nbsp;";
-$pAdminEdit_admin_password_text = "&nbsp;";
 $pCreate_alias_address_text = "&nbsp;";
 $pCreate_alias_goto_text = "&nbsp;";
 $pCreate_mailbox_name_text = "&nbsp;";


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.