From c4f06d98304e9d6b3b97fb4a1638a4b82bfd8cdf Mon Sep 17 00:00:00 2001 From: arkon Date: Sat, 16 Oct 2021 10:21:15 -0400 Subject: [PATCH] Rename app updating classes So I stop confusing it for updaters of other things. (cherry picked from commit 21e647017b629d1ec07a32b548d27342810a2c6b) # Conflicts: # app/src/main/java/eu/kanade/tachiyomi/Migrations.kt --- app/src/main/AndroidManifest.xml | 2 +- .../java/eu/kanade/tachiyomi/Migrations.kt | 17 +++++++++++------ .../{UpdaterJob.kt => AppUpdateJob.kt} | 19 +++++++++---------- ...pdaterNotifier.kt => AppUpdateNotifier.kt} | 13 ++++--------- ...{UpdaterService.kt => AppUpdateService.kt} | 12 ++++++------ .../tachiyomi/data/updater/GithubRelease.kt | 4 ++-- .../ui/more/NewUpdateDialogController.kt | 4 ++-- app/src/main/java/exh/EXHMigrations.kt | 4 ++-- 8 files changed, 37 insertions(+), 38 deletions(-) rename app/src/main/java/eu/kanade/tachiyomi/data/updater/{UpdaterJob.kt => AppUpdateJob.kt} (72%) rename app/src/main/java/eu/kanade/tachiyomi/data/updater/{UpdaterNotifier.kt => AppUpdateNotifier.kt} (91%) rename app/src/main/java/eu/kanade/tachiyomi/data/updater/{UpdaterService.kt => AppUpdateService.kt} (93%) diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml index 6f3fe3abe..8b140281f 100755 --- a/app/src/main/AndroidManifest.xml +++ b/app/src/main/AndroidManifest.xml @@ -184,7 +184,7 @@ android:exported="false" /> ( + val request = PeriodicWorkRequestBuilder( 7, TimeUnit.DAYS, 3, diff --git a/app/src/main/java/eu/kanade/tachiyomi/data/updater/UpdaterNotifier.kt b/app/src/main/java/eu/kanade/tachiyomi/data/updater/AppUpdateNotifier.kt similarity index 91% rename from app/src/main/java/eu/kanade/tachiyomi/data/updater/UpdaterNotifier.kt rename to app/src/main/java/eu/kanade/tachiyomi/data/updater/AppUpdateNotifier.kt index cf8735b7b..fc914f759 100644 --- a/app/src/main/java/eu/kanade/tachiyomi/data/updater/UpdaterNotifier.kt +++ b/app/src/main/java/eu/kanade/tachiyomi/data/updater/AppUpdateNotifier.kt @@ -12,12 +12,7 @@ import eu.kanade.tachiyomi.data.notification.Notifications import eu.kanade.tachiyomi.util.system.notificationBuilder import eu.kanade.tachiyomi.util.system.notificationManager -/** - * DownloadNotifier is used to show notifications when downloading and update. - * - * @param context context of application. - */ -internal class UpdaterNotifier(private val context: Context) { +internal class AppUpdateNotifier(private val context: Context) { private val notificationBuilder = context.notificationBuilder(Notifications.CHANNEL_COMMON) @@ -31,8 +26,8 @@ internal class UpdaterNotifier(private val context: Context) { } fun promptUpdate(url: String) { - val intent = Intent(context, UpdaterService::class.java).apply { - putExtra(UpdaterService.EXTRA_DOWNLOAD_URL, url) + val intent = Intent(context, AppUpdateService::class.java).apply { + putExtra(AppUpdateService.EXTRA_DOWNLOAD_URL, url) } val pendingIntent = PendingIntent.getService(context, 0, intent, PendingIntent.FLAG_UPDATE_CURRENT) with(notificationBuilder) { @@ -125,7 +120,7 @@ internal class UpdaterNotifier(private val context: Context) { addAction( R.drawable.ic_refresh_24dp, context.getString(R.string.action_retry), - UpdaterService.downloadApkPendingService(context, url) + AppUpdateService.downloadApkPendingService(context, url) ) addAction( R.drawable.ic_close_24dp, diff --git a/app/src/main/java/eu/kanade/tachiyomi/data/updater/UpdaterService.kt b/app/src/main/java/eu/kanade/tachiyomi/data/updater/AppUpdateService.kt similarity index 93% rename from app/src/main/java/eu/kanade/tachiyomi/data/updater/UpdaterService.kt rename to app/src/main/java/eu/kanade/tachiyomi/data/updater/AppUpdateService.kt index 916debfe8..e89910242 100755 --- a/app/src/main/java/eu/kanade/tachiyomi/data/updater/UpdaterService.kt +++ b/app/src/main/java/eu/kanade/tachiyomi/data/updater/AppUpdateService.kt @@ -25,7 +25,7 @@ import logcat.LogPriority import uy.kohesive.injekt.injectLazy import java.io.File -class UpdaterService : Service() { +class AppUpdateService : Service() { private val network: NetworkHelper by injectLazy() @@ -34,12 +34,12 @@ class UpdaterService : Service() { */ private lateinit var wakeLock: PowerManager.WakeLock - private lateinit var notifier: UpdaterNotifier + private lateinit var notifier: AppUpdateNotifier override fun onCreate() { super.onCreate() - notifier = UpdaterNotifier(this) + notifier = AppUpdateNotifier(this) wakeLock = acquireWakeLock(javaClass.name) startForeground(Notifications.ID_UPDATER, notifier.onDownloadStarted().build()) @@ -139,7 +139,7 @@ class UpdaterService : Service() { * @return true if the service is running, false otherwise. */ private fun isRunning(context: Context): Boolean = - context.isServiceRunning(UpdaterService::class.java) + context.isServiceRunning(AppUpdateService::class.java) /** * Downloads a new update and let the user install the new version from a notification. @@ -149,7 +149,7 @@ class UpdaterService : Service() { */ fun start(context: Context, url: String, title: String = context.getString(R.string.app_name)) { if (!isRunning(context)) { - val intent = Intent(context, UpdaterService::class.java).apply { + val intent = Intent(context, AppUpdateService::class.java).apply { putExtra(EXTRA_DOWNLOAD_TITLE, title) putExtra(EXTRA_DOWNLOAD_URL, url) } @@ -164,7 +164,7 @@ class UpdaterService : Service() { * @return [PendingIntent] */ internal fun downloadApkPendingService(context: Context, url: String): PendingIntent { - val intent = Intent(context, UpdaterService::class.java).apply { + val intent = Intent(context, AppUpdateService::class.java).apply { putExtra(EXTRA_DOWNLOAD_URL, url) } return PendingIntent.getService(context, 0, intent, PendingIntent.FLAG_UPDATE_CURRENT) diff --git a/app/src/main/java/eu/kanade/tachiyomi/data/updater/GithubRelease.kt b/app/src/main/java/eu/kanade/tachiyomi/data/updater/GithubRelease.kt index 8672e496f..fa68b30c8 100755 --- a/app/src/main/java/eu/kanade/tachiyomi/data/updater/GithubRelease.kt +++ b/app/src/main/java/eu/kanade/tachiyomi/data/updater/GithubRelease.kt @@ -12,7 +12,7 @@ import kotlinx.serialization.Serializable * @param assets assets of latest release. */ @Serializable -class GithubRelease( +data class GithubRelease( @SerialName("tag_name") val version: String, @SerialName("body") val info: String, @SerialName("assets") private val assets: List @@ -30,5 +30,5 @@ class GithubRelease( * @param downloadLink download url. */ @Serializable - class Assets(@SerialName("browser_download_url") val downloadLink: String) + data class Assets(@SerialName("browser_download_url") val downloadLink: String) } diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/more/NewUpdateDialogController.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/more/NewUpdateDialogController.kt index cf99f4fab..7d1521ce8 100644 --- a/app/src/main/java/eu/kanade/tachiyomi/ui/more/NewUpdateDialogController.kt +++ b/app/src/main/java/eu/kanade/tachiyomi/ui/more/NewUpdateDialogController.kt @@ -6,7 +6,7 @@ import androidx.core.os.bundleOf import com.google.android.material.dialog.MaterialAlertDialogBuilder import eu.kanade.tachiyomi.R import eu.kanade.tachiyomi.data.updater.AppUpdateResult -import eu.kanade.tachiyomi.data.updater.UpdaterService +import eu.kanade.tachiyomi.data.updater.AppUpdateService import eu.kanade.tachiyomi.ui.base.controller.DialogController class NewUpdateDialogController(bundle: Bundle? = null) : DialogController(bundle) { @@ -24,7 +24,7 @@ class NewUpdateDialogController(bundle: Bundle? = null) : DialogController(bundl if (appContext != null) { // Start download val url = args.getString(URL_KEY) ?: "" - UpdaterService.start(appContext, url) + AppUpdateService.start(appContext, url) } } .setNegativeButton(R.string.update_check_ignore, null) diff --git a/app/src/main/java/exh/EXHMigrations.kt b/app/src/main/java/exh/EXHMigrations.kt index 7c00356d3..23dfbf6c3 100644 --- a/app/src/main/java/exh/EXHMigrations.kt +++ b/app/src/main/java/exh/EXHMigrations.kt @@ -19,7 +19,7 @@ import eu.kanade.tachiyomi.data.library.LibraryUpdateJob import eu.kanade.tachiyomi.data.preference.PreferenceKeys import eu.kanade.tachiyomi.data.preference.PreferencesHelper import eu.kanade.tachiyomi.data.track.TrackManager -import eu.kanade.tachiyomi.data.updater.UpdaterJob +import eu.kanade.tachiyomi.data.updater.AppUpdateJob import eu.kanade.tachiyomi.extension.ExtensionUpdateJob import eu.kanade.tachiyomi.network.PREF_DOH_CLOUDFLARE import eu.kanade.tachiyomi.source.Source @@ -72,7 +72,7 @@ object EXHMigrations { preferences.ehLastVersionCode().set(BuildConfig.VERSION_CODE) if (BuildConfig.INCLUDE_UPDATER) { - UpdaterJob.setupTask(context) + AppUpdateJob.setupTask(context) } ExtensionUpdateJob.setupTask(context) LibraryUpdateJob.setupTask(context)