CODEWITHSUNDEEP
asp.net-mvcasp.net-mvc-4
Heidel
Heidel

Reputation: 3254

asp.net mvc Send form from site to email

I have feedback form on my mvc site, it looks like

enter image description here

I created model for my form

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;

namespace ComponentTrading.Web.Models
{
public class FeedbackForm
    {
    public string Name { get; set; } 
    public string Email { get; set; } 
    public string Phone { get; set; }
    public string Message { get; set; }
    }
}

I created view for my form

 @using (Html.BeginForm(null, null, FormMethod.Post, new { id = "contact-form" }))
                    {
                    <fieldset>
                    @Html.TextBoxFor(model => model.Name, new { @Value = "Name" })
                    @Html.TextBoxFor(model => model.Email, new { @Value = "E-mail" })
                    @Html.TextBoxFor(model => model.Phone, new { @Value = "Phone" })
                    @Html.TextAreaFor(model => model.Message, new { @class = "img-shadow"})
                    <input class="form-button" data-type="reset" value="Clear" />
                    <input class="form-button" data-type="submit" type="submit" value="Send" />


}

and now i try to do a sending a letter to email, but it doesn work, when I push "send-button" it happens nothing and I dont get email

my controller

    [HttpGet]
    public ActionResult Contacts()
    {
        FeedbackForm temp = new FeedbackForm();
        temp.Message = "Message";
        return View(temp);
    }


    [HttpPost]
    public ActionResult Contacts(FeedbackForm Model)
    {
        string Text = "<html> <head> </head>" +
        " <body style= \" font-size:12px; font-family: Arial\">"+
        Model.Message+
        "</body></html>";

        SendEmail("[email protected]", Text);
        FeedbackForm temp = new FeedbackForm();
        return View(temp);
    }


    public static bool SendEmail(string SentTo, string Text)
    {
        MailMessage msg = new MailMessage();

        msg.From = new MailAddress("[email protected]");
        msg.To.Add(SentTo);
        msg.Subject = "Password";
        msg.Body = Text;
        msg.Priority = MailPriority.High;
        msg.IsBodyHtml = true;

        SmtpClient client = new SmtpClient("smtp.mail.ru", 25);



        client.UseDefaultCredentials = false;
        client.EnableSsl = false;
        client.Credentials = new NetworkCredential("TestLogin", "TestPassword");
        client.DeliveryMethod = SmtpDeliveryMethod.Network;
        //client.EnableSsl = true;

        try
        {
            client.Send(msg);
        }
        catch (Exception)
        {
            return false;
        }
        return true;
    }

what's wrong?

Upvotes: 1

Views: 2192

Answers (1)

AliRıza Adıyahşi
AliRıza Adıyahşi

Reputation: 15866

Use

// if you dont pass any parameter
// BeginForm posted to the action that
// has name as view name.
// so no need to write any parameters
@using (Html.BeginForm())
{
    ...

    <input type="submit" value="Send">
}

OR

@using (Html.BeginForm("Contacts", "SomeController", FormMethod.Post, new { id = "contact-form" }))
{
    ...

    <input type="submit" value="Send">
}

You have a few question about mvc and there is more important thing of them is incorrect overload methods. My suggestion, first, you should learn html-helpers overload methods. And MVC model binding strategies...

Upvotes: 1

Related Questions