From 2ba9925d86b64fc3b9767215920a140509b853ee Mon Sep 17 00:00:00 2001
From: Loic Masi <loic.masi@diiage.org>
Date: Tue, 7 Apr 2026 10:03:34 +0200
Subject: [PATCH] #199 : Correction de bug sur l'ajout des commentaires si le
 ModelState n'est pas valide.

---
 .../Controllers/TitreController.cs            | 67 ++++++++++++-------
 .../Filters/ValidationActionFilter.cs         | 11 ++-
 .../Views/Titre/Index.cshtml                  | 19 +++---
 3 files changed, 60 insertions(+), 37 deletions(-)

diff --git a/Webzine.WebApplication/Controllers/TitreController.cs b/Webzine.WebApplication/Controllers/TitreController.cs
index 8f00a16..fbab2ae 100644
--- a/Webzine.WebApplication/Controllers/TitreController.cs
+++ b/Webzine.WebApplication/Controllers/TitreController.cs
@@ -38,6 +38,7 @@ namespace Webzine.WebApplication.Controllers
         /// Affiche le detail d'un titre specifique.
         /// </summary>
         /// <param name="id">Identifiant du titre.</param>
+        /// <param name="model">Model de donnée pour un commentaire.</param>
         /// <returns>Vue des details ou 404 si introuvable.</returns>
         public IActionResult Index(int id)
         {
@@ -48,34 +49,11 @@ namespace Webzine.WebApplication.Controllers
             if (titre == null)
             {
                 this.logger.LogWarning("Titre avec ID {Id} introuvable.", id);
-                return this.RedirectToAction("Index");
+                return this.RedirectToAction("Index", "Accueil");
             }
 
             this.titreRepository.IncrementNbLectures(titre);
-
-            var vm = new TitreDetail
-            {
-                Details = new TitreContent
-                {
-                    IdTitre = titre.IdTitre,
-                    Libelle = titre.Libelle,
-                    Chronique = titre.Chronique,
-                    DateSortie = titre.DateSortie,
-                    NbLikes = titre.NbLikes,
-                    UrlJaquette = titre.UrlJaquette,
-                    UrlEcoute = titre.UrlEcoute,
-                    UrlEmbedEcoute = BuildSpotifyEmbedUrl(titre.UrlEcoute),
-                    ArtisteNom = titre.Artiste.Nom,
-                    Styles = titre.Styles,
-                    Commentaires = titre.Commentaires,
-                },
-                CommentForm = new TitreComment
-                {
-                    IdTitre = titre.IdTitre,
-                },
-            };
-
-            return this.View(vm);
+            return this.View(this.BuildTitreDetailViewModel(titre));
         }
 
         /// <summary>
@@ -121,8 +99,20 @@ namespace Webzine.WebApplication.Controllers
         /// <param name="model">Donnees du commentaire.</param>
         /// <returns>Redirection vers la page detail.</returns>
         [HttpPost]
