SF.net SVN: postfixadmin:[1099] trunk

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

Revision: 1099
          http://postfixadmin.svn.sourceforge.net/postfixadmin/?rev=1099&view=rev
Author:   christian_boltz
Date:     2011-07-20 23:19:17 +0000 (Wed, 20 Jul 2011)

Log Message:
-----------
create-admin.php
- renamed $pAdminCreate_admin_username_text to $pAdminCreate_admin_username_text_error
- renamed $pAdminCreate_admin_password_text to $pAdminCreate_admin_password_text_error
- replaced $tMessage with $infoMessage (displayed via flash_info)
- additional change compared to Dale's patch: removed unused variable 
  $pAdminCreate_admin_username_text

variables.inc.php:
- removed now unused variables $pAdminCreate_admin_password_text and
  $pAdminCreate_admin_username_text

templates/admin_create-admin.tpl:
- added 4th column for error messages
- replaced <h3> with <th>
- marked labels with <label>
- align button with input fields
- remove 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/create-admin.php
    trunk/templates/admin_create-admin.tpl
    trunk/variables.inc.php

Modified: trunk/create-admin.php
===================================================================
--- trunk/create-admin.php	2011-07-20 22:39:31 UTC (rev 1098)
+++ trunk/create-admin.php	2011-07-20 23:19:17 UTC (rev 1099)
@@ -19,7 +19,6 @@
  *
  * Template Variables:
  *
- * tMessage
  * tUsername
  * tDomains
  *
@@ -36,10 +35,11 @@
 authentication_require_role('global-admin');
 $list_domains = list_domains ();
 $tDomains = array();
+$pAdminCreate_admin_username_text_error = "";
+$pAdminCreate_admin_password_text_error = "";
 
 if ($_SERVER['REQUEST_METHOD'] == "GET")
 {
-   $pAdminCreate_admin_username_text = $PALANG['pAdminCreate_admin_username_text'];
    $tDomains = array ();
 }
 
@@ -50,18 +50,22 @@
    if (isset ($_POST['fPassword2'])) $fPassword2 = escape_string ($_POST['fPassword2']);
    $fDomains = array();
    if (!empty ($_POST['fDomains'])) $fDomains = $_POST['fDomains'];
-   list ($error, $tMessage, $pAdminCreate_admin_username_text, $pAdminCreate_admin_password_text) = create_admin($fUsername, $fPassword, $fPassword2, $fDomains);
+   list ($error, $infoMessage, $pAdminCreate_admin_username_text_error, $pAdminCreate_admin_password_text_error) = create_admin($fUsername, $fPassword, $fPassword2, $fDomains);
 
    if ($error != 0) {
       if (isset ($_POST['fUsername'])) $tUsername = escape_string ($_POST['fUsername']);
       if (isset ($_POST['fDomains'])) $tDomains = $_POST['fDomains'];
    }
+   
+   if(!empty($infoMessage))
+	flash_info($infoMessage);
+   
 }
 
 $smarty->assign ('tUsername', $tUsername);
-$smarty->assign ('pAdminCreate_admin_username_text', $pAdminCreate_admin_username_text, false);
-$smarty->assign ('pAdminCreate_admin_password_text', $pAdminCreate_admin_password_text, false);
-$smarty->assign ('tMessage', $tMessage, false);
+$smarty->assign ('pAdminCreate_admin_username_text', $PALANG['pAdminCreate_admin_username_text'], false);
+$smarty->assign ('pAdminCreate_admin_username_text_error', $pAdminCreate_admin_username_text_error, false);
+$smarty->assign ('pAdminCreate_admin_password_text_error', $pAdminCreate_admin_password_text_error, false);
 $smarty->assign ('select_options', select_options ($list_domains, $tDomains), false);
 
 $smarty->assign ('smarty_template', 'admin_create-admin');

Modified: trunk/templates/admin_create-admin.tpl
===================================================================
--- trunk/templates/admin_create-admin.tpl	2011-07-20 22:39:31 UTC (rev 1098)
+++ trunk/templates/admin_create-admin.tpl	2011-07-20 23:19:17 UTC (rev 1099)
@@ -2,36 +2,37 @@
 <form name="create_admin" method="post" action="">
 <table>
 	<tr>
-		<td colspan="3"><h3>{$PALANG.pAdminCreate_admin_welcome}</h3></td>
+		<th colspan="4">{$PALANG.pAdminCreate_admin_welcome}</th>
 	</tr>
 	<tr>
-		<td>{$PALANG.pAdminCreate_admin_username}:</td>
+		<td class="label"><label>{$PALANG.pAdminCreate_admin_username}:</label></td>
 		<td><input class="flat" type="text" name="fUsername" value="{$tUsername}" /></td>
 		<td>{$pAdminCreate_admin_username_text}</td>
+		<td><span class="error_msg">{$pAdminCreate_admin_username_text_error}</span></td>
 	</tr>
 	<tr>
-		<td>{$PALANG.pAdminCreate_admin_password}:</td>
+		<td class="label"><label>{$PALANG.pAdminCreate_admin_password}:</label></td>
 		<td><input class="flat" type="password" name="fPassword" /></td>
-		<td>{$pAdminCreate_admin_password_text}</td>
+		<td>&nbsp;</td>
+		<td><span class="error_msg">{$pAdminCreate_admin_password_text_error}</span></td>
 	</tr>
 	<tr>
-		<td>{$PALANG.pAdminCreate_admin_password2}:</td>
+		<td class="label"><label>{$PALANG.pAdminCreate_admin_password2}:</label></td>
 		<td><input class="flat" type="password" name="fPassword2" /></td>
-
+		<td colspan="2">&nbsp;</td>
 	</tr>
 	<tr>
-		<td>{$PALANG.pAdminCreate_admin_address}:</td>
+		<td class="label"><label>{$PALANG.pAdminCreate_admin_address}:</label></td>
 		<td>
 			<select name="fDomains[]" size="10" multiple="multiple">{$select_options}</select>
 		</td>
-
+		<td colspan="2">&nbsp;</td>
 	</tr>
 	<tr>
-		<td colspan="3" class="hlp_center"><input class="button" type="submit" name="submit" value="{$PALANG.pAdminCreate_admin_button}" /></td>
+		<td>&nbsp;</td>
+		<td><input class="button" type="submit" name="submit" value="{$PALANG.pAdminCreate_admin_button}" /></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-20 22:39:31 UTC (rev 1098)
+++ trunk/variables.inc.php	2011-07-20 23:19:17 UTC (rev 1099)
@@ -77,8 +77,6 @@
 $tUsername = "";
 $tTransport = "";
 
-$pAdminCreate_admin_password_text = "&nbsp;";
-$pAdminCreate_admin_username_text = "&nbsp;";
 $pAdminCreate_domain_defaultaliases_text = "&nbsp;";
 $pAdminCreate_domain_domain_text = "&nbsp;";
 $pCreate_alias_address_text = "&nbsp;";


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