Update labs visibility

changelog.d/1019.misc

@@ -0,0 +1 @@
+Update labs visibility.

vector-app/src/debug/java/im/vector/app/features/debug/di/FeaturesModule.kt

@@ -22,6 +22,7 @@ import dagger.Module
 import dagger.Provides
 import dagger.hilt.InstallIn
 import dagger.hilt.components.SingletonComponent
+import im.vector.app.core.resources.AppNameProvider
 import im.vector.app.core.resources.BooleanProvider
 import im.vector.app.core.resources.StringArrayProvider
 import im.vector.app.features.DefaultVectorFeatures
@@ -44,8 +45,8 @@ interface FeaturesModule {
     companion object {
 
         @Provides
-        fun providesDefaultVectorFeatures(stringArrayProvider: StringArrayProvider, booleanProvider: BooleanProvider): DefaultVectorFeatures {
-            return DefaultVectorFeatures(stringArrayProvider, booleanProvider)
+        fun providesDefaultVectorFeatures(appNameProvider: AppNameProvider, stringArrayProvider: StringArrayProvider, booleanProvider: BooleanProvider): DefaultVectorFeatures {
+            return DefaultVectorFeatures(appNameProvider, stringArrayProvider, booleanProvider)
         }
 
         @Provides

vector-app/src/debug/java/im/vector/app/features/debug/features/DebugVectorFeatures.kt

@@ -48,6 +48,8 @@ class DebugVectorFeatures(
 
     override fun tchapIsThreadEnabled() = vectorFeatures.tchapIsThreadEnabled()
 
+    override fun tchapIsLabsVisible(domain: String) = vectorFeatures.tchapIsLabsVisible(domain)
+
     override fun onboardingVariant(): OnboardingVariant {
         return readPreferences().getEnum<OnboardingVariant>() ?: vectorFeatures.onboardingVariant()
     }

vector-app/src/release/java/im/vector/app/core/di/FeaturesModule.kt

@@ -32,8 +32,8 @@ import im.vector.app.features.VectorOverrides
 object FeaturesModule {
 
     @Provides
-    fun providesFeatures(stringArrayProvider: StringArrayProvider, booleanProvider: BooleanProvider): VectorFeatures {
-        return DefaultVectorFeatures(stringArrayProvider, booleanProvider)
+    fun providesFeatures(appNameProvider: AppNameProvider, stringArrayProvider: StringArrayProvider, booleanProvider: BooleanProvider): VectorFeatures {
+        return DefaultVectorFeatures(appNameProvider, stringArrayProvider, booleanProvider)
     }
 
     @Provides

vector-config/src/btchap/res/values/config-settings.xml

@@ -1,4 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
-    <bool name="tchap_settings_root_labs_visible">true</bool>
+    <!-- Level 0: Root -->
+    <bool name="settings_root_labs_visible">true</bool>
 </resources>

vector-config/src/debug/res/values/config-settings.xml

@@ -1,6 +1,9 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
 
+    <!-- Level 0: Root -->
+    <bool name="settings_root_labs_visible">true</bool>
+
     <!-- Level 1: Advanced settings -->
     <bool name="tchap_settings_advanced_developer_mode_visible">true</bool> <!-- Tchap: Show developer mode in debug -->
 

vector-config/src/devTchap/res/values/config-settings.xml

@@ -1,4 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
-    <bool name="tchap_settings_root_labs_visible">true</bool>
+    <!-- Level 0: Root -->
+    <bool name="settings_root_labs_visible">true</bool>
 </resources>

vector-config/src/main/res/values/config-settings.xml

@@ -17,8 +17,7 @@
     <!--    <bool name="settings_root_voice_video_visible">true</bool>-->
 
     <bool name="settings_root_security_privacy_visible">true</bool>
-    <!-- Tchap: use gradle config -->
-    <!--    <bool name="settings_root_labs_visible">true</bool>-->
+    <bool name="settings_root_labs_visible">false</bool>
     <bool name="settings_root_advanced_visible">true</bool>
     <bool name="settings_root_help_about_visible">true</bool>
     <bool name="settings_root_legals_visible">true</bool>

vector/src/main/java/im/vector/app/features/VectorFeatures.kt

@@ -19,6 +19,7 @@ package im.vector.app.features
 import im.vector.app.R
 import im.vector.app.config.Config
 import im.vector.app.config.OnboardingVariant
+import im.vector.app.core.resources.AppNameProvider
 import im.vector.app.core.resources.BooleanProvider
 import im.vector.app.core.resources.StringArrayProvider
 import im.vector.app.features.settings.VectorPreferences
@@ -30,6 +31,7 @@ interface VectorFeatures {
     fun tchapIsCrossSigningEnabled(): Boolean
     fun tchapIsKeyBackupEnabled(): Boolean
     fun tchapIsThreadEnabled(): Boolean
+    fun tchapIsLabsVisible(domain: String): Boolean
     fun onboardingVariant(): OnboardingVariant
     fun isOnboardingAlreadyHaveAccountSplashEnabled(): Boolean
     fun isOnboardingSplashCarouselEnabled(): Boolean
@@ -56,6 +58,7 @@ interface VectorFeatures {
 }
 
 class DefaultVectorFeatures @Inject constructor(
+        private val appNameProvider: AppNameProvider,
         private val stringArrayProvider: StringArrayProvider,
         private val booleanProvider: BooleanProvider
 ) : VectorFeatures {
@@ -66,6 +69,8 @@ class DefaultVectorFeatures @Inject constructor(
     override fun tchapIsCrossSigningEnabled() = booleanProvider.getBoolean(R.bool.tchap_is_cross_signing_enabled)
     override fun tchapIsKeyBackupEnabled() = booleanProvider.getBoolean(R.bool.tchap_is_key_backup_enabled)
     override fun tchapIsThreadEnabled() = booleanProvider.getBoolean(R.bool.tchap_is_thread_enabled)
+    override fun tchapIsLabsVisible(domain: String) = booleanProvider.getBoolean(R.bool.settings_root_labs_visible)
+            || domain == appNameProvider.getAppName()
     override fun onboardingVariant() = Config.ONBOARDING_VARIANT
     override fun isOnboardingAlreadyHaveAccountSplashEnabled() = true
     override fun isOnboardingSplashCarouselEnabled() = false // Tchap: no carousel

vector/src/main/java/im/vector/app/features/settings/VectorSettingsRootFragment.kt

@@ -19,16 +19,24 @@ package im.vector.app.features.settings
 import android.os.Bundle
 import androidx.preference.Preference
 import dagger.hilt.android.AndroidEntryPoint
+import fr.gouv.tchap.core.utils.TchapUtils
 import im.vector.app.R
 import im.vector.app.core.preference.VectorPreference
 import im.vector.app.core.utils.FirstThrottler
 import im.vector.app.core.utils.openUrlInChromeCustomTab
+import im.vector.app.features.VectorFeatures
 import im.vector.app.features.analytics.plan.MobileScreen
+import im.vector.app.features.displayname.getBestName
+import org.matrix.android.sdk.api.session.getUserOrDefault
+import org.matrix.android.sdk.api.util.toMatrixItem
+import javax.inject.Inject
 
 @AndroidEntryPoint
 class VectorSettingsRootFragment :
         VectorSettingsBaseFragment() {
 
+    @Inject lateinit var vectorFeatures: VectorFeatures
+
     override var titleRes: Int = R.string.title_activity_settings
     override val preferenceXmlRes = R.xml.vector_settings_root
 
@@ -50,6 +58,11 @@ class VectorSettingsRootFragment :
             }
             false
         }
+
+        // Tchap: Manage labs entry.
+        val myUserDisplayName = session.getUserOrDefault(session.myUserId).toMatrixItem().getBestName()
+        findPreference<VectorPreference>(VectorPreferences.SETTINGS_LABS_PREFERENCE_KEY)!!
+                .isVisible = vectorFeatures.tchapIsLabsVisible(TchapUtils.getDomainFromDisplayName(myUserDisplayName))
     }
 
     private fun tintIcons() {

vector/src/main/res/xml/vector_settings_root.xml

@@ -34,9 +34,10 @@
 
     <im.vector.app.core.preference.VectorPreference
         android:icon="@drawable/ic_settings_root_labs"
+        android:key="SETTINGS_LABS_PREFERENCE_KEY"
         android:title="@string/room_settings_labs_pref_title"
         app:fragment="im.vector.app.features.settings.labs.VectorSettingsLabsFragment"
-        app:isPreferenceVisible="@bool/tchap_settings_root_labs_visible" />
+        app:isPreferenceVisible="@bool/settings_root_labs_visible" />
 
     <im.vector.app.core.preference.VectorPreference
         android:icon="@drawable/ic_settings_root_advanced"