Merge branch 'j2/feat/artiste_repository' into j2/refactor/controler-style-titre
This commit is contained in:
@@ -1,21 +1,26 @@
|
||||
using Microsoft.AspNetCore.Mvc;
|
||||
using Webzine.Entity.Fixtures;
|
||||
|
||||
namespace Webzine.WebApplication.Controllers
|
||||
namespace Webzine.WebApplication.Controllers
|
||||
{
|
||||
using Microsoft.AspNetCore.Mvc;
|
||||
using Webzine.Entity.Fixtures;
|
||||
using Webzine.Repository.Contracts;
|
||||
using Webzine.WebApplication.ViewModels.Artiste;
|
||||
|
||||
public class ArtisteController : Controller
|
||||
{
|
||||
// Injection du logger via le constructeur
|
||||
private readonly ILogger<ArtisteController> logger;
|
||||
private readonly ILogger<ArtisteController> _logger;
|
||||
private readonly IArtisteRepository _artisteRepository;
|
||||
|
||||
/// <summary>
|
||||
/// Initialise une nouvelle instance de la classe <see cref="ArtisteController"/>.
|
||||
/// Initialise une nouvelle instance du <see cref="ArtisteController"/>. avec un service de journalisation injecté.
|
||||
/// </summary>
|
||||
/// <param name="logger">Service de journalisation injecté pour enregistrer les événements et les erreurs.</param>
|
||||
public ArtisteController(ILogger<ArtisteController> logger)
|
||||
public ArtisteController(ILogger<ArtisteController> logger,
|
||||
IArtisteRepository artisteRepository)
|
||||
{
|
||||
this.logger = logger;
|
||||
this.logger.LogDebug(1, "initialisation du ArtisteController");
|
||||
this._logger = logger;
|
||||
this._logger.LogDebug("Initialisation du ArtisteController");
|
||||
this._artisteRepository = artisteRepository;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
@@ -30,8 +35,8 @@ namespace Webzine.WebApplication.Controllers
|
||||
|
||||
if (string.IsNullOrEmpty(nom))
|
||||
{
|
||||
this.logger.LogWarning("Nom de l'artiste manquant dans la requête.");
|
||||
return this.RedirectToAction("Index", "Accueil");
|
||||
this._logger.LogWarning("Nom de l'artiste manquant dans la requête.");
|
||||
return RedirectToAction("Index");
|
||||
}
|
||||
|
||||
// On transforme "fatal-bazooka" en "Fatal Bazooka" pour la factory
|
||||
@@ -39,11 +44,28 @@ namespace Webzine.WebApplication.Controllers
|
||||
.ToTitleCase(nom.Replace("-", " "));
|
||||
|
||||
// On appelle la factory pour obtenir l'artiste unique
|
||||
var artiste = ArtisteFactory.SeedArtisteByName(nomPropre);
|
||||
var artiste = this._artisteRepository.FindByName(nomPropre);
|
||||
|
||||
this.logger.LogInformation("Artiste trouvé : {NomArtiste}", nom);
|
||||
if (artiste == null)
|
||||
{
|
||||
this._logger.LogWarning("Artiste non trouvé pour le nom : {NomArtiste}", nomPropre);
|
||||
return RedirectToAction("Index");
|
||||
}
|
||||
var viewModel = new ArtisteDetailsViewModel
|
||||
{
|
||||
IdArtiste = artiste.IdArtiste,
|
||||
Nom = artiste.Nom,
|
||||
Biographie = artiste.Biographie,
|
||||
// On effectue le groupement ici une bonne fois pour toutes
|
||||
AlbumsGroupes = artiste.Titres
|
||||
.OrderBy(t => t.Libelle)
|
||||
.GroupBy(t => t.Album)
|
||||
.OrderBy(g => g.Key),
|
||||
};
|
||||
|
||||
return this.View(artiste);
|
||||
this._logger.LogInformation("Artiste trouvé : {NomArtiste}", nom);
|
||||
|
||||
return View(viewModel);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user