-        public IActionResult Comment(TitreComment model)
+        public IActionResult Comment([Bind(Prefix = "CommentForm")] TitreComment model)
         {
+            if (!this.ModelState.IsValid)
+            {
+                var titre = this.titreRepository.Find(model.IdTitre);
+                if (titre == null)
+                {
+                    this.logger.LogWarning("Titre avec ID {Id} introuvable pour ajout de commentaire.", model.IdTitre);
+                    return this.RedirectToAction("Index", "Accueil");
+                }
+
+                return this.View("Index", this.BuildTitreDetailViewModel(titre, model));
+            }
+
             var titreToUpdate = this.titreRepository.Find(model.IdTitre);
             if (titreToUpdate != null)
             {
@@ -141,6 +131,31 @@ namespace Webzine.WebApplication.Controllers
             return this.RedirectToAction("Index", new { id = model.IdTitre });
         }
 
+        private TitreDetail BuildTitreDetailViewModel(Titre titre, TitreComment? commentForm = null)
+        {
+            return new TitreDetail
+            {
+                Details = new TitreContent
+                {
+                    IdTitre = titre.IdTitre,
+                    Libelle = titre.Libelle,
+                    Chronique = titre.Chronique,
+                    DateSortie = titre.DateSortie,
+                    NbLikes = titre.NbLikes,
+                    UrlJaquette = titre.UrlJaquette,
+                    UrlEcoute = titre.UrlEcoute,
+                    UrlEmbedEcoute = BuildSpotifyEmbedUrl(titre.UrlEcoute),
+                    ArtisteNom = titre.Artiste.Nom,
+                    Styles = titre.Styles,
+                    Commentaires = titre.Commentaires,
+                },
+                CommentForm = commentForm ?? new TitreComment
+                {
+                    IdTitre = titre.IdTitre,
+                },
+            };
+        }
+
         /// <summary>
         /// Mappe une entite Titre vers un item de la liste de titres pour l'affichage dans la vue de style.
         /// </summary>
diff --git a/Webzine.WebApplication/Filters/ValidationActionFilter.cs b/Webzine.WebApplication/Filters/ValidationActionFilter.cs
index 94d0efa..f3613ec 100644
--- a/Webzine.WebApplication/Filters/ValidationActionFilter.cs
+++ b/Webzine.WebApplication/Filters/ValidationActionFilter.cs
@@ -23,6 +23,15 @@ public class ValidationActionFilter : IActionFilter
     /// <inheritdoc/>
     public void OnActionExecuting(ActionExecutingContext context)
     {
+        string controllerName = context.RouteData.Values["controller"]?.ToString() ?? string.Empty;
+        string actionName = context.RouteData.Values["action"]?.ToString() ?? string.Empty;
+
+        if (controllerName.Equals("Titre", StringComparison.OrdinalIgnoreCase)
+            && actionName.Equals("Comment", StringComparison.OrdinalIgnoreCase))
+        {
+            return;
+        }
+
         if (!context.ModelState.IsValid)
         {
             var erreurs = context.ModelState
@@ -35,8 +44,6 @@ public class ValidationActionFilter : IActionFilter
                 context.ActionDescriptor.DisplayName,
                 string.Join(" | ", erreurs));
 
-            string actionName = context.RouteData.Values["action"]?.ToString() ?? string.Empty;
-
             // cas spécial: titre details
             if (actionName.Equals("Index", StringComparison.OrdinalIgnoreCase))
             {
diff --git a/Webzine.WebApplication/Views/Titre/Index.cshtml b/Webzine.WebApplication/Views/Titre/Index.cshtml
index c5a1c6f..75bc6d3 100644
--- a/Webzine.WebApplication/Views/Titre/Index.cshtml
+++ b/Webzine.WebApplication/Views/Titre/Index.cshtml
@@ -125,30 +125,31 @@
             <h4 class="mb-4">Donne ton avis sur le titre</h4>
 
             <form asp-action="Comment" asp-controller="Titre" asp-route-id="@Model.Details.IdTitre" method="post">
-                <input type="hidden" name="IdTitre" value="@Model.Details.IdTitre"/>
+                <input asp-for="CommentForm.IdTitre" type="hidden" />
+                <div asp-validation-summary="ModelOnly" class="text-danger mb-3"></div>
 
                 <div class="row mb-3 align-items-center">
-                    <label class="col-sm-2 col-form-label">
+                    <label asp-for="CommentForm.Auteur" class="col-sm-2 col-form-label">
                         Nom<span class="text-danger">*</span>
                     </label>
                     <div class="col">
-                        <input name="Auteur"
+                        <input asp-for="CommentForm.Auteur"
                                class="form-control input-full"
-                               placeholder="Votre nom"
-                               required/>
+                               placeholder="Votre nom" />
+                        <span asp-validation-for="CommentForm.Auteur" class="text-danger"></span>
                     </div>
                 </div>
 
                 <div class="row mb-3 align-items-start">
-                    <label class="col-sm-2 col-form-label">
+                    <label asp-for="CommentForm.Contenu" class="col-sm-2 col-form-label">
                         Commentaire<span class="text-danger">*</span>
                     </label>
                     <div class="col">
-                        <textarea name="Contenu"
+                        <textarea asp-for="CommentForm.Contenu"
                                   rows="3"
                                   class="form-control input-full"
-                                  placeholder="Votre commentaire..."
-                                  required></textarea>
+                                  placeholder="Votre commentaire..."></textarea>
+                        <span asp-validation-for="CommentForm.Contenu" class="text-danger"></span>
                     </div>
                 </div>