Avoid throw as it is slow expensive operations
This commit is contained in:
parent
1613d561c1
commit
9547311d7d
@ -313,16 +313,18 @@ class LibraryUpdateService(
|
||||
manga,
|
||||
) { manga ->
|
||||
try {
|
||||
if (MANGA_NON_COMPLETED in restrictions && manga.status == SManga.COMPLETED) {
|
||||
throw SkipUpdateException(getString(R.string.skipped_reason_completed))
|
||||
when {
|
||||
MANGA_NON_COMPLETED in restrictions && manga.status == SManga.COMPLETED -> {
|
||||
skippedUpdates.add(manga to getString(R.string.skipped_reason_completed))
|
||||
}
|
||||
if (MANGA_HAS_UNREAD in restrictions && manga.unreadCount != 0) {
|
||||
throw SkipUpdateException(getString(R.string.skipped_reason_not_caught_up))
|
||||
MANGA_HAS_UNREAD in restrictions && manga.unreadCount != 0 -> {
|
||||
skippedUpdates.add(manga to getString(R.string.skipped_reason_not_caught_up))
|
||||
}
|
||||
if (MANGA_NON_READ in restrictions && manga.totalChapters > 0 && !manga.hasStarted) {
|
||||
throw SkipUpdateException(getString(R.string.skipped_reason_not_started))
|
||||
MANGA_NON_READ in restrictions && manga.totalChapters > 0 && !manga.hasStarted -> {
|
||||
skippedUpdates.add(manga to getString(R.string.skipped_reason_not_started))
|
||||
}
|
||||
|
||||
else -> {
|
||||
// Convert to the manga that contains new chapters
|
||||
val (newChapters, _) = updateManga(manga)
|
||||
|
||||
if (newChapters.isNotEmpty()) {
|
||||
@ -337,8 +339,8 @@ class LibraryUpdateService(
|
||||
.toTypedArray()
|
||||
)
|
||||
}
|
||||
} catch (e: SkipUpdateException) {
|
||||
skippedUpdates.add(manga to e.message)
|
||||
}
|
||||
}
|
||||
} catch (e: Throwable) {
|
||||
val errorMessage = when (e) {
|
||||
is NoChaptersException -> {
|
||||
@ -587,5 +589,3 @@ class LibraryUpdateService(
|
||||
|
||||
private const val MANGA_PER_SOURCE_QUEUE_WARNING_THRESHOLD = 60
|
||||
private const val ERROR_LOG_HELP_URL = "https://tachiyomi.org/help/guides/troubleshooting"
|
||||
|
||||
private class SkipUpdateException(override val message: String) : RuntimeException()
|
||||
|
Loading…
Reference in New Issue
Block a user