From dfab8488604d29fb332700a390b7d241483aca07 Mon Sep 17 00:00:00 2001
From: Joseph Milazzo <joseph.v.milazzo@gmail.com>
Date: Sat, 28 Jun 2025 07:52:58 -0500
Subject: [PATCH] Default to a list just in case and tweaked the logging for
 Koreader.

---
 API/Controllers/KoreaderController.cs   | 2 +-
 API/Entities/Chapter.cs                 | 3 ++-
 API/Entities/Metadata/SeriesMetadata.cs | 2 +-
 API/Services/KoreaderService.cs         | 2 +-
 4 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/API/Controllers/KoreaderController.cs b/API/Controllers/KoreaderController.cs
index 1ce5e3202..566b190ce 100644
--- a/API/Controllers/KoreaderController.cs
+++ b/API/Controllers/KoreaderController.cs
@@ -94,7 +94,7 @@ public class KoreaderController : BaseApiController
         {
             var userId = await GetUserId(apiKey);
             var response = await _koreaderService.GetProgress(ebookHash, userId);
-            _logger.LogDebug("Koreader response progress: {Progress}", response.Progress);
+            _logger.LogDebug("Koreader response progress for User ({UserId}): {Progress}", userId, response.Progress);
 
             return Ok(response);
         }
diff --git a/API/Entities/Chapter.cs b/API/Entities/Chapter.cs
index 30cd2cf91..fe3646943 100644
--- a/API/Entities/Chapter.cs
+++ b/API/Entities/Chapter.cs
@@ -126,10 +126,11 @@ public class Chapter : IEntityDate, IHasReadTimeEstimate, IHasCoverImage, IHasKP
     /// </summary>
     public string WebLinks { get; set; } = string.Empty;
     public string ISBN { get; set; } = string.Empty;
+
     /// <summary>
     /// Tracks which metadata has been set by K+
     /// </summary>
-    public IList<MetadataSettingField> KPlusOverrides { get; set; }
+    public IList<MetadataSettingField> KPlusOverrides { get; set; } = [];
 
     /// <summary>
     /// (Kavita+) Average rating from Kavita+ metadata
diff --git a/API/Entities/Metadata/SeriesMetadata.cs b/API/Entities/Metadata/SeriesMetadata.cs
index 1ab7c7534..8bb33fdc0 100644
--- a/API/Entities/Metadata/SeriesMetadata.cs
+++ b/API/Entities/Metadata/SeriesMetadata.cs
@@ -46,7 +46,7 @@ public class SeriesMetadata : IHasConcurrencyToken, IHasKPlusMetadata
     /// <summary>
     /// Tracks which metadata has been set by K+
     /// </summary>
-    public IList<MetadataSettingField> KPlusOverrides { get; set; }
+    public IList<MetadataSettingField> KPlusOverrides { get; set; } = [];
 
     #region Locks
 
diff --git a/API/Services/KoreaderService.cs b/API/Services/KoreaderService.cs
index 69b3948ed..db8522b25 100644
--- a/API/Services/KoreaderService.cs
+++ b/API/Services/KoreaderService.cs
@@ -39,7 +39,7 @@ public class KoreaderService : IKoreaderService
     /// <param name="userId"></param>
     public async Task SaveProgress(KoreaderBookDto koreaderBookDto, int userId)
     {
-        _logger.LogDebug("Saving Koreader progress for {UserId}: {KoreaderProgress}", userId, koreaderBookDto.Progress);
+        _logger.LogDebug("Saving Koreader progress for User ({UserId}): {KoreaderProgress}", userId, koreaderBookDto.Progress);
         var file = await _unitOfWork.MangaFileRepository.GetByKoreaderHash(koreaderBookDto.Document);
         if (file == null) throw new KavitaException(await _localizationService.Translate(userId, "file-missing"));