Changeset 12

Timestamp:

11/12/07 12:17:35 (10 months ago)

Author:

david

Message:

fixes to ensure variables are defined

Location:

trunk

Files:

• postfixadmin_forward.php (modified) (2 diffs)
• postfixadmin_vacation.php (modified) (2 diffs)

trunk/postfixadmin_forward.php

@@ -30 +30 @@
 }
 
+$tMessage = '';
 
 include_once (SM_PATH . 'plugins/postfixadmin/config.php');
@@ -104 +105 @@
     $array = preg_split ('/,/', $goto);
 
+    $error = 0;
     // array should be a list of valid email addresses...
     foreach($array as $email_address) 

trunk/postfixadmin_vacation.php

@@ -92 +92 @@
     else
     {
+        $tMessage = '';
         bindtextdomain('postfixadmin', SM_PATH . 'plugins/postfixadmin/locale');
         textdomain('postfixadmin');
@@ -126 +127 @@
 if ($_SERVER['REQUEST_METHOD'] == "POST")
 {
-    $fSubject = db_escape($_POST['fSubject']);
-    $fBody = db_escape($_POST['fBody']);
-    $fAway = db_escape($_POST['fAway']);
-    $fBack = db_escape($_POST['fBack']);
+    $fBack = null;
+    $fAway = null;
+    foreach(array('fBack', 'fAway', 'fSubject', 'fBody') as $key) {
+        $$key = null;
+        if(isset($_POST[$key])) {
+            $$key = db_escape($_POST[$key]);
+        }
+    }
+    
     $USERID_USERNAME = db_escape($USERID_USERNAME);
     $USERID_DOMAIN = db_escape($USERID_DOMAIN);