mail() HELP!!

015653

Hi,

I am trying to have the following script sent via e-maile.
This info will be gathered from a form. I am trying to add a loop that will alow 22 different rows, and rather than typing out 22 different rows I am trying to create a loop that will increase the

        ."<tr>"
            ."<td align=\"left\">$pn"
            ."<td align=\"left\">$desc"
            ."<td align=\"left\">$rec"
            ."<td align=\"left\">$qty"
           ."</tr>"

up to
."<tr>"
."<td align=\"left\">$pn22"
."<td align=\"left\">$desc22"
."<td align=\"left\">$rec22"
."<td align=\"left\">$qty22"
."</tr>"

I have tried the following but continue getting an error message:

Parse error: parse error, unexpected T_FOR in c:\www\safreq.php on line 66
ERROR ON LINE: (for($i=1;$i<22;$i++) {)

for($i=1;$i<22;$i++) {
if($POST['qty'.$i] AND ($POST['pn'.$i] AND ($POST['desc'.$i] AND ($POST['rec'.$i])))) {

      echo    "<tr>"
                ."<td align=\"left\">".$_POST['pn'.$i]
                ."<td align=\"left\">".$_POST['desc'.$i]
                ."<td align=\"left\">".$_POST['rec'.$i]
                ."<td align=\"left\">".$_POST['qty'.$i];

}

Can anyone please let me know what I am doing wrong?
The original script is below.

<?
$to = "billy.murray@avnet.com";
$subject = "Supplies Request Form";
$headers = "MIME-Version: 1.0\r\n";
$headers .= "Content-type: text/html; charset=iso-8859-1\r\n";
$headers .= "Cc: billy.murray@avnet.com\r\n";
$mailed = mail($to, $subject,

"<html>"
."<head>"
."<title>Supplies Request</title>"
."</head>"
."<body>"
."<left>"
."<h2><b>Safety Supplies Request</h2>"
."</left>"
."<br><br>"
."<br>"

."</b>"
."<b>A Supplies request was submitted by:\n</b>"
."<br><br>"
."<TABLE border=\"1\" cellPadding=\"0\" cellSpacing=\"0\">"
."<TR>"
."<TD align=\"right\"><B>&nbspRequest Date:&nbsp</td>"
."<td align=\"left\">$date_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspDept/Location:&nbsp</td>"
."<td align=\"left\">$dept_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspRequested By:&nbsp</td>"
."<td align=\"left\">$from_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspE-mail address:nbsp</td>"
."<td align=\"left\">$email_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspPhone:&nbsp</td>"
."<td align=\"left\">$phone_required"
."</td>"
."</tr>"
."</BODY>"
."</TABLE>"
."<br><br><b>Item(s) requested:"
."</b><br><br>"
."<table border=\"2\" cellspacing=\"5\" cellpadding=\"2\">"
."<table border=\"2\" cellspacing=\"5\" cellpadding=\"2\">"
."<tr>"
."<th align=\"center\">Part Number"
."<th align=\"center\">Description"
."<th align=\"center\">Recommended Qty"
."<th align=\"center\">Qty Ordered"

."<tr>"
."<td align=\"left\">$pn"
."<td align=\"left\">$desc"
."<td align=\"left\">$rec"
."<td align=\"left\">$qty"
."</tr>"

for($i=1;$i<22;$i++) {
if($POST['qty'.$i] AND ($POST['pn'.$i] AND ($POST['desc'.$i] AND ($POST['rec'.$i])))) {

      echo    "<tr>"
                ."<td align=\"left\">".$_POST['pn'.$i]
                ."<td align=\"left\">".$_POST['desc'.$i]
                ."<td align=\"left\">".$_POST['rec'.$i]
                ."<td align=\"left\">".$_POST['qty'.$i];

}

            ."</table>" 
            ."<br>"
            ."Comments:\n"                   
            .$Comments
            ."</tr>" ,$headers);

if($mailed) {
echo "You have succesfully submitted your request";
}
?>

Thank you!!!
😕 😕

LordShryku

You get an unexpected t_for because your previous string does not end

// Change

."<tr>" 
."<td align=\"left\">$pn" 
."<td align=\"left\">$desc" 
."<td align=\"left\">$rec" 
."<td align=\"left\">$qty" 
."</tr>" 

for($i=1;$i<22;$i++) { 

// To...

."<tr>" 
."<td align=\"left\">".$pn 
."<td align=\"left\">".$desc 
."<td align=\"left\">".$rec
."<td align=\"left\">".$qty 
."</tr>";

for($i=1;$i<22;$i++) {

015653

Thank you for your reply.
But I am still gettign the following error.

Parse error: parse error, unexpected ';' in c:\www\safreq.php on line 63
Line(."</tr>"😉

Could this not work due to the loop inside the body of the mail() function?????

Here is my entire page. The loop on the bottom works fine. It is just the top loop that is taking me for a loop, No pun intended...

🙂

$to = "billy.murray@avnet.com";
$subject = "Supplies Request Form";
$headers = "MIME-Version: 1.0\r\n";
$headers .= "Content-type: text/html; charset=iso-8859-1\r\n";
$headers .= "Cc: billy.murray@avnet.com\r\n";
$mailed = mail($to, $subject,

"<html>"
."<head>"
."<title>Supplies Request</title>"
."</head>"
."<body>"
."<left>"
."<h2><b>Safety Supplies Request</h2>"
."</left>"
."<br><br>"
."<br>"

."</b>"
."<b>A Supplies request was submitted by:\n</b>"
."<br><br>"
."<TABLE border=\"1\" cellPadding=\"0\" cellSpacing=\"0\">"
."<TR>"
."<TD align=\"right\"><B>&nbspRequest Date:&nbsp</td>"
."<td align=\"left\">$date_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspDept/Location:&nbsp</td>"
."<td align=\"left\">$dept_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspRequested By:&nbsp</td>"
."<td align=\"left\">$from_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspE-mail address:nbsp</td>"
."<td align=\"left\">$email_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspPhone:&nbsp</td>"
."<td align=\"left\">$phone_required"
."</td>"
."</tr>"
."</BODY>"
."</TABLE>"
."<br><br><b>Item(s) requested:"
."</b><br><br>"
."<table border=\"2\" cellspacing=\"5\" cellpadding=\"2\">"
."<tr>"
."<th align=\"center\">Part Number"
."<th align=\"center\">Description"
."<th align=\"center\">Recommended Qty"
."<th align=\"center\">Qty Ordered"

."<tr>"

."<td align=\"left\">".$pn

."<td align=\"left\">".$desc

."<td align=\"left\">".$rec
."<td align=\"left\">".$qty

."</tr>";

for($i=1;$i<22;$i++) {

 if($_POST['qty'.$i] AND ($_POST['pn'.$i] AND ($_POST['desc'.$i] AND ($_POST['rec'.$i])))) { 

    echo "<tr>" 
          ."<td align=\"left\">".$_POST['pn'.$i] 
          ."<td align=\"left\">".$_POST['desc'.$i] 
          ."<td align=\"left\">".$_POST['rec'.$i] 
          ."<td align=\"left\">".$_POST['qty'.$i];

}

            ."</table>" 
            ."<br>"
            ."Comments:\n"                   
            .$Comments
            ."</tr>" ,$headers);

if($mailed) {

echo "Thank you for your order.<br>You will receive an E-mail shortly. If you have any questions please contct Brenneva Parris\n"
."<A href=\"Mailto:brenneva.parris@avnet.com?subject=CHANDLER SUPPLIES QUESTION\">E-mail<br></a><br>"
."<b>You have submitted a request for the following:<BR></b>"
."<TABLE border=\"1\" cellPadding=\"0\" cellSpacing=\"0\">"
."<TR>"
."<TD align=\"right\"><B>&nbspRequest Date:&nbsp</td>"
."<td align=\"left\">$date_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspDept/Location:&nbsp</td>"
."<td align=\"left\">$dept_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspRequested By:&nbsp</td>"
."<td align=\"left\">$from_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspE-mail address:nbsp</td>"
."<td align=\"left\">$email_required"
."</td>"
."</tr>"
."<TR>"
."<TD align=\"right\"><B>&nbspPhone:&nbsp</td>"
."<td align=\"left\">$phone_required"
."</td>"
."</tr>"
."</BODY>"
."</TABLE>"
."<br><br><b>Item(s) requested:"
."</b><br>"
."<table border=\"2\" cellspacing=\"5\" cellpadding=\"2\">"
."<br>"
."<tr>"
."<th align=\"center\">Part Number"
."<th align=\"center\">Description"
."<th align=\"center\">Recommended Qty"
."<th align=\"center\">Qty Ordered"

."<tr>"
."<td align=\"left\">$pn"
."<td align=\"left\">$desc"
."<td align=\"left\">$rec"
."<td align=\"left\">$qty"
."</tr>";

for($i=1;$i<22;$i++) {
if($POST['qty'.$i] AND ($POST['pn'.$i] AND ($POST['desc'.$i] AND ($POST['rec'.$i])))) {

      echo    "<tr>"
                ."<td align=\"left\">".$_POST['pn'.$i]
                ."<td align=\"left\">".$_POST['desc'.$i]
                ."<td align=\"left\">".$_POST['rec'.$i]
                ."<td align=\"left\">".$_POST['qty'.$i];

  }

}

Please advise,

Thnak you!

Bill Murray

LordShryku

It's because you're not closing your mail() statement.

Set the body of your email up as a variable, then set your mailed statement like

$mailed = mail($to, $subject, $body, $headers);

drew010

i dont think putting the for loop in the mail function is a good idea either. i would make up some variable like $content and do the for loop before it sends the mail then do something like this:
for(....) {
$content .= "something";
$content .= "something else";
}
mail($to,$subject,$message . $content,$headers);
unset($content); //needed if you are sending mail in a loop.