Merge pull request 'j2/feat/admin_CRUD' (#119) from j2/feat/admin_CRUD into dev

Reviewed-on: https://10.4.0.131/gitea/DI1-P4-E1/Webzine/pulls/119

Webzine.EntitiesContext/WebzineDbContext.cs

@@ -49,7 +49,7 @@ namespace Webzine.EntitiesContext
                 entity.HasMany(a => a.Titres)
                     .WithOne(t => t.Artiste)
                     .HasForeignKey(t => t.IdArtiste)
-                    .OnDelete(DeleteBehavior.Restrict);
+                    .OnDelete(DeleteBehavior.Cascade);
             });
 
             modelBuilder.Entity<Style>(entity =>

Webzine.Repository.Contracts/IArtisteRepository.cs

@@ -24,6 +24,7 @@ namespace Webzine.Repository.Contracts
         /// <param name="id">L'identifiant de l'artiste.</param>
         /// <returns></returns>
         Artiste Find(int id);
+
         /// <summary>
         /// Récupère un artiste par son nom. Si aucun artiste n'est trouvé, retourne null.
         /// </summary>

Webzine.Repository/DbArtisteRepository.cs

@@ -63,6 +63,7 @@ namespace Webzine.Repository
 
                 this.context.Artistes.Remove(artiste);
                 this.context.SaveChanges();
+                this.logger.LogDebug("L'artiste {IdArtiste} a bien été supprimé", artiste.IdArtiste);
             }
             catch (DbUpdateException dbex)
             {
@@ -124,7 +125,7 @@ namespace Webzine.Repository
             {
                 // .AsNoTracking() rend la requête beaucoup plus rapide pour de la lecture
                 var artistes = this.context.Artistes.AsNoTracking().ToList();
-                this.logger.LogInformation("{Count} artistes récupérés de la base.", artistes.Count);
+                this.logger.LogDebug("{Count} artistes récupérés de la base.", artistes.Count);
                 return artistes;
             }
             catch (Exception ex)
@@ -146,7 +147,7 @@ namespace Webzine.Repository
             {
                 this.context.Artistes.Update(artiste);
                 this.context.SaveChanges();
-                this.logger.LogInformation("Artiste {Id} ({Nom}) mis à jour avec succès.", artiste.IdArtiste, artiste.Nom);
+                this.logger.LogDebug("Artiste {Id} ({Nom}) mis à jour avec succès.", artiste.IdArtiste, artiste.Nom);
             }
             catch (DbUpdateException ex)
             {

Webzine.Repository/DbTitreRepository.cs

@@ -294,6 +294,7 @@ public class DbTitreRepository : ITitreRepository
             var titres = this.context.Titres
                 .Include(t => t.Artiste)
                 .Include(t => t.Styles)
+                .Include(t => t.Commentaires)
                 .OrderBy(t => t.Libelle)
                 .ToList();
 

Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs

@@ -1,29 +1,33 @@
 using Microsoft.AspNetCore.Mvc;
 using Webzine.Entity;
-using Webzine.Entity.Fixtures;
 using Webzine.Repository.Contracts;
 using Webzine.WebApplication.Areas.Administration.ViewModels.Artiste;
 
 namespace Webzine.WebApplication.Areas.Administration.Controllers;
 
+/// <summary>
+/// Contrôleur pour la gestion des artistes dans l'administration du webzine.
+/// </summary>
 [Area("Administration")]
 public class ArtisteController : Controller
 {
     // Injection du logger via le constructeur
     private readonly ILogger<ArtisteController> logger;
-    private readonly IArtisteRepository _artisteRepository;
+    private readonly IArtisteRepository artisteRepository;
-    private readonly List<Artiste> _artistes = new List<Artiste>();
 
-
+    /// <summary>
-    public ArtisteController(ILogger<ArtisteController> logger,
+    /// Initialise une nouvelle instance de la classe <see cref="ArtisteController"/>.
+    /// </summary>
+    /// <param name="logger">Logger.</param>
+    /// <param name="artisteRepository">Repository pour les artistes.</param>
+    public ArtisteController(
+        ILogger<ArtisteController> logger,
         IArtisteRepository artisteRepository)
     {
         this.logger = logger;
-        this.logger.LogDebug(1, "initialisation du ArtisteController d'administration");
+        this.artisteRepository = artisteRepository;
-        this._artisteRepository = artisteRepository;
-        this._artistes.AddRange(this._artisteRepository.FindAll());
     }
-    
+
     /// <summary>
     /// Affiche la liste des artistes. Pour l'instant, les artistes sont générés à partir de noms prédéfinis via la méthode SeedArtisteByName de la classe ArtisteFactory.
     /// Chaque artiste est ensuite ajouté à une liste d'artistes qui est passée à la vue.
@@ -31,10 +35,11 @@ public class ArtisteController : Controller
     /// <returns>Redirection.</returns>
     public IActionResult Index()
     {
+        IEnumerable<Artiste> artistes = this.artisteRepository.FindAll();
 
-        var _artistes_ordre = this._artistes.OrderBy(t => t.Nom).ToList();
+        var artistes_ordre = artistes.OrderBy(t => t.Nom).ToList();
 
-        return View(_artistes_ordre);
+        return this.View(artistes_ordre);
     }
 
     /// <summary>
@@ -47,10 +52,10 @@ public class ArtisteController : Controller
         {
             Id = 0,
             Nom = string.Empty,
-            Biographie = string.Empty
+            Biographie = string.Empty,
         };
 
-        return View(model);
+        return this.View(model);
     }
 
     /// <summary>
@@ -60,32 +65,51 @@ public class ArtisteController : Controller
     /// <returns>Redirection.</returns>
     public IActionResult Edit(int id)
     {
-        var artiste = this._artistes.First(t => t.IdArtiste == id);
+        var artiste = this.artisteRepository.Find(id);
 
         var model = new AdminArtisteForm
         {
             Id = artiste.IdArtiste,
             Nom = artiste.Nom,
-            Biographie = artiste.Biographie
+            Biographie = artiste.Biographie,
         };
-        
+
-        return View(model);
+        return this.View(model);
     }
 
     /// <summary>
     /// Renvoie à la page supprimer un artiste.
     /// </summary>
     /// <param name="id">L'identifiant de l'artiste à supprimer. </param>
-    /// <returns>Redirection.></returns>
+    /// <returns>Redirection.</returns>
     public IActionResult Delete(int id)
     {
-        var artiste = this._artistes.First(t => t.IdArtiste == id);
+        var artiste = this.artisteRepository.Find(id);
         var model = new AdminArtisteForm
         {
             Id = id,
             Nom = artiste.Nom,
-            Biographie = artiste.Biographie
+            Biographie = artiste.Biographie,
         };
-        return View(model);
+        return this.View(model);
+    }
+
+    /// <summary>
+    /// Méthode POST pour supprimer un artiste.
+    /// </summary>
+    /// <param name="model">L'artiste à supprimer.</param>
+    /// <returns>Redirige vers la page d'index d'admin artiste.</returns>
+    [HttpPost]
+    public IActionResult Delete(AdminArtisteForm model)
+    {
+        var artiste = this.artisteRepository.Find(model.Id);
+
+        if (artiste != null)
+        {
+            this.artisteRepository.Delete(artiste);
+        }
+
+        // 3. Redirect back to the list (or wherever you want them to go after)
+        return this.RedirectToAction("Index");
     }
 }

Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs

@@ -1,6 +1,4 @@
 using Microsoft.AspNetCore.Mvc;
-using Webzine.Entity;
-using Webzine.Entity.Fixtures;
 using Webzine.Repository.Contracts;
 using Webzine.WebApplication.Areas.Administration.ViewModels.Style;
 
@@ -13,20 +11,22 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
     public class StyleController : Controller
     {
         private readonly ILogger<StyleController> logger;
-        private readonly IStyleRepository styles;
+        private readonly IStyleRepository styleRepository;
 
         /// <summary>
         /// Initialise une nouvelle instance de la classe <see cref="StyleController"/>.
         /// </summary>
         /// <param name="logger">Service de journalisation injecté.</param>
         /// <param name="styles">Repository des styles injecté.</param>
-        public StyleController(ILogger<StyleController> logger, IStyleRepository styles)
+        public StyleController(
+            ILogger<StyleController> logger,
+            IStyleRepository styleRepository)
         {
             this.logger = logger;
 
             this.logger.LogInformation("Initialisation du contrôleur StyleController.");
 
-            this.styles = styles;
+            this.styleRepository = styleRepository;
         }
 
         /// <summary>
@@ -35,7 +35,7 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
         /// <returns>La vue Index avec le ViewModel contenant la liste des styles.</returns>
         public IActionResult Index()
         {
-            var listeStyles = this.styles.FindAll().Take(10);
+            var listeStyles = this.styleRepository.FindAll().Take(10);
 
             return this.View(listeStyles);
         }
@@ -56,7 +56,7 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
         /// <returns>La vue de confirmation de suppression avec le ViewModel contenant les détails du style à supprimer, ou une redirection vers l'index si le style n'existe pas.</returns>
         public IActionResult Delete(int id)
         {
-            var style = this.styles.Find(id);
+            var style = this.styleRepository.Find(id);
 
             var vm = new StyleDeleteViewModel
             {
@@ -67,6 +67,24 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
             return this.View(vm);
         }
 
+        /// <summary>
+        /// Méthode POST pour supprimer un style.
+        /// </summary>
+        /// <param name="model">Le style à supprimer.</param>
+        /// <returns>Redirige vers la page d'index d'admin style.</returns>
+        [HttpPost]
+        public IActionResult Delete(StyleEditViewModel model)
+        {
+            var style = this.styleRepository.Find(model.IdStyle);
+
+            if (style != null)
+            {
+                this.styleRepository.Delete(style);
+            }
+
+            return this.RedirectToAction("Index");
+        }
+
         /// <summary>
         /// Affiche la vue d'édition d'un style existant, en récupérant les détails du style à éditer à partir de l'identifiant fourni.
         /// </summary>
@@ -75,7 +93,7 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
         [HttpGet]
         public IActionResult Edit(int id)
         {
-            var style = styles.Find(id);
+            var style = styleRepository.Find(id);
 
             var model = new StyleEditViewModel
             {

Webzine.WebApplication/Areas/Administration/Controllers/TitreController.cs

@@ -29,9 +29,6 @@ public class TitreController : Controller
     public TitreController(ILogger<TitreController> logger, ITitreRepository titreRepository, IArtisteRepository artisteRepository, IStyleRepository styleRepository)
     {
         this.logger = logger;
-
-        this.logger.LogInformation("Initialisation du contrôleur TitreController pour l'Administration.");
-
         this.titreRepository = titreRepository;
         this.artisteRepository = artisteRepository;
         this.styleRepository = styleRepository;
@@ -140,4 +137,21 @@ public class TitreController : Controller
 
         return this.View(model);
     }
+
+    /// <summary>
+    /// Méthode POST pour supprimer un titre.
+    /// </summary>
+    /// <param name="model">Le titre à supprimer.</param>
+    /// <returns>Redirige vers la page d'index d'admin titre.</returns>
+    [HttpPost]
+    public IActionResult Delete(AdminTitreDelete model)
+    {
+        var titre = this.titreRepository.Find(model.Id);
+        if (titre != null)
+        {
+            this.titreRepository.Delete(titre);
+        }
+
+        return this.RedirectToAction("Index");
+    }
 }

Webzine.WebApplication/Areas/Administration/Views/Artiste/Index.cshtml

@@ -33,7 +33,9 @@
                                 <i class="fa fa-edit"></i>
                             </a>
 
-                            <a asp-action="Delete" asp-route-id="@artiste.IdArtiste">
+                            <a asp-action="Delete"
+                                asp-controller="Artiste"
+                                asp-route-id="@artiste.IdArtiste">
                                 <i class="fa fa-trash"></i>
                             </a>