From be2a4489e2b0f3604e5400616d6ef7a7d6d7b939 Mon Sep 17 00:00:00 2001 From: Chris Narkiewicz Date: Sun, 24 May 2020 01:39:15 +0100 Subject: [PATCH] Fix contacts backup job interval Job interval was set in ms, when API requred minutes. Use of invalid time unit resulted in period of ~164 years. This change removes ms interval constants and sets proper contact backup job period. Fixes #6041 Signed-off-by: Chris Narkiewicz --- .../com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt b/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt index aa2ab66eab66..2c0f3713b7d4 100644 --- a/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt +++ b/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt @@ -85,11 +85,8 @@ internal class BackgroundJobManagerImpl( const val TAG_PREFIX_START_TIMESTAMP = "timestamp" val PREFIXES = setOf(TAG_PREFIX_NAME, TAG_PREFIX_USER, TAG_PREFIX_START_TIMESTAMP) const val NOT_SET_VALUE = "not set" + const val PERIODIC_CONTACTS_BACKUP_INTERVAL_MINUTES = 24 * 60L const val DEFAULT_PERIODIC_JOB_INTERVAL_MINUTES = 15L - const val INTERVAL_SECOND = 1000L - const val INTERVAL_MINUTE = 60L * INTERVAL_SECOND - const val INTERVAL_HOUR = 60 * INTERVAL_MINUTE - const val INTERVAL_24H = 24L * INTERVAL_HOUR const val DEFAULT_IMMEDIATE_JOB_DELAY_SEC = 3L fun formatNameTag(name: String, user: User? = null): String { @@ -231,7 +228,7 @@ internal class BackgroundJobManagerImpl( val request = periodicRequestBuilder( jobClass = ContactsBackupWork::class, jobName = JOB_PERIODIC_CONTACTS_BACKUP, - intervalMins = INTERVAL_24H, + intervalMins = PERIODIC_CONTACTS_BACKUP_INTERVAL_MINUTES, user = user ).setInputData(data).build()