Per Checkbox Kopie an Absender - Problem

[rob]

Hi!

http://dislabs.de/index.php?ac=labor&sub=5&id=16
Und ich glaub, daß dieses das das sicherste Kontaktformular ist, was es momentan gibt.

Das glaube ich zwar nicht, aber es werden einige Prüfungen durchgeführt und eventuell gefährliche Daten bereinigt.

Aber ich sehe trotzdem bei diesem Script einiges, was ich verbessern würde:

if ( (!(eregi('^[1]+@[a-z0-9_-]+.[a-z0-9_.-]+$',$email))) && (strlen($email)>0) || $email=="")

Statt mit eregi() würde ich mit preg_match() arbeiten, da dieses deutlich schneller arbeitet.
Und der reguläre Ausruck gefällt mir auch nicht so wirklich. Da solltest du vielleicht besser den einsetzen, den du vorher schon genutzt hast.

Content-Type: text/html; charset=ISO-8859-1;";

Das Script versendet HTML-Mails anstelle von reinem Text.
Ich würde das ändern. Aber vielleicht bevorzugst du ja HTML-Mails...

$pagename="formmailer.php";
header("Location:$pagename?success");

Der Location-Header benötigt einen vollständigen URL (mit http://), was hier nicht der Fall ist.

Ich habe allerdings nur einen kurzen Blick auf den Code geworfen, der auf der Website gezeigt wird. Die eigentlichen Dateien habe ich mir nicht runtergeladen. Eventuell wurde da noch was geändert, wenn ich mir die Kommentare so anschaue...

Aber nun ja...
Jetzt funktioniert alles bei dir? Keine Probleme mehr?

Schöner Gruß,
rob

---

1. a-z0-9_.- ↩︎