Problems with E-mail user function

[PeetaGTS]

Hey Guys , when i goto the vbulletin 2.3.2 admin menu and click on either e-mail users or build mailing list under Users.. I get this error:

Parse error: parse error in /home/virtual/site1/fst/var/www/html/forums/admin/email.php on line 113

any suggestions? Thanks!

[MindTrix]

Chances are you have installed a hack recently that modifys that file.

Post the code from the admin/email.php from line 113, and also post the surrounding code at the same time.

[PeetaGTS]

if ($DB_site->num_rows($users)) {

$page = $startat/$perpage+1;
$totalpages = ceil($counter[total]/$perpage);

echo "<p><b>Emailing users. Page $page/$totalpages (".number_format($counter[total])." users total).</b></p>";

if (strpos($message,"\$activateid")) {
$hasactivateid=1;
} else {
$hasactivateid=0;
}

while ($user=$DB_site->fetch_array($users)) {

echo "$user[userid] - $user[username] .... \n";
flush();

$userid=$user[userid];
$sendmessage=$message;
$sendmessage=str_replace("\$email",$user[email],$sendmessage);

Line 113 - while ($user=$DB_site->fetch_array($users)) {

Many thanx.

PS - no hax installed recently.

[MindTrix]

PHP Code:

if ($DB_site->num_rows($users)) {

$page = $startat/$perpage+1;
$totalpages = ceil($counter[total]/$perpage);

echo "<p><b>Emailing users. Page $page/$totalpages (".number_format($counter[total])." users total).</b></p>";

if (strpos($message,"\$activateid")) {
$hasactivateid=1;
} else {
$hasactivateid=0;
}

while ($user==$DB_site->fetch_array($users)) {

echo "$user[userid] - $user[username] .... \n";
flush();

$userid=$user[userid];
$sendmessage=$message;
$sendmessage=str_replace("\$email",$user[email],$sendmessage); 


Try that

[MindTrix]

Any luck?

[PeetaGTS]

Hey! Thanks alot! the script worked! If you don't mind me asking, what exactly did you change? Thank you so much.

[MindTrix]

PHP Code:

while ($user=$DB_site->fetch_array($users)) { 


That line should have been

PHP Code:

while ($user==$DB_site->fetch_array($users)) { 


Basically there should have been == not =

[PeetaGTS]

Something else...

I wasn't able to massmail my users. I got this message:

Emailing users. Page 1/1 (341 users total).

m-dd hh:mm:ss, leave blank for today)","lastpostbefore"); makeinputcode("and Number of Posts is greater than","postslower"); makeinputcode("and Number of Posts is less than","postsupper"); doformfooter("Send Email");

any advice????

many thanx

[MindTrix]

What was the exact error you received??? Also place the php code inside these tags ---

[ php ] [ /php ]

Minus the spaces of course. Thanks.

[PeetaGTS]

It doesn't state that its an error but for some reason it outputs a chunk of the coding. This is what i got:

Emailing users. Page 1/1 (341 users total).

m-dd hh:mm:ss, leave blank for today)","lastpostbefore"); makeinputcode("and Number of Posts is greater than","postslower"); makeinputcode("and Number of Posts is less than","postsupper"); doformfooter("Send Email");