Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ad6caf70 authored by Vincent Bourgmayer's avatar Vincent Bourgmayer
Browse files

chore: Apply suggestion. rename method & const in WorkRequestFactory

parent 50005766
Loading
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -39,7 +39,7 @@ class WorkLauncher private constructor(context: Context) {
    }

    fun enqueuePeriodicUserInfoFetching() {
        val request = WorkRequestFactory.getPeriodicWorkRequest(PeriodicWorkType.FETCH_USER_INFO)
        val request = WorkRequestFactory.createPeriodicWorkRequest(PeriodicWorkType.FETCH_USER_INFO)
        workManager.enqueueUniquePeriodicWork(
            AccountUserInfoWorker.UNIQUE_WORK_NAME,
            ExistingPeriodicWorkPolicy.UPDATE,
@@ -48,7 +48,7 @@ class WorkLauncher private constructor(context: Context) {
    }

    fun enqueuePeriodicFullScan() {
        val request = WorkRequestFactory.getPeriodicWorkRequest(PeriodicWorkType.PERIODIC_SCAN)
        val request = WorkRequestFactory.createPeriodicWorkRequest(PeriodicWorkType.PERIODIC_SCAN)
        workManager.enqueueUniquePeriodicWork(PeriodicScanWorker.UNIQUE_WORK_NAME,
            ExistingPeriodicWorkPolicy.KEEP,
            request)
+6 −6
Original line number Diff line number Diff line
@@ -54,15 +54,15 @@ enum class OneTimeWorkType {

object WorkRequestFactory {

    private const val PERIODIC_WORK_REPEAT_INTERVAL = 30L
    private const val PERIODIC_SCAN_FLEX_TIME = 5L
    private const val PERIODIC_WORK_REPEAT_INTERVAL_IN_MIN = 30L
    private const val PERIODIC_SCAN_FLEX_TIME_IN_MIN = 5L

    /**
     * Build an instance of PeriodicWorkRequest depending of the work type specified
     * @param type WorkType. Should be PERIODIC_SCAN or PERIODIC_USER_INFO
     * @return Periodic WorkRequest
     */
    fun getPeriodicWorkRequest(type: PeriodicWorkType): PeriodicWorkRequest {
    fun createPeriodicWorkRequest(type: PeriodicWorkType): PeriodicWorkRequest {
        return when (type) {
            PeriodicWorkType.FETCH_USER_INFO -> createPeriodicGetUserInfoWorkRequest()
            PeriodicWorkType.PERIODIC_SCAN -> createPeriodicScanWorkRequest()
@@ -227,8 +227,8 @@ object WorkRequestFactory {
            .build()

        return PeriodicWorkRequestBuilder<PeriodicScanWorker>(
            PERIODIC_WORK_REPEAT_INTERVAL, MINUTES,
            PERIODIC_SCAN_FLEX_TIME,MINUTES)
            PERIODIC_WORK_REPEAT_INTERVAL_IN_MIN, MINUTES,
            PERIODIC_SCAN_FLEX_TIME_IN_MIN,MINUTES)
            .setConstraints(constraints)
            .addTag(WORK_GENERIC_TAG)
            .build()
@@ -240,7 +240,7 @@ object WorkRequestFactory {
            .build()

        return PeriodicWorkRequestBuilder<AccountUserInfoWorker>(
            PERIODIC_WORK_REPEAT_INTERVAL, MINUTES)
            PERIODIC_WORK_REPEAT_INTERVAL_IN_MIN, MINUTES)
            .setConstraints(constraint)
            .addTag(WORK_GENERIC_TAG)
            .build()