Emails not getting generated

Question

0.00/5 (No votes)

See more:

VS2008

Hi, I'm trying to setup email options for a web form.However if i don't put any condition and try to send a general email, it works, but if at all i try to put any condition as to send it different recipents depending on the report type.The page loads up again but no email is sent.Here is the piece of code for sending the email.It would really helpful if you could help me out with this.

C#

foreach (GridViewRow row in reportTable.Rows)
            {
                foreach (dbTypes.report report in reports)
                {

                    CheckBox cb = (CheckBox)row.Cells[8].FindControl("cb");
                    cb = (CheckBox)reportTable.Rows[i].Cells[8].FindControl("cb");
                    testBench b = new testBench();

                    if (cb.Checked && b.costPos == "123")
                    {
                        MailMessage message = new MailMessage("no-reply@gmail.com",
               "tony.romo@gmail.com", "Kostenstelle  : 123",
               "Kostenstelle : 123");


                        /* create SMTP Client and add credentials */
                        SmtpClient smtpClient = new SmtpClient("SERVER");
                        smtpClient.UseDefaultCredentials = true;
                        /* Email with Authentication */
                        //smtpClient.Credentials = new NetworkCredential("userID",
                        //"password", "domainName");

                        /*Send the message */
                        smtpClient.Send(message);
                    }

Posted 20-Aug-12 21:44pm

Member 9196002

Updated 20-Aug-12 21:54pm

v3

Add a Solution

Comments

Mario Majčica 21-Aug-12 4:09am

Are you sure that the condition is met? If you set a break point on smtpClient.Send(message); does it hit?

Member 9196002 21-Aug-12 4:31am

you were right..the condition is not met.

3 solutions

Solution 1

hii refer this ,

www.codeproject.com/Articles/12511/Send-Email-in-ASP-Net-2-0-Feed-back-Form[^]

Posted 20-Aug-12 22:11pm

Ganesh Nikam

Solution 2

Well, what I see, you didn't provide any user name/password, if your SMTP server supports relay.

Check the code,

smtpClient = new SmtpClient();
System.Net.NetworkCredential credentials = null;
smtpClient.Port = int_port;
smtpClient.Host = str_server;
if (str_ssl == "SSL")
{
    smtpClient.EnableSsl = true;
}
else
{
    smtpClient.EnableSsl = false;
}
if (str_isauthenticate == "Yes")
{
    smtpClient.UseDefaultCredentials = false;
    credentials =
            new System.Net.NetworkCredential(str_user, str_password);

    smtpClient.Credentials = credentials;
}
else
{
    smtpClient.UseDefaultCredentials = true;
    credentials =
            new System.Net.NetworkCredential();
    smtpClient.Credentials = credentials;

}

Hope this helps.
cheers

Posted 20-Aug-12 22:15pm

Sandip.Nascar

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)

**AmitGajjar** · Accepted Answer · 2012-08-20T22:23:00

Solution 3

Hi,

Possible issues:

1) Exception occur at cb.Checked if checkbox not found.
2) Your condition is not met and it is not able to execute if code.

Resolution:

1) Add try ...catch block and check if any exception occur.
2) add else condition and prompt appropriate message.

Thanks
-Amit Gajjar

Posted 20-Aug-12 22:23pm

AmitGajjar

Comments

Member 9196002 21-Aug-12 4:31am

the problem is that the condition is not meeting

AmitGajjar 21-Aug-12 4:33am

ok then you need to check two things,
1) is b.costPos value is 123 ?
2) is cb is not null and cb is checked ?

Member 9196002 21-Aug-12 4:40am

Oh ok.Does it make a difference if you are referencing b.costPos to a value from a datatable? 123 is the value stored in the document.
and cb is not null..i made sure its checked

AmitGajjar 21-Aug-12 4:42am

just your b.costPos value should match your datatable value. and add breakpoint inside your if condition and check if you got hit there ? if not then your b.costPos must be different value then 123.

Member 9196002 21-Aug-12 4:46am

ok!!i shall check that..i guess it should work!! thanks a lot!! :)

AmitGajjar 21-Aug-12 4:47am

your welcome dear.

Member 9196002 21-Aug-12 6:49am

i figured out what the issue was..Costpos is of the type String but the value in the document i was referring to which was 123 is of int..is there a workaround for this comparision?

AmitGajjar 21-Aug-12 6:50am

You can use int.TryParse method to check if you are getting correct integer value.

Member 9196002 21-Aug-12 6:59am

ok!! :)thanks again!!

Member 9196002 22-Aug-12 2:48am

Well.this is what i have been trying to do but it doesnt seem to be meeting the first condition itself.
int value;
int.TryParse(bench.costPos, out value);
if (value == 123)
{
MailMessage message = new MailMessage("no-reply@gmail.com",
"tony.romo@gmail.com", "Kostenstelle : 123",
"Kostenstelle : 123");
/* create SMTP Client and add credentials */
SmtpClient smtpClient = new SmtpClient("SERVER");
smtpClient.UseDefaultCredentials = true;
/*Send the message */
smtpClient.Send(message);
}
else
{
MailMessage message = new MailMessage("no-reply@gmail.com",
"victoria.ray@gmail.com", "Kostenstelle : 456",
"Kostenstelle : 456");
/* create SMTP Client and add credentials */
SmtpClient smtpClient = new SmtpClient("SERVER");
smtpClient.UseDefaultCredentials = true;
/*Send the message */
smtpClient.Send(message);

AmitGajjar 22-Aug-12 2:50am

it means your value is not 123.

Member 9196002 22-Aug-12 2:51am

yes..but even when i select the checkbox where the value is 123 it doesn't meet that condition.

AmitGajjar 22-Aug-12 2:53am

you need to place your code to get value from your checkbox. can you post that code? where you set bench.costPos value ?

Member 9196002 22-Aug-12 3:00am

userManager user = Global.sessions.getUser(Request.UserHostName);
ArrayList reports = (ArrayList)Session["reportSelect"];
foreach (GridViewRow row in reportTable.Rows)
{
CheckBox cb = (CheckBox)row.Cells[8].FindControl("cb");

if (cb.Checked)
{
testBench bench = new testBench();

and then the rest of the code follows

AmitGajjar 22-Aug-12 3:02am

ok but where you set bench.costPos ?

Member 9196002 22-Aug-12 3:18am

the namespaces have been compiled as reference assemblies and loaded onto the project and costPos is one of the identifier in that namespace

AmitGajjar 22-Aug-12 4:09am

ok i have doubt about the value of costPos. can you check what value you are getting ?

Member 9196002 22-Aug-12 5:57am

its done!! :)there was no need of any parsing required i directly referred to the cell entry in the datatable and now its working fine!!thanks a lot!! :)

AmitGajjar 22-Aug-12 7:23am

great... :)