Auch mit Traditionen sollte man irgendwann mal brechen:

Ich habe diesen uralten regulaeren Ausdruck, den ich mal aus dem SELFHTML Forum
mitgenommen und immer mal wieder verbessert habe, durch die PHP-Funktion
filter_var() ersetzt.
Die PHP-Versionen (< 5.2.0), in denen die Filter-Funktionen nicht verfuegbar
sind, sollte sowieso niemand mehr ernsthaft benutzen.

diff --git a/form-email/mailer.php b/form-email/mailer.php
index 3adbaadf3816ad55e7264cdff8efc9358b7a980e..3c1d3ce355c22668f6a6c3d1ef9f08f2b37db41c 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
 # geschrieben wurde.\r
 #\r
 # Autor:            Patrick Canterino <patrick@patshaping.de>\r

-# Letzte Aenderung: 12.12.2011\r
+# Letzte Aenderung: 15.12.2011\r

 #\r
 # Copyright (C) 2002-2011 Patrick Canterino\r
 #\r
 #\r
 # Copyright (C) 2002-2011 Patrick Canterino\r
 #\r


@@ -84,13 +84,8 @@ if($subject_max && strlen($subject) > $subject_max) show_user_error($err_subject
 if($text_max    && strlen($text)    > $text_max)    show_user_error($err_text_long);\r
 \r
 # Pruefen, ob eingegebene E-Mail-Adresse gueltig ist\r
 if($text_max    && strlen($text)    > $text_max)    show_user_error($err_text_long);\r
 \r
 # Pruefen, ob eingegebene E-Mail-Adresse gueltig ist\r

-# Danke an "ich" aus dem SELFHTML Forum fuer die Muehe\r

 \r
 \r

-$mail_regexp = ($no_tld)\r
-             ? '/^([a-z0-9]+([\.]?|[-]?|[_]?)?)*([a-z0-9]+([\.]?[_]?)?)@([a-z0-9]+([\.]?|[-]?)?)*[a-z0-9]$/i'\r
-             : '/^([a-z0-9]+([\.]?|[-]?|[_]?)?)*([a-z0-9]+([\.]?[_]?)?)@([a-z0-9]+([\.]?|[-]?)?)*[a-z0-9]+\.{1}[a-z]{2,6}$/i';\r
-\r
-if(!preg_match($mail_regexp,$email)) show_user_error($err_invalid_email);\r
+if(!filter_var($email,FILTER_VALIDATE_EMAIL)) show_user_error($err_invalid_email);\r

 \r
 # Individuelle Felder einlesen\r
 \r
 \r
 # Individuelle Felder einlesen\r
 \r