PHP-Frage Formmailer Einstellungen

[amethyst]

Hi Ihr,
ich will auf meinen Seiten eine Newsletter-Bestellung einbauen. Das klappt auch, aber, es klappt leider auch dann, wenn die Seite nur aufgerufen wird. Ergo ich mach was falsch. ;-))

Ich hoffe Ihr könnt mir helfen, hier der Code:

QUELLTEXT

<?php
 $admin= 'amethyst@amethyst-edv.de';
 $subject= 'Newsletter-Bestellung';
 $message= 'Nachrichten an '.$admin.' von '.$email.":\n\n";
 foreach($HTTP_POST_VARS as $name=> $value) {
   $message.= "$name:$value\n";
 }
 mail($admin, $subject, $message, "From: $email");

?>
<form method="post" action="mail.php">
<table height="120" width="150" class="news" border="0" >
<tr>
<td align="center"> <span style="text-align:center;color:white;font-weight:bold;font-size:x-small;">E-Mail:</span><br>
 <input class="text" size="10" type="text" name="email"><br><br>

 <input class="senden" type="submit" name="senden" value="Senden"> </td>
</tr>
</table>
</form>

Ich krieg also bei jedem Seitenaufruf eine Bestellung, logischerweise ohne email-adresse. Irgendwas fehlt da, wenn eine Adresse eingetragen wird, wird auf eine andere Seite weitergeleitet, wo es dann auch wieder zurück auf die Ursprungsseite geht, Erfolg? Eine leere Newsletterbestellung.

Ihr habt sicher die Lösung, was da nicht richtig ist, bzw. was da fehlt.

Lieber Gruß Uschi

Der Beitrag wurde von amethyst bearbeitet: 20.02.2005 - 2:08

[Auge]

Hallo Uschi

QUELLTEXT

<?php
 $admin= 'amethyst@amethyst-edv.de';
 $subject= 'Newsletter-Bestellung';
 $message= 'Nachrichten an '.$admin.' von '.$email.":\n\n";
 foreach($HTTP_POST_VARS as $name=> $value) {
   $message.= "$name:$value\n";
 }
 mail($admin, $subject, $message, "From: $email");

?>

Ich krieg also bei jedem Seitenaufruf eine Bestellung, logischerweise ohne email-adresse. Irgendwas fehlt da, wenn eine Adresse eingetragen wird, wird auf eine andere Seite weitergeleitet, wo es dann auch wieder zurück auf die Ursprungsseite geht, Erfolg? Eine leere Newsletterbestellung.

Ihr habt sicher die Lösung, was da nicht richtig ist, bzw. was da fehlt.

Was da fehlt ist eine Bedingung, unter der die Email versandt wird.

QUELLTEXT

<?php
if (isset($_POST["send"]) and $_POST["send"]=="Eintragen")
  {
  // Vorbereitung zum mailen
  mail($to,$subject,$message,$header);
  }
?>
<form ...>
<!-- das Newsletteranmeldeformular -->
...
<input type="submit" name="send" value="Eintragen">
</form>

So wie du es geschrieben hast, wird die Email, im wahrsten Sinne des Wortes, unbedingt versandt.

Tschö, Auge

Der Beitrag wurde von Auge bearbeitet: 20.02.2005 - 13:06

[amethyst]

Hallo Ihr,
danke für die Codes, ich melde mich morgen, ob es jetzt klappt und wie ich es gelöst habe ;-))

Ich bin mir nicht ganz sicher, ob der Code von Auge alles ist oder ob da meiner noch dazu muss ;-)) Ich bin nämlich völlig ahnungslos, was php betrifft. Aber ich werde alle Eure Tipps ausprobieren ;-)) Nur nicht mehr jetzt ;-))

Ich hab keine Benachrichtigungen bekommen, ich schau jetzt aber mal in mein Profil, ob ich da was umstellen muss. Ich hatte mich schon gewundert, dass sich so gar nichts tat.

Lieber Gruß Uschi

[Auge]

Ich bin mir nicht ganz sicher, ob der Code von Auge alles ist oder ob da meiner noch dazu muss ) Ich bin nämlich völlig ahnungslos, was php betrifft. Aber ich werde alle Eure Tipps ausprobieren ) Nur nicht mehr jetzt )

Hallo Uschi

Mein Code ist kein vollständiges Skript. Die Codeteile, die den Versand der Email vorbereiten, z.B. das Zusammensetzen des Emailbodys müsste dort stehen, wo ich den Kommentar hingeschrieben habe.
Ich wollte dir nur aufzeigen, dass es einer Bedingung bedarf, festzulegen, wann der Versand der Email erfolgt. Die Bedingung ist das Vorhandensein der Variable $_POST["send"] und der erfolgreiche Vergleich mit dem String "Eintragen", wenn so auch die Beschriftung des Buttons zum Absenden lautet und dieser mit "send" benamst ist.

Wenn beides zutrifft, solltest du sicher sein können, dass das Formular soeben abgesandt wurde. Somit kannst du die Überprüfung der Werte, den Eintrag und den folgenden Versand der Email anwerfen.

Tschö, Auge

[amethyst]

Danke an alle,
an Conny für die Mühe alles auszukommentieren, an Auge für die Unterstützung und an hondocrx.

Ich habe es gelöst und gereicht hat wirklich die eine Zeile von hondocrx.

@ Conny

Dank Dir ganz herzlich, dieses Script werde ich verwenden, wenn ich jemals dahin komme, dass meine Kurse online gebucht werden können ) Für den Newsletterversand war es mir zu umfangreich, ich hätte die Hälfte oder mehr löschen müssen und dann, wie ich mich kenne, das Falsche gelöscht.

@ Auge
Ich hab zwar verstanden, was Du mir sagen wolltest, aber ich konnte es nicht umsetzen, was wahrscheinlich daran liegt, dass ich vorher schon getrickst hatte )

Ich hab nämlich, weil ich irgendwann geschnallt habe, dass auf die gleiche Seite zurück gegangen wird und ich das nicht wollte, eine 2. Seite geschaffen mit dem gleichen code und auf der Seite als Inhalt den dank für die Besteller untergebracht und dann mit einem winzigen Java-Script (nicht erschlagen) zur Ausgangsseite zurück verlinkt wird.

Der Grund dafür ist, dass die Newsletterbestellung auf jeder Seite unter dem Menü platziert, aber nur auf eine Seite zugegriffen werden soll, sonst funktioniert das nämlich nicht als include ) und das sollte es, ich will so wenig wie möglich von Hand bei den Bestandteilen, die nicht verändert werden, einfügen, ich kenn mich, ich würde es ständig vergessen. Mir reicht schon, dass ich beim versteckten counter so vorgehen muss.

@ hondocrx
Danke für die einfache und doch praktikable Lösung )

Lieber Gruß Uschi

P.S. Wenn mir jetzt noch einer verrät, wie ich mein Profil so einstellen muss, dass ich wieder Mail-Benachrichtigungen erhalte, bin ich erstmal (lange sicher nicht ) wunschlos glücklich )

Der Beitrag wurde von amethyst bearbeitet: 21.02.2005 - 23:45

[Auge]

@ hondocrx
Danke für die einfache und doch praktikable Lösung )

Denke aber dran, dass so nur geprüft wird, ob das Feld für die Email nicht leer übergeben wurde.

P.S. Wenn mir jetzt noch einer verrät, wie ich mein Profil so einstellen muss, dass ich wieder Mail-Benachrichtigungen erhalte, bin ich erstmal (lange sicher nicht ) wunschlos glücklich )

Tja, das ist mir momentan auch ein wenig suspekt. Ich bekomme auch keine Emails mehr, auch meine Cookies werden vom Server gelegentlich vergessen, so dass ich mich neu einloggen muss, auch wenn ich mich beim letzten Besuch nicht ausgeloggt habe. und selbst nach dem neuen einloggen bei erfolgtem ausloggen werden mir gelegentlich bereits gelesene Postings als neu präsentiert.

Irgendwie scheint in der neuen Boardversion der Wurm drin zu sein. Allerdings ist das für mich nicht unbedingt ein Beinbruch, da ich fast täglich vorbeischaue, manchmal auch mehrmals. Ist dann nur ein wenig lästig.

Tschö, Auge

[Andreas]

Tja, das ist mir momentan auch ein wenig suspekt. Ich bekomme auch keine Emails mehr,

$irgendwer hat in den individuellen Maileinstellungen offensichtlich die Benachrichtigung ausgestellt. War bei mir auch so - dachte natürlich, ich wäre ein Einzelfall

Einfach wieder umstellen ...

[Auge]

Hallo Andreas

$irgendwer...

Notice: Call to undefined variable ''$irgendwer´' on line 1 in ...

Einfach wieder umstellen ...

Schon geschehen.

Tschö, Auge

Der Beitrag wurde von Auge bearbeitet: 22.02.2005 - 23:21