Bevor eine Fehlermeldung fuer ein individuelles Formularfeld ausgegeben wird,

wird geprueft, ob die Fehlermeldung ueberhaupt definiert wurde.
Mir gefaellt die Loesung nicht, sollte man vielleicht mal ueberarbeiten (mach
ich dann aber sowieso nicht...).

diff --git a/form-email/config.php b/form-email/config.php
index a3af2a10c9b89426097e6df36b589b46cc0850c5..a4208e97590f66c18d4ec2a6e770383fd99fdbde 100644 (file)
--- a/form-email/config.php
+++ b/form-email/config.php
@@ -6,7 +6,7 @@
 # Konfiguration fuer Form E-Mail\r
 #\r
 # Autor:            Patrick Canterino <patrick@patshaping.de>\r
-# Letzte Aenderung: 2.12.2011\r
+# Letzte Aenderung: 12.12.2011\r
 #\r
 # Copyright (C) 2002-2011 Patrick Canterino\r
 #\r
@@ -120,6 +120,8 @@ $err_send_error     = 'Beim Versenden der Mail ist ein Fehler aufgetreten!';
 $err_only_post      = 'Dieser Form-Mailer kann nur mit POST aufgerufen werden.';\r
 $err_referer_not_ok = 'Der im HTTP-Referer angegebene Host darf keine Mails versenden.';\r
 \r
+$err_indiv_errmsg_miss = 'Die Fehlermeldung <em>{ERRMSG}</em> für das individuelle Formularfeld <em>{INDIVIDUAL}</em> wurde nicht definiert!';\r
+\r
 $err_captcha_incorrect       = 'Der eingegebene Sicherheitscode ist nicht korrekt!';\r
 $err_captcha_max             = 'Sie haben den Sicherheitscode zu oft falsch eingegeben!';\r
 $err_captcha_disabled        = 'Captchas sind nicht aktiviert!';\r

diff --git a/form-email/functions.php b/form-email/functions.php
index d8d6f71773f93980ca5433bab4a09138aed03d18..2374b7bcc6066fd1338605927b679cdbcaa7420e 100644 (file)
--- a/form-email/functions.php
+++ b/form-email/functions.php
@@ -7,7 +7,7 @@
 # geschrieben wurde.\r
 #\r
 # Autor:            Patrick Canterino <patrick@patshaping.de>\r
-# Letzte Aenderung: 3.12.2011\r
+# Letzte Aenderung: 12.12.2011\r
 #\r
 # Copyright (C) 2002-2011 Patrick Canterino\r
 #\r
@@ -103,13 +103,21 @@ function semicolon_split($string) {
 #\r
 # Parameter: Fehlermeldung\r
 \r
-function show_fatal(&$errmsg) {\r
+function show_fatal(&$errmsg,$vars=array()) {\r
     global $tpl_fatal;\r
 \r
     $tpl = new Template;\r
     $tpl->read_file($tpl_fatal);\r
 \r
-    $tpl->fillin('ERROR',$errmsg);\r
+    if(sizeof($vars) > 0) {\r
+        while(list($var,$content) = each($vars)) {\r
+            $errmsg = str_replace('{'.htmlspecialchars($var).'}',htmlspecialchars($content),$errmsg);\r
+        }\r
+    }\r
+\r
+    $tpl->set_var('ERROR',$errmsg);\r
+\r
+    $tpl->parse();\r
 \r
     print $tpl->get_template();\r
     exit;\r

diff --git a/form-email/mailer.php b/form-email/mailer.php
index 11061d40f811254fd25681393b470f7ac454ba65..ff431129b89265199abe1236fb219d987436d8c7 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: 4.12.2011\r
+# Letzte Aenderung: 12.12.2011\r
 #\r
 # Copyright (C) 2002-2011 Patrick Canterino\r
 #\r
@@ -116,13 +116,23 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
     \r
     if(isset($user_field_data['min']) && $user_field_data['min']) {\r
         if(strlen($user_field_content) < $user_field_data['min']) {\r
-            show_user_error($user_field_data['errors']['too_short']);\r
+            if(isset($user_field_data['errors']['too_short'])) {\r
+                show_user_error($user_field_data['errors']['too_short']);\r
+            }\r
+            else {\r
+                show_fatal($err_indiv_errmsg_miss,array('ERRMSG' => 'too_short', 'INDIVIDUAL' => $user_field));\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
-            show_user_error($user_field_data['errors']['too_long']);\r
+            if(isset($user_field_data['errors']['too_long'])) {\r
+                show_user_error($user_field_data['errors']['too_long']);\r
+            }\r
+            else {\r
+                show_fatal($err_indiv_errmsg_miss,array('ERRMSG' => 'too_long', 'INDIVIDUAL' => $user_field));\r
+            }\r
         }\r
     }\r
     \r
@@ -130,7 +140,12 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
 \r
     if(isset($user_field_data['check']) && $user_field_data['check'] != '') {\r
         if(function_exists($user_field_data['check']) && call_user_func($user_field_data['check'],$user_field_content) === false) {\r
-            show_user_error($user_field_data['errors']['check_fail']);\r
+            if(isset($user_field_data['errors']['check_fail'])) {\r
+                show_user_error($user_field_data['errors']['check_fail']);\r
+            }\r
+            else {\r
+                show_fatal($err_indiv_errmsg_miss,array('ERRMSG' => 'check_fail', 'INDIVIDUAL' => $user_field));\r
+            }\r
         }\r
     }\r
     \r
@@ -138,7 +153,12 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
 \r
     if(isset($user_field_data['regex']) && $user_field_data['regex'] != '') {\r
         if(!preg_match($user_field_data['regex'],$user_field_content)) {\r
-            show_user_error($user_field_data['errors']['match_fail']);\r
+            if(isset($user_field_data['errors']['match_fail'])) {\r
+                show_user_error($user_field_data['errors']['match_fail']);\r
+            }\r
+            else {\r
+                show_fatal($err_indiv_errmsg_miss,array('ERRMSG' => 'match_fail', 'INDIVIDUAL' => $user_field));\r
+            }\r
         }\r
     }\r
     \r