Formailer

[Gehirndoktor]

Also ich hab ein Problem bei dem Formailer denn ich geschrieben habe, denn er gibt immer beide Meldungen aus

//Mail Kontrolle
if($abschicken)
{
if(strlen($homepage)==0){
$error_msg.="Bitte geben sie ihre Homepage Adresse ein.<br>";
}
if(strlen($box)==0){
$error_msg.="Bitte geben sie ihre Box4Promo ID ein.<br>";
}
if($error_msg){
//Falls einer der Felder nicht ausgefüllt wurde
echo "Ihre Promo Bestellung konnte aus folgendem Grund nicht abgeschickt werden:<br><br>";
echo $error_msg;
}

//Mail von Box4Promo
$recipient="xxx@xxx.de";
$subject="Promo Credits";
$header="From: " . $emailadresse . "\n";
$mail_body ="Das Anfrage-Formular wurde am " . date("d.m.Y") . " um " . date("H:i") . "h ausgefuehrt.\n";
$mail_body.="Folgende Werte wurden eingetragen:\n\n";
$mail_body.="ID: " . $user[1] . "\n";
$mail_body.="Homepage: " . $homepage . "\n";
$mail_body.="Box4Promo ID: " . $box . "\n";
#
mail($recipient,$subject,$mail_body,$header);

//Formular ausgeführt - Meldung ausgeben
$post.="<b>Anmeldung bei Box4Promo.de</b><br><br>Unser Team wird jetzt ihre Anmeldung bei Box4Promo.de überprüfen. Falls alle Daten korrekt eingegeben wurden und der HTML Code auf ihrer Seite eingebunden ist, erhalten sie die Credits innerhalb von 48 Stunden auf ihr Konto überwiesen.";
}else{
$post.="Hier kannst du durch eine Anmeldung bei einem unserer Partner Promos Verdienen, die nach der überprüfung auf dein Konto gutgeschrieben werden.<br><br>";

weiter das formular

[BloodReaver]

hm...du machsd ja immer ne neue if abfrage!

du mussd das entweder so mache:

if(...)
{

}
else
{

} 

oder halt

if(...)
{

}
elseif(...)
{

}
else
{

} 

würd ich et mal so ganz spontan und angetrunken sagen...


MfG Blood

[martin_de]

es würde auch schon ausreichen das skript durch ein exit; zu beenden wenn ein fehler auftritt.

aber auf formulardaten geift man per $_POST['feldname'] oder $_GET['feldname'] zu - je nachdem, ob dus per POST oder GET überträgst.

du solltest also z.b. nicht if ($homepage == blalba...) verwenden, sondern if ($_POST['homepage'] == blalbla)

desweiteren wäre es dringend notwendig zu überprüfen, ob die angaben dem notwendigen format entsprechen und da nicht irgendein müll angegeben wurde. in verbindung mit sicherheitsproblemen einiger php versionen in der mail() funktion kann sowas schnell übel ausgehen.