Newsletterservice funktioniert nicht!

AnTrOn · 23. März 2010

Hi,

ich hab hier ein Newsletterscript das wie der Name E-Mail an Interessenten schicken soll! Es wird auch alles ordentlich gemacht blos die E-Mail werden dann immer nur an den ersten in der Datenbank verschickt an keine weiteren was mach ich falsch???

PHP

<?php
require("config.php");
if (isset($_POST['text'])) {
$text=$_POST['text'];
};
if (isset($_POST['betreff'])) {
$betreff=$_POST['betreff'];
};
if (isset($_POST['eig_email'])) {
$eig_email=$_POST['eig_email'];
};
session_start();
if (isset($_POST['user'])) {
$user=$_POST['user'];
};
if (isset($_SESSION['user'])) {
$user=$_SESSION['user'];
};
if (isset($_POST['passw'])) {
$passw=$_POST['passw'];
};
if (isset($_SESSION['passw'])) {
$passw=$_SESSION['passw'];
};
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"  "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>Newsletter versenden</title>
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1" />
<link rel="stylesheet" type="text/css" href="style.css" />
</head>
<body>
<h1>Newsletter schreiben</h1>
<?php
if (isset($user) && isset($passw)) {
if (($user == $benutzer) && ($passw == $kennwort)) {
require("sql.php");
$_SESSION['user']=$user;
$_SESSION['passw']=$passw;
print "
<table width=\"100%\">
<tr>
<td style=\"vertical-align:top; width:200px;\">
<ul>
<li><a href=\"schreiben.php\">Newsletter schreiben</a></li>
<li><a href=\"mitgliedverwalt.php\">Mitglieder verwalten</a></li>
</ul>
</td>
<td>
";
print "<p>Zum <a href=\"logout.php\">ausloggen</a> aus dem passwortgesch&uuml;tzten Bereich.</p>";
if (isset($_POST['submit'])) {
/* MySQL Datenbank für E-Mailadressen auslesen, in array $email[] abspeichern */
$zeiger=mysql_connect ($sqlip,$sqlbenutzer,$sqlpassw) or die ("Konnte Datenbank nicht &ouml;ffnen");
mysql_select_db ("$sqlname");
$anfrage="SELECT EMail FROM newsletter";
$ergebnis=mysql_query($anfrage,$zeiger);
$pruef_korr=mysql_num_rows($ergebnis);
while ($datensatz=mysql_fetch_row($ergebnis)) {
$email[]=$datensatz[0];
};
mysql_close($zeiger);
/* E-Mail an alle Eintrager versenden */
$text=stripslashes($text);
foreach ($email as $temp) {
mail ("$temp","$betreff","$text","From: $eig_email") or die ("Konnte Newsletter nicht versenden");
};
mail ("$eig_email","versendet: $betreff","Sie haben folgenden Text als Newsletter versendet:\n\n$text");
print "<p><strong>Der Newsletter wurde verschickt!</strong></p>";
};
print "<form action=\"$_SERVER[PHP_SELF]\" method=\"post\">
<div>
Von Absender-Emailadresse: <input type=\"text\" name=\"eig_email\" value=\"$eig_email\" size=\"30\" maxlength=\"45\" /><br />
Betreff: <input type=\"text\" name=\"betreff\" value=\"$betreff\" size=\"30\" maxlength=\"40\" /><br />
<br />
<textarea rows=\"15\" cols=\"70\" name=\"text\">\n\n\n$ende</textarea><br />
<input type=\"submit\" name=\"submit\" value=\"Absenden\" />&nbsp;<input type=\"reset\" value=\"Abbrechen\" /><br />
</div>
</form>";
print "
</td>
</tr>
</table>
";
} else {
print "<p><strong>Das Passwort ist falsch!</strong> Gehen Sie <a href=\"admin.html\">zur&uuml;ck</a> und versuchen sie es noch mal.</p>";
};
} else { print "<p><strong>Sie haben kein Passwort angegeben!</strong> Gehen Sie <a href=\"admin.html\">zur&uuml;ck</a> und versuchen sie es noch mal.</p>"; };
?>
</body>
</html>

Alles anzeigen

schon ma vielen Dank

maulepan · 23. März 2010

ich sehe das problem in dieser zeile....
$email[]=$datensatz[0];

Mit $datensatz[0] lädst du in das array immer den ersten datensatz.
Am einfachsten wäre es so zu lösen.

Code

$datensatz=mysql_fetch_row($ergebnis)); 
while($datensatz){
$text=stripslashes($text);
mail ("$datensatz->mail","$betreff","$text","From: $eig_email") or die ("Konnte Newsletter nicht versenden");
};

Damit umgehst du das übertragen des arrays einfach.

AnTrOn · 23. März 2010

jetzt sagt er immer das die Nachricht nicht versendet werden konnte! halt den fehler wie er angegeben ist

maulepan · 23. März 2010

und was ist nun der fehler...
überprüfe doch mal den inhalt von $datensatz['mail']

AnTrOn · 25. März 2010

sorry das bissl lange gedauert hat!

das ergebnis ist Array

maulepan · 26. März 2010

check ich irgendwie net.... schreibt nochma deinen von dir (jetzt) verwendeten code....

AnTrOn · 26. März 2010

also hier nochma der code

PHP

<?php
require("config.php");
if (isset($_POST['text'])) {
$text=$_POST['text'];
};
if (isset($_POST['betreff'])) {
$betreff=$_POST['betreff'];
};
if (isset($_POST['eig_email'])) {
$eig_email=$_POST['eig_email'];
};
session_start();
if (isset($_POST['user'])) {
$user=$_POST['user'];
};
if (isset($_SESSION['user'])) {
$user=$_SESSION['user'];
};
if (isset($_POST['passw'])) {
$passw=$_POST['passw'];
};
if (isset($_SESSION['passw'])) {
$passw=$_SESSION['passw'];
};
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"  "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>Newsletter versenden</title>
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1" />
<link rel="stylesheet" type="text/css" href="style.css" />
</head>
<body>
<h1>Newsletter schreiben</h1>
<?php
if (isset($user) && isset($passw)) {
if (($user == $benutzer) && ($passw == $kennwort)) {
require("sql.php");
$_SESSION['user']=$user;
$_SESSION['passw']=$passw;
print "
<table width=\"100%\">
<tr>
<td style=\"vertical-align:top; width:200px;\">
<ul>
<li><a href=\"schreiben.php\">Newsletter schreiben</a></li>
<li><a href=\"mitgliedverwalt.php\">Mitglieder verwalten</a></li>
</ul>
</td>
<td>
";
print "<p>Zum <a href=\"logout.php\">ausloggen</a> aus dem passwortgesch&uuml;tzten Bereich.</p>";
if (isset($_POST['submit'])) {
/* MySQL Datenbank für E-Mailadressen auslesen, in array $email[] abspeichern */
$zeiger=mysql_connect ($sqlip,$sqlbenutzer,$sqlpassw) or die ("Konnte Datenbank nicht &ouml;ffnen");
mysql_select_db ("$sqlname");
$anfrage="SELECT EMail FROM newsletter";
$ergebnis=mysql_query($anfrage,$zeiger);
$pruef_korr=mysql_num_rows($ergebnis);
$datensatz=mysql_fetch_object($ergebnis);
while($datensatz){
$text=stripslashes($text);
mail ("$datensatz->EMail","$betreff","$text","From: $eig_email") or die ("Konnte Newsletter nicht versenden");
};
mail ("$eig_email","versendet: $betreff","Sie haben folgenden Text als Newsletter versendet:\n\n$text");
print "<p><strong>Der Newsletter wurde verschickt!</strong></p>";
};
print "<form action=\"$_SERVER[PHP_SELF]\" method=\"post\">
<div>
Von Absender-Emailadresse: <input type=\"text\" name=\"eig_email\" value=\"$eig_email\" size=\"30\" maxlength=\"45\" /><br />
Betreff: <input type=\"text\" name=\"betreff\" value=\"$betreff\" size=\"30\" maxlength=\"40\" /><br />
<br />
<textarea rows=\"15\" cols=\"70\" name=\"text\">\n\n\n$ende</textarea><br />
<input type=\"submit\" name=\"submit\" value=\"Absenden\" />&nbsp;<input type=\"reset\" value=\"Abbrechen\" /><br />
</div>
</form>";
print "
</td>
</tr>
</table>
";
} else {
print "<p><strong>Das Passwort ist falsch!</strong> Gehen Sie <a href=\"admin.html\">zur&uuml;ck</a> und versuchen sie es noch mal.</p>";
};
} else { print "<p><strong>Sie haben kein Passwort angegeben!</strong> Gehen Sie <a href=\"admin.html\">zur&uuml;ck</a> und versuchen sie es noch mal.</p>"; };
?>
</body>
</html>

Alles anzeigen

und wenn ich jetzt in die while schlweifen nochma echo mit der variable datensatz eingeb zeigt er an "Array" anstatt die mailadressen die eigendlich kommen sollten

maulepan · 26. März 2010

mach mal bitte statt

$datensatz=mysql_fetch_object($ergebnis);
while($datensatz){

einfach ein
while($datensatz=mysql_fetch_object($ergebnis)){

weiss nicht genau, ob eine while bei einem array so funktioniert wie ich dachte^^

AnTrOn · 26. März 2010

ja jetzt verschickt es die ersten paar in der datenbank aba nich alle und zeigt wieder fehler beim versenden

maulepan · 26. März 2010

dann ist bestimmt eine mailadresse fehlerhaft.... wenn es die ersten versendet, dann liegst ja auf jedenfall nicht am php code

AnTrOn · 26. März 2010

hast du einen code mit dem man die mail auf bestimmte zeichen einschränken kann?

maulepan · 26. März 2010

google mal....

Basi · 27. März 2010

Zitat von "maulepan"

google mal....

Ohaaaa, produktiv, muss ich sagen^^

Reguläre ausdrücke:

Ich weiss nicht genau wie du es meinst, also 2 Möglichkeiten:

PHP

<?php




// Alle Zeichen die im esten parameter von preg_replace() stehen sind erlaubt




$var = "<<<<<<<<<<æþ&{}()Hallo!>>>>>>></æþ&{}()>";




$var = preg_replace("/[^a-zA-Z0-9&auml;&ouml;&uuml;&Auml;&Ouml;&Uuml;äöüÄÖÜ\!\.\,\:\;\& ]/" , "" , $var);




print $var."<br />";




// prüft länge




$var = "123456789123";  // 12 zeichen




if(strlen($var) > 10){
      print "Zu lang!<br />";
} elseif(strlen($var) <= 10){
      print "Nicht zu lang!<br />";
}




/*************
*
*  Gibt aufgrund von 12 Zeichen länge "Zu lang<br />" aus!
*
**************/




?>

Alles anzeigen

AnTrOn · 27. März 2010

cool danke basiii maulepan aba auch

seit echt hilfreich thx

Newsletterservice funktioniert nicht!

Jetzt mitmachen!

Teilen