null2264/yokai
1
0
Fork 0
mirror of https://github.com/null2264/yokai.git synced 2025-06-21 10:44:42 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Improve beta check logic

Browse source 
Check the last 10 releases, as stable versions may come out after the beta
Also fix the app thinking a beta on the same version is a new release, also also fix checking if one beta is newer than another
This commit is contained in:
Jays2Kings 2023-03-20 00:40:15 -04:00
parent 475abad6f6
commit 1293adf624
1 changed files with 18 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

24
app/src/main/java/eu/kanade/tachiyomi/data/updater/AppUpdateChecker.kt
View file

@ -32,11 +32,12 @@ class AppUpdateChecker {
.await()
.parseAs<List<GithubRelease>>()
.let {
val release = it.firstOrNull() ?: return@let AppUpdateResult.NoNewUpdate
val releases = it.take(10)
// Check if any of the latest versions are newer than the current version
val release = releases.find { release -> isNewVersion(release.version) }
preferences.lastAppCheck().set(Date().time)
// Check if latest version is different from current version
if (isNewVersion(release.version)) {
if (release != null) {
AppUpdateResult.NewUpdate(release)
} else {
AppUpdateResult.NoNewUpdate
@ -50,7 +51,7 @@ class AppUpdateChecker {
.let {
preferences.lastAppCheck().set(Date().time)
// Check if latest version is different from current version
// Check if latest version is newer than the current version
if (isNewVersion(it.version)) {
AppUpdateResult.NewUpdate(it)
} else {
@ -89,8 +90,19 @@ class AppUpdateChecker {
return if (newSemVer.size > oldSemVer.size) {
true
} else {
// For production versions from beta (new: 1.2.3 vs old: 1.2.3-b1, return true)
(newPreReleaseVer.getOrNull(1) != null) != (oldPreReleaseVer.getOrNull(1) != null)
// If the version numbers match, check the beta versions
val newPreVersion =
newPreReleaseVer.getOrNull(1)?.replace("[^\\d.-]".toRegex(), "")?.toIntOrNull()
val oldPreVersion =
oldPreReleaseVer.getOrNull(1)?.replace("[^\\d.-]".toRegex(), "")?.toIntOrNull()
when {
// For prod, don't bother with betas (current: 1.2.3 vs new: 1.2.3-b1)
oldPreVersion == null -> false
// For betas, always use prod builds (current: 1.2.3-b1 vs new: 1.2.3)
newPreVersion == null -> true
// For betas, higher beta ver is newer (current: 1.2.3-b1 vs new: 1.2.3-b2)
else -> (oldPreVersion < newPreVersion)
}
}
}
}