Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

ASP.NET MVC Redirect with model

I currently have a method in my controller which accepts a form collection, saves the data, and then displays the data in a 'Details' page. At the moment, the code currently looks something like:

[HttpPost]
public ActionResult Create(PersonModel person)
{
    if (person.IsValid())
    {
        person.additionalData = "Person created successfully";
        return View("Details", person);
    }
}

The problem is that returning the Details view in this manner retains the URL mysite/Person/Create - ideally I would like the URL to be mysite/Person/Details/{PersonId}.

Im sure this must be possible. Obviously, I could use Response.Redirect, however this doesn't allow me to pass the model to the view. As far as I can tell, I can't pass the model using RedirectToAction?

Thanks for the help.

EDIT: To confirm - the model produced by the Create action is different to the default one created by the Details action. Therefore, doing a straight redirect to Action and passing the Id does not work as the model produced is not correct. To give more context, the model from the Create action contains an additional value saying 'Person created successfully', if I redirect to the Details action, this message is not present in the model.

like image 339
user460667 Avatar asked Oct 25 '10 10:10

user460667


People also ask

Can we pass model to RedirectToAction in MVC?

When a Button is clicked, the Model object is populated with values and passed to the RedirectToAction method along with the name of the Controller and its Action method in ASP.Net MVC Razor. In this article I will explain with an example, how to redirect to Action method with Model data in ASP.Net MVC Razor.

How redirect a specific view from controller in MVC?

You can use the RedirectToAction() method, then the action you redirect to can return a View. The easiest way to do this is: return RedirectToAction("Index", model); Then in your Index method, return the view you want.


3 Answers

Straight from my own app:

public ActionResult Create(Booking item)
{
    if (ModelState.IsValid)
    {
        int newID = _tasks.Create(item);
        // NEW section to emulate model being populated for use in Details view
        TempData["additionalData"] = "Person created successfully";
        return RedirectToAction("Details", new { id = newID });
    }
    else
    {
        return View();
    }
}

then, couldn't your "Details" action be like this:

public ActionResult Details(int id)
{
    var item = _tasks.GetByKey(id);
    var additionalData = TempData["additionalData"];
    if(item != null) {
        if(additonalMessage!=null)
        {
            item.additionalData = additionalData;
        }
        return View(item);
    }
    else
        return View("Notfound");
}

couldn't you adopt a similar approach??

You could just do the redirect as per convention and have a flag set (on tempdata as above) that gives this message? The tempadata flag would ONLY be set inside the Create action, therefore would only ever happen on Creating a new 'person' object. thus the Details action would only ever show it as a consequence of the Create action

like image 60
jim tollan Avatar answered Oct 08 '22 15:10

jim tollan


You could supplement what has been offered (use RedirectToAction and routing) with the use of TempData

[HttpPost]
public virtual ActionResult Create(IEnumerable<OrderItem> orderItems)
    {
        if (orderItems.Count() == 0)
        {
            return RedirectToAction("NoOrderItems");
        }
        else
        {
            TempData["orderItems"] = orderItems;
            return RedirectToAction("Confirm");
        }
    }

    [HttpGet]
    public virtual ActionResult Confirm()
    {
        var orderItems = TempData["orderItems"] as IEnumerable<OrderItem>;
        if (orderItems == null || orderItems.Count() == 0)
        {
            this.InvokeHttp404(ControllerContext.HttpContext);
        }

        return View(orderItems);
    }

I use this for items that I might not want to create again on subsequent requests or persist quite yet in the database yet. With this, I don't need null checks in my view, as the Confirm page can only be "gotten" if there is data for it.

like image 31
Jonathan Bates Avatar answered Oct 08 '22 16:10

Jonathan Bates


This should take you to the Details Model, passing the ID with it.

return RedirectToAction("Details", new { id = person.PersonID });
like image 39
Marko Avatar answered Oct 08 '22 16:10

Marko