From d04d53c8fde7688ed7b4483d4ad6f83e5465289b Mon Sep 17 00:00:00 2001
From: Loic Masi <loic.masi@diiage.org>
Date: Sun, 5 Apr 2026 10:31:00 +0200
Subject: [PATCH 1/2] #199 : Patch bug StyleController.

---
 .../Controllers/StyleController.cs            | 277 +++++++++---------
 1 file changed, 139 insertions(+), 138 deletions(-)

diff --git a/Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs b/Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs
index 51b9d80..9cb8f1b 100644
--- a/Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs
+++ b/Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs
@@ -1,174 +1,175 @@
-namespace Webzine.WebApplication.Areas.Administration.Controllers;
-
-using Microsoft.AspNetCore.Mvc;
-
-using ViewModels.Styles;
-
-using Webzine.Entity;
-using Webzine.Repository.Contracts;
-
-/// <summary>
-/// Controleur pour la gestion des styles dans l'administration du webzine.
-/// </summary>
-[Area("Administration")]
-public class StyleController : Controller
+namespace Webzine.WebApplication.Areas.Administration.Controllers
 {
-    private readonly ILogger<StyleController> logger;
-    private readonly IStyleRepository styleRepository;
-    private readonly IConfiguration configuration;
+    using Microsoft.AspNetCore.Mvc;
+
+    using Webzine.Entity;
+    using Webzine.Repository.Contracts;
+    using Webzine.WebApplication.Areas.Administration.ViewModels.Style;
+    using Webzine.WebApplication.Areas.Administration.ViewModels.Styles;
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="StyleController"/> class.
+    /// Controleur pour la gestion des styles dans l'administration du webzine.
     /// </summary>
-    /// <param name="logger">Service de journalisation injecte.</param>
-    /// <param name="styleRepository">Repository des styles injecte.</param>
-    /// <param name="configuration">Service de configuration injecte pour acceder aux parametres de configuration.</param>
-    public StyleController(
-        ILogger<StyleController> logger,
-        IStyleRepository styleRepository,
-        IConfiguration configuration)
+    [Area("Administration")]
+    public class StyleController : Controller
     {
-        this.logger = logger;
-        this.styleRepository = styleRepository;
-        this.configuration = configuration;
+        private readonly ILogger<StyleController> logger;
+        private readonly IStyleRepository styleRepository;
+        private readonly IConfiguration configuration;
 
-        this.logger.LogInformation("Initialisation du controleur StyleController.");
-    }
-
-    /// <summary>
-    /// Affiche la liste des styles dans la vue Index.
-    /// </summary>
-    /// <param name="page">Le numero de page pour la pagination des styles (par defaut a 0).</param>
-    /// <returns>La vue Index avec la liste des styles.</returns>
-    public IActionResult Index(int page = 0)
-    {
-        int styles_par_page = this.configuration.GetValue<int>("Webzine:NombreDeLignesAdministration");
-        IEnumerable<Style> listeStyles = this.styleRepository.FindStyles(page * styles_par_page, styles_par_page);
-
-        int totalStyles = this.styleRepository.Count();
-
-        var model = new StyleIndexViewModel
+        /// <summary>
+        /// Initializes a new instance of the <see cref="StyleController"/> class.
+        /// </summary>
+        /// <param name="logger">Service de journalisation injecte.</param>
+        /// <param name="styleRepository">Repository des styles injecte.</param>
+        /// <param name="configuration">Service de configuration injecte pour acceder aux parametres de configuration.</param>
+        public StyleController(
+            ILogger<StyleController> logger,
+            IStyleRepository styleRepository,
+            IConfiguration configuration)
         {
-            Styles = listeStyles,
-            Page = page,
-            TotalPages = (int)Math.Ceiling((double)totalStyles / styles_par_page),
-        };
-        return this.View(model);
-    }
+            this.logger = logger;
+            this.styleRepository = styleRepository;
+            this.configuration = configuration;
 
-    /// <summary>
-    /// Affiche la vue de creation d'un nouveau style.
-    /// </summary>
-    /// <returns>La vue Create pour ajouter un nouveau style.</returns>
-    public IActionResult Create()
-    {
-        var model = new StyleCreateViewModel
+            this.logger.LogInformation("Initialisation du controleur StyleController.");
+        }
+
+        /// <summary>
+        /// Affiche la liste des styles dans la vue Index.
+        /// </summary>
+        /// <param name="page">Le numero de page pour la pagination des styles (par defaut a 0).</param>
+        /// <returns>La vue Index avec la liste des styles.</returns>
+        public IActionResult Index(int page = 0)
         {
-            Libelle = string.Empty,
-        };
+            int styles_par_page = this.configuration.GetValue<int>("Webzine:NombreDeLignesAdministration");
+            IEnumerable<Style> listeStyles = this.styleRepository.FindStyles(page * styles_par_page, styles_par_page);
 
-        return this.View(model);
-    }
+            int totalStyles = this.styleRepository.Count();
 
