Skip to content

Commit 593fde6

Browse files
muhomorrthestinger
authored andcommitted
fixup! SudConfigProvider: return sysprop setupwizard.theme if set
1 parent c7e0fba commit 593fde6

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/com/android/settings/sudconfig/SudConfigProvider.kt

+6-3
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@ package com.android.settings.sudconfig
22

33
import android.os.Build
44
import android.os.Bundle
5-
import android.os.SystemProperties
65
import android.provider.Settings
76
import android.text.TextUtils
87
import android.util.Log
98
import com.android.settings.R;
9+
import java.util.Optional
1010

1111
/**
1212
* Provides system-wide config for setup wizard screens.
@@ -30,8 +30,11 @@ class SudConfigProvider : NonRelationalProvider() {
3030
private lateinit var defaultThemeString: String
3131

3232
override fun onCreate(): Boolean {
33-
defaultThemeString = SystemProperties.get("setupwizard.theme")
34-
if (TextUtils.isEmpty(defaultThemeString)) defaultThemeString = "glif_v4_light"
33+
// returns value of setupwizard.theme sysprop
34+
val theme: Optional<String> = android.sysprop.SetupWizardProperties.theme()
35+
// setupwizard.theme should always be set to prevent inconsistencies in setupdesign UIs
36+
check(theme.isPresent) { "missing setupwizard.theme sysprop" }
37+
defaultThemeString = theme.get()
3538
return true
3639
}
3740

0 commit comments

Comments
 (0)