ASP.NET MVC - Comment préserver les erreurs de ModelState à travers RedirectToAction ?

Question

J'ai les deux méthodes d'action suivantes (simplifiées pour la question) :

[HttpGet]
public ActionResult Create(string uniqueUri)
{
   // get some stuff based on uniqueuri, set in ViewData.  
   return View();
}

[HttpPost]
public ActionResult Create(Review review)
{
   // validate review
   if (validatedOk)
   {
      return RedirectToAction("Details", new { postId = review.PostId});
   }  
   else
   {
      ModelState.AddModelError("ReviewErrors", "some error occured");
      return RedirectToAction("Create", new { uniqueUri = Request.RequestContext.RouteData.Values["uniqueUri"]});
   }   
}

Ainsi, si la validation passe, je redirige vers une autre page (confirmation).

Si une erreur se produit, je dois afficher la même page avec l'erreur.

Si je le fais return View() l'erreur est affichée, mais si je fais return RedirectToAction (comme ci-dessus), il perd les erreurs de modèle.

Je ne suis pas surpris par ce problème, mais je me demande comment vous gérez ça.

Je pourrais bien sûr simplement renvoyer la même vue au lieu de la redirection, mais j'ai une logique dans la méthode "Create" qui alimente les données de la vue, que je devrais dupliquer.

Des suggestions ?

Answer 1

J'ai dû résoudre moi-même ce problème aujourd'hui, et je suis tombé sur cette question.

Certaines des réponses sont utiles (utilisation de TempData), mais ne répondent pas vraiment à la question posée.

Le meilleur conseil que j'ai trouvé se trouve dans cet article de blog :

http://www.jefclaes.be/2012/06/persisting-model-state-when-using-prg.html

Fondamentalement, utilisez TempData pour enregistrer et restaurer l'objet ModelState. Cependant, c'est beaucoup plus propre si vous abstrayez tout cela dans des attributs.

Par exemple

public class SetTempDataModelStateAttribute : ActionFilterAttribute
{
    public override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        base.OnActionExecuted(filterContext);         
        filterContext.Controller.TempData["ModelState"] = 
           filterContext.Controller.ViewData.ModelState;
    }
}

public class RestoreModelStateFromTempDataAttribute : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        base.OnActionExecuting(filterContext);
        if (filterContext.Controller.TempData.ContainsKey("ModelState"))
        {
            filterContext.Controller.ViewData.ModelState.Merge(
                (ModelStateDictionary)filterContext.Controller.TempData["ModelState"]);
        }
    }
}

Ensuite, comme dans votre exemple, vous pourriez sauvegarder / restaurer le ModelState comme suit :

[HttpGet]
[RestoreModelStateFromTempData]
public ActionResult Create(string uniqueUri)
{
    // get some stuff based on uniqueuri, set in ViewData.  
    return View();
}

[HttpPost]
[SetTempDataModelState]
public ActionResult Create(Review review)
{
    // validate review
    if (validatedOk)
    {
        return RedirectToAction("Details", new { postId = review.PostId});
    }  
    else
    {
        ModelState.AddModelError("ReviewErrors", "some error occured");
        return RedirectToAction("Create", new { uniqueUri = Request.RequestContext.RouteData.Values["uniqueUri"]});
    }   
}

Si vous voulez également transmettre le modèle dans TempData (comme bigb l'a suggéré), vous pouvez toujours le faire.

Answer 2

Vous devez avoir la même instance de Review sur votre HttpGet action. Pour ce faire, vous devez enregistrer un objet Review review dans une variable temporaire sur votre HttpPost et le restaurer ensuite sur HttpGet action.

[HttpGet]
public ActionResult Create(string uniqueUri)
{
   //Restore
   Review review = TempData["Review"] as Review;            

   // get some stuff based on uniqueuri, set in ViewData.  
   return View(review);
}
[HttpPost]
public ActionResult Create(Review review)
{
   //Save your object
   TempData["Review"] = review;

   // validate review
   if (validatedOk)
   {
      return RedirectToAction("Details", new { postId = review.PostId});
   }  
   else
   {
      ModelState.AddModelError("ReviewErrors", "some error occured");
      return RedirectToAction("Create", new { uniqueUri = Request.RequestContext.RouteData.Values["uniqueUri"]});
   }   
}

Si vous voulez que cela fonctionne même si le navigateur est rafraîchi après la première exécution de la commande HttpGet action, vous pourriez faire ça :

  Review review = TempData["Review"] as Review;  
  TempData["Review"] = review;

Sinon, sur l'objet bouton de rafraîchissement review sera vide parce qu'il n'y aurait pas de données dans TempData["Review"] .

Answer 3

Pourquoi ne pas créer une fonction privée avec la logique de la méthode "Create" et appeler cette méthode à partir des méthodes "Get" et "Post" et faire simplement "return View()".

Answer 4

Je pourrais utiliser TempData["Errors"]

Les données temporaires sont transmises entre les actions en préservant les données une fois.

Answer 5

Je vous suggère de retourner la vue, et d'éviter la duplication via un attribut sur l'action. Voici un exemple de remplissage des données de la vue. Vous pourriez faire quelque chose de similaire avec la logique de votre méthode de création.

public class GetStuffBasedOnUniqueUriAttribute : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        var filter = new GetStuffBasedOnUniqueUriFilter();

        filter.OnActionExecuting(filterContext);
    }
}

public class GetStuffBasedOnUniqueUriFilter : IActionFilter
{
    #region IActionFilter Members

    public void OnActionExecuted(ActionExecutedContext filterContext)
    {

    }

    public void OnActionExecuting(ActionExecutingContext filterContext)
    {
        filterContext.Controller.ViewData["somekey"] = filterContext.RouteData.Values["uniqueUri"];
    }

    #endregion
}

Voici un exemple :

[HttpGet, GetStuffBasedOnUniqueUri]
public ActionResult Create()
{
    return View();
}

[HttpPost, GetStuffBasedOnUniqueUri]
public ActionResult Create(Review review)
{
    // validate review
    if (validatedOk)
    {
        return RedirectToAction("Details", new { postId = review.PostId });
    }

    ModelState.AddModelError("ReviewErrors", "some error occured");
    return View(review);
}