Discussion :

[Finality]

Bonjour tout le monde,

je travaille actuellement sur une petite application web, surtout pour me faire la main sur ASP et MVC.

Je suis actuellement en train de mettre en place un CRUD pour celle-ci. J'utilise les vues fortement typées, et actuellement je travaille sur l'Edit.

Le problème est que j'ai un peu de mal avec la validation de mon formulaire. Je m'explique : dans le listings de mes produits, j'ai un lien qui pointe vers l'action Edit de mon controller, déclarée telle quelle :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
public ActionResult Edit(Models.Product product)
{
   Models.productsEntities model = new Models.productsEntities();
   return View(model.Product.Where(x => x.ID == product.ID).First());
}

Une fois passé par l'action, j'ai bien mon formulaire qui s'affiche =) Ma question concerne le traitement de mon formulaire.

1ère question : dois-je pointer vers une autre action de mon controller, spécifique pour le traitement de mon formulaire ?

2ème question : j'essaye de gérer correctement les validation du formulaire. Pour cela, j'utilise l'annotation suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
[Required(ErrorMessage="Vous devez renseigner le champ xxx")]

Dans mon listing, j'utilise le code suivant pour accéder à mon formulaire d'édition :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
Html.ActionLink("Edit", "Edit", new { id=item.ID })

Le problème est que dans ce cas là, quand le formulaire s'affiche, j'ai une erreur au niveau des champs que j'ai marqué comme requis. Je me demande donc si c'est pas du au fait que j'essaie de tout faire passer par la même action

Je vous remercie donc d'avance pour vos éclaircissements

Finality

[Feez]

1ère question : dois-je pointer vers une autre action de mon controller, spécifique pour le traitement de mon formulaire ?

Pas necessairement.

Normalement, dans ton Controller, tu devrais avoir quelque chose qui ressemble à ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
//
// GET: /Article/Edit/5
public ActionResult Edit(int id)
{
  return View(articleRepository.Find(id));
}
//
// POST: /Article/Edit/5
[HttpPost]
public ActionResult Edit(Article article)
{
 if (ModelState.IsValid) {
  articleRepository.InsertOrUpdate(article);
  articleRepository.Save();
  return RedirectToAction("Index");
 } else {
  return View();
 }
}

C'est la seconde action "Edit" , précédée de [HttpPost], qui traitera la soumission de ton fourmulaire.
En supposant que dans ta vue tu as :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Html.BeginForm()

Sinon tu peux spécifier une autre action pout traiter ton formulaire : voir les surcharges de la méthode BeginForm()

Pour la seconde question, il faudrai plus de précision. quelle erreur as tu? un exemple de code ? ou mets tu ton [Required] ?

[Finality]

Coucou Feez,

tout d'abord merci pour ton aide j'ai donc appliqué l'idée du [HttpPost] pour l'action traitant le formulaire, et ça fonctionne correctement

Par contre maintenant j'ai un petit problème. Dans les attributs de mon objet, j'ai un champ ImageUrl qui contient l'image du produit. Dans mon formulaire, je souhaite l'afficher, pour cela je fais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
<img src="<%: Html.DisplayTextFor(model => model.ImageUrl) %>" />

la première fois que mon formulaire s'affiche, tout fonctionne parfaitement, mon image est bien présente. Mais si je valide mon formulaire en ne remplissant pas tous les champs marqués comme requis, j'ai bien le formulaire qui s'affiche avec les messages d'erreurs, mais cette fois-ci plus d'image...
Aurais-tu une idée d'où cela peut venir ?

Encore merci pour ton aide
Finality

[Feez]

Sans grande conviction, je te dirai de rajouter ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
Html.HiddenFor(model => model.ImageUrl)

au dessus de ton <img />.

[ncheboi]

Pas necessairement.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
//
// GET: /Article/Edit/5
public ActionResult Edit(int id)
{
  return View(articleRepository.Find(id));
}
//
// POST: /Article/Edit/5
[HttpPost]
public ActionResult Edit(Article article)
{
 if (ModelState.IsValid) {
  articleRepository.InsertOrUpdate(article);
  articleRepository.Save();
  return RedirectToAction("Index");
 } else {
  return View();
 }
}

En cas d'erreur à l'ajout, t'as méthode // POST: /Article/Edit/5 fait
return View();
Du coup tu ne passes pas d'informations à ta vue edit donc il ne peux pas afficher l'image. D'ailleurs, avec ce que t'as fait, tous tes champs doivent être vides alors que si tu repasse ton article à la vue:
return View(article);
ca devrait garder ce que l'utilisateur avait mis.

Sinon, je te conseil le tuto NerdDinner de Scott Guthrie (tu dois le trouver sur google) qui reprends tout...

[Feez]

En cas d'erreur à l'ajout, t'as méthode // POST: /Article/Edit/5 fait
return View();
Du coup tu ne passes pas d'informations à ta vue edit donc il ne peux pas afficher l'image. D'ailleurs, avec ce que t'as fait, tous tes champs doivent être vides alors que si tu repasse ton article à la vue:
return View(article);
ca devrait garder ce que l'utilisateur avait mis.

Je ne pense pas que le problème vienne de là.
La réponse est (presque) dans la question de Finality.
Lors de la soumission du formulaire, que devient ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
<img src="<%: Html.DisplayTextFor(model => model.ImageUrl) %>" />

[Finality]

Bonjour tous les deux,

tout d'abord désolé du retard, mais j'ai été extrèmement débordé ces derniers temps.

ncheboi, le problème ne vient pas du "return View();". En effet, quand je valide un formulaire avec une erreur, tous les champs retrouvent la valeur qui a été saisie. L'erreur est donc celle qu'indique Feez : que devient le HtmlDisplayTextFor ?

Pour l'instant, j'ai trouvé une solution pas très belle qui fonctionne :

Quand j'affiche mon formulaire, j'ajoute un champ caché qui contiendra le lien de l'image. Ainsi, une fois le formulaire ré affiché avec les erreurs, je n'ai plus qu'à afficher cette variable. Mais je trouve que cela fait un peu (beaucoup ?) bricolage...

[ncheboi]

Mea culpa