From: Patrick Canterino <patrick@patshaping.de>
Date: Mon, 12 Dec 2011 20:51:18 +0000 (+0000)
Subject: Bevor eine Fehlermeldung fuer ein individuelles Formularfeld ausgegeben wird,
X-Git-Tag: 3.5~49
X-Git-Url: https://git.p6c8.net/form-email.git/commitdiff_plain/c715b5507a92eea0180b640db399e0144e100bf6

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 a3af2a1..a4208e9 100644
--- a/form-email/config.php
+++ b/form-email/config.php
@@ -6,7 +6,7 @@
 # Konfiguration fuer Form E-Mail
 #
 # Autor:            Patrick Canterino <patrick@patshaping.de>
-# Letzte Aenderung: 2.12.2011
+# Letzte Aenderung: 12.12.2011
 #
 # Copyright (C) 2002-2011 Patrick Canterino
 #
@@ -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.';
 $err_referer_not_ok = 'Der im HTTP-Referer angegebene Host darf keine Mails versenden.';
 
+$err_indiv_errmsg_miss = 'Die Fehlermeldung <em>{ERRMSG}</em> für das individuelle Formularfeld <em>{INDIVIDUAL}</em> wurde nicht definiert!';
+
 $err_captcha_incorrect       = 'Der eingegebene Sicherheitscode ist nicht korrekt!';
 $err_captcha_max             = 'Sie haben den Sicherheitscode zu oft falsch eingegeben!';
 $err_captcha_disabled        = 'Captchas sind nicht aktiviert!';
diff --git a/form-email/functions.php b/form-email/functions.php
index d8d6f71..2374b7b 100644
--- a/form-email/functions.php
+++ b/form-email/functions.php
@@ -7,7 +7,7 @@
 # geschrieben wurde.
 #
 # Autor:            Patrick Canterino <patrick@patshaping.de>
-# Letzte Aenderung: 3.12.2011
+# Letzte Aenderung: 12.12.2011
 #
 # Copyright (C) 2002-2011 Patrick Canterino
 #
@@ -103,13 +103,21 @@ function semicolon_split($string) {
 #
 # Parameter: Fehlermeldung
 
-function show_fatal(&$errmsg) {
+function show_fatal(&$errmsg,$vars=array()) {
     global $tpl_fatal;
 
     $tpl = new Template;
     $tpl->read_file($tpl_fatal);
 
-    $tpl->fillin('ERROR',$errmsg);
+    if(sizeof($vars) > 0) {
+        while(list($var,$content) = each($vars)) {
+            $errmsg = str_replace('{'.htmlspecialchars($var).'}',htmlspecialchars($content),$errmsg);
+        }
+    }
+
+    $tpl->set_var('ERROR',$errmsg);
+
+    $tpl->parse();
 
     print $tpl->get_template();
     exit;
diff --git a/form-email/mailer.php b/form-email/mailer.php
index 11061d4..ff43112 100644
--- a/form-email/mailer.php
+++ b/form-email/mailer.php
@@ -7,7 +7,7 @@
 # geschrieben wurde.
 #
 # Autor:            Patrick Canterino <patrick@patshaping.de>
-# Letzte Aenderung: 4.12.2011
+# Letzte Aenderung: 12.12.2011
 #
 # Copyright (C) 2002-2011 Patrick Canterino
 #
@@ -116,13 +116,23 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
     
     if(isset($user_field_data['min']) && $user_field_data['min']) {
         if(strlen($user_field_content) < $user_field_data['min']) {
-            show_user_error($user_field_data['errors']['too_short']);
+            if(isset($user_field_data['errors']['too_short'])) {
+                show_user_error($user_field_data['errors']['too_short']);
+            }
+            else {
+                show_fatal($err_indiv_errmsg_miss,array('ERRMSG' => 'too_short', 'INDIVIDUAL' => $user_field));
+            }
         }
     }
     
     if(isset($user_field_data['min']) && $user_field_data['max']) {
         if(strlen($user_field_content) > $user_field_data['max']) {
-            show_user_error($user_field_data['errors']['too_long']);
+            if(isset($user_field_data['errors']['too_long'])) {
+                show_user_error($user_field_data['errors']['too_long']);
+            }
+            else {
+                show_fatal($err_indiv_errmsg_miss,array('ERRMSG' => 'too_long', 'INDIVIDUAL' => $user_field));
+            }
         }
     }
     
@@ -130,7 +140,12 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
 
     if(isset($user_field_data['check']) && $user_field_data['check'] != '') {
         if(function_exists($user_field_data['check']) && call_user_func($user_field_data['check'],$user_field_content) === false) {
-            show_user_error($user_field_data['errors']['check_fail']);
+            if(isset($user_field_data['errors']['check_fail'])) {
+                show_user_error($user_field_data['errors']['check_fail']);
+            }
+            else {
+                show_fatal($err_indiv_errmsg_miss,array('ERRMSG' => 'check_fail', 'INDIVIDUAL' => $user_field));
+            }
         }
     }
     
@@ -138,7 +153,12 @@ while(list($user_field,$user_field_data) = each($user_fields)) {
 
     if(isset($user_field_data['regex']) && $user_field_data['regex'] != '') {
         if(!preg_match($user_field_data['regex'],$user_field_content)) {
-            show_user_error($user_field_data['errors']['match_fail']);
+            if(isset($user_field_data['errors']['match_fail'])) {
+                show_user_error($user_field_data['errors']['match_fail']);
+            }
+            else {
+                show_fatal($err_indiv_errmsg_miss,array('ERRMSG' => 'match_fail', 'INDIVIDUAL' => $user_field));
+            }
         }
     }