fix: series genre is not imported from metadata

gotson · Aug 27, 2020 · d0815dd · d0815dd
1 parent f4451bf
commit d0815dd
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/komga/src/main/kotlin/org/gotson/komga/domain/model/SeriesMetadataPatch.kt b/komga/src/main/kotlin/org/gotson/komga/domain/model/SeriesMetadataPatch.kt
@@ -9,7 +9,7 @@ data class SeriesMetadataPatch(
   val publisher: String?,
   val ageRating: Int?,
   val language: String?,
-  val genres: Set<String>,
+  val genres: Set<String>?,
 
   val collections: List<String>
 )
diff --git a/komga/src/main/kotlin/org/gotson/komga/domain/service/MetadataLifecycle.kt b/komga/src/main/kotlin/org/gotson/komga/domain/service/MetadataLifecycle.kt
@@ -134,7 +134,7 @@ class MetadataLifecycle(
               title = patches.mostFrequent { it.title },
               titleSort = patches.mostFrequent { it.titleSort },
               status = patches.mostFrequent { it.status },
-              genres = patches.flatMap { it.genres }.toSet(),
+              genres = patches.mapNotNull { it.genres }.flatten().toSet(),
               language = patches.mostFrequent { it.language },
               summary = null,
               readingDirection = patches.mostFrequent { it.readingDirection },

diff --git a/...a/src/main/kotlin/org/gotson/komga/infrastructure/metadata/comicinfo/ComicInfoProvider.kt b/...a/src/main/kotlin/org/gotson/komga/infrastructure/metadata/comicinfo/ComicInfoProvider.kt
@@ -73,7 +73,7 @@ class ComicInfoProvider(
         else -> null
       }
 
-      val genres = comicInfo.genre?.split(',')?.map { it.trim() }?.toSet() ?: emptySet()
+      val genres = comicInfo.genre?.split(',')?.map { it.trim() }?.toSet()
 
       return SeriesMetadataPatch(
         title = comicInfo.series,

diff --git a/komga/src/main/kotlin/org/gotson/komga/infrastructure/metadata/epub/EpubMetadataProvider.kt b/komga/src/main/kotlin/org/gotson/komga/infrastructure/metadata/epub/EpubMetadataProvider.kt
@@ -88,8 +88,8 @@ class EpubMetadataProvider(
         publisher = publisher,
         ageRating = null,
         summary = null,
-        language = if(language != null && BCP47TagValidator.isValid(language)) language else null,
-        genres = if(genre != null) setOf(genre) else emptySet(),
+        language = if (language != null && BCP47TagValidator.isValid(language)) language else null,
+        genres = genre?.let { setOf(genre) },
         collections = emptyList()
       )
     }