#117 Mise en place de la suppression d'un artiste. La DbContext a été modifiée pour supprimer les titres d'un artiste à la suppression d'un artiste.

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.LogInformation("L'artiste {IdArtiste} a bien été supprimé", artiste.IdArtiste);
             }
             catch (DbUpdateException dbex)
             {

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 List<Artiste> _artistes = new List<Artiste>();
+    private readonly IArtisteRepository artisteRepository;
 
-
-    public ArtisteController(ILogger<ArtisteController> logger,
+    /// <summary>
+    /// 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._artistes.AddRange(this._artisteRepository.FindAll());
+        this.artisteRepository = artisteRepository;
     }
-    
+
     /// <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 View(artistes_ordre);
     }
 
     /// <summary>
@@ -47,7 +52,7 @@ public class ArtisteController : Controller
         {
             Id = 0,
             Nom = string.Empty,
-            Biographie = string.Empty
+            Biographie = string.Empty,
         };
 
         return View(model);
@@ -60,15 +65,15 @@ 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);
     }
 
@@ -79,13 +84,32 @@ public class ArtisteController : Controller
     /// <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);
     }
+
+    /// <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/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>