Why don't send email

Question

1.00/5 (1 vote)

See more:

When Paypal notifyback will trigger my methed in my controller.
Yes the method is indeed trigger, And except the send email function don't work

if (amountPaid == price)
               {
                   if (payment_status.ToLower() == &completed)
                   {
                       //update the database table function over there 
                       //it works
                      //Send Parse Email to company  system                    

                        SendParseEmail(clientID);//this didn't work
////send a comfirmation email withe QR code
                        ////send email to the client
                       
                            SendClientEmail(clientID);
                       }
               ///follow code all working correcting
//....


  public void SendParseEmail(int clientID)
        {
            //there are a lot of code just for get information from database               

                     var mailMessage = new MailMessage();
                     mailMessage.From = new MailAddress("xxxx@xxx.xxx");
                     mailMessage.To.Add("xxx@xxx.xxx");// 

                     mailMessage.Body = mm_Form.ToString();
                     mailMessage.IsBodyHtml = false;
                
                 
                 Thread parseEmail = new Thread(delegate()
                 {
                     var mysmtpParse = new SmtpClient();
                     mysmtpParse.Send(mailMessage);
                 });
                 parseEmail.IsBackground = true;
                 parseEmail.Start();

             }
        }

 public void SendClientEmail(int clientID)
        {
            string fileName = Server.MapPath("~/Common/ClientEmail.html");
            string mailBody = System.IO.File.ReadAllText(fileName);
            if(book.ContactName==null)
                mailBody = mailBody.Replace("##ContactName##", book.UserName);
            else
                mailBody = mailBody.Replace("##ContactName##", book.ContactName);
            mailBody = mailBody.Replace("##Email##", book.Email);
            if(book.Phone2==null)
                mailBody = mailBody.Replace("##HomePhone##", book.Phone);
            else
                mailBody = mailBody.Replace("##HomePhone##", book.Phone2);
            mailBody = mailBody.Replace("##Pickup##", PickupInfo);
            mailBody = mailBody.Replace("##Destination##", DropoffInfo);
            mailBody = mailBody.Replace("##Vehicle##", "Shuttle");
            mailBody = mailBody.Replace("##numAdults##", numOfPaxStr);
            mailBody = mailBody.Replace("##UnitAdultPrice##", unitOfPriceStr);            
            mailBody = mailBody.Replace("##TotalPrice##", totalPrice);
            mailBody = mailBody.Replace("##PaidDate##", paidDay); 
            mailBody = mailBody.Replace("##PaidAmount##", amountPaid.ToString()); 


            var myMessage = new MailMessage();

            myMessage.Subject = "Shuttle Booking:" + book.Ref + "(" + cityName + "): Tax Invoice, Travel Details & Receipt";
            myMessage.Body =  mailBody;
            myMessage.IsBodyHtml = true;

            myMessage.From = new MailAddress("xxx@xxx.xxx", "Royale Airport Transfer Bookings");//enquiries@royalecoach.com.au
            myMessage.To.Add(new MailAddress(payer_email, userName));
            
           
            Thread clientEmail = new Thread(delegate()
            {
                var mysmtpClient = new SmtpClient();
                mysmtpClient.Send(myMessage);
            });
            clientEmail.IsBackground = true;
            clientEmail.Start();

        }

Posted 26-Nov-14 14:34pm

SandyZhang2014

Updated 26-Nov-14 14:47pm

v2

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Sergey Alexandrovich Kryukov · Answer 1 · 2014-11-26T17:15:00

It's possible that you are sending it nowhere. The SmtpClient constructor you are using is based on assumption that you chose you SMTP server and all the parameter by writing appropriate configuration file:
http://msdn.microsoft.com/en-us/library/k1c4h6e2%28v=vs.110%29.aspx[^].

Who knows what did you write in that configuration file, if it even exists? :-)

To explicitly choose the server or server with the port, you would need to use other constructors:
http://msdn.microsoft.com/en-us/library/k0y6s613(v=vs.110).aspx[^],
http://msdn.microsoft.com/en-us/library/67w4as51(v=vs.110).aspx[^].

Besides, if everything is done, I cannot be sure that you used correct parameters for the server and authentication. You can do this: try the server you have with some available mail client program, and when you make sure it really sends messages, mimic all the parameters in your code.

—SA

SandyZhang2014 · Answer 2 · 2014-11-27T12:23:00

Solution 2

It has been sovled by myself, I have some parameter is not correct

Posted 27-Nov-14 12:23pm