SF.net SVN: postfixadmin: [144] trunk/admin/create-domain.php

[<Gin...@us...>]

Revision: 144
          http://postfixadmin.svn.sourceforge.net/postfixadmin/?rev=144&view=rev
Author:   GingerDog
Date:     2007-10-09 09:53:09 -0700 (Tue, 09 Oct 2007)

Log Message:
-----------
create-domain.php: fix bug reported by ricflomag - see http://sourceforge.net/tracker/index.php?func=detail&aid=1752057&group_id=191583&atid=937964

Modified Paths:
--------------
    trunk/admin/create-domain.php

Modified: trunk/admin/create-domain.php
===================================================================
--- trunk/admin/create-domain.php	2007-10-09 16:23:09 UTC (rev 143)
+++ trunk/admin/create-domain.php	2007-10-09 16:53:09 UTC (rev 144)
@@ -57,13 +57,14 @@
 if ($_SERVER['REQUEST_METHOD'] == "POST")
 {
    if (isset ($_POST['fDomain'])) $fDomain = escape_string ($_POST['fDomain']);
-   !empty ($_POST['fDescription']) ? $fDescription = escape_string ($_POST['fDescription']) : $fDescription = "";
-   if (isset ($_POST['fAliases'])) $fAliases = escape_string ($_POST['fAliases']);
-   if (isset ($_POST['fMailboxes'])) $fMailboxes = escape_string ($_POST['fMailboxes']);
-   !empty ($_POST['fMaxquota']) ? $fMaxquota = escape_string ($_POST['fMaxquota']) : $fMaxquota = "0";
-   !empty ($_POST['fTransport']) ? $fTransport = escape_string ($_POST['fTransport']) : $fTransport = "virtual";
-   if (isset ($_POST['fDefaultaliases'])) $fDefaultaliases = escape_string ($_POST['fDefaultaliases']);
-   isset ($_POST['fBackupmx']) ? $fBackupmx = escape_string ($_POST['fBackupmx']) : $fBackupmx = "0";
+   foreach(array('fDescription' => '', 'fAliases' => '0', 'fMailboxes' => '0', 
+      'fMaxquota' => '0', 'fTransport' => 'virtual', 
+      'fDefaultaliases' => '0', 'fBackupmx' => '0') as $key => $default) {
+      if(isset($_POST[$key]) && !empty($POST[$key])) {
+         $$key = escape_string($_POST[$key]);
+      }
+      $$key = $default;
+   }
 
    if (empty ($fDomain) or domain_exist ($fDomain) or !check_domain ($fDomain))
    {
@@ -99,8 +100,8 @@
          $fBackupmx = 0;
          $sqlBackupmx = ('pgsql'==$CONF['database_type']) ? 'false' : 0;
       }
-      
-      $result = db_query ("INSERT INTO $table_domain (domain,description,aliases,mailboxes,maxquota,transport,backupmx,created,modified) VALUES ('$fDomain','$fDescription',$fAliases,$fMailboxes,$fMaxquota,'$fTransport',$sqlBackupmx,NOW(),NOW())");
+      $sql_query = "INSERT INTO $table_domain (domain,description,aliases,mailboxes,maxquota,transport,backupmx,created,modified) VALUES ('$fDomain','$fDescription',$fAliases,$fMailboxes,$fMaxquota,'$fTransport',$sqlBackupmx,NOW(),NOW())";
+      $result = db_query($sql_query);
       if ($result['rows'] != 1)
       {
          $tMessage = $PALANG['pAdminCreate_domain_result_error'] . "<br />($fDomain)<br />";


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