fix: sidecars would not be deleted with library

gotson · Jun 17, 2021 · 49f83b7 · 49f83b7
1 parent 1a1f1cd
commit 49f83b7
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 0 deletions.
diff --git a/komga/src/flyway/resources/db/migration/sqlite/V20210617114814__sidecar_cleanup.sql b/komga/src/flyway/resources/db/migration/sqlite/V20210617114814__sidecar_cleanup.sql
@@ -0,0 +1,6 @@
+DELETE
+FROM SIDECAR
+WHERE LIBRARY_ID NOT IN (
+    SELECT ID
+    FROM LIBRARY
+    )
diff --git a/komga/src/main/kotlin/org/gotson/komga/domain/persistence/SidecarRepository.kt b/komga/src/main/kotlin/org/gotson/komga/domain/persistence/SidecarRepository.kt
@@ -10,4 +10,5 @@ interface SidecarRepository {
   fun save(libraryId: String, sidecar: Sidecar)
 
   fun deleteByLibraryIdAndUrls(libraryId: String, urls: Collection<URL>)
+  fun deleteByLibraryId(libraryId: String)
 }
diff --git a/komga/src/main/kotlin/org/gotson/komga/domain/service/LibraryLifecycle.kt b/komga/src/main/kotlin/org/gotson/komga/domain/service/LibraryLifecycle.kt
@@ -8,6 +8,7 @@ import org.gotson.komga.domain.model.Library
 import org.gotson.komga.domain.model.PathContainedInPath
 import org.gotson.komga.domain.persistence.LibraryRepository
 import org.gotson.komga.domain.persistence.SeriesRepository
+import org.gotson.komga.domain.persistence.SidecarRepository
 import org.springframework.stereotype.Service
 import java.io.FileNotFoundException
 import java.nio.file.Files
@@ -19,6 +20,7 @@ class LibraryLifecycle(
   private val libraryRepository: LibraryRepository,
   private val seriesLifecycle: SeriesLifecycle,
   private val seriesRepository: SeriesRepository,
+  private val sidecarRepository: SidecarRepository,
   private val taskReceiver: TaskReceiver
 ) {
 
@@ -73,6 +75,7 @@ class LibraryLifecycle(
 
     val seriesIds = seriesRepository.findAllByLibraryId(library.id).map { it.id }
     seriesLifecycle.deleteMany(seriesIds)
+    sidecarRepository.deleteByLibraryId(library.id)
 
     libraryRepository.delete(library.id)
   }

diff --git a/komga/src/main/kotlin/org/gotson/komga/infrastructure/jooq/SidecarDao.kt b/komga/src/main/kotlin/org/gotson/komga/infrastructure/jooq/SidecarDao.kt
@@ -41,6 +41,12 @@ class SidecarDao(
       .execute()
   }
 
+  override fun deleteByLibraryId(libraryId: String) {
+    dsl.deleteFrom(sc)
+      .where(sc.LIBRARY_ID.eq(libraryId))
+      .execute()
+  }
+
   private fun SidecarRecord.toDomain() =
     SidecarStored(
       url = URL(url),