ASP.NET MVC 2 - Object passed to delete method controller is empty

Question

I'm trying to delete an entry from the database. This is my controller method:

   public ActionResult Delete(Models.CommentEntry entry)
    {
        _db.Entries.Remove(entry);
        _db.SaveChanges();
        return RedirectToAction("Index");
    }

And this is my .cshtml:

         @foreach (var entry in ViewBag.Entries)
            {
                <section class="contact">
                    <header>
                        <h3>@entry.Message</h3>
                    </header>
                    <p>
                        Posted by @entry.Name on @entry.DateAdded.ToLongDateString()
                    </p>
                </section>
                <form method="post" action="Delete">
                    <input type="submit" value="Delete" class="btn btn-danger" />
                </form>
            }

This is how it looks right now. As you can see i want whenever the user presses delete, to delete the corresponding comment. I'm using foreach to access them all and was hoping that this way I can pass the comment object to the controller.

As you can see the object doesn't get passed down and always has initial values, therefore i'm getting a dump. Where is my mistake?

Thank you.

Farshad Delavarpour · Accepted Answer · 2021-04-24 18:02:59Z

1

You must enter the entry parameters between the form tag. for each parameter in the entry model, declare an input tag with name and value. change the form part code as follow:

<form method="post" action="Delete">
<input type="hidden" name="Message" value="@entry.Message" /> 
<input type="hidden" name="Id" value="@entry.Id" />
 .
 .
<input type="submit" class="btn btn-danger" />
</form>

For more information about sending data to the controller, you can read this article: https://www.c-sharpcorner.com/UploadFile/3d39b4/getting-data-from-view-to-controller-in-mvc/

edited Apr 24, 2021 at 18:02

answered Apr 24, 2021 at 15:03

Farshad Delavarpour

2051 silver badge7 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Radoslav Pavlov Over a year ago

Thanks for your answer. Unfortunately it's still not working, parameter is still not being passed down to controller, it's still null.

Farshad Delavarpour Over a year ago

Try this, for each parameter in the entry model, declare an input tag with name and value. for example: <input type="hidden" name="Message" value="@entry.Message" /> <input type="hidden" name="Id" value="@entry.Id" /> . . . It certainly works.

Radoslav Pavlov Over a year ago

Thank you, it worked now, i pass the value correctly. Now for some reason it doesn't find it in my objectstatemanager, even though i'm positive it exists with the correct ID, but i guess tha's a seperate problem i'll look into.

mohammad mobasher · Accepted Answer · 2021-04-30 15:02:07Z

first of all i think you are here from laravel (i figure out it from you action). here is my code and i explain all:

public async Task<ActionResult> Delete(int Id)
{
    var entry = _db.Entries(x=>x.Id == Id);
    if(entry != null)
    {
        _db.Entries.RemoveAsync(entry);
        _db.SaveChangesAsync();
    }
    return RedirectToAction("Index");
}

now you have to fix you'r cshtml file too, for sending Id to this action

@foreach (var entry in ViewBag.Entries)
{
    <section class="contact">
        <header>
            <h3>@entry.Message</h3>
        </header>
        <p>
            Posted by @entry.Name on @entry.DateAdded.ToLongDateString()
        </p>
    </section>
    <form  action="Delete">
        <input type="hidden" name="Id" value="@entry.Id" />
        <input type="submit" value="Delete" class="btn btn-danger" />
    </form>
}

Alireza heidari · Accepted Answer · 2021-04-24 14:59:03Z

0

this not wrong question

<section class="contact">
                <header>
                    <h3>@entry.Message</h3>
                </header>
                <p>
                    Posted by @entry.Name on @entry.DateAdded.ToLongDateString()
                </p>
                 <form method="post" action="Delete"><!-- must have name and value input for form tag -->
            </section>
                <input type="submit" value="Delete" class="btn btn-danger" />
            </form>

Please read this https://www.w3schools.com/tags/tag_form.asp

edited Apr 24, 2021 at 14:59

answered Apr 24, 2021 at 14:52

Alireza heidari

457 bronze badges

Comments

Sagar Shanks · Accepted Answer · 2021-04-26 09:27:19Z

0

   // you can use entryId
    
     public ActionResult Delete(int entryId)
        {
             var entry = _db.Entries(x=>x.Id == entryId);
            _db.Entries.Remove(entry);
            _db.SaveChanges();
            return RedirectToAction("Index");
        }
//Get the Id From SubmitButton in View
    @Html.ActionLink("Delete","ControllerName",new{entryid==entry.Id})

answered Apr 26, 2021 at 9:27

Sagar Shanks

11 bronze badge

Collectives™ on Stack Overflow

ASP.NET MVC 2 - Object passed to delete method controller is empty

4 Answers 4

3 Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

3 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related