Pruefen, ob $user_field_data['optional'] korrekt gesetzt wurde

[form-email.git] / form-email / mailer.php

diff --git a/form-email/mailer.php b/form-email/mailer.php
index 3c1d3ce355c22668f6a6c3d1ef9f08f2b37db41c..1156d30a70981216d38824ef4ae7c780a30967eb 100644 (file)
--- a/form-email/mailer.php
+++ b/form-email/mailer.php
@@ -7,7 +7,7 @@
 # geschrieben wurde.\r
 #\r
 # Autor:            Patrick Canterino <patrick@patshaping.de>\r
-# Letzte Aenderung: 15.12.2011\r
+# Letzte Aenderung: 18.12.2011\r
 #\r
 # Copyright (C) 2002-2011 Patrick Canterino\r
 #\r
@@ -93,22 +93,22 @@ $prepared_user_fields = array();
 \r
 while(list($user_field,$user_field_data) = each($user_fields)) {\r
     $user_field_content = formdata($user_field);\r
-    \r
+\r
     # Pruefung, ob das Feld gesetzt wurde und ob es optional ist\r
-    \r
-    if((!$user_field_data['optional']) && (!$user_field_content)) {\r
+\r
+    if(!(isset($user_field_data['optional']) && $user_field_data['optional']) && (!$user_field_content)) {\r
         show_user_error($user_field_data['errors']['not_set']);\r
     }\r
-    elseif($user_field_data['optional'] && (!$user_field_content)) {\r
+    elseif(isset($user_field_data['optional']) && $user_field_data['optional'] && (!$user_field_content)) {\r
         # Wenn das Feld optional ist und nicht gesetzt wurde, brechen wir hier ab\r
         # und machen mit dem naechsten Feld weiter\r
-        \r
+\r
         $prepared_user_fields[$user_field_data['tpl_var']] = '';\r
         continue;\r
     }\r
-    \r
+\r
     # Laengen-Pruefung\r
-    \r
+\r
     if(isset($user_field_data['min']) && $user_field_data['min']) {\r
         if(strlen($user_field_content) < $user_field_data['min']) {\r
             if(isset($user_field_data['errors']['too_short'])) {\r
@@ -119,7 +119,7 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
             }\r
         }\r
     }\r
-    \r
+\r
     if(isset($user_field_data['min']) && $user_field_data['max']) {\r
         if(strlen($user_field_content) > $user_field_data['max']) {\r
             if(isset($user_field_data['errors']['too_long'])) {\r
@@ -130,7 +130,7 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
             }\r
         }\r
     }\r
-    \r
+\r
     # Pruefung gegen frei definierbare Pruef-Funktion\r
 \r
     if(isset($user_field_data['check']) && $user_field_data['check'] != '') {\r
@@ -143,7 +143,7 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
             }\r
         }\r
     }\r
-    \r
+\r
     # Pruefung gegen frei definierbaren regulaeren Ausdruck\r
 \r
     if(isset($user_field_data['regex']) && $user_field_data['regex'] != '') {\r
@@ -156,7 +156,7 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
             }\r
         }\r
     }\r
-    \r
+\r
     # Formularfeld durch frei definierbare Funktion bearbeiten\r
 \r
     if(isset($user_field_data['prepare']) && $user_field_data['prepare'] != '') {\r
@@ -164,7 +164,7 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
             $user_field_content = call_user_func($user_field_data['prepare'],$user_field_content);\r
         }\r
     }\r
-    \r
+\r
     # Name der Template-Variable ermitteln\r
 \r
     if(isset($user_field_data['tpl_var']) && $user_field_data['tpl_var'] != '') {\r
@@ -173,7 +173,7 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
     else {\r
         $tpl_var = 'USER_'.$user_field;\r
     }\r
-    \r
+\r
     # Fertige Formulardaten wegkopieren und fuer spaeter zur Verfuegung stellen\r
 \r
     $prepared_user_fields[$tpl_var] = $user_field_content;\r
@@ -192,7 +192,7 @@ if($captcha_enable) {
         else {\r
             if(!$captcha_input || empty($captcha_input) || strtolower($captcha_input) != strtolower($_SESSION['captcha'])) $captcha_incorrect = true;\r
         }\r
-  \r
+\r
         if($captcha_incorrect) {\r
             if($captcha_max) {\r
                 if(isset($_SESSION['captcha_failed']) && $_SESSION['captcha_failed']) {\r
@@ -206,7 +206,7 @@ if($captcha_enable) {
             if($captcha_max && isset($_SESSION['captcha_failed']) && $_SESSION['captcha_failed'] >= $captcha_max) {\r
                 show_fatal($err_captcha_max);\r
             }\r
-            else {   \r
+            else {\r
                 show_user_error($err_captcha_incorrect);\r
             }\r
         }\r
@@ -348,14 +348,15 @@ if($captcha_enable) session_destroy();
 if($return_url) {\r
     # Per Redirect auf Seite umleiten\r
 \r
-    if(!preg_match('!://!',$return_url)) {\r
+    if(strpos($return_url,'://') === false) {\r
         # URL vervollstaendigen\r
 \r
         $http_host = ($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : $_SERVER['SERVER_NAME'];\r
         $protocol  = '';\r
         $port      = '';\r
+        $path      = '';\r
 \r
-        if($_SERVER['HTTPS']) {\r
+        if(isset($_SERVER['HTTPS'])) {\r
             # SSL-verschluesseltes HTTP\r
 \r
             $protocol = 'https://';\r
@@ -368,13 +369,19 @@ if($return_url) {
             if($_SERVER['SERVER_PORT'] != 80) $port = ':'.$_SERVER['SERVER_PORT'];\r
         }\r
 \r
-        if(preg_match('!^/!',$return_url)) {\r
+        if(substr($return_url,0,1) == '/') {\r
             $return_url = $protocol.$http_host.$port.$return_url;\r
         }\r
         else {\r
-            if(preg_match('!/$!',$_SERVER['SCRIPT_NAME'])) $path = substr($_SERVER['SCRIPT_NAME'],0,-1);\r
+            if(substr($_SERVER['SCRIPT_NAME'],-1,1) == '/') {\r
+                $path = substr($_SERVER['SCRIPT_NAME'],0,-1);\r
+            }\r
+            else {\r
+                $path = $_SERVER['SCRIPT_NAME'];\r
+            }\r
+\r
             $path = substr($path,0,strrpos($path,'/')+1);\r
-            if(!preg_match('!^/!',$path)) $path = '/'.$path;\r
+            if(substr($path,0,1) != '/') $path = '/'.$path;\r
 \r
             $return_url = $protocol.$http_host.$port.$path.$return_url;\r
         }\r