Merge branch 'dev' into j2/test/pr

# Conflicts:
#	Webzine.WebApplication/appsettings.json
This commit is contained in:
mirage
2026-03-27 13:05:56 +01:00
8 changed files with 285 additions and 48 deletions

View File

@@ -11,6 +11,7 @@
<PackageReference Include="Microsoft.EntityFrameworkCore" Version="10.0.5" />
<PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="10.0.5" />
<PackageReference Include="NLog" Version="6.1.1" />
<PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="10.0.1" />
<PackageReference Include="StyleCop.Analyzers" Version="1.1.118">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>

View File

@@ -43,6 +43,8 @@ namespace Webzine.EntitiesContext
modelBuilder.Entity<Artiste>(entity =>
{
entity.ToTable("Artistes");
entity.HasKey(a => a.IdArtiste)
.HasName("PK_Artiste");
@@ -54,12 +56,16 @@ namespace Webzine.EntitiesContext
modelBuilder.Entity<Style>(entity =>
{
entity.ToTable("Styles");
entity.HasKey(s => s.IdStyle)
.HasName("PK_Style");
});
modelBuilder.Entity<Commentaire>(entity =>
{
entity.ToTable("Commentaires");
entity.HasKey(c => c.IdCommentaire)
.HasName("PK_Commentaire");
@@ -71,6 +77,8 @@ namespace Webzine.EntitiesContext
modelBuilder.Entity<Titre>(entity =>
{
entity.ToTable("Titres");
entity.HasKey(t => t.IdTitre)
.HasName("PK_Titre");

View File

@@ -44,8 +44,8 @@ namespace Webzine.Entity.Fixtures
Faker<Titre> faker = new Faker<Titre>("fr")
.RuleFor(t => t.Libelle, f => f.Lorem.Sentence(3).Replace(".", string.Empty))
.RuleFor(t => t.Chronique, f => f.Lorem.Paragraphs(3))
.RuleFor(t => t.DateCreation, f => f.Date.Recent(120))
.RuleFor(t => t.DateSortie, (f, t) => f.Date.Past(10, t.DateCreation))
.RuleFor(t => t.DateCreation, f => DateTime.SpecifyKind(f.Date.Recent(120), DateTimeKind.Utc))
.RuleFor(t => t.DateSortie, (f, t) => DateTime.SpecifyKind(f.Date.Past(10, t.DateCreation), DateTimeKind.Utc))
.RuleFor(t => t.Duree, f => f.Random.Int(120, 420))
.RuleFor(t => t.UrlJaquette, f => $"https://picsum.photos/seed/{Guid.NewGuid():N}/640/640")
.RuleFor(t => t.UrlEcoute, f => $"https://example.com/listen/{Guid.NewGuid():N}")
@@ -129,7 +129,7 @@ namespace Webzine.Entity.Fixtures
Faker<Commentaire> faker = new Faker<Commentaire>("fr")
.RuleFor(c => c.Auteur, f => f.Internet.UserName())
.RuleFor(c => c.Contenu, f => f.Lorem.Sentences(2))
.RuleFor(c => c.DateCreation, f => f.Date.Recent(60))
.RuleFor(c => c.DateCreation, f => DateTime.SpecifyKind(f.Date.Recent(60), DateTimeKind.Utc))
.RuleFor(c => c.Titre, _ => titre)
.RuleFor(c => c.IdTitre, _ => titre.IdTitre);

View File

@@ -0,0 +1,139 @@
using Microsoft.EntityFrameworkCore;
using Webzine.EntitiesContext;
using Microsoft.Extensions.Logging;
using Webzine.Entity;
using Webzine.Repository.Contracts;
namespace Webzine.Repository;
/// <summary>
/// Classe qui implémente le repository pour les commentaires en utilisant une base de données.
/// </summary>
public class DbCommentaireRepository : ICommentaireRepository
{
private readonly ILogger<DbCommentaireRepository> logger;
private readonly WebzineDbContext context;
/// <summary>
/// Initialisation de <see cref="DbCommentaireRepository"/>.
/// </summary>
/// <param name="logger">Le service de journalisation injecté pour suivre les opérations du repository.</param>
/// <param name="context">Le contexte de base de données injecté.</param>
public DbCommentaireRepository(ILogger<DbCommentaireRepository> logger, WebzineDbContext context)
{
this.logger = logger;
this.context = context;
this.logger.LogDebug("NLog injecté dans DbCommentaireRepository");
}
/// <inheritdoc/>
public void Add(Commentaire commentaire)
{
try
{
this.logger.LogDebug("Ajout d'un nouveau commentaire de l'auteur : {Auteur}", commentaire.Auteur);
this.context.Commentaires.Add(commentaire);
this.context.SaveChanges();
this.logger.LogDebug("Commentaire ajouté avec l'id : {Id}", commentaire.IdCommentaire);
}
catch (DbUpdateException dbex)
{
this.logger.LogError(dbex, "Erreur de base de données lors de l'ajout du commentaire de l'auteur : {Auteur}", commentaire?.Auteur);
throw;
}
catch (Exception ex)
{
this.logger.LogError(ex, "Une erreur est survenue lors de l'ajout d'un commentaire.");
throw new Exception("Une erreur est survenue lors de l'ajout du commentaire.", ex);
}
}
/// <inheritdoc/>
public void Delete(Commentaire commentaire)
{
try
{
if (commentaire == null)
{
throw new ArgumentNullException(nameof(commentaire), "Le commentaire à supprimer ne peut pas être null.");
}
this.context.Commentaires.Remove(commentaire);
this.context.SaveChanges();
this.logger.LogDebug("Le commentaire {IdCommentaire} a bien été supprimé", commentaire.IdCommentaire);
}
catch (DbUpdateException dbex)
{
this.logger.LogError(dbex, "Erreur de base de données lors de la suppression du commentaire : {Id}", commentaire.IdCommentaire);
throw;
}
catch (Exception ex)
{
this.logger.LogError(ex, "Une erreur est survenue lors de la suppression du commentaire {Id}.", commentaire?.IdCommentaire);
throw new Exception("Une erreur est survenue lors de la suppression du commentaire.", ex);
}
}
/// <inheritdoc/>
public int Count()
{
var count = this.context.Commentaires.Count();
this.logger.LogDebug("Compte total des commentaires : {Count}", count);
return count;
}
/// <inheritdoc/>
public Commentaire Find(int idCommentaire)
{
this.logger.LogDebug("Recherche du commentaire avec l'id : {Id}", idCommentaire);
// On inclut le titre car il est souvent affiché avec le commentaire
return this.context.Commentaires
.Include(c => c.Titre)
.FirstOrDefault(c => c.IdCommentaire == idCommentaire);
}
/// <inheritdoc/>
public IEnumerable<Commentaire> FindAll()
{
this.logger.LogDebug("Récupération de tous les commentaires");
var commentaires = this.context.Commentaires
.Include(c => c.Titre)
.OrderByDescending(c => c.DateCreation)
.ToList();
this.logger.LogDebug("Nombre de commentaires trouvés : {Count}", commentaires.Count);
return commentaires;
}
/// <inheritdoc/>
public IEnumerable<Commentaire> FindCommentaires(int offset, int limit)
{
this.logger.LogDebug("Recherche paginée des commentaires (offset : {Offset}, limit : {Limit})", offset, limit);
var commentaires = this.context.Commentaires
.Include(c => c.Titre)
.OrderByDescending(c => c.DateCreation)
.Skip(offset)
.Take(limit)
.ToList();
this.logger.LogDebug("{Count} commentaires trouvés pour cette page", commentaires.Count);
return commentaires;
}
/// <inheritdoc/>
public IEnumerable<Commentaire> FindByIdTitre(int idTitre)
{
this.logger.LogDebug("Recherche des commentaires pour le titre ID : {IdTitre}", idTitre);
var commentaires = this.context.Commentaires
.Where(c => c.Titre.IdTitre == idTitre)
.OrderByDescending(c => c.DateCreation)
.ToList();
this.logger.LogDebug($"{commentaires.Count} commentaires trouvés pour l'ID de titre : {idTitre}");
return commentaires;
}
}

View File

@@ -2,42 +2,96 @@
// Copyright (c) PlaceholderCompany. All rights reserved.
// </copyright>
namespace Webzine.Repository
{
using System;
using System.Collections.Generic;
using System.Linq;
using Microsoft.Extensions.Logging;
using Webzine.Entity;
using Webzine.Repository.Contracts;
namespace Webzine.Repository
{
/// <summary>
/// Initialise une classe <see cref="LocalCommentaireRepository"/> qui implémente l'interface <see cref="ICommentaireRepository"/> pour gérer les opérations liées aux commentaires.
/// Utilise <see cref="ICommentaireRepository"/> en injection de dépendances.
/// </summary>
public class LocalCommentaireRepository : ICommentaireRepository
{
private readonly ILogger<LocalCommentaireRepository> logger;
private readonly InMemoryDataStore dataStore;
public LocalCommentaireRepository(InMemoryDataStore dataStore)
/// <summary>
/// Initialise une nouvelle instance du <see cref="LocalCommentaireRepository"/> .
/// Est liée à un magasin de données en mémoire et utilise un logger pour enregistrer les opérations.
/// </summary>
/// <param name="dataStore">Le magasin de données en mémoire. Ne peut pas être null.</param>
/// <param name="logger">Le logger à utiliser pour enregistrer les messages de journalisation. Ne peut pas être null.</param>
public LocalCommentaireRepository(InMemoryDataStore dataStore, ILogger<LocalCommentaireRepository> logger)
{
this.logger = logger;
this.dataStore = dataStore;
}
/// <inheritdoc/>
public void Add(Commentaire commentaire)
{
throw new NotSupportedException();
throw new NotSupportedException("Mode Local");
}
/// <inheritdoc/>
public void Delete(Commentaire commentaire)
{
throw new NotSupportedException();
throw new NotSupportedException("Mode Local");
}
/// <inheritdoc/>
public Commentaire Find(int id)
public int Count()
{
return this.dataStore.Commentaires.Find(c => c.IdCommentaire == id);
return this.dataStore.Commentaires.Count;
}
/// <inheritdoc/>
public Commentaire Find(int idCommentaire)
{
var commentaire = this.dataStore.Commentaires.FirstOrDefault(c => c.IdCommentaire == idCommentaire);
if (commentaire == null)
{
return new Commentaire();
}
return commentaire;
}
/// <inheritdoc/>
public IEnumerable<Commentaire> FindAll()
{
return this.dataStore.Commentaires.ToList();
return this.dataStore.Commentaires
.OrderByDescending(c => c.DateCreation)
.ToList();
}
/// <inheritdoc/>
public IEnumerable<Commentaire> FindCommentaires(int offset, int limit)
{
if (offset < 0 || limit <= 0)
{
return Enumerable.Empty<Commentaire>();
}
return this.dataStore.Commentaires
.OrderByDescending(c => c.DateCreation)
.Skip(offset)
.Take(limit)
.ToList();
}
/// <inheritdoc/>
public IEnumerable<Commentaire> FindByIdTitre(int idTitre)
{
return this.dataStore.Commentaires
.Where(c => c.Titre != null && c.Titre.IdTitre == idTitre)
.OrderByDescending(c => c.DateCreation)
.ToList();
}
}
}

View File

@@ -1,6 +1,6 @@
using Microsoft.AspNetCore.Mvc;
using Webzine.Entity;
using Webzine.Entity.Fixtures;
using Webzine.Repository.Contracts;
using Webzine.WebApplication.Areas.Administration.ViewModels.Commentaire;
namespace Webzine.WebApplication.Areas.Administration.Controllers
@@ -9,59 +9,48 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
public class CommentaireController : Controller
{
private readonly ILogger<CommentaireController> logger;
private readonly List<Commentaire> _commentaires;
private readonly ICommentaireRepository commentaireRepository;
/// <summary>
/// Initialise une nouvelle instance de la classe <see cref="CommentaireController"/>.
/// Initialise une nouvelle instance du <see cref="CommentaireController"/>.
/// Les données sont générées dynamiquement via <see cref="DataFactory"/>.
/// Utilise l'injection de dépendances pour récupérer le repository.
/// </summary>
/// <param name="logger">Service de journalisation injecté.</param>
public CommentaireController(ILogger<CommentaireController> logger)
/// <param name="commentaireRepository">Le repository des commentaires injecté.</param>
public CommentaireController(ILogger<CommentaireController> logger, ICommentaireRepository commentaireRepository)
{
this.logger = logger;
this.commentaireRepository = commentaireRepository;
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
var _artistes = factory.GenerateArtists(10);
var _styles = factory.GenerateStyles(10);
var _titres = factory.GenerateTitres(30, _artistes, _styles);
_commentaires = factory.GenerateCommentaires(50, _titres);
this.logger.LogInformation("Données fictives générées avec succès.");
}
/// <summary>
/// Affiche la liste des commentaires dans la vue Index.
/// </summary>
/// <returns>>La vue Index avec le ViewModel contenant la liste des commentaires.</returns>
/// <returns>La vue Index avec le ViewModel contenant la liste des commentaires.</returns>
public IActionResult Index()
{
// Création de données "bouchon" (mock) pour tester l'affichage
// Récupération des commentaires depuis le repository
var commentaires = this.commentaireRepository.FindAll();
// Initialisation du ViewModel
var viewModel = new CommentaireViewModel
{
Commentaires = _commentaires
Commentaires = commentaires
};
return View(viewModel);
}
/// <summary>
/// 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.
/// Affiche la vue de confirmation de suppression d'un commentaire, en récupérant les détails à partir de l'identifiant fourni.
/// </summary>
/// <param name="id">L'identifiant du commentaire à supprimer.</param>
/// <returns>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.</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)
{
var commentaire = _commentaires
.FirstOrDefault(c => c.IdCommentaire == id);
var commentaire = this.commentaireRepository.Find(id);
if (commentaire == null)
{
@@ -80,5 +69,27 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
return View(vm);
}
/// <summary>
/// Effectue la suppression réelle du commentaire (accessible via un simple lien sans HttpPost).
/// </summary>
/// <param name="id">L'identifiant du commentaire à supprimer.</param>
/// <returns>Redirection vers la vue Index après suppression.</returns>
public IActionResult DeleteConfirm(int id)
{
var commentaire = this.commentaireRepository.Find(id);
if (commentaire != null)
{
this.commentaireRepository.Delete(commentaire);
this.logger.LogInformation("Commentaire {Id} supprimé avec succès.", id);
}
else
{
this.logger.LogWarning("Échec de la suppression : Commentaire avec ID {Id} introuvable.", id);
}
return RedirectToAction("Index");
}
}
}

View File

@@ -23,22 +23,30 @@ try
// Necessite le package Nuget Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.
.AddRazorRuntimeCompilation();
builder.Services.AddDbContext<WebzineDbContext>(options =>
options.UseSqlite(builder.Configuration.GetConnectionString("DefaultConnection")));
// NLog: Setup NLog for Dependency injection
builder.Logging.ClearProviders();
builder.Host.UseNLog();
// En fonction de la configuration, utilise soit les repositories basés sur une base de données, soit les repositories basés sur des listes locales.
bool useDatabase = builder.Configuration.GetValue<bool>("UseDatabase");
bool isSQLite = builder.Configuration.GetValue<bool>("IsSQLite");
if (useDatabase)
{
if (isSQLite)
{
builder.Services.AddDbContext<WebzineDbContext>(options =>
options.UseSqlite(builder.Configuration.GetConnectionString("SqliteConnection")));
}
else
{
builder.Services.AddDbContext<WebzineDbContext>(options =>
options.UseNpgsql(builder.Configuration.GetConnectionString("PostGreSQLConnection")));
}
builder.Services.AddScoped<DbEntityRepository>();
builder.Services.AddScoped<ITitreRepository, DbTitreRepository>();
builder.Services.AddScoped<IStyleRepository, DbStyleRepository>();
builder.Services.AddScoped<IArtisteRepository, DbArtisteRepository>();
//builder.Services.AddScoped<ICommentaireRepository, DbCommentaireRepository>();
builder.Services.AddScoped<DbEntityRepository>();
builder.Services.AddScoped<ICommentaireRepository, DbCommentaireRepository>();
}
else
{
@@ -57,13 +65,28 @@ try
if (useDatabase)
{
using (var scope = app.Services.CreateScope())
if (isSQLite)
{
var db = scope.ServiceProvider.GetRequiredService<WebzineDbContext>();
db.Database.EnsureDeleted();
db.Database.EnsureCreated();
var repo = scope.ServiceProvider.GetRequiredService<DbEntityRepository>();
repo.SeedBaseDeDonnees();
using (var scope = app.Services.CreateScope())
{
var db = scope.ServiceProvider.GetRequiredService<WebzineDbContext>();
db.Database.EnsureDeleted();
db.Database.EnsureCreated();
var repo = scope.ServiceProvider.GetRequiredService<DbEntityRepository>();
repo.SeedBaseDeDonnees();
}
}
else
{
using (var scope = app.Services.CreateScope())
{
// TODO : A modifier pour ne pas supprimer la base de donnée en prod
var db = scope.ServiceProvider.GetRequiredService<WebzineDbContext>();
db.Database.EnsureDeleted();
db.Database.EnsureCreated();
var repo = scope.ServiceProvider.GetRequiredService<DbEntityRepository>();
repo.SeedBaseDeDonnees();
}
}
}
else

View File

@@ -6,13 +6,14 @@
}
},
"Webzine": {
// TODO : préciser les modes environnement et repo
"NombreDerniereChronique": 3,
"NombreDeTopTitres": 3
},
"UseDatabase": true,
"IsSQLite": true,
"ConnectionStrings": {
"DefaultConnection": "Data Source=Data/webzine.sqlite"
"SqliteConnection": "Data Source=Data/webzine.sqlite",
"PostGreSQLConnection" : "Host=localhost;Port=5432;Username=admin;Password=admin123;Database=webzine_db"
},
"AllowedHosts": "*"
}