# 123 fix 1

This commit is contained in:
b.nodon
2026-03-27 11:22:52 +01:00
parent a3db0059ab
commit b53fa189a5
3 changed files with 15 additions and 21 deletions

View File

@@ -31,12 +31,7 @@ public class DbCommentaireRepository : ICommentaireRepository
{ {
try try
{ {
if (commentaire == null) this.logger.LogDebug("Ajout d'un nouveau commentaire de l'auteur : {Auteur}", commentaire.Auteur);
{
throw new ArgumentNullException(nameof(commentaire), "Le commentaire à ajouter ne peut pas être null.");
}
this.logger.LogInformation("Ajout d'un nouveau commentaire de l'auteur : {Auteur}", commentaire.Auteur);
this.context.Commentaires.Add(commentaire); this.context.Commentaires.Add(commentaire);
this.context.SaveChanges(); this.context.SaveChanges();
this.logger.LogDebug("Commentaire ajouté avec l'id : {Id}", commentaire.IdCommentaire); this.logger.LogDebug("Commentaire ajouté avec l'id : {Id}", commentaire.IdCommentaire);
@@ -65,7 +60,7 @@ public class DbCommentaireRepository : ICommentaireRepository
this.context.Commentaires.Remove(commentaire); this.context.Commentaires.Remove(commentaire);
this.context.SaveChanges(); this.context.SaveChanges();
this.logger.LogInformation("Le commentaire {IdCommentaire} a bien été supprimé", commentaire.IdCommentaire); this.logger.LogDebug("Le commentaire {IdCommentaire} a bien été supprimé", commentaire.IdCommentaire);
} }
catch (DbUpdateException dbex) catch (DbUpdateException dbex)
{ {
@@ -131,7 +126,7 @@ public class DbCommentaireRepository : ICommentaireRepository
/// <inheritdoc/> /// <inheritdoc/>
public IEnumerable<Commentaire> FindByIdTitre(int idTitre) public IEnumerable<Commentaire> FindByIdTitre(int idTitre)
{ {
this.logger.LogInformation("Recherche des commentaires pour le titre ID : {IdTitre}", idTitre); this.logger.LogDebug("Recherche des commentaires pour le titre ID : {IdTitre}", idTitre);
var commentaires = this.context.Commentaires var commentaires = this.context.Commentaires
.Where(c => c.Titre.IdTitre == idTitre) .Where(c => c.Titre.IdTitre == idTitre)

View File

@@ -53,7 +53,6 @@ namespace Webzine.Repository
/// <inheritdoc/> /// <inheritdoc/>
public Commentaire Find(int idCommentaire) public Commentaire Find(int idCommentaire)
{ {
var commentaire = this.dataStore.Commentaires.FirstOrDefault(c => c.IdCommentaire == idCommentaire); var commentaire = this.dataStore.Commentaires.FirstOrDefault(c => c.IdCommentaire == idCommentaire);
if (commentaire == null) if (commentaire == null)
{ {

View File

@@ -8,8 +8,8 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
[Area("Administration")] [Area("Administration")]
public class CommentaireController : Controller public class CommentaireController : Controller
{ {
private readonly ILogger<CommentaireController> _logger; private readonly ILogger<CommentaireController> logger;
private readonly ICommentaireRepository _commentaireRepository; private readonly ICommentaireRepository commentaireRepository;
/// <summary> /// <summary>
/// Initialise une nouvelle instance de la classe <see cref="CommentaireController"/>. /// Initialise une nouvelle instance de la classe <see cref="CommentaireController"/>.
@@ -19,10 +19,10 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
/// <param name="commentaireRepository">Le repository des commentaires injecté.</param> /// <param name="commentaireRepository">Le repository des commentaires injecté.</param>
public CommentaireController(ILogger<CommentaireController> logger, ICommentaireRepository commentaireRepository) public CommentaireController(ILogger<CommentaireController> logger, ICommentaireRepository commentaireRepository)
{ {
this._logger = logger; this.logger = logger;
this._commentaireRepository = commentaireRepository; this.commentaireRepository = commentaireRepository;
this._logger.LogInformation("Initialisation du contrôleur CommentaireController."); this.logger.LogInformation("Initialisation du contrôleur CommentaireController.");
} }
/// <summary> /// <summary>
@@ -32,7 +32,7 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
public IActionResult Index() public IActionResult Index()
{ {
// Récupération des commentaires depuis le repository // Récupération des commentaires depuis le repository
var commentaires = this._commentaireRepository.FindAll().ToList(); var commentaires = this.commentaireRepository.FindAll().ToList();
// Initialisation du ViewModel // Initialisation du ViewModel
var viewModel = new CommentaireViewModel var viewModel = new CommentaireViewModel
@@ -50,11 +50,11 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
/// <returns>La vue de confirmation de suppression avec le ViewModel contenant les détails, ou une redirection vers l'index si introuvable.</returns> /// <returns>La vue de confirmation de suppression avec le ViewModel contenant les détails, ou une redirection vers l'index si introuvable.</returns>
public IActionResult Delete(int id) public IActionResult Delete(int id)
{ {
var commentaire = this._commentaireRepository.Find(id); var commentaire = this.commentaireRepository.Find(id);
if (commentaire == null) if (commentaire == null)
{ {
this._logger.LogWarning("Commentaire avec ID {Id} introuvable pour suppression.", id); this.logger.LogWarning("Commentaire avec ID {Id} introuvable pour suppression.", id);
return RedirectToAction("Index"); return RedirectToAction("Index");
} }
@@ -77,16 +77,16 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
/// <returns>Redirection vers la vue Index après suppression.</returns> /// <returns>Redirection vers la vue Index après suppression.</returns>
public IActionResult DeleteConfirm(int id) public IActionResult DeleteConfirm(int id)
{ {
var commentaire = this._commentaireRepository.Find(id); var commentaire = this.commentaireRepository.Find(id);
if (commentaire != null) if (commentaire != null)
{ {
this._commentaireRepository.Delete(commentaire); this.commentaireRepository.Delete(commentaire);
this._logger.LogInformation("Commentaire {Id} supprimé avec succès.", id); this.logger.LogInformation("Commentaire {Id} supprimé avec succès.", id);
} }
else else
{ {
this._logger.LogWarning("Échec de la suppression : Commentaire avec ID {Id} introuvable.", id); this.logger.LogWarning("Échec de la suppression : Commentaire avec ID {Id} introuvable.", id);
} }
return RedirectToAction("Index"); return RedirectToAction("Index");