-    /// <summary>
-    /// Cree un nouveau style.
-    /// </summary>
-    /// <param name="model">Nouveau style.</param>
-    /// <returns>IActionResult.</returns>
-    [HttpPost]
-    public IActionResult Create(StyleCreateViewModel model)
-    {
-        var style = new Style
+            var model = new StyleIndexViewModel
+            {
+                Styles = listeStyles,
+                Page = page,
+                TotalPages = (int)Math.Ceiling((double)totalStyles / styles_par_page),
+            };
+            return this.View(model);
+        }
+
+        /// <summary>
+        /// Affiche la vue de creation d'un nouveau style.
+        /// </summary>
+        /// <returns>La vue Create pour ajouter un nouveau style.</returns>
+        public IActionResult Create()
         {
-            Libelle = model.Libelle,
-        };
+            var model = new StyleCreateViewModel
+            {
+                Libelle = string.Empty,
+            };
 
-        this.styleRepository.Add(style);
+            return this.View(model);
+        }
 
-        return this.RedirectToAction("Index");
-    }
-
-    /// <summary>
-    /// Affiche la vue de confirmation de suppression d'un style.
-    /// </summary>
-    /// <param name="id">L'identifiant du style a supprimer.</param>
-    /// <returns>La vue de confirmation ou une redirection vers l'index si le style n'existe pas.</returns>
-    public IActionResult Delete(int id)
-    {
-        var style = this.styleRepository.Find(id);
-
-        if (style == null || style.IdStyle == 0)
+        /// <summary>
+        /// Cree un nouveau style.
+        /// </summary>
+        /// <param name="model">Nouveau style.</param>
+        /// <returns>IActionResult.</returns>
+        [HttpPost]
+        public IActionResult Create(StyleCreateViewModel model)
         {
+            var style = new Style
+            {
+                Libelle = model.Libelle,
+            };
+
+            this.styleRepository.Add(style);
+
             return this.RedirectToAction("Index");
         }
 
-        var model = new StyleDeleteViewModel
+        /// <summary>
+        /// Affiche la vue de confirmation de suppression d'un style.
+        /// </summary>
+        /// <param name="id">L'identifiant du style a supprimer.</param>
+        /// <returns>La vue de confirmation ou une redirection vers l'index si le style n'existe pas.</returns>
+        public IActionResult Delete(int id)
         {
-            IdStyle = style.IdStyle,
-            Libelle = style.Libelle,
-        };
+            var style = this.styleRepository.Find(id);
 
-        return this.View(model);
-    }
+            if (style == null || style.IdStyle == 0)
+            {
+                return this.RedirectToAction("Index");
+            }
 
