Skip to content

Commit f8bc86b

Browse files
committed
Fix CI issues
1 parent 7a00931 commit f8bc86b

File tree

2 files changed

+12
-8
lines changed

2 files changed

+12
-8
lines changed

malicious-site-protection/malicious-site-protection-impl/src/main/kotlin/com/duckduckgo/malicioussiteprotection/impl/MaliciousSiteProtectionFiltersUpdateWorker.kt

+5-3
Original file line numberDiff line numberDiff line change
@@ -111,9 +111,11 @@ class MaliciousSiteProtectionFiltersUpdateWorkerScheduler @Inject constructor(
111111
}
112112

113113
private fun enqueuePeriodicScamWork() {
114-
if (maliciousSiteProtectionFeature.isFeatureEnabled().not() || maliciousSiteProtectionFeature.canUpdateDatasets().not() || maliciousSiteProtectionFeature.scamProtectionEnabled().not()) {
115-
workManager.cancelUniqueWork(MALICIOUS_SITE_PROTECTION_FILTERS_UPDATE_WORKER_SCAM_TAG)
116-
return
114+
with(maliciousSiteProtectionFeature) {
115+
if (isFeatureEnabled().not() || canUpdateDatasets().not() || scamProtectionEnabled().not()) {
116+
workManager.cancelUniqueWork(MALICIOUS_SITE_PROTECTION_FILTERS_UPDATE_WORKER_SCAM_TAG)
117+
return
118+
}
117119
}
118120
enqueueWorker(SCAM, tag = MALICIOUS_SITE_PROTECTION_FILTERS_UPDATE_WORKER_SCAM_TAG)
119121
}

malicious-site-protection/malicious-site-protection-impl/src/main/kotlin/com/duckduckgo/malicioussiteprotection/impl/MaliciousSiteProtectionHashPrefixesUpdateWorker.kt

+7-5
Original file line numberDiff line numberDiff line change
@@ -111,11 +111,12 @@ class MaliciousSiteProtectionHashPrefixesUpdateWorkerScheduler @Inject construct
111111
}
112112

113113
private fun enqueuePeriodicScamWork() {
114-
if (maliciousSiteProtectionFeature.isFeatureEnabled().not() || maliciousSiteProtectionFeature.canUpdateDatasets().not() || maliciousSiteProtectionFeature.scamProtectionEnabled().not()) {
115-
workManager.cancelUniqueWork(MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_SCAM_TAG)
116-
return
114+
with(maliciousSiteProtectionFeature) {
115+
if (isFeatureEnabled().not() || canUpdateDatasets().not() || scamProtectionEnabled().not()) {
116+
workManager.cancelUniqueWork(MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_SCAM_TAG)
117+
return
118+
}
117119
}
118-
119120
enqueueWorker(SCAM, tag = MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_SCAM_TAG)
120121
}
121122

@@ -137,6 +138,7 @@ class MaliciousSiteProtectionHashPrefixesUpdateWorkerScheduler @Inject construct
137138

138139
companion object {
139140
private const val MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_TAG = "MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_TAG"
140-
private const val MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_SCAM_TAG = "MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_SCAM_TAG"
141+
private const val MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_SCAM_TAG =
142+
"MALICIOUS_SITE_PROTECTION_HASH_PREFIXES_UPDATE_WORKER_SCAM_TAG"
141143
}
142144
}

0 commit comments

Comments
 (0)