Apply suggestions and corrections from code review

app/src/main/java/com/aurora/store/view/ui/preferences/NetworkPreference.kt

@@ -114,10 +114,7 @@ class NetworkPreference : PreferenceFragmentCompat() {
                 }
         }
 
-        val vendingVersion: Preference? =
+        findPreference<Preference>(Preferences.PREFERENCE_VENDING_VERSION)?.let {
-            findPreference(Preferences.PREFERENCE_VENDING_VERSION)
-
-        vendingVersion?.let {
             it.setOnPreferenceChangeListener { _, newValue ->
                 save(Preferences.PREFERENCE_VENDING_VERSION, Integer.parseInt(newValue.toString()))
                 runOnUiThread {

app/src/main/java/com/aurora/store/viewmodel/auth/AuthViewModel.kt

@@ -291,17 +291,22 @@ class AuthViewModel(application: Application) : BaseAndroidViewModel(application
             authData.locale = Locale.getDefault()
         }
 
-        val versionId = Preferences.getInteger(getApplication(), Preferences.PREFERENCE_VENDING_VERSION)
+        val versionId =
-        if(versionId > 0) {
+            Preferences.getInteger(getApplication(), Preferences.PREFERENCE_VENDING_VERSION)
+        if (versionId != 0) {
             val resources = (getApplication() as Context).resources
 
-            authData.deviceInfoProvider?.properties?.setProperty(
+            authData.deviceInfoProvider?.properties?.let {
-                "Vending.version",
+                it.setProperty(
-                resources.getStringArray(R.array.pref_vending_version_codes)[versionId])
+                    "Vending.version",
+                    resources.getStringArray(R.array.pref_vending_version_codes)[versionId]
+                )
 
-            authData.deviceInfoProvider?.properties?.setProperty(
+                it.setProperty(
-                "Vending.versionString",
+                    "Vending.versionString",
-                resources.getStringArray(R.array.pref_vending_version)[versionId])
+                    resources.getStringArray(R.array.pref_vending_version)[versionId]
+                )
+            }
         }
 
         if (authData.authToken.isNotEmpty() && authData.deviceConfigToken.isNotEmpty()) {

app/src/main/res/values-ru/strings.xml

@@ -274,8 +274,6 @@
     <string name="account_anonymous_insecure">Анонимный (небезопасно)</string>
     <string name="installer_am_unavailable">Установите Менеджер Приложений или смените установщик.</string>
     <string name="pref_install_mode_am">Установщик МП.</string>
-    <string name="pref_vending_version_title">Версия Google Play</string>
-    <string name="pref_vending_version_device">По умолчанию (с конфигурации устройства)</string>
     <string name="ui_accent_1">Синий</string>
     <string name="ui_accent_2">Пурпурный</string>
     <string name="ui_accent_3">Красный</string>

app/src/main/res/values-uk/strings.xml

@@ -274,8 +274,6 @@
     <string name="pref_install_mode_am">Інсталятор менеджера застосунків.</string>
     <string name="account_anonymous_insecure">Анонімно (незахищено)</string>
     <string name="installer_am_unavailable">Встановити Менеджер застосунків або змінити встановлювач.</string>
-    <string name="pref_vending_version_title">Версія Google Play</string>
-    <string name="pref_vending_version_device">За замовчуванням (з конфігурації пристрою)</string>
     <string name="ui_accent_1">Синій</string>
     <string name="ui_accent_4">Золотий</string>
     <string name="ui_accent_2">Пурпуровий</string>

app/src/main/res/xml/preferences_network.xml

@@ -56,4 +56,4 @@
         app:key="PREFERENCE_VENDING_VERSION"
         app:title="@string/pref_vending_version_title"
         app:useSimpleSummaryProvider="true" />
-</PreferenceScreen>
+</PreferenceScreen>