ASP.NET MVC Redirect with model

Question

Hi,

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)<br>
{
    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/IdGoesHere.

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.

Answer 1

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

return RedirectToAction("Details", new { id = person.PersonID });

Answer 2

Use RedirectToAction and provide routing there.

Answer 3

user460667,

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??

[edit]:

user, re your EDIT. couldn't you 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

Answer 4

Mind you could also take the type-safe approach of MvcContrib and do

return this.RedirectToAction<MyController>(c => c.Details(person.PersonID));

Answer 5

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.