[PHP] Email senden

[Luckie]

Und wie mache ich das? Ich glaube, das übersteigt etwas meine PHP Kenntnisse.

[Daniel]

Ach so. Sorry.

"\n" in einem String repräsentiert einen Zeilenumbruch. Subject wird ja mehr oder weniger ein String sein und dann könntest Du zum Beispiel mit

$subject = string_replace( "\n", '', $_POST['subject'] );

solches Zeug rausfiltern.


Man kann den Spieß auch herumdrehen und über einen regulären Ausdruck alles rausnehmen, was nicht in Ordnung ist:

$subject = preg_replace( "[^a-zA-Z0-9 ]", '', $_POST['subject'] );

Das würde alles rauslöschen, was kein Buchstabe, Zahl oder Leerzeichen ist.

[alcaeus]

Hallo Daniel,

gut dass bald Wochenende ist, oder?

Zitat von Daniel:

$subject = string_replace( "\n", '', $_POST['subject'] );

Die Funktion heisst in PHP str_replace(), sprechende Namen wie "string_replace" sind eher selten

Du kannst aber auch den Titel ablehnen wenn \n oder \r im Subject vorkommen; dies kann mit strpos() ueberprueft werden (und schoen brav === bzw. !== verwenden!)

Zitat von Daniel:

$subject = preg_replace( "[^a-zA-Z0-9 ]", '', $_POST['subject'] );

Das würde alles rauslöschen, was kein Buchstabe, Zahl oder Leerzeichen ist.

Ja, wuerde es, wenn du denn die Delimiter in die Regexp eingebaut haettest

Code:

$subject = preg_replace('#[^a-zA-Z0-9 ]#i', '', $_POST['subject']);

Greetz
alcaeus

[Luckie]

Ah, besten Dank. Mit diesen Replace-Funktionen habe ich schon mal gekämpft. Ich werde es einbauen.

[Daniel]

Ach Andreas,

nu' lass' doch dem Michael auch noch was zum Knobeln.

[Luckie]

Ich hätte esd ann jetzt so:

Code:

$headers = "MIME-Version: 1.0\n".
   "Content-Type: text/plain; charset=iso-8859-1\n".
   "From: \"".$_POST['yourname']."\" <".$_POST['from'].">\n".
   "To: ".TO." <".TO.">\n".
   "Date: ".date("r")."\n";
   
  $subject = preg_replace('#[^a-zA-Z0-9 ]#i', '', $_POST['subject']);

  $msg = "Name     : ".$_POST['yourname']."\n".
         "E-Mail   : ".$_POST['from']."\n".
         "Betreff  : ".$subject."\n\n".$text;
  $msg = stripslashes($msg);

[Daniel]

Das, was Du jetzt mit der Subject-Zeile gemacht hast, solltest Du auch noch mit den anderen Daten (from und yourname) machen, weil dort das gleiche Problem existiert.

[Luckie]

OK. An was man alles denken muss.

[Luckie]

Der vollständigkeit halber:

Code:

$headers = "MIME-Version: 1.0\n".
   "Content-Type: text/plain; charset=iso-8859-1\n".
   "From: \"".$_POST['yourname']."\" <".$_POST['from'].">\n".
   "To: ".TO." <".TO.">\n".
   "Date: ".date("r")."\n";
   
  $name = preg_replace('#[^a-zA-Z0-9 ]#i', '', $_POST['yourname']);
  $from = preg_replace('#[^a-zA-Z0-9 ]#i', '', $_POST['from']);
  $subject = preg_replace('#[^a-zA-Z0-9 ]#i', '', $_POST['subject']);

  $msg = "Name     : ".$name."\n".
         "E-Mail   : ".$from."\n".
         "Betreff  : ".$subject."\n\n".$text;
  $msg = stripslashes($msg);

Halt, stopp. Jetzt wird ja auch in der Adresse (from) das @ und der Punkt rausgefiltert. Das darf natürlich nicht sein. Desweiteren wird die Absendeadresse ja hier:

Code:

if ((empty($from) || empty($text)) || (!validate_email($_POST['from'])))
  {
     echo "

Bitte füllen Sie mindestens die Felder 'E-Mail Adresse' und 'Text' aus und überprüfen Sie ihre E-Mail Adresse
     auf Gültigkeit.</p>";
  }
  else if (!mail($to, $_POST['subject'], $msg, $headers))
  {
    echo '

Es ist ein Fehler beim Senden der E-Mail aufgetreten.</p>';
  }
  else
  {
    echo '

Die E-Mail wurde erfolgreich gesendet.</p>';
  }

Code:

function validate_email($email)
  {
  
     // Create the syntactical validation regular expression
     $regexp = "^([_a-z0-9-]+)(\.[_a-z0-9-]+)*@([a-z0-9-]+)(\.[a-z0-9-]+)*(\.[a-z]{2,4})$";
  
     // Presume that the email is invalid
     $valid = 0;
  
     // Validate the syntax
     if (eregi($regexp, $email))
     {
        list($username,$domaintld) = split("@",$email);
        // Validate the domain
        if (getmxrr($domaintld,$mxrecords))
           $valid = 1;
     } else {
        $valid = 0;
     }
     return $valid;
  }

überprüft.