php error

krizzone

Hi, I'm new to the whole php world but I have a script that I'm having trouble with.

I have a website with a contact form and I tested to see if the form would actually send the submitted info to my email but I get this error...

Warning: mail(): Bad parameters to mail() function, mail not sent. in /home/content/g/o/l/goldenhills/html/email_us.php on line 62

Here's the php code:

<?php include "header.php"; ?>

<tr>

<td width=150 background="images/bg_pix.jpg"></td>

<td width=600 bgcolor=#FFFFFF>

  <?php

// multiple recipients

$to = get_db_setting('email_from') . "\r\n"; // note the comma

// subject

$subject = $_POST['subject'];

// message

//echo "$data[0]";

$message = $_POST['message'];
//$message = get_db_setting('email_text');

// To send HTML mail, the Content-type header must be set
// andy: NOT HTML email
//$headers = 'MIME-Version: 1.0' . "\r\n";

//$headers .= 'Content-type: text/html; charset=iso-8859-1' . "\r\n";

// Additional headers

//$headers .= 'To: <$to>' . "\r\n";

$headers .= 'From: ' . $_POST['emailFrom'] . "\r\n";

// Mail it

mail($to, $subject, $message, $headers);

  <h2>Thanks!</h2>

  Email has been sent to:

<?php echo nl2br($to); ?>

</H2>

</td>

<td width=150 background="images/bg_pix.jpg"></td>

</tr>

</table>

<body>

<?php include "footer.php"; ?>

devinemke

echo the $to, $subject, $message, and $headers variables before sending them to mail(). what do you see?

krizzone

How exactly do I do that?

I'm using Adobe GoLive to edit my php files. I'm really new with this. Can you tell me step by step plz? Thanks!

I think the problem is with this line...

$to = get_db_setting('email_from') . "\r\n"; // note the comma

I'm not sure though.

Any ideas how to troubleshoot this problem?

ClarkF1

echo $to;
echo $from;
etc...

krizzone

This is the error I get from testing the contact form on my website:

Warning: mail(): Bad parameters to mail() function, mail not sent. in /home/content/g/o/l/goldenhills/html/email_us.php on line 62

Thanks!
Email has been sent to:
admin@example.com

here's my contact.php:

<?php include "header.php"; ?>

<tr>

<td width=159 background="images/grid.gif" style="border-left: 1px solid #423837;">&nbsp;</td>

<td width=25 bgcolor=#CCCCCC>&nbsp;</td>

<td width=600 bgcolor=#CCCCCC><br>

  <h2>Contact us </h2>

  <p>Please contact us using the form below: 
  <form action="email_us.php" method="post" name="form1" onSubmit="MM_validateForm('emailFrom2','','RisEmail','subject','','R','message','','R');return document.MM_returnValue">
    <table border="0">
      <tr>
        <td>            Your e-mail address:</td>
        <td>
        <input name="emailFrom" type="text" id="emailFrom2" value="" size="50"></td>
      </tr>
      <tr>
        <td>Subject:</td>
        <td>              <input name="subject" type="text" id="subject" value="<?=$_REQUEST['subject']; ?>" size="50">             </td>
      </tr>
      <tr>
        <td>Message:</td>
        <td>              <textarea name="message" cols="60" rows="6" id="message"></textarea>            </td>
      </tr>
    </table>
    <p align="center">
      <input type="submit" name="Submit" value="Send">

</p>
<p align="center">All fields are required</p>
</form> <p>
<strong>Note:</strong><br>
Your email address will not be misused nor provided to a third party.
<p>

<p>
<p align="center">

<p align="center">

<p>

  <p> 

  <p> 

  <p> 

  <p> 

  <p> 

  <p> 

  <p> 

  <p> 

  <p> 

  <p> 

  <p> 

  <p> 

  <p> </td>

<td width="150" background="images/grid.gif" style="border-right: 1px solid #423837;">&nbsp;</td>

</tr>

</table>

<?php include "footer.php"; ?>

Where do I put those codes exactly? I don't know how to "echo" anything. All I know is that the email is being sent which it states but I'm not receiving the email because of that error. Do I have to configure something else in some other file?

ClarkF1

Put them just before the line beginning mail.

Make each line echo "\$to = $to<br>";
etc.

change $to for each variable obviously

get_db_setting is not a PHP function.

Try changing that line to $to = test@test.com (change it to your email address obviously)

krizzone

I wish I knew what you meant by changing all of that but I don't. I'm trying to learn though.

I tried changing that line you suggested and this is what I get when I test it.

Parse error: parse error, unexpected '@' in /home/content/g/o/l/goldenhills/html/email_us.php on line 30

ClarkF1

Just add the lines:

echo "\$to = $to<br>"; 
echo "\$subject = $subject<br>"; 
echo "\$message = $message<br>"; 
echo "\$headers = $headers<br>";

immediately before the line beginning mail( and print the output here

Try it both with the change to the $to line I mentioned before and with the original version referring to get_db_setting

krizzone

Is this right or wrong??

// multiple recipients

$to = "admin@example.com";

// subject

$subject = $_POST['subject'];

// message

//echo "$data[0]";

$message = $_POST['message'];
//$message = get_db_setting('email_text');

// To send HTML mail, the Content-type header must be set
// andy: NOT HTML email
//$headers = 'MIME-Version: 1.0' . "\r\n";

//$headers .= 'Content-type: text/html; charset=iso-8859-1' . "\r\n";

// Additional headers

//$headers .= 'To: <$to>' . "\r\n";

$headers .= 'From: ' . $_POST['emailFrom'] . "\r\n";

// Mail it

mail(echo "\$to = $to<br>";
echo "\$subject = $subject<br>";
echo "\$message = $message<br>";
echo "\$headers = $headers<br>"😉;

krizzone

I did it this way and it works.

but....is that the "right" way to do it?

<?php include "header.php"; ?>

<tr>

<td width=150 background="images/bg_pix.jpg"></td>

<td width=600 bgcolor=#FFFFFF>

  <?php

// multiple recipients

$to = 'admin@example.com' . ' , '; // note the comma

// subject

$subject = $_POST['subject'];

// message

//echo "$data[0]";

$message = $_POST['message'];
//$message = get_db_setting('email_text');

// To send HTML mail, the Content-type header must be set
// andy: NOT HTML email
//$headers = 'MIME-Version: 1.0' . "\r\n";

//$headers .= 'Content-type: text/html; charset=iso-8859-1' . "\r\n";

// Additional headers

//$headers .= 'To: <$to>' . "\r\n";

$headers .= 'From: ' . $_POST['emailFrom'] . "\r\n";

// Mail it

mail($to, $subject, $message, $headers);

  <h2>Thanks!</h2>

  Email has been sent to: <br> <?php echo nl2br($to); ?> <br> <br> <br> <br> </H2> 

</td>

<td width=150 background="images/bg_pix.jpg"></td>

</tr>

</table>

<body>

<?php include "footer.php"; ?>

I guess what I'm trying to accomplish is sending the email to the designated email address in my database settings instead of putting in my email address.

In the RED field is my example email address which use to be

$to = get_db_setting('email_from') . "\r\n"; // note the comma

Any ideas? Could it be a missing comma? It says note the comma but I don't see a comma.