¿Cómo puedo mantener ModelState con RedirectToAction?

Question

¿Cómo puedo devolver el resultado de una acción diferente o mover al usuario a una acción diferente si hay un error en mi ModelState sin perder la información de mi ModelState?

El escenario es; La acción de eliminación acepta una POST de un formulario DELETE rendido por mi acción/vista de índice. Si hay un error en la eliminación, deseo mover al usuario de nuevo a la acción/vista de índice y mostrar los errores almacenados por la acción Eliminar en el ViewData.ModelState. ¿Cómo se puede hacer esto en ASP.NET MVC?

[AcceptVerbs(HttpVerbs.Post | HttpVerbs.Delete)] 
public ActionResult Delete([ModelBinder(typeof(RdfUriBinder))] RdfUri graphUri) 
{ 
    if (!ModelState.IsValid) 
     return Index(); //this needs to be replaced with something that works :) 

    return RedirectToAction("Index"); 
} 

Answer 1

debe almacenar datos de vista en TempData y recuperarla desde allí en su acción index, si es que existe.

... 
    if (!ModelState.IsValid) 
     TempData["ViewData"] = ViewData; 

    RedirectToAction("Index"); 
} 

public ActionResult Index() 
{ 
    if (TempData["ViewData"] != null) 
    { 
     ViewData = (ViewDataDictionary)TempData["ViewData"]; 
    } 

    ... 
} 

[EDIT] he comprobado la fuente en línea para MVC y parece que la ViewData en el controlador es ajustable, por lo que es probablemente más fácil sólo para transferir todos los ViewData, incluyendo el ModelState, a la Acción del índice

Answer 2

quizás tratar

return View("Index"); 

en lugar de

return Index(); 

Answer 3

Tenga en cuenta que la solución de tvanfosson no siempre funciona, aunque en la mayoría de los casos, debe estar muy bien.

El problema con esa solución particular es que si ya tiene alguna ViewData o ModelState que terminan sobrescribir todo con el estado de la solicitud anterior. Por ejemplo, la nueva solicitud puede tener algunos errores de estado del modelo relacionados con parámetros no válidos que se pasan a la acción, pero que terminarían ocultos porque se sobrescriben.

Otra situación en la que podría no funcionar como se espera es si ha tenido una acción de filtrado que inicializa algunos errores ViewData o ModelState. De nuevo, serían sobrescritos por ese código.

Estamos pensando en algunas soluciones para ASP.NET MVC que permitirían a combinar más fácilmente el estado de las dos peticiones, así que estad atentos para eso.

Gracias, Eilon

Answer 4

Filtros Uso de acción (PRG) (patrón tan fácil como usar atributos)

mencionado here y here.

Answer 5

En caso de que esto es útil para cualquier persona que solía solución recomendada @bob @ s utilizando PRG:

tema, véase 13 ->link.

tuve la emisión adicional de los mensajes que se pasa en el VeiwBag a la vista que se escriben y se comprueba/carga manual de TempData en las acciones del controlador cuando se hace una RedirectToAction("Action"). En un intento de simplificar (y también hacer que se pueda mantener) extendí ligeramente este enfoque para verificar y almacenar/cargar otros datos también.Mis métodos de acción parecían algo así como:

[AcceptVerbs(HttpVerbs.Post)] 
[ExportModelStateToTempData] 
public ActionResult ChangePassword(ProfileViewModel pVM) { 
     bool result = MyChangePasswordCode(pVM.ChangePasswordViewModel); 
     if (result) { 
      ViewBag.Message = "Password change success"; 
     else { 
      ModelState.AddModelError("ChangePassword", "Some password error"); 
     } 
     return RedirectToAction("Index"); 
    } 

Y mi Índice de Acción:

[ImportModelStateFromTempData] 
public ActionResult Index() { 
    ProfileViewModel pVM = new ProfileViewModel { //setup } 
    return View(pVM); 
} 

El código en la acción Filtros:

// Following best practices as listed here for storing/restoring model data: 
// http://weblogs.asp.net/rashid/archive/2009/04/01/asp-net-mvc-best-practices-part-1.aspx#prg 
public abstract class ModelStateTempDataTransfer : ActionFilterAttribute { 
    protected static readonly string Key = typeof(ModelStateTempDataTransfer).FullName; 
} 

:

public class ExportModelStateToTempData : ModelStateTempDataTransfer { 
    public override void OnActionExecuted(ActionExecutedContext filterContext) { 
     //Only export when ModelState is not valid 
     if (!filterContext.Controller.ViewData.ModelState.IsValid) { 
      //Export if we are redirecting 
      if ((filterContext.Result is RedirectResult) || (filterContext.Result is RedirectToRouteResult)) { 
       filterContext.Controller.TempData[Key] = filterContext.Controller.ViewData.ModelState; 
      } 
     } 
     // Added to pull message from ViewBag 
     if (!string.IsNullOrEmpty(filterContext.Controller.ViewBag.Message)) { 
      filterContext.Controller.TempData["Message"] = filterContext.Controller.ViewBag.Message; 
     } 

     base.OnActionExecuted(filterContext); 
    } 
} 

:

public class ImportModelStateFromTempData : ModelStateTempDataTransfer { 
    public override void OnActionExecuted(ActionExecutedContext filterContext) { 
     ModelStateDictionary modelState = filterContext.Controller.TempData[Key] as ModelStateDictionary; 

     if (modelState != null) { 
      //Only Import if we are viewing 
      if (filterContext.Result is ViewResult) { 
       filterContext.Controller.ViewData.ModelState.Merge(modelState); 
      } else { 
       //Otherwise remove it. 
       filterContext.Controller.TempData.Remove(Key); 
      } 
     } 
     // Restore Viewbag message 
     if (!string.IsNullOrEmpty((string)filterContext.Controller.TempData["Message"])) { 
      filterContext.Controller.ViewBag.Message = filterContext.Controller.TempData["Message"]; 
     } 

     base.OnActionExecuted(filterContext); 
    } 
} 

realizo mis cambios aquí son una extensión bastante obvio de lo que ya se está haciendo con el ModelState por el código @ el enlace proporcionado por @bob - pero tenía que tropezar en este hilo antes de que yo pensaba de la manipulación de esta manera.