From 1d6704124751b5e625b38973ba28f4a5d8f621c9 Mon Sep 17 00:00:00 2001
From: Hamid Reza Mohammadi <mohammadi.hr@gmail.com>
Date: Fri, 14 Aug 2026 19:39:48 +0330
Subject: [PATCH] #498 bug fix

---
 .../Areas/Admin/Pages/PublicDataImport.cshtml |  3 +++
 .../GanjoorService-Centuries.cs               | 26 ++++++++++++++-----
 .../GanjoorService-PublicDataImport.cs        | 11 ++++++++
 3 files changed, 33 insertions(+), 7 deletions(-)

diff --git a/GanjooRazor/Areas/Admin/Pages/PublicDataImport.cshtml b/GanjooRazor/Areas/Admin/Pages/PublicDataImport.cshtml
index 4aebfac4..f9e4e40f 100644
--- a/GanjooRazor/Areas/Admin/Pages/PublicDataImport.cshtml
+++ b/GanjooRazor/Areas/Admin/Pages/PublicDataImport.cshtml
@@ -26,6 +26,9 @@
     <br />
     این فرایند در پس‌زمینه اجرا می‌شود؛ برای پیگیری وضعیت و درصد پیشرفت به
     <a asp-area="Admin" asp-page="LongRunningJobs">صفحهٔ کارها</a> مراجعه کنید.
+    <br />
+    گروه‌بندی سخنوران بر اساس قرن هم پس از پایان درون‌ریزی به‌صورت خودکار بازسازی می‌شود؛ نیازی به
+    اجرای جداگانهٔ آن از صفحهٔ سخنوران نیست.
 </div>
 
 <script>
diff --git a/RMuseum/Services/Implementation/GanjoorService-Partials/GanjoorService-Centuries.cs b/RMuseum/Services/Implementation/GanjoorService-Partials/GanjoorService-Centuries.cs
index 9d700fbe..9d0e5bf7 100644
--- a/RMuseum/Services/Implementation/GanjoorService-Partials/GanjoorService-Centuries.cs
+++ b/RMuseum/Services/Implementation/GanjoorService-Partials/GanjoorService-Centuries.cs
@@ -79,13 +79,25 @@ namespace RMuseum.Services.Implementation
         /// </summary>
         /// <returns></returns>
         public async Task<RServiceResult<bool>> RegenerateHalfCenturiesAsync()
+        {
+            return await _RegenerateHalfCenturies(_context);
+        }
+
+        /// <summary>
+        /// same as <see cref="RegenerateHalfCenturiesAsync"/>, but against an explicitly-passed
+        /// context instead of the injected <see cref="_context"/> — needed so background jobs
+        /// that construct their own short-lived <see cref="RMuseumDbContext"/> (because the
+        /// request-scoped <see cref="_context"/> may already be disposed by the time a background
+        /// job runs) can call this too, e.g. right after a public-data import finishes.
+        /// </summary>
+        private async Task<RServiceResult<bool>> _RegenerateHalfCenturies(RMuseumDbContext context)
         {
             try
             {
-                await _context.Database.ExecuteSqlRawAsync("DELETE FROM GanjoorCenturyPoet");
-                var oldOnes = await _context.GanjoorCenturies.ToArrayAsync();
-                _context.RemoveRange(oldOnes);
-                await _context.SaveChangesAsync();
+                await context.Database.ExecuteSqlRawAsync("DELETE FROM GanjoorCenturyPoet");
+                var oldOnes = await context.GanjoorCenturies.ToArrayAsync();
+                context.RemoveRange(oldOnes);
+                await context.SaveChangesAsync();
 
 
                 var periods = new List<GanjoorCentury>
@@ -191,7 +203,7 @@ namespace RMuseum.Services.Implementation
 
                 };
 
-                var poets = await _context.GanjoorPoets.AsNoTracking().Where(p => p.Published && p.BirthYearInLHijri != 0).OrderBy(p => p.BirthYearInLHijri).ToArrayAsync();
+                var poets = await context.GanjoorPoets.AsNoTracking().Where(p => p.Published && p.BirthYearInLHijri != 0).OrderBy(p => p.BirthYearInLHijri).ToArrayAsync();
 
                 foreach (var poet in poets)
                 {
@@ -236,8 +248,8 @@ namespace RMuseum.Services.Implementation
                 {
                     if (period.Poets != null)
                     {
-                        _context.Add(period);
-                        await _context.SaveChangesAsync();
+                        context.Add(period);
+                        await context.SaveChangesAsync();
                     }
                 }
 
diff --git a/RMuseum/Services/Implementation/GanjoorService-Partials/GanjoorService-PublicDataImport.cs b/RMuseum/Services/Implementation/GanjoorService-Partials/GanjoorService-PublicDataImport.cs
index 9c7bffa1..b1ec2ee0 100644
--- a/RMuseum/Services/Implementation/GanjoorService-Partials/GanjoorService-PublicDataImport.cs
+++ b/RMuseum/Services/Implementation/GanjoorService-Partials/GanjoorService-PublicDataImport.cs
@@ -88,6 +88,17 @@ namespace RMuseum.Services.Implementation
                                     await ImportPoetFromPublicData(context, source, poetEntry.Id, poetEntry.FullUrl);
                                 }
 
+                                // GanjooRazor's home page groups poets by century (GetCenturiesAsync /
+                                // the "PoetGroups" the view renders) — that grouping is a materialized
+                                // table (GanjorCentury/GanjoorCenturyPoet), not computed on the fly, and
+                                // newly-imported poets aren't in it until this runs. Skipping this step
+                                // is exactly what leaves PoetGroups empty and throws on the home page
+                                // (see Index.cshtml.cs's IsDatabaseEffectivelyEmpty check) even though
+                                // poets now exist.
+                                await jobProgressServiceEF.UpdateJob(job.Id, 99, "Regenerating century groupings");
+                                await _RegenerateHalfCenturies(context);
+                                _memoryCache.Remove("ganjoor/centuries"); // same cache key GetCenturiesAsync/the "periods" endpoint use
+
                                 await jobProgressServiceEF.UpdateJob(job.Id, 100, "", true);
                             }
                             catch (Exception exp)