diff --git a/Webzine.Entity/Artiste.cs b/Webzine.Entity/Artiste.cs index 8f1df08..5cd404e 100644 --- a/Webzine.Entity/Artiste.cs +++ b/Webzine.Entity/Artiste.cs @@ -1,8 +1,4 @@ -using System; -using System.Collections.Generic; -using System.ComponentModel.DataAnnotations; -using System.Text; -using System.Timers; +using System.ComponentModel.DataAnnotations; namespace Webzine.Entity { diff --git a/Webzine.Entity/Commentaire.cs b/Webzine.Entity/Commentaire.cs index baaa852..b29c14e 100644 --- a/Webzine.Entity/Commentaire.cs +++ b/Webzine.Entity/Commentaire.cs @@ -1,7 +1,4 @@ -using System; -using System.Collections.Generic; -using System.ComponentModel.DataAnnotations; -using System.Text; +using System.ComponentModel.DataAnnotations; namespace Webzine.Entity { diff --git a/Webzine.Entity/Style.cs b/Webzine.Entity/Style.cs index 628dfec..fcb010c 100644 --- a/Webzine.Entity/Style.cs +++ b/Webzine.Entity/Style.cs @@ -1,7 +1,4 @@ -using System; -using System.Collections.Generic; -using System.ComponentModel.DataAnnotations; -using System.Text; +using System.ComponentModel.DataAnnotations; namespace Webzine.Entity { diff --git a/Webzine.Entity/Titre.cs b/Webzine.Entity/Titre.cs index a5fa1ae..ccb8c63 100644 --- a/Webzine.Entity/Titre.cs +++ b/Webzine.Entity/Titre.cs @@ -1,7 +1,4 @@ -using System; -using System.Collections.Generic; -using System.ComponentModel.DataAnnotations; -using System.Text; +using System.ComponentModel.DataAnnotations; namespace Webzine.Entity { diff --git a/Webzine.Repository/LocalEntityRepository.cs b/Webzine.Repository/LocalEntityRepository.cs index 8095e32..f9684e7 100644 --- a/Webzine.Repository/LocalEntityRepository.cs +++ b/Webzine.Repository/LocalEntityRepository.cs @@ -20,8 +20,8 @@ public class LocalEntityRepository : ITitreRepository /// Service de journalisation injecte pour suivre les operations du repository. public LocalEntityRepository(ILogger logger) { - _logger = logger; - _logger.LogDebug(1, "NLog injected into LocalEntityRepository"); + this._logger = logger; + this._logger.LogDebug(1, "NLog injected into LocalEntityRepository"); var factory = new DataFactory(); var artistes = factory.GenerateArtists(10); diff --git a/Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs b/Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs index 05bfeb0..d059092 100644 --- a/Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs +++ b/Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs @@ -1,9 +1,7 @@ using Microsoft.AspNetCore.Mvc; -using Microsoft.AspNetCore.Mvc.Rendering; using Webzine.Entity; using Webzine.Entity.Fixtures; using Webzine.WebApplication.Areas.Administration.ViewModels.Artiste; -using Webzine.WebApplication.Areas.Administration.ViewModels.Titre; namespace Webzine.WebApplication.Areas.Administration.Controllers; @@ -17,15 +15,16 @@ public class ArtisteController : Controller public ArtisteController(ILogger logger) { - _logger = logger; + this._logger = logger; this._logger.LogDebug(1, "initialisation du ArtisteController d'administration"); var factory = new DataFactory(); _artistes = factory.GenerateArtists(10); } + /// - /// 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 via un objet GroupeArtisteViewModel. + /// 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. /// /// Redirection. public IActionResult Index() @@ -33,14 +32,9 @@ public class ArtisteController : Controller var _artistes_ordre = _artistes.OrderBy(t => t.Nom).ToList(); - _logger.LogInformation("Initialisation du contrôleur TitreController pour l'Administration."); + this._logger.LogInformation("Initialisation du contrôleur TitreController pour l'Administration."); - GroupeArtisteViewModel groupeArtisteModel = new GroupeArtisteViewModel - { - Artistes = _artistes_ordre - }; - - return View(groupeArtisteModel); + return View(_artistes_ordre); } /// diff --git a/Webzine.WebApplication/Areas/Administration/Controllers/CommentaireController.cs b/Webzine.WebApplication/Areas/Administration/Controllers/CommentaireController.cs index 7c1a17f..e8d4ce3 100644 --- a/Webzine.WebApplication/Areas/Administration/Controllers/CommentaireController.cs +++ b/Webzine.WebApplication/Areas/Administration/Controllers/CommentaireController.cs @@ -18,9 +18,9 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers /// Service de journalisation injecté. public CommentaireController(ILogger logger) { - _logger = logger; + this._logger = logger; - _logger.LogInformation("Initialisation du contrôleur CommentaireController."); + this._logger.LogInformation("Initialisation du contrôleur CommentaireController."); var factory = new DataFactory(); // TODO injecter le factory via DI pour éviter de le recréer à chaque fois // faire une classe statique @@ -31,7 +31,7 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers _commentaires = factory.GenerateCommentaires(50, _titres); - _logger.LogInformation("Données fictives générées avec succès."); + this._logger.LogInformation("Données fictives générées avec succès."); } /// @@ -84,15 +84,17 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers /// Affiche la vue de confirmation de suppression d'un commentaire, en récupérant les détails du commentaire à supprimer à partir de l'identifiant fourni. /// /// L'identifiant du commentaire à supprimer. - /// La vue de confirmation de suppression avec le ViewModel contenant les détails du commentaire à supprimer, ou une réponse NotFound si le commentaire n'existe pas. + /// La vue de confirmation de suppression avec le ViewModel contenant les détails du commentaire à supprimer, ou une redirection vers l'index si le commentaire n'existe pas. public IActionResult Delete(int id) { var commentaire = _commentaires .FirstOrDefault(c => c.IdCommentaire == id); if (commentaire == null) - return NotFound(); // TODO faire une page d'erreur personnalisée pour les 404, et pas juste un message d'erreur dans la console - // mettre des accolades dans les if pour éviter les erreurs de maintenance, même pour une seule ligne de code + { + this._logger.LogWarning("Commentaire avec ID {Id} introuvable pour suppression.", id); + return RedirectToAction("Index"); + } var vm = new CommentaireDeleteViewModel { diff --git a/Webzine.WebApplication/Areas/Administration/Controllers/DashboardController.cs b/Webzine.WebApplication/Areas/Administration/Controllers/DashboardController.cs index 128f8bb..a2ac740 100644 --- a/Webzine.WebApplication/Areas/Administration/Controllers/DashboardController.cs +++ b/Webzine.WebApplication/Areas/Administration/Controllers/DashboardController.cs @@ -20,9 +20,9 @@ public class DashboardController : Controller /// Service de journalisation injecté. public DashboardController(ILogger logger) { - _logger = logger; + this._logger = logger; - _logger.LogInformation("Initialisation du contrôleur TitreController."); + this._logger.LogInformation("Initialisation du contrôleur TitreController."); var factory = new DataFactory(); @@ -32,7 +32,7 @@ public class DashboardController : Controller factory.GenerateCommentaires(50, _titres); - _logger.LogInformation("Données fictives générées avec succès."); + this._logger.LogInformation("Données fictives générées avec succès."); } /// diff --git a/Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs b/Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs index 6eb3c39..34f6cd1 100644 --- a/Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs +++ b/Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs @@ -21,39 +21,22 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers /// Service de journalisation injecté. public StyleController(ILogger logger) { - _logger = logger; + this._logger = logger; - _logger.LogInformation("Initialisation du contrôleur StyleController."); + this._logger.LogInformation("Initialisation du contrôleur StyleController."); var factory = new DataFactory(); _styles = factory.GenerateStyles(10); - _logger.LogInformation("Données fictives générées avec succès."); + this._logger.LogInformation("Données fictives générées avec succès."); } // GET: Administration/Styles public IActionResult Index() { // Création de données "bouchon" (mock) pour tester l'affichage - var listeStyles = new List