Revert "Always update uninitialized library entries (#3179)"

This reverts commit 8c0419657b.

Closes #4081
This commit is contained in:
AntsyLich
2026-10-12 13:56:33 +06:00
parent 9dbb7148c5
commit 010ac71a84
3 changed files with 4 additions and 7 deletions
+1
View File
@@ -45,6 +45,7 @@ The format is a modified version of [Keep a Changelog](https://keepachangelog.co
- Fixed installing or updating an extension sometimes cancelling the install of another one ([@AntsyLich](https://github.com/AntsyLich)) - Fixed installing or updating an extension sometimes cancelling the install of another one ([@AntsyLich](https://github.com/AntsyLich))
- Fixed updating from v0.20.x taking up to hours on libraries with many duplicate entries ([@AntsyLich](https://github.com/AntsyLich)) - Fixed updating from v0.20.x taking up to hours on libraries with many duplicate entries ([@AntsyLich](https://github.com/AntsyLich))
- Fixed the app staying on the updating screen for good when updating its data failed, it now shows what went wrong, and updating failing on leftover data of deleted entries ([@AntsyLich](https://github.com/AntsyLich)) - Fixed the app staying on the updating screen for good when updating its data failed, it now shows what went wrong, and updating failing on leftover data of deleted entries ([@AntsyLich](https://github.com/AntsyLich))
- Fixed library updates ignoring the smart update restrictions for entries whose details were never fetched ([@AntsyLich](https://github.com/AntsyLich))
## [v0.21.0] - 2026-10-12 ## [v0.21.0] - 2026-10-12
### Added ### Added
@@ -192,9 +192,6 @@ class LibraryUpdateWorker(private val context: Context, workerParams: WorkerPara
mangaToUpdate = listToUpdate mangaToUpdate = listToUpdate
.filter { .filter {
when { when {
// Entries added without opening them have no details yet
!it.manga.initialized -> true
it.manga.updateStrategy == UpdateStrategy.ONLY_FETCH_ONCE && it.totalChapters > 0L -> { it.manga.updateStrategy == UpdateStrategy.ONLY_FETCH_ONCE && it.totalChapters > 0L -> {
skippedUpdates.add( skippedUpdates.add(
it.manga to context.stringResource(MR.strings.skipped_reason_not_always_update), it.manga to context.stringResource(MR.strings.skipped_reason_not_always_update),
@@ -357,7 +354,7 @@ class LibraryUpdateWorker(private val context: Context, workerParams: WorkerPara
val update = updateMangaFromRemote( val update = updateMangaFromRemote(
source = source, source = source,
manga = manga, manga = manga,
fetchDetails = !manga.initialized || libraryPreferences.autoUpdateMetadata.get(), fetchDetails = libraryPreferences.autoUpdateMetadata.get(),
fetchChapters = true, fetchChapters = true,
fetchWindow = fetchWindow, fetchWindow = fetchWindow,
) )
@@ -69,7 +69,7 @@ class UpdateMangaFromRemote(
fetchDetails = fetchDetails, fetchDetails = fetchDetails,
fetchChapters = fetchChapters, fetchChapters = fetchChapters,
) )
awaitUpdateFromSource(manga, update.manga, fetchDetails, manualFetch) awaitUpdateFromSource(manga, update.manga, manualFetch)
val newChapters = syncChaptersWithSource.await( val newChapters = syncChaptersWithSource.await(
rawSourceChapters = update.chapters, rawSourceChapters = update.chapters,
manga = manga, manga = manga,
@@ -89,7 +89,6 @@ class UpdateMangaFromRemote(
private suspend fun awaitUpdateFromSource( private suspend fun awaitUpdateFromSource(
localManga: Manga, localManga: Manga,
remoteManga: SManga, remoteManga: SManga,
fetchDetails: Boolean,
manualFetch: Boolean, manualFetch: Boolean,
): Boolean { ): Boolean {
val remoteTitle = try { val remoteTitle = try {
@@ -135,7 +134,7 @@ class UpdateMangaFromRemote(
thumbnailUrl = thumbnailUrl, thumbnailUrl = thumbnailUrl,
status = remoteManga.status.toLong(), status = remoteManga.status.toLong(),
updateStrategy = remoteManga.update_strategy, updateStrategy = remoteManga.update_strategy,
initialized = localManga.initialized || fetchDetails, initialized = true,
memo = remoteManga.memo, memo = remoteManga.memo,
), ),
) )