Merge remote-tracking branch 'origin/j2/refactor/controler-style-titre' into j2/ajout_repo
This commit is contained in:
@@ -2,6 +2,8 @@
|
||||
// Copyright (c) Equipe 1 - . All rights reserved.
|
||||
// </copyright>
|
||||
|
||||
using Webzine.Repository.Contracts;
|
||||
|
||||
namespace Webzine.WebApplication.Controllers
|
||||
{
|
||||
using Microsoft.AspNetCore.Mvc;
|
||||
@@ -30,6 +32,7 @@ namespace Webzine.WebApplication.Controllers
|
||||
{
|
||||
this.logger = logger;
|
||||
this.configuration = configuration;
|
||||
this.titreRepository = titreRepository;
|
||||
this.logger.LogDebug(1, "initialisation du AccueilController");
|
||||
this.titreRepository = titreRepository;
|
||||
}
|
||||
|
||||
@@ -4,26 +4,16 @@ namespace Webzine.WebApplication.Controllers;
|
||||
|
||||
public class ApiController : ControllerBase
|
||||
{
|
||||
private readonly ILogger<ApiController> _logger;
|
||||
private readonly ILogger<ApiController> logger;
|
||||
|
||||
/// <summary>
|
||||
/// Initialise une nouvelle instance du <see cref="ApiController"/> avec un service de journalisation injecté.
|
||||
/// Initialise une nouvelle instance de la classe <see cref="ApiController"/>.
|
||||
/// </summary>
|
||||
/// <param name="logger">Service de journalisation injecté pour enregistrer les événements et les erreurs.</param>
|
||||
public ApiController(ILogger<ApiController> logger)
|
||||
{
|
||||
this._logger = logger;
|
||||
this._logger.LogDebug(1, "initialisation du ApiController");
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Endpoint de test pour vérifier que l'API fonctionne correctement. Retourne une chaîne de caractères "Hello World !".
|
||||
/// </summary>
|
||||
/// <returns>Une chaîne de caractères "Hello World !".</returns>
|
||||
[HttpGet]
|
||||
public string HelloWorld()
|
||||
{
|
||||
return "Hello World !";
|
||||
this.logger = logger;
|
||||
this.logger.LogDebug(1, "initialisation du ApiController");
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
@@ -33,7 +23,7 @@ public class ApiController : ControllerBase
|
||||
[HttpGet]
|
||||
public IActionResult Version()
|
||||
{
|
||||
this._logger.LogInformation("Get Version was called");
|
||||
this.logger.LogInformation("Get Version was called");
|
||||
|
||||
return Ok(new
|
||||
{
|
||||
|
||||
@@ -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 IArtisteRepository _artisteRepository;
|
||||
|
||||
/// <summary>
|
||||
/// Initialise une nouvelle instance du <see cref="ArtisteController"/> avec un service de journalisation injecté.
|
||||
/// 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.LogDebug("Initialisation du ArtisteController");
|
||||
this._artisteRepository = artisteRepository;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
@@ -26,12 +31,12 @@ namespace Webzine.WebApplication.Controllers
|
||||
[HttpGet("/artiste/{nom}")]
|
||||
public IActionResult Index(string nom)
|
||||
{
|
||||
this._logger.LogInformation("Tentative d'accès à l'artiste avec le nom : {NomArtiste}", nom);
|
||||
this.logger.LogInformation("Tentative d'accès à l'artiste avec le nom : {NomArtiste}", nom);
|
||||
|
||||
if (string.IsNullOrEmpty(nom))
|
||||
{
|
||||
this._logger.LogWarning("Nom de l'artiste manquant dans la requête.");
|
||||
return RedirectToAction("Index", "Accueil");
|
||||
return RedirectToAction("Index");
|
||||
}
|
||||
|
||||
// On transforme "fatal-bazooka" en "Fatal Bazooka" pour la factory
|
||||
@@ -39,17 +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);
|
||||
|
||||
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),
|
||||
};
|
||||
|
||||
this._logger.LogInformation("Artiste trouvé : {NomArtiste}", nom);
|
||||
|
||||
return View(artiste);
|
||||
return View(viewModel);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -8,16 +8,16 @@ namespace Webzine.WebApplication.Controllers
|
||||
public class ContactController : Controller
|
||||
{
|
||||
// Injection du logger via le constructeur
|
||||
private readonly ILogger<ContactController> _logger;
|
||||
private readonly ILogger<ContactController> logger;
|
||||
|
||||
/// <summary>
|
||||
/// Initialise une nouvelle instance du <see cref="ContactController"/> avec un service de journalisation injecté.
|
||||
/// Initialise une nouvelle instance de la classe <see cref="ContactController"/>.
|
||||
/// </summary>
|
||||
/// <param name="logger">Service de journalisation injecté pour enregistrer les événements et les erreurs.</param>
|
||||
public ContactController(ILogger<ContactController> logger)
|
||||
{
|
||||
this._logger = logger;
|
||||
this._logger.LogDebug(1, "initialisation du ContactController");
|
||||
this.logger = logger;
|
||||
this.logger.LogDebug(1, "initialisation du ContactController");
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
@@ -26,7 +26,7 @@ namespace Webzine.WebApplication.Controllers
|
||||
/// <returns>La vue Index de la page de contact.</returns>
|
||||
public IActionResult Index()
|
||||
{
|
||||
return View();
|
||||
return this.View();
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -17,7 +17,7 @@ public class TitreController : Controller
|
||||
private readonly ITitreRepository titreRepository;
|
||||
|
||||
/// <summary>
|
||||
/// Initialise une nouvelle instance du <see cref="TitreController"/>.
|
||||
/// Initialise une nouvelle instance de la classe <see cref="TitreController"/>.
|
||||
/// </summary>
|
||||
/// <param name="logger">Service de journalisation injecte.</param>
|
||||
/// <param name="titreRepository">Repository des titres injecte.</param>
|
||||
@@ -58,7 +58,7 @@ public class TitreController : Controller
|
||||
NbLikes = titre.NbLikes,
|
||||
UrlJaquette = titre.UrlJaquette,
|
||||
UrlEcoute = titre.UrlEcoute,
|
||||
ArtisteNom = titre.Artiste?.Nom,
|
||||
ArtisteNom = titre.Artiste.Nom,
|
||||
Styles = titre.Styles,
|
||||
Commentaires = titre.Commentaires,
|
||||
},
|
||||
@@ -81,7 +81,11 @@ public class TitreController : Controller
|
||||
{
|
||||
this.logger.LogInformation("Recherche des titres pour le style : {Style}.", style);
|
||||
|
||||
<<<<<<< HEAD
|
||||
var titresFiltres = this.titreRepository.SearchByStyle(style).ToList();
|
||||
=======
|
||||
var titresFiltres = this._titreRepository.SearchByStyle(style).ToList();
|
||||
>>>>>>> origin/j2/refactor/controler-style-titre
|
||||
|
||||
var vm = new TitreStyle
|
||||
{
|
||||
@@ -106,7 +110,11 @@ public class TitreController : Controller
|
||||
|
||||
if (titre == null)
|
||||
{
|
||||
<<<<<<< HEAD
|
||||
this.logger.LogWarning("Impossible d'ajouter un like. Titre ID {Id} introuvable.", model.IdTitre);
|
||||
=======
|
||||
this._logger.LogWarning("Impossible d'ajouter un like. Titre ID {Id} introuvable.", model.IdTitre);
|
||||
>>>>>>> origin/j2/refactor/controler-style-titre
|
||||
return this.RedirectToAction("Index");
|
||||
}
|
||||
|
||||
@@ -125,7 +133,11 @@ public class TitreController : Controller
|
||||
{
|
||||
if (!this.ModelState.IsValid)
|
||||
{
|
||||
<<<<<<< HEAD
|
||||
this.logger.LogWarning("Echec de validation du modele de commentaire pour le titre ID {Id}.", model.IdTitre);
|
||||
=======
|
||||
this._logger.LogWarning("Echec de validation du modele de commentaire pour le titre ID {Id}.", model.IdTitre);
|
||||
>>>>>>> origin/j2/refactor/controler-style-titre
|
||||
return this.RedirectToAction("Details", new { id = model.IdTitre });
|
||||
}
|
||||
|
||||
@@ -133,7 +145,11 @@ public class TitreController : Controller
|
||||
|
||||
if (titre == null)
|
||||
{
|
||||
<<<<<<< HEAD
|
||||
this.logger.LogWarning("Impossible d'ajouter le commentaire. Titre ID {Id} introuvable.", model.IdTitre);
|
||||
=======
|
||||
this._logger.LogWarning("Impossible d'ajouter le commentaire. Titre ID {Id} introuvable.", model.IdTitre);
|
||||
>>>>>>> origin/j2/refactor/controler-style-titre
|
||||
return this.RedirectToAction("Index");
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user