-    /// <summary>
-    /// Methode POST pour supprimer un style.
-    /// </summary>
-    /// <param name="model">Le style a supprimer.</param>
-    /// <returns>Redirige vers la page d'index d'admin style.</returns>
-    [HttpPost]
-    public IActionResult Delete(StyleDeleteViewModel model)
-    {
-        var style = this.styleRepository.Find(model.IdStyle);
+            var model = new StyleDeleteViewModel
+            {
+                IdStyle = style.IdStyle,
+                Libelle = style.Libelle,
+            };
 
-        if (style != null)
-        {
-            this.styleRepository.Delete(style);
+            return this.View(model);
         }
 
-        return this.RedirectToAction("Index");
-    }
-
-    /// <summary>
-    /// Affiche la vue d'edition d'un style existant.
-    /// </summary>
-    /// <param name="id">L'identifiant du style a editer.</param>
-    /// <returns>La vue d'edition ou une redirection vers l'index si le style n'existe pas.</returns>
-    public IActionResult Edit(int id)
-    {
-        var style = this.styleRepository.Find(id);
-
-        if (style == null || style.IdStyle == 0)
+        /// <summary>
+        /// Methode POST pour supprimer un style.
+        /// </summary>
+        /// <param name="model">Le style a supprimer.</param>
+        /// <returns>Redirige vers la page d'index d'admin style.</returns>
+        [HttpPost]
+        public IActionResult Delete(StyleDeleteViewModel model)
         {
+            var style = this.styleRepository.Find(model.IdStyle);
+
+            if (style != null)
+            {
+                this.styleRepository.Delete(style);
+            }
+
             return this.RedirectToAction("Index");
         }
 
-        var model = new StyleEditViewModel
+        /// <summary>
+        /// Affiche la vue d'edition d'un style existant.
+        /// </summary>
+        /// <param name="id">L'identifiant du style a editer.</param>
+        /// <returns>La vue d'edition ou une redirection vers l'index si le style n'existe pas.</returns>
+        public IActionResult Edit(int id)
         {
-            IdStyle = style.IdStyle,
-            Libelle = style.Libelle,
-        };
+            var style = this.styleRepository.Find(id);
 
-        return this.View(model);
-    }
+            if (style == null || style.IdStyle == 0)
+            {
+                return this.RedirectToAction("Index");
+            }
 
-    /// <summary>
-    /// Met a jour un style existant.
-    /// </summary>
-    /// <param name="model">Donnees du style a modifier.</param>
-    /// <returns>Redirige vers la page d'index d'admin style.</returns>
-    [HttpPost]
-    public IActionResult Edit(StyleEditViewModel model)
-    {
-        var style = this.styleRepository.Find(model.IdStyle);
-        if (style == null)
-        {
-            return this.RedirectToAction("Index");
+            var model = new StyleEditViewModel
+            {
+                IdStyle = style.IdStyle,
+                Libelle = style.Libelle,
+            };
+
+            return this.View(model);
         }
 
-        style.Libelle = model.Libelle;
-        this.styleRepository.Update(style);
+        /// <summary>
+        /// Met a jour un style existant.
+        /// </summary>
+        /// <param name="model">Donnees du style a modifier.</param>
+        /// <returns>Redirige vers la page d'index d'admin style.</returns>
+        [HttpPost]
+        public IActionResult Edit(StyleEditViewModel model)
+        {
+            var style = this.styleRepository.Find(model.IdStyle);
+            if (style == null)
+            {
+                return this.RedirectToAction("Index");
+            }
 
-        return this.RedirectToAction("Index");
+            style.Libelle = model.Libelle;
+            this.styleRepository.Update(style);
+
+            return this.RedirectToAction("Index");
+        }
     }
 }
\ No newline at end of file

From cbefaad9d7d561b1701284317054497dd38f8e83 Mon Sep 17 00:00:00 2001
From: Loic Masi <loic.masi@diiage.org>
Date: Sun, 5 Apr 2026 10:56:53 +0200
Subject: [PATCH 2/2] =?UTF-8?q?#190=20:=20Application=20d'une=20m=C3=A9tho?=
 =?UTF-8?q?de=20g=C3=A9n=C3=A9rique=20pour=20la=20pagination.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Webzine.Repository/DbArtisteRepository.cs     |  3 +-
 Webzine.Repository/DbCommentaireRepository.cs |  3 +-
 Webzine.Repository/DbStyleRepository.cs       |  3 +-
 Webzine.Repository/DbTitreRepository.cs       |  3 +-
 Webzine.Repository/LocalArtisteRepository.cs  |  3 +-
 .../LocalCommentaireRepository.cs             |  3 +-
 Webzine.Repository/LocalStyleRepository.cs    |  3 +-
 Webzine.Repository/LocalTitreRepository.cs    |  3 +-
 .../RepositoryPaginationExtensions.cs         | 56 +++++++++++++++++++
 9 files changed, 64 insertions(+), 16 deletions(-)
 create mode 100644 Webzine.Repository/RepositoryPaginationExtensions.cs

