can get php to process my form, why?

[enrique]

I'm trying to use the following script to process a form (below) I've been racking my brains over this one and am at breaking point....it just doesn't seem to work can anyone help please?

<?php
// Get submitted data
$name = $HTTP_POST_VARS['name'];
$company = 'From: '.$HTTP_POST_VARS['company'];
$phone = $HTTP_POST_VARS['phone'];
$message = $HTTP_POST_VARS['message'];

// Put data into readable format
$to = 'enrique@hotmail.co.uk';
$body = 'Name: '.$name.'\n'
.'Company: '.$company.'\n'
.'email: '.$email.'\n'
.'Message: \n \n'.$message;

// Send email
mail($to, $body, $email);
header( "Location: http://www.webhotdesign.co.uk/whd/pages/thanks.htm" );
?>



<form name="form" id="form" method="post" action="form.php">
<!--DWLayoutTable-->
<tr>
<td width="32" rowspan="9" valign="top"><!--DWLayoutEmptyCell-->&nbsp;</td>
<td height="66" colspan="2" valign="middle"><!--DWLayoutEmptyCell-->&nbsp;</td>
<td width="1"></td>
<td width="26">&nbsp;</td>
<td width="334">&nbsp;</td>
<td width="106">&nbsp;</td>
</tr>
<tr>
<td width="69" height="21" valign="top">name<br> </td>
<td colspan="2" rowspan="8" valign="top"> <p>
<input name="name" type="text" id="name" size="15" maxlength="20" />
<br>
<br>
<input name="phone" type="text" id="phone" size="15" maxlength="20" />
<br>
<br>
<input name="email" type="text" id="email" size="15" maxlength="25" />
<br>
<br>
<textarea name="message" cols="15" rows="4" wrap="VIRTUAL"></textarea>
</p>
<p>
<input class=aceButton type="submit" value="submit">
</td>
<td>&nbsp;</td>
<td rowspan="4" valign="top"> For all initial enquiries please use the form
<br><br> Tel: 07782 216 212 <br>
Email <a href="mailto:enrique@webhotdesign.co.uk">enrique@w ebhotdesign.co.uk</a>
</td>
<td>&nbsp;</td>
</tr>
<tr>
<td height="20" valign="top"><!--DWLayoutEmptyCell-->&nbsp;</td>
<td>&nbsp;</td>
<td>&nbsp;</td>
</tr>
<tr>
<td height="21" valign="top">phone</td>
<td>&nbsp;</td>
<td>&nbsp;</td>
</tr>
<tr>
<td rowspan="2" valign="top"><!--DWLayoutEmptyCell-->&nbsp;</td>
<td height="14"></td>
<td></td>
</tr>
<tr>
<td height="6"></td>
<td></td>
<td></td>
</tr>
<tr>
<td height="21" valign="top">email</td>
<td></td>
<td></td>
<td></td>
</tr>
<tr>
<td height="20" valign="top"><!--DWLayoutEmptyCell-->&nbsp;</td>
<td></td>
<td></td>
<td></td>
</tr>
<tr>
<td height="164" valign="top"><br>
message</td>
<td></td>
<td></td>
<td></td>
</tr>
<tr>
<td height="1"></td>
<td></td>
<td width="182"></td>
<td></td>
<td></td>
<td></td>
<td></td>
</tr></form>
</table>

[masonbarge]

Code: Select all

 $company = 'From:  '.$HTTP_POST_VARS['company'];

I'd rewrite this. I'm not on firm ground here, but echo syntax doesn't work for variable assignments sometimes. I'd just write $company = $HTTP_POST_VARS['company']; and concat $company as needed.

Probably not it, wish I could be more help.

[gustava32]

Is it because you have not told it when it needs to run the PHP that processes it?So it just runs it straight away. I think you need to change it too something like this.

PHP: Select all

 <?php
if($HTTP_POST_VARS[submit]) //This means it will not process this section until 
{                                             //the submit button is pressed.
//  Get submitted data
$name = $HTTP_POST_VARS['name'];
$company = 'From:  '.$HTTP_POST_VARS['company'];
$phone = $HTTP_POST_VARS['phone'];
$message  = $HTTP_POST_VARS['message']; 

// Put data into readable format
$to =  'enrique@hotmail.co.uk';
$body = 'Name: '.$name.'\n'
.'Company:  '.$company.'\n'
.'email: '.$email.'\n'
.'Message: \n \n'.$message;  

// Send email
mail($to, $body, $email);
header( "Location:  http://www.webhotdesign.co.uk/whd/pages/thanks.htm"  );
}
?>

Unless I am getting the wrong end of the stick.

Hope this helps.
Gareth

[enrique]

cheers guys for the suggestions, but i've taken everything on board and tried it but it still doesn't work - the header part works which is basically
the thank you page, but that's it....

any other suggestions?

[gustava32]

What exactly is it doing?

What is is supposed to be doing?

[enrique]

It's supposed to get the info of the form and send it over to my hotmail account but it doesn't do it.
the header (the thankyou page) appears and that's it......

[gustava32]

How about changing the body bit to this and see if that works:

PHP: Select all

$body = "Name: $name\nCompany: $company\nemail: $email\nMessage: \n \n$message"; 

See if that helps!

[enrique]

hi there,

I tried that but still nothing - Is there anything else I can try or maybe start from scratch...

Once again the setup is that I have :-

the formpage
the php page
and the thank you page (header)

three name fields: name, company and phone as well as a textarea (message)

can you suggest something ?? getting desperate now. Didn't think that it would be so troublesome..

[masonbarge]

1)What version of php are you using?

2) What mail program are you using, i.e. check php_ini for sendmail. If you're using some other program you need to make adjustments.

3) Try taking the newline markers out of the $body, that will screw it up sometimes. Actually do that first as it is the easiest thing to try.

Actually, don't you need a subject in the second parameter spot? No great expert, just trying to help you think of something.

[enrique]

hi,

Thanks for the tips I have tried them but still nothing.

The server which is hosting my site supports php4 if that helps...
with regards to what i'm using locally on my machine well nothing.
I just upload it straight to server and test it there...

The header the thank you page appears but that's it.

[gustava32]

If only the header to the thankyou page displays then there is something wrong with your PHP in the thankyou page.

Post that here as well and I will see if can spot that problem.

[gustava32]

Also i have just been looking through your code again and I see that you are using things that dont even exist which might not help.

For example you are trying to get data from

PHP: Select all

<?php $company = $HTTP_POST_VARS['company'] ;?>

when you dont even have a company field. I suggest you rewrite the page from scratch and also do it manually as the dreamweaver template seems pretty messed up too me!

[gustava32]

I have just tweaked it a little bit more and managed to get it too work for me!
Here is the code:

PHP: Select all

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Test Page</title>
</head>

<body>
<?php
// Get submitted data
if ($HTTP_POST_VARS['submit'])
{
$name = $HTTP_POST_VARS['name'];
$company = 'From: '.$HTTP_POST_VARS['company'];
$phone = $HTTP_POST_VARS['phone'];
$message = $HTTP_POST_VARS['message'];
$email = $HTTP_POST_VARS['email'];

// Put data into readable format
$to = 'enrique@hotmail.co.uk';
$subject = 'test email';
$body = "Name: $name.\n Company: $company\n email: $email\n Message: \n \n$message";

// Send email
if(mail($to, $subject, $body, $company ))
{

//header( "Location: http://www.webhotdesign.co.uk/whd/pages/thanks.htm" );
echo "Mail Sent";
}
else
{
    echo "There was an error sending your email.";
}
}
?>



<form name="form" id="form" method="post" action="form.php">
<!--DWLayoutTable-->
<tr>
<td width="32" rowspan="9" valign="top"><br> </td>
</tr>
<tr>Name</tr>
<tr><td width="69" height="21" valign="top">&nbsp;</td>
<td colspan="2" rowspan="8" valign="top"> <p>
<input name="name" type="text" id="name" size="15" maxlength="20" />
<br>
<br>
<input name="phone" type="text" id="phone" size="15" maxlength="20" />
<br>
<br>
<input name="company" type="text" id="company" size="25" maxlength="25" />
<br>
<br>
<input name="email" type="text" id="email" size="15" maxlength="25" />
<br>
<br>
<textarea name="message" cols="15" rows="4" wrap="VIRTUAL"></textarea>
</p>
<p>
<input class=aceButton type="submit" value="submit" name="submit">
</td>
<td>&nbsp;</td>
<td rowspan="4" valign="top"> For all initial enquiries please use the form
<br><br> Tel: 07782 216 212 <br>
Email <a href="mailto:enrique@webhotdesign.co.uk">enrique@w ebhotdesign.co.uk</a>
</td>
<td>&nbsp;</td>
</tr>
<tr>
<td height="20" valign="top"><!--DWLayoutEmptyCell-->&nbsp;</td>
<td>&nbsp;</td>
<td>&nbsp;</td>
</tr>
<tr>
<td height="21" valign="top">phone</td>
<td>&nbsp;</td>
<td>&nbsp;</td>
</tr>
<tr>
<td rowspan="2" valign="top"><!--DWLayoutEmptyCell-->&nbsp;</td>
<td height="14"></td>
<td></td>
</tr>
<tr>
<td height="6"></td>
<td></td>
<td></td>
</tr>
<tr>
<td height="21" valign="top">email</td>
<td></td>
<td></td>
<td></td>
</tr>
<tr>
<td height="20" valign="top"><!--DWLayoutEmptyCell-->&nbsp;</td>
<td></td>
<td></td>
<td></td>
</tr>
<tr>
<td height="164" valign="top"><br>
message</td>
<td></td>
<td></td>
<td></td>
</tr>
<tr>
<td height="1"></td>
<td></td>
<td width="182"></td>
<td></td>
<td></td>
<td></td>
<td></td>
</tr></form>
</table>
</body>
</html>

I would still suggest completely re-organising it though!

[masonbarge]

Having the mandatory 'subject' parameter probably helps

[JacobHaug]

Does that work?

[enrique]

hi gustava, for your last comment...


This time I tried intergrating the code within the contact page itself, as you had laid it out and changed the form action to PHP_$SELF - I guess thats the right thing to do but still no joy..

Any other suggestions ?