Hallo mensen!

Ik heb een mass mailer op internet gevonden, en hij ziet er goed uit.
Het gaat goed als je een mailtje naar 1 iemand wilt sturen.

Als je (bvb. 3) email-adressen invult zegt hij: sent 3 mails
Maar hij verstuurd er geen 1, ziet iemand waar dit door komt?
bedankt!

PHP/Html code: http://pastebin.com/1zNvNm8q
Kan je de relevante code tot +/- 20 regels hier tonen i.p.v. een pastebin die opeens uit het zicht kan verdwijnen?
Ik heb iets dergelijks gemaakt voor mezelf.
<?php
foreach ($_POST['to_email'] as $to_email){
$to = $to_email;
$subject = 'We have some new updates for you';
$message = 'Jouw bericht ';
$headers = 'MIME-Version: 1.0' . "\r\n";
$headers .= 'Content-type: text/html; charset=iso-8859-1' . "\r\n";
$headers .= 'From: bloopp-it <[email protected]>' . "\r\n";
mail($to, $subject, $message, $headers);
}
?>

Misschien werkt het met 'foreach'?
Zorg wel voor goede foutafhandeling op je mail(), of gebruik bij voorkeur phpMailer.
ereg_replace gaat je in ieder geval (normaal gesproken) een (fout)melding opleveren.

ereg_replace() is deprecated as of PHP 5.3.0. preg_replace() is the suggested alternative to this function.
@Sven: Regel 3 is overbodig. Maar jij zet de post-waarde zo (ongecontroleerd) in de mail. Dat is heerlijk voor spammers.
dit is 1 tot en met 20

<?php

if(isset($_POST['action'] ) ){
$action=$_POST['action'];
$message=$_POST['message'];
$emaillist=$_POST['emaillist'];
$from=$_POST['from'];
$replyto=$_POST['replyto'];
$subject=$_POST['subject'];
$realname=$_POST['realname'];
$file_name=$_POST['file'];
$contenttype=$_POST['contenttype'];

$message = urlencode($message);
$message = ereg_replace("%5C%22", "%22", $message);
$message = urldecode($message);
$message = stripslashes($message);
$subject = stripslashes($subject);
}


?>
Regel 3: Of er gepost is doe je met if($_SERVER['REQUEST_METHOD'] == "POST").
Regel 4 t/m 12: Overbodig. Het zit al in een var en dit kost alleen extra geheugen.
Regel 14: Waarom urlencode()?
Regel 15: ereg_replace() is antiek. En volgens mij wat jij wilt kan zo te zien ook met stripslashes().
Regel 16: ??
Ik ben nieuw in php, dus als je zou kunnen schrijven hoe het wel zou moeten zou voor mij handiger zijn
Kijk eens o.a naar de genoemde functies op php.net.
Ik kan dat echt niet fixen, dus als iemand eens dat stukje zou kunnen verbeteren :/

Reageren