[ANDROAPP-6162] Remove COMPOSE_FORM option

Signed-off-by: andresmr <[email protected]>
dhis2 · May 14, 2024 · e8c5166 · e8c5166
1 parent 3ac7aad
commit e8c5166
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 20 deletions.
diff --git a/app/src/androidTest/java/org/dhis2/usescases/BaseTest.kt b/app/src/androidTest/java/org/dhis2/usescases/BaseTest.kt
@@ -169,12 +169,13 @@ open class BaseTest {
     }
 
     private fun disableComposeForms() {
+        preferencesRobot.saveValue(SET_FROM_TESTING, true)
         preferencesRobot.saveValue(Feature.COMPOSE_FORMS.name, false)
     }
 
 
     fun enableComposeForms() {
-        preferencesRobot.saveValue("SET_FROM_DEVELOPMENT", true)
+        preferencesRobot.saveValue(SET_FROM_TESTING, true)
         preferencesRobot.saveValue(Feature.COMPOSE_FORMS.name, true)
     }
 
@@ -184,5 +185,6 @@ open class BaseTest {
         val disableAnimationsTestRule = DisableAnimations()
         const val MOCK_SERVER_URL = "http://127.0.0.1:8080"
         const val API = "api"
+        const val SET_FROM_TESTING = "SET_FROM_TESTING"
     }
 }
diff --git a/commons/src/main/java/org/dhis2/commons/featureconfig/data/FeatureConfigRepositoryImpl.kt b/commons/src/main/java/org/dhis2/commons/featureconfig/data/FeatureConfigRepositoryImpl.kt
@@ -3,45 +3,36 @@ package org.dhis2.commons.featureconfig.data
 import org.dhis2.commons.featureconfig.model.Feature
 import org.dhis2.commons.featureconfig.model.FeatureState
 import org.dhis2.commons.prefs.PreferenceProvider
-import org.hisp.dhis.android.core.D2
-import org.hisp.dhis.android.core.settings.ExperimentalFeature
 import javax.inject.Inject
 
 class FeatureConfigRepositoryImpl @Inject constructor(
     val preferences: PreferenceProvider,
-    val d2: D2,
 ) : FeatureConfigRepository {
 
-    val SET_FROM_DEVELOPMENT = "SET_FROM_DEVELOPMENT"
-
     override val featuresList: List<FeatureState>
-        get() = Feature.entries.map {
-            FeatureState(it, isFeatureEnable(it))
-        }
+        get() = Feature.entries
+            .filter { it.name != Feature.COMPOSE_FORMS.name }
+            .map { FeatureState(it, isFeatureEnable(it)) }
 
     override fun updateItem(featureState: FeatureState) {
-        if (featureState.feature.name == Feature.COMPOSE_FORMS.name) {
-            preferences.setValue(SET_FROM_DEVELOPMENT, true)
-        }
         preferences.setValue(featureState.feature.name, !featureState.enable)
     }
 
     override fun isFeatureEnable(feature: Feature): Boolean {
         return when {
             feature.name == Feature.COMPOSE_FORMS.name -> {
-                val fromDevelopment = preferences.getBoolean(SET_FROM_DEVELOPMENT, false)
+                val fromDevelopment = preferences.getBoolean("SET_FROM_TESTING", false)
                 if (fromDevelopment) {
                     preferences.getBoolean(feature.name, false)
-                } else if (d2.settingModule().generalSetting().blockingExists()) {
-                    d2.settingModule().generalSetting()
-                        .hasExperimentalFeature(ExperimentalFeature.NewFormLayout).blockingGet()
                 } else {
                     true
                 }
             }
+
             preferences.contains(feature.name) -> {
                 preferences.getBoolean(feature.name, false)
             }
+
             else -> false
         }
     }

diff --git a/commons/src/main/java/org/dhis2/commons/featureconfig/di/FeatureConfigModule.kt b/commons/src/main/java/org/dhis2/commons/featureconfig/di/FeatureConfigModule.kt
@@ -5,15 +5,12 @@ import dagger.Provides
 import org.dhis2.commons.featureconfig.data.FeatureConfigRepository
 import org.dhis2.commons.featureconfig.data.FeatureConfigRepositoryImpl
 import org.dhis2.commons.prefs.PreferenceProvider
-import org.hisp.dhis.android.core.D2Manager
 
 @Module
 class FeatureConfigModule {
 
     @Provides
     fun provideRepository(preferenceProvider: PreferenceProvider): FeatureConfigRepository {
-        return FeatureConfigRepositoryImpl(preferenceProvider, provideD2())
+        return FeatureConfigRepositoryImpl(preferenceProvider)
     }
-
-    private fun provideD2() = D2Manager.getD2()
 }