Newsletterservice funktioniert nicht!

AnTrOn

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

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

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

maulepan

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

AnTrOn

sorry das bissl lange gedauert hat!

das ergebnis ist Array

maulepan

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

AnTrOn

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

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

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

maulepan

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

AnTrOn

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

maulepan

google mal....

Basi

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

cool danke basiii maulepan aba auch

seit echt hilfreich thx

Jetzt mitmachen!