From 352328f44e9ad62a151d4c401c63db87f801b7b6 Mon Sep 17 00:00:00 2001
From: "josephine.vetu" <josephinevetu.mj@gmail.com>
Date: Thu, 26 Mar 2026 18:51:38 +0100
Subject: [PATCH] =?UTF-8?q?#117=20Mise=20en=20place=20de=20la=20suppressio?=
 =?UTF-8?q?n=20d'un=20artiste.=20La=20DbContext=20a=20=C3=A9t=C3=A9=20modi?=
 =?UTF-8?q?fi=C3=A9e=20pour=20supprimer=20les=20titres=20d'un=20artiste=20?=
 =?UTF-8?q?=C3=A0=20la=20suppression=20d'un=20artiste.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Webzine.EntitiesContext/WebzineDbContext.cs   |  2 +-
 .../IArtisteRepository.cs                     |  1 +
 Webzine.Repository/DbArtisteRepository.cs     |  1 +
 .../Controllers/ArtisteController.cs          | 58 +++++++++++++------
 .../Administration/Views/Artiste/Index.cshtml |  4 +-
 5 files changed, 47 insertions(+), 19 deletions(-)

diff --git a/Webzine.EntitiesContext/WebzineDbContext.cs b/Webzine.EntitiesContext/WebzineDbContext.cs
index 2ec5aa1..df64945 100644
--- a/Webzine.EntitiesContext/WebzineDbContext.cs
+++ b/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 =>
diff --git a/Webzine.Repository.Contracts/IArtisteRepository.cs b/Webzine.Repository.Contracts/IArtisteRepository.cs
index 2f26de5..135eb4b 100644
--- a/Webzine.Repository.Contracts/IArtisteRepository.cs
+++ b/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>
diff --git a/Webzine.Repository/DbArtisteRepository.cs b/Webzine.Repository/DbArtisteRepository.cs
index 9091e2a..45e6a56 100644
--- a/Webzine.Repository/DbArtisteRepository.cs
+++ b/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)
             {
diff --git a/Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs b/Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs
index 1e34b15..f1c3e88 100644
--- a/Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs
+++ b/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");
+    }
 }
\ No newline at end of file
diff --git a/Webzine.WebApplication/Areas/Administration/Views/Artiste/Index.cshtml b/Webzine.WebApplication/Areas/Administration/Views/Artiste/Index.cshtml
index 73aabac..bb36638 100644
--- a/Webzine.WebApplication/Areas/Administration/Views/Artiste/Index.cshtml
+++ b/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>