diff --git a/Webzine.Repository/DbArtisteRepository.cs b/Webzine.Repository/DbArtisteRepository.cs
index de34724..0692fbb 100644
--- a/Webzine.Repository/DbArtisteRepository.cs
+++ b/Webzine.Repository/DbArtisteRepository.cs
@@ -217,8 +217,7 @@ namespace Webzine.Repository
                     .AsNoTracking()
                     .OrderBy(a => a.Nom)
                     .Include(t => t.Titres)
-                    .Skip(offset)
-                    .Take(limit);
+                    .Paginate(offset, limit);
                 this.logger.LogDebug("Page {PageNumber} d'artistes récupérée avec {PageSize} artistes par page.", offset, limit);
                 return artistes;
             }
diff --git a/Webzine.Repository/DbCommentaireRepository.cs b/Webzine.Repository/DbCommentaireRepository.cs
index 087fa0f..7d669b2 100644
--- a/Webzine.Repository/DbCommentaireRepository.cs
+++ b/Webzine.Repository/DbCommentaireRepository.cs
@@ -103,8 +103,7 @@ public class DbCommentaireRepository : ICommentaireRepository
                 .AsNoTracking()
                 .Include(c => c.Titre)
                 .OrderByDescending(c => c.DateCreation)
-                .Skip(offset)
-                .Take(limit);
+                .Paginate(offset, limit);
 
             return commentaires;
         }
diff --git a/Webzine.Repository/DbStyleRepository.cs b/Webzine.Repository/DbStyleRepository.cs
index d831ead..5bcdf6d 100644
--- a/Webzine.Repository/DbStyleRepository.cs
+++ b/Webzine.Repository/DbStyleRepository.cs
@@ -191,8 +191,7 @@ public class DbStyleRepository : IStyleRepository
             var styles = this.context.Styles
                 .AsNoTracking()
                 .OrderBy(s => s.Libelle)
-                .Skip(offset)
-                .Take(limit);
+                .Paginate(offset, limit);
 
             this.logger.LogDebug("La liste paginée de styles a été récupérée.");
             return styles;
diff --git a/Webzine.Repository/DbTitreRepository.cs b/Webzine.Repository/DbTitreRepository.cs
index 049715c..918d543 100644
--- a/Webzine.Repository/DbTitreRepository.cs
+++ b/Webzine.Repository/DbTitreRepository.cs
@@ -107,8 +107,7 @@ public class DbTitreRepository : ITitreRepository
                 .ThenBy(t => t.Libelle)
                 .Include(t => t.Artiste)
                 .Include(t => t.Styles)
-                .Skip(offset)
-                .Take(limit)
+                .Paginate(offset, limit)
                 .AsNoTracking();
 
             return titres;
diff --git a/Webzine.Repository/LocalArtisteRepository.cs b/Webzine.Repository/LocalArtisteRepository.cs
index 8443258..a1e95b9 100644
--- a/Webzine.Repository/LocalArtisteRepository.cs
+++ b/Webzine.Repository/LocalArtisteRepository.cs
@@ -112,8 +112,7 @@ namespace Webzine.Repository
         {
             return this.dataStore.Artistes
                 .OrderBy(a => a.Nom)
-                .Skip(offset)
-                .Take(limit)
+                .Paginate(offset, limit)
                 .ToList();
         }
     }
diff --git a/Webzine.Repository/LocalCommentaireRepository.cs b/Webzine.Repository/LocalCommentaireRepository.cs
index d24a1c7..23372b5 100644
--- a/Webzine.Repository/LocalCommentaireRepository.cs
+++ b/Webzine.Repository/LocalCommentaireRepository.cs
@@ -63,8 +63,7 @@ namespace Webzine.Repository
         {
             return this.dataStore.Commentaires
                 .OrderByDescending(c => c.DateCreation)
-                .Skip(offset)
-                .Take(limit);
+                .Paginate(offset, limit);
         }
 
         /// <inheritdoc/>
diff --git a/Webzine.Repository/LocalStyleRepository.cs b/Webzine.Repository/LocalStyleRepository.cs
index 549a6c1..d064c88 100644
--- a/Webzine.Repository/LocalStyleRepository.cs
+++ b/Webzine.Repository/LocalStyleRepository.cs
@@ -77,8 +77,7 @@ public class LocalStyleRepository : IStyleRepository
     {
         return this.dataStore.Styles
         .OrderBy(s => s.Libelle)
-        .Skip(offset)
-        .Take(limit)
+        .Paginate(offset, limit)
         .ToList();
     }
 }
\ No newline at end of file
diff --git a/Webzine.Repository/LocalTitreRepository.cs b/Webzine.Repository/LocalTitreRepository.cs
index 5dc0355..605a5c2 100644
--- a/Webzine.Repository/LocalTitreRepository.cs
+++ b/Webzine.Repository/LocalTitreRepository.cs
@@ -50,8 +50,7 @@ public class LocalTitreRepository : ITitreRepository
         return this.dataStore.Titres
             .OrderByDescending(t => t.DateCreation)
             .ThenBy(t => t.Libelle)
-            .Skip(offset)
-            .Take(limit);
+            .Paginate(offset, limit);
     }
 
     /// <inheritdoc/>
diff --git a/Webzine.Repository/RepositoryPaginationExtensions.cs b/Webzine.Repository/RepositoryPaginationExtensions.cs
new file mode 100644
index 0000000..add2a68
--- /dev/null
+++ b/Webzine.Repository/RepositoryPaginationExtensions.cs
@@ -0,0 +1,56 @@
+namespace Webzine.Repository;
+
+/// <summary>
+/// Fournit des méthodes génériques pour paginer des collections dans la couche Repository.
+/// </summary>
+public static class RepositoryPaginationExtensions
+{
+    /// <summary>
+    /// Retourne une portion paginée d'une requête.
+    /// </summary>
+    /// <typeparam name="T">Type des éléments paginés.</typeparam>
+    /// <param name="source">Source à paginer.</param>
+    /// <param name="offset">Nombre d'éléments à ignorer.</param>
+    /// <param name="limit">Nombre maximal d'éléments à retourner.</param>
+    /// <returns>La requête paginée.</returns>
+    public static IQueryable<T> Paginate<T>(this IQueryable<T> source, int offset, int limit)
+    {
+        ArgumentNullException.ThrowIfNull(source);
+        ValidatePaginationArguments(offset, limit);
+
+        return source
+            .Skip(offset)
+            .Take(limit);
+    }
+
+    /// <summary>
+    /// Retourne une portion paginée d'une collection en mémoire.
+    /// </summary>
+    /// <typeparam name="T">Type des éléments paginés.</typeparam>
+    /// <param name="source">Source à paginer.</param>
+    /// <param name="offset">Nombre d'éléments à ignorer.</param>
+    /// <param name="limit">Nombre maximal d'éléments à retourner.</param>
+    /// <returns>La collection paginée.</returns>
+    public static IEnumerable<T> Paginate<T>(this IEnumerable<T> source, int offset, int limit)
+    {
+        ArgumentNullException.ThrowIfNull(source);
+        ValidatePaginationArguments(offset, limit);
+
+        return source
+            .Skip(offset)
+            .Take(limit);
+    }
+
+    private static void ValidatePaginationArguments(int offset, int limit)
+    {
+        if (offset < 0)
+        {
+            throw new ArgumentOutOfRangeException(nameof(offset), "L'offset doit être supérieur ou égal à 0.");
+        }
+
+        if (limit <= 0)
+        {
+            throw new ArgumentOutOfRangeException(nameof(limit), "La limite doit être strictement supérieure à 0.");
+        }
+    }
+}
\ No newline at end of file