Want to protect your cyber security and still get fast solutions? Ask a secure question today.Go Premium

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 493
  • Last Modified:

Conflict between PhpMailer and phpmail

Hi Am using the following code to send two different emails with the same form.  am using phpmailer to send the mail with attachment and phpmail to send without the attachment to two different recepients.

This code works fine when the size of the attachment is smaill like 3-10kb but if the size is a bigger one something like 245kb only the first mail goes through and second mail doesn't.

Please let me know what is wrong here.
$uploaddir = 'c:\\ABC\\';
$uploadfile = $uploaddir . basename($_FILES['file']['name']);
move_uploaded_file($_FILES['file']['tmp_name'], $uploadfile); 
 
$mail = new PHPMailer();
 
$mail->IsSMTP();                      
$mail->Host = "X.X.X.X"; 
 
 
$mail->From = $_POST['email'];
$mail->FromName = "Submissioin";
$mail->AddAddress("abc@abc.com");                 
                              
$mail->AddAttachment($uploadfile); 
$mail->IsHTML(true);        
$mail->Subject = "Submission Form";
 
$mail_body = "Test mail";
$mail->Body    = $mail_body;
 
 
if(!$mail->Send())
{
   echo "Message could not be sent. <p>";
   echo "Mailer Error: " . $mail->ErrorInfo;
   exit;
}
else
{
$mail->ClearAddresses();
$mail->ClearAttachments();
 
//sending email using phpmailer===============================================================
$Name =  $_POST['fName'] . ' ' . $_POST['Lname']; 
$email = $_POST['email']; 
$recipient = "xyz@xyz.com"; 
$mail_body = "Attachment recieved";                       
$subject = "Submission Form"; //subject 
$header = "From: ". $Name . " <" . $email . ">\r\n"; //optional headerfields  
$mail_body = strip_tags( preg_replace('#<br\s*/?>#i',"\n",$mail_body) );
mail($recipient, $subject, $mail_body, $header); 
 
//======================================================================
 
 
header("Location: http://somewhere.ca.com/partnerescalation/reqkd.php");
exit();}

Open in new window

0
Shiva_Kumar
Asked:
Shiva_Kumar
  • 2
1 Solution
 
wildzeroCommented:
Hi there,

This block of code here

if(!$mail->Send())
{
   echo "Message could not be sent. <p>";
   echo "Mailer Error: " . $mail->ErrorInfo;
   exit;
}
else
{

Means that it's only ever going to send 1 email. It will try and do PHPMailer, and only if that fails it will use the PHP Mail. If you want it to always send two emails then you can use this. The code below will not stop on errors.

You should also check and make your capitilization consistant
$Name =  $_POST['fName'] . ' ' . $_POST['Lname'];
is it fName or Fname? Lname or lName


$uploaddir = 'c:\\ABC\\';
$uploadfile = $uploaddir . basename($_FILES['file']['name']);
move_uploaded_file($_FILES['file']['tmp_name'], $uploadfile); 
 
$mail = new PHPMailer();
 
$mail->IsSMTP();                      
$mail->Host = "X.X.X.X"; 
 
 
$mail->From = $_POST['email'];
$mail->FromName = "Submissioin";
$mail->AddAddress("abc@abc.com");                 
                              
$mail->AddAttachment($uploadfile); 
$mail->IsHTML(true);        
$mail->Subject = "Submission Form";
 
$mail_body = "Test mail";
$mail->Body    = $mail_body;
 
$mail->Send();
$mail->ClearAddresses();
$mail->ClearAttachments();
 
//sending email using phpmailer=================================
$Name =  $_POST['fName'] . ' ' . $_POST['Lname']; 
$email = $_POST['email']; 
$recipient = "xyz@xyz.com"; 
$mail_body = "Attachment recieved";                       
$subject = "Submission Form"; //subject 
$header = "From: ". $Name . " <" . $email . ">\r\n"; //optional headerfields  
$mail_body = strip_tags( preg_replace('#<br\s*/?>#i',"\n",$mail_body) );
mail($recipient, $subject, $mail_body, $header); 
 
//======================================================================
 
 
header("Location: http://somewhere.ca.com/partnerescalation/reqkd.php");
exit();

Open in new window

0
 
Marcus BointonCommented:
> Means that it's only ever going to send 1 email. It will try and do PHPMailer, and only if that fails it will use the PHP Mail.

That's not true. There's a ! in that if that means it will only exit if it fails to send.

It's a complete mystery to me why you'd use PHPMailer to send the first one but not the second. Using mail() directly is generally just asking for trouble, as you're seeing. I'm not going to debug that code because the right fix is to use PHPMailer to send that as well.
0
 
Shiva_KumarAuthor Commented:
Thank you, it worked
0
 
Marcus BointonCommented:
What worked? In the selected answer, the logic is completely wrong and it gets around the problem of there being an error by removing the error checking!
0

Featured Post

What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now