I paid a PHP programmer and he could not figure it out.

[agisthos]

I think I have been taken for a ride. A PHP programmer for my fathers website supposedly spent '2 hours' going over this code and could not get it to work on our webhost. Please check it and tell me what you think. Below is a test php script that works fine.

<?php
$to = "[e-mail]";
$subject = "Php test mail";
$body = "Test mail for php mail()";
if (mail($to, $subject, $body)) {
  echo("<p>Yes you have done it</p>");
 } else {
  echo("<p>Oops try once more..</p>");
 }
?>

The code that does not work required is below and required php pear to be installed. When using this form it just hangs and then says could not connect to the host via SMTP. All details have been checked and the form works on another host.

Does some hosts have problem with a forms smtp authentication procedure?

How do I change the below code to remove the smtp login e.t.c and use the method of emailing as in the test script above? The test script does not use any external server login, it just does it from the server directly.

<?php

ini_set("include_path", ".:/usr/lib/php:/usr/local/lib/php:/home/harore/php");



require_once "Mail.php";



// <<< begin form handling

if ($_POST["PA_Name"] != "")

{

$from = "Website Enquiry <[e-mail]>";

$to = "Haro Removals <[e-mail]>";



$subject = "Website Quotation";

$host = "mail.example.com"; 

$port = "26";

$username = "[e-mail]";

$password = "pass";

$body = "Hi,\r\n\r\nHere is my request:\r\n\r\nPickup From:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "PA_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nDelivery To:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "DA_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nAnything else we should know:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "AE_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nBedrooms:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "BE_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nLaundry:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "LA_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nLounge:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "LO_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nHall:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "HA_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nKitchen:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "KI_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nDining:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "DI_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nSundries:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "SU_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nFamily / Study:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "FA_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nOutside:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "OU_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nCartoons:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "CA_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$body = $body."\r\n\r\nAdditional Items:\r\n\r\n";



foreach ($_POST as $k => $v)

if (substr($k, 0, 3) == "AD_")

    $body = $body.substr($k, 3).": ".$v."\r\n";



$headers = array ('From' => $from,

  'To' => $to,

  'Subject' => $subject);

$smtp = Mail::factory('smtp',

  array ('host' => $host,

'port' => $port,

'auth' => true,

'username' => $username,

'password' => $password));



$mail = $smtp->send($to, $headers, $body);



if (PEAR::isError($mail)) {

  echo("<p>" . $mail->getMessage() . "</p>");

 } else {

  echo("<p>Message successfully sent!</p>");

 }

}

// end form handling >>>

?>

[bradgrafelman]

Try changing the port to 25, or possibly 587. Either way, port '26' isn't a standard SMTP port. You can confirm this by asking your host/email provider.

[agisthos]

Hi Brad,

Yes I tried port 25, and also tried running it throught a gmail smtp account using port 465 and 587, it still hung.

That is why I think the problem may be with the webhost, and want to try remove the whole smtp authentication process from the code. I have tried it myself, but get errors.

[bradgrafelman]

Well you can't simply remove the SMTP authentication - it's required by SMTP providers to prevent spamming.

Try contacting your host and ask how they suggest you are to send e-mails via PHP scripts.

[agisthos]

The hosting provider gave me that test script, and because it works (and sends an email directly from the server) they have washed their hands of the issue and say the problem is with my code.

The coder charges me and claims the problem is with the server.

So here I am. That test php script does send an email without any sort of smtp login or authentication (the email 'from' field is a combination of the account username and server number itself)

That is why I need help to plug it in to the main code.

[bradgrafelman]

Well in that case, switch the host to "localhost", get rid of the SMTP authentication, and change the 'From' header to an address on the server.

It's possible that they're blocking connections to external SMTP servers, though you might check with them if this is the case and you really need to use an external SMTP server.

[onicsoft]

A simple hack job of doing it really quick and letting it use the mail instead of the smtp would be this..

Before doing this make a backup copy!

FIND

$mail = $smtp->send($to, $headers, $body);

REPLACE WITH

if (mail($to, $subject, $body,$headers)) {
  echo("<p>Yes you have done it</p>");
} else {
  echo("<p>Oops try once more..</p>");
} 

REMOVE/DELETE

if (PEAR::isError($mail)) {
  echo("<p>" . $mail->getMessage() . "</p>");
} else {
  echo("<p>Message successfully sent!</p>");

} 

[agisthos]

Thanks guys that did it. It works now.