Skip to content
Snippets Groups Projects
Unverified Commit eeda5d13 authored by Matthias Urhahn's avatar Matthias Urhahn Committed by GitHub
Browse files

Reduce code smell debt within RiskLevelChange detektor (DEV) #3151

parent 349957fa
No related branches found
No related tags found
No related merge requests found
......@@ -113,17 +113,13 @@ class RiskLevelChangeDetector @Inject constructor(
) {
// Save riskLevelTurnedRedTime if not already set before for high risk detection
Timber.i("riskLevelTurnedRedTime=%s", testResultDonorSettings.riskLevelTurnedRedTime.value)
if (testResultDonorSettings.riskLevelTurnedRedTime.value == null) {
if (newEwRiskState.isIncreasedRisk) {
testResultDonorSettings.riskLevelTurnedRedTime.update {
newEwRiskState.calculatedAt
}
Timber.i(
"riskLevelTurnedRedTime: newRiskState=%s, riskLevelTurnedRedTime=%s",
newEwRiskState.riskState,
newEwRiskState.calculatedAt
)
}
if (testResultDonorSettings.riskLevelTurnedRedTime.value == null && newEwRiskState.isIncreasedRisk) {
testResultDonorSettings.riskLevelTurnedRedTime.update { newEwRiskState.calculatedAt }
Timber.i(
"riskLevelTurnedRedTime: newRiskState=%s, riskLevelTurnedRedTime=%s",
newEwRiskState.riskState,
newEwRiskState.calculatedAt
)
}
// Save most recent date of high or low risks
......@@ -136,9 +132,7 @@ class RiskLevelChangeDetector @Inject constructor(
lastRiskEncounterAt
)
testResultDonorSettings.mostRecentDateWithHighOrLowRiskLevel.update {
lastRiskEncounterAt
}
testResultDonorSettings.mostRecentDateWithHighOrLowRiskLevel.update { lastRiskEncounterAt }
}
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment