[Resolved] Parse error again

[slimjim]

Ok I am trying to make a script that will allow the admin to send email to either one member or all of them at same. I am sure ot have more issues until if figure this all out. Right now I am getting this error : Parse error: parse error, unexpected $ in /home/virtual/site3/fst/var/www/html/brenda-redirect/admin/spam.php on line 66
Now line 66 is the very last line and its not making sense.

<?

require_once "common.php";
checklogin();
dbconnect();


  if($action == "spam")
  {
		if($subject == "" && $content == "")
		{
			echo "Invalid parameters entered!";
		}
		else
		{

		if($mail_to != "all")
		{
			sendmail($mail_to,$email,$subject,$content);
			echo "<b>Mailing done!</b><br><br>";
		}
		else
		{
				$query = "SELECT member_id, email, username from member order by username";

				while($username = _fetch($query))
                sendmail($mail_to,$email,$subject,$content);
                {
				echo "<b>Mailing done!</b><br><br>";
		}
	}
  }
  echo "<H3>Mailing clients</H3>";
?>

<?adminheader()?>
<TABLE>
	<FORM ACTION="spam.php?spam" method="post">
   <TR>
	   <TD><FONT SIZE="-1">Mail to:</FONT></TD>
		<TD>
			 <SELECT NAME="mail_to">
				<option value="all">All
				<?
					$query = "SELECT member_id, email, username from member order by username";

				while($username = _fetch($query))

			?>
		 </SELECT>
	</TD>
   </TR>
	<TR>
		<TD><FONT SIZE="-1">Subject:</FONT></TD>
		<TD><INPUT TYPE="text" NAME="subject" SIZE="30"></TD>
	</TR>
	<TR>
		<TD valign=top><FONT SIZE="-1">Letter's body:</FONT></TD>
		<TD><textarea name="content" cols="50" rows="10"></textarea></TD>
	</TR>
	<TR>
		<TD></TD>
		<TD><INPUT TYPE="submit" VALUE="Send letters"></TD>
	</TR>
	</FORM>
</TABLE>

[Moonglobe]

it means you have mismatched brackets. this is one of the few error messages that DONT make sense. count your brackets, see where you left one out, and all will be good.

[tekky]

I've been seeing this alot lately...

    while($username = _fetch($query)) 

delete the space infront of fetch($query)) and re-add it

for some reason some text editors SOMETIMES add a space that gets interpretted as an _ or something in PHP, I've seen this MOSTLY from MAC users.... 😕

also...

<?adminheader()?> 

(you could put that in the PHP tags above it.. save some file size... but it still needs a closing semi-colon ; 🙂

[Moonglobe]

nope it doesnt, not if its the last thing before a ?>

[tekky]

Originally posted by Moonglobe
nope it doesnt, not if its the last thing before a ?>

wow, didnt know that :eek:

[Moonglobe]

of course you didnt 😉

and i believe the problem is here:

else
            {
                    $query = "SELECT member_id, email, username from member order by username";

                while($username = _fetch($query))
                sendmail($mail_to,$email,$subject,$content);
                {
                echo "<b>Mailing done!</b><br><br>";
        } 

the while never closes.

[tekky]

I bet thats right... although I suspect the _ in that one control structure will cause some errors too...

anywho gonna go do some training at work...

WOOHOO! Clank and Ratchet! (hand eye coordination training 😃 -- I've got such a boring job at times.... 🙁)

[Moonglobe]

you work the late shift or somethin? its past midnight out there, isn't it?

[slimjim]

Well i wasnt able to get it to work the first way. But this way worked for me. So I am calling this Soved.

$res="";
$res2="";

  if($action == "spam")
  {
		if($subject == "" && $content == "")
		{
			$res="<b>Message is Empty or Invalid chars</b>";
		}
		else
		{

		if($mail_to != "all")
		{
			$query = "SELECT member_id, email, username from member where member_id=$mail_to";
			$result = mysql_query($query) or die( "ERROR: " . mysql_error() . "\n");	
			while($myrow = mysql_fetch_array($result)) {
      				  sendmail($myrow["email"],$admin_email,$subject,$content);
       			}
			$res2="<b>Mailing done !</b>";
		}
		else
		{
				$query = "SELECT member_id, email, username from member order by username";
				$result = mysql_query($query) or die( "ERROR: " . mysql_error() . "\n");	
				while($myrow = mysql_fetch_array($result)) {
              				  sendmail($myrow["email"],$admin_email,$subject,$content);
               			}
				$res2="<b>Mailing done 2!</b>";
		}
	}
  }

[tekky]

Originally posted by Moonglobe
you work the late shift or somethin? its past midnight out there, isn't it?

somit like that... 6p-6a this month 🙂

nice quiet shift, we play much games and watch many DVD's 😃