PHP -Kontaktformular-Script Fehler

funnie01

Hallo,

bin ein absoluter Neuling und wollte folgendes php-Kontaktformular-script in meine HP einbinden.

PHP

<?php


$mail_to="Sebastian-...";


$from_name=$_POST['fromname'];
$from_mail=strtolower($_POST['frommail']);
$mail_subject=$_POST['mailsubject'];
$mail_text=$_POST['mailtext'];
$send=$_POST['s'];


//*** Dieser Text soll u.a. unter der Textbox erscheinen***


if(trim($from_name)=="") $err_text.="Bitte Namen angeben.
";
if(trim($from_mail)=="") $err_text.="Bitte E-Mail-Adresse angeben.
";
else
  if(!ereg("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,6})$",$from_mail))
    $err_text.="Bitte eine gültige E-Mail-Adresse angeben.
";
if(trim($mail_subject)=="") $err_text.="Bitte Betreff eingeben.
";
if(trim($mail_text)=="") $err_text.="Bitte Nachrichtentext eingeben.
";




if(strlen($mail_text)>1000) {
  $mail_text=substr($mail_text,0,1000)."... (Text wurde gekürzt!)";
}




$from_name=str_replace(chr(34),"''",$from_name);
$mail_subject=str_replace(chr(34),"''",$mail_subject);
$from_name=stripslashes($from_name);
$from_mail=stripslashes($from_mail);
$mail_subject=stripslashes($mail_subject);
$mail_text=stripslashes($mail_text);


if(($send=="1") && (isset($err_text))) {
  echo "


[b]Fehler:[/b]
";
  echo "$err_text</p>";
}


if(($send!="1") || (isset($err_text)))
{
?>


<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<table border=0 cellspacing=2 cellpadding=0 width=100%>
<tr><td nowrap align=right class="Stil6">Dein Name:</td><td></td>
<td width=100%><input type="text" name="fromname" size=25 maxlength=120 value="<?php echo $from_name; ?>"></td></tr>
<tr><td nowrap align=right class="Stil6">E-Mail Adresse:</td><td></td>
<td width=100%><input type="text" name="frommail" size=25 maxlength=120 value="<?php echo $from_mail; ?>"></td></tr>
<tr><td nowrap align=right class="Stil6">Betreff:</td><td></td>
<td width=100%><input type="text" name="mailsubject" size=25 maxlength=120 value="<?php echo $mail_subject; ?>"></td></tr>
<tr><td nowrap align=right valign=top class="Stil6">Textmitteilung:
[i](max. 1.000 Zeichen)[/i]</td><td></td>
<td width=100%><textarea cols="40" rows="8" name="mailtext"><?php echo $mail_text; ?></textarea></td></tr>
<tr><td></td><td></td><td>
<input type="hidden" value="1" name="s">
<input type="submit" value="Nachricht versenden" name="submit">
</td></tr>
</table>
</form>


<?php
} else {
  $header="From: $from_name <$from_mail>\n";
  $header.="Reply-To: $from_mail\n";
  $header.="X-Mailer: PHP-FormMailer \n";
  $header.="Content-Type: text/plain";
  $mail_date=gmdate("D, d M Y H:i:s")." +0000";
  $send=0;
  if(@mail($mail_to,$mail_subject,$mail_text,$header))
  {
    echo "


[b]Deine E-Mail wurde abgesendet.[/b]</p>";
    echo "


<a href=\"".$_SERVER['PHP_SELF']."?from_name=$from_name&from_mail=$from_mail\"> [b] [Zurück] [/b] </a></p>";
  }else{
    echo "


[b]Beim Versenden der E-Mail ist ein Fehler aufgetreten![/b]</p>";
    echo "


<a href=\"".$_SERVER['PHP_SELF']."?from_name=$from_name&from_mail=$from_mail&mail_subject=$mail_subject&mail_text=";
    echo urlencode($mail_text)."\"> <stong> [Zurück] [/b] </a></p>";
  }
}
?>

Alles anzeigen

Da ich keine Ahnung von diesem PHP-Script habe würde ich jetzt ganz gerne wissen, wie ich

(1) Fehlermeldungen anstatt über den Kontaktfeldern unter dem Textfeld anzeigen lassen kann
(2) Fehlermeldungen wie "Bitte eine gültige E-Mail-Adresse angeben!" ohne Zeilenumbruch anzeigen lassen kann
(3) den Fehlerhinweis ($err_text.) rot formatieren kann ("Fehler:" soll rot sein)
(4) einzelne Textpassagen in den Meldungen formatieren kann z.B. spezielle echo-Texte.

Wenn mir jemand mit meinem kleinen Problem weiterhelfen könnte, wäre ich dafür äusserst dankbar.

Wer das Problem in der Realität erleben möchte, kann es sich unter http://www.sl-x.info/Kontakt.php ansehen, dann wird auch deutlich warum ich Hilfe benötige.

Vielen Dank im Voraus!

yngvar

Was für Textpassagen willst du ändern können?

Hier der restliche Code angepasst auf deinen Zweck:

PHP

<?php


$mail_to="Sebastian-...";


$from_name=$_POST['fromname'];
$from_mail=strtolower($_POST['frommail']);
$mail_subject=$_POST['mailsubject'];
$mail_text=$_POST['mailtext'];
$send=$_POST['s'];


//*** Dieser Text soll u.a. unter der Textbox erscheinen***


if(trim($from_name)=="") $err_text.="Bitte Namen angeben.
";
if(trim($from_mail)=="") $err_text.="Bitte E-Mail-Adresse angeben.
";
else
  if(!ereg("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,6})$",$from_mail))
    $err_text.="Bitte eine gültige E-Mail-Adresse angeben.
";
if(trim($mail_subject)=="") $err_text.="Bitte Betreff eingeben.
";
if(trim($mail_text)=="") $err_text.="Bitte Nachrichtentext eingeben.
";




if(strlen($mail_text)>1000) {
  $mail_text=substr($mail_text,0,1000)."... (Text wurde gekürzt!)";
}




$from_name=str_replace(chr(34),"''",$from_name);
$mail_subject=str_replace(chr(34),"''",$mail_subject);
$from_name=stripslashes($from_name);
$from_mail=stripslashes($from_mail);
$mail_subject=stripslashes($mail_subject);
$mail_text=stripslashes($mail_text);


if(($send=="1") && (isset($err_text))) {
  $err_text = "


[b]<font color=\"red\">Fehler:</font>[/b] ".$err_text."</p>";
}


if(($send!="1") || (isset($err_text)))
{
?>


<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<table border=0 cellspacing=2 cellpadding=0 width=100%>
<tr><td nowrap align=right class="Stil6">Dein Name:</td><td></td>
<td width=100%><input type="text" name="fromname" size=25 maxlength=120 value="<?php echo $from_name; ?>"></td></tr>
<tr><td nowrap align=right class="Stil6">E-Mail Adresse:</td><td></td>
<td width=100%><input type="text" name="frommail" size=25 maxlength=120 value="<?php echo $from_mail; ?>"></td></tr>
<tr><td nowrap align=right class="Stil6">Betreff:</td><td></td>
<td width=100%><input type="text" name="mailsubject" size=25 maxlength=120 value="<?php echo $mail_subject; ?>"></td></tr>
<tr><td nowrap align=right valign=top class="Stil6">Textmitteilung:
[i](max. 1.000 Zeichen)[/i]</td><td></td>
<td width=100%><textarea cols="40" rows="8" name="mailtext"><?php echo $mail_text; ?></textarea></td></tr>
<tr><td></td><td></td><td>
<input type="hidden" value="1" name="s">
<input type="submit" value="Nachricht versenden" name="submit">
</td></tr>
</table>
</form>


<?php
echo $err_text;
} else {
  $header="From: $from_name <$from_mail>\n";
  $header.="Reply-To: $from_mail\n";
  $header.="X-Mailer: PHP-FormMailer \n";
  $header.="Content-Type: text/plain";
  $mail_date=gmdate("D, d M Y H:i:s")." +0000";
  $send=0;
  if(@mail($mail_to,$mail_subject,$mail_text,$header))
  {
    echo "


[b]Deine E-Mail wurde abgesendet.[/b]</p>";
    echo "


<a href=\"".$_SERVER['PHP_SELF']."?from_name=$from_name&from_mail=$from_mail\"> [b] [Zurück] [/b] </a></p>";
  }else{
    echo "


[b]Beim Versenden der E-Mail ist ein Fehler aufgetreten![/b]</p>";
    echo "


<a href=\"".$_SERVER['PHP_SELF']."?from_name=$from_name&from_mail=$from_mail&mail_subject=$mail_subject&mail_text=";
    echo urlencode($mail_text)."\"> <stong> [Zurück] [/b] </a></p>";
  }
}
?>

Alles anzeigen

funnie01

Ähm, irgendwie bekomme ich jetzt von vornherein alle Fehler angezeigt...

Sieht man von diesem Missstand ab, steht jetzt die Fehlermeldung wie gewünscht unter der Textbox ... und der Text "Fehler:" wird rot dargestellt. Super!

Aber: Wie bekomme ich es hin, dass mir in der Fehlermeldung kein Zeilenumbruch eingebaut wird...

Bsp. Bei der Eingabe einer ungültigen E-Mail Adresse kommt der Text:

"Bitte eine gültige E-Mail Adresse eingeben!"

Diese Zeile wird aber aus irgendwelchen Gründen nach "Adresse" gebrochen.

Diesen Fehler in der Anzeige möchte ich vermeiden.

Ferner wäre es schön, wenn ich mir einzelne Anzeigetexte farblich individuell formatieren könnte (z.B. die verschiedenen Fehlermeldungen). Die Html-Textformatierung scheint da nicht zu wirken.

Vielen Dank für eure Hilfe!

P.s. Zur Veranschaulichung der Problematik unter http://www.sl-x.info/Kontakt.php einfach eine falsche E-Mail Adresse eingeben!

yngvar

Hier nochmal der Code:

PHP

<?php


$mail_to="Sebastian-...";


$from_name=$_POST['fromname'];
$from_mail=strtolower($_POST['frommail']);
$mail_subject=$_POST['mailsubject'];
$mail_text=$_POST['mailtext'];
$send=$_POST['s'];




if(trim($from_name)=="") $err_text.="  Bitte Namen angeben.
\n";
if(trim($from_mail)=="") $err_text.="  Bitte E-Mail-Adresse angeben.
\n";
else
  if(!ereg("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,6})$",$from_mail))
    $err_text.="  Bitte eine gültige E-Mail-Adresse angeben.
\n";
if(trim($mail_subject)=="") $err_text.="  Bitte Betreff eingeben.
\n";
if(trim($mail_text)=="") $err_text.="  Bitte Nachrichtentext eingeben.
\n";




if(strlen($mail_text)>1000) {
  $mail_text=substr($mail_text,0,1000)."... (Text wurde gekürzt!)";
}




$from_name=str_replace(chr(34),"''",$from_name);
$mail_subject=str_replace(chr(34),"''",$mail_subject);
$from_name=stripslashes($from_name);
$from_mail=stripslashes($from_mail);
$mail_subject=stripslashes($mail_subject);
$mail_text=stripslashes($mail_text);


if(($send=="1") && (isset($err_text))) {
  $err_text = "


\n  [b]<font color=\"red\">Fehler:</font>[/b]
\n".$err_text."</p>\n";
}


if(($send!="1") || (isset($err_text)))
{
?>


<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<table border=0 cellspacing=2 cellpadding=0 width=100%>
  <tr>
    <td nowrap align=right class="Stil6">Dein Name:</td>
    <td></td>
    <td width=100%><input type="text" name="fromname" size=25 maxlength=120 value="<?php echo $from_name; ?>"></td>
  </tr>
  <tr>
    <td nowrap align=right class="Stil6">E-Mail Adresse:</td>
    <td></td>
    <td width=100%><input type="text" name="frommail" size=25 maxlength=120 value="<?php echo $from_mail; ?>"></td>
  </tr>
  <tr>
    <td nowrap align=right class="Stil6">Betreff:</td>
    <td></td>
    <td width=100%><input type="text" name="mailsubject" size=25 maxlength=120 value="<?php echo $mail_subject; ?>"></td>
  </tr>
  <tr>
    <td nowrap align=right valign=top class="Stil6">Textmitteilung:
[i](max. 1.000 Zeichen)[/i]</td>
    <td></td>
    <td width=100%><textarea cols="40" rows="8" name="mailtext"><?php echo $mail_text; ?></textarea></td>
  </tr>
  <tr>
    <td></td>
    <td></td>
    <td>
      <input type="hidden" value="1" name="s">
      <input type="submit" value="Nachricht versenden" name="submit">
    </td>
  </tr>
</table>
</form>


<?php
if ($send == 1) echo $err_text;
} else {
  $header="From: $from_name <$from_mail>\n";
  $header.="Reply-To: $from_mail\n";
  $header.="X-Mailer: PHP-FormMailer \n";
  $header.="Content-Type: text/plain";
  $mail_date=gmdate("D, d M Y H:i:s")." +0000";
  $send=0;
  if(@mail($mail_to,$mail_subject,$mail_text,$header))
  {
    echo "


[b]Deine E-Mail wurde abgesendet.[/b]</p>";
    echo "


<a href=\"".$_SERVER['PHP_SELF']."?from_name=$from_name&from_mail=$from_mail\"> [b] [Zurück] [/b] </a></p>";
  }else{
    echo "


[b]Beim Versenden der E-Mail ist ein Fehler aufgetreten![/b]</p>";
    echo "


<a href=\"".$_SERVER['PHP_SELF']."?from_name=$from_name&from_mail=$from_mail&mail_subject=$mail_subject&mail_text=";
    echo urlencode($mail_text)."\"> <stong> [Zurück] [/b] </a></p>";
  }
}
?>

Alles anzeigen

Die individiuelle Formatierung musst du schon selbst machen. Suche in der Datei einfach den Text den du bearbeiten willst und bearbeite ihn. Achte dabei darauf, wenn du <a href="LINK... schreibst, dass du <a href=\"LINK... schreibst.

Desweiteren kannste dir hier die Datei mal anschauen: http://yngvarlap.no-ip.biz/forum-hilfe_001.php

yngvar

Woran es liegt, dass auf deiner Seite ein Zeilenumbruch an dieser Stelle gemacht wird liegt an folgendem:

Ein paar Zeile bevor du den PHP-Code einfügst steht folgendes:

Code

<div id="Layer7" style="position: absolute; width: 200px; height: 115px; z-index: 17; left: 250px; top: 222px;" class="Stil6">

Der Parameter width=200px muss mehr werden, stell ihn auf 500 oder sowas in der Art.

Gruß Heinrich

funnie01

Dein Code geht auf... nur der verdammte Zeilenumbruch bleibt auf meiner HP bestehen! ...

Woran mag das liegen...

habe das layout der seite jetzt geändert ... so dass es nicht so gravierend ist, aber ohne zeilenumbrauch wäre es besser...

Danke!

yngvar

wie ich schon sagte, ich hab deine seite mal local gespeichert und die obige zeile den width wert auf 500px gesetzt, dann hat es einwandfrei funktioniert.

funnie01

Vielen, vielen Dank für deine Hilfe!

Die Probleme sind gelöst.

Noch frohes Schaffen und Grüße

P.s. hatte deine Nachricht gelesen und meine Antwort zwar geschrieben aber vergessen abzuschicken.
Dies geschah scheinbar erst einige Zeit später, deshalb kam es zu der zeitlichen Überschneidung.

Bandit

Hallo,

hier noch einige Anregungen zu dem Script:

Code

$from_name=$_POST['fromname'];

Hier wird einfach auf die Post-Variable zugegriffen. Beim ersten Aufruf existiert diese Variable aber nicht. Deshalb sollte man das so schreiben:

Code

$from_name = (isset($_POST['fromname'])) ? $_POST['fromname'] : "";

$_SERVER['PHP_SELF'] sollte man nicht benutzen, google mal mit php_self und xss füttern.

Gruß
bandit

K.Y.L.T.

man sollte aber auch nicht nach isset Prüfen, sondern ob das Element leer ist, gesetzt wird das Element ja schon wenn du das Formular abschickst.

funnie01

Also ändere ich den code mal besser nicht ab?

Irgendwie habe ich das Gefühl, dass die Nachrichten zwar ordentlich an meinen E-Mail Service weitergeleitet werden, ich aber auf die E-Mails nicht ordentlich antworten kann.

Weder manuel noch mit einem Responder...

Woran kann das wohl liegen?

Von der Aufmachung sollte das Script doch funktionieren oder nicht?

Für Tips bin ich äußerst dankbar!

Bandit

Hi,

Zitat

Also ändere ich den code mal besser nicht ab?

doch, auf jeden fall!

die sauberste lösung ist:

PHP

<?php
   $errorText = "";
   $send = 0;


   if (isset($_POST['fromname']))
   {
      // Hier die Daten verarbeiten.
   }


   if(($send != 1) || strlen($errorText)
   {
       // Formular anzeigen
   }
?>

Alles anzeigen

Gruß
bandit

yngvar

@bandit600:
Er wird mit dienem Code-Schnipsel nichts anfangen können.

Zitat von funnie01

bin ein absoluter Neuling

Bevor ich den Code ncohmal bearbeite würd ich gern wissen, was du damit meinst: "ich aber auf die E-Mails nicht ordentlich antworten kann. "?

Heinrich

Tags