Fix other wrong usage of repeatOnLifecycle

vector/src/main/java/im/vector/app/core/extensions/TextInputLayout.kt

@@ -21,10 +21,9 @@ import android.text.Editable
 import android.view.View
 import android.view.inputmethod.EditorInfo
 import androidx.autofill.HintConstants
-import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.LifecycleOwner
 import androidx.lifecycle.lifecycleScope
-import androidx.lifecycle.repeatOnLifecycle
+import androidx.lifecycle.withResumed
 import com.google.android.material.textfield.TextInputLayout
 import im.vector.app.core.platform.SimpleTextWatcher
 import kotlinx.coroutines.flow.launchIn
@@ -88,7 +87,7 @@ fun TextInputLayout.setOnImeDoneListener(action: () -> Unit) {
 fun TextInputLayout.setOnFocusLostListener(lifecycleOwner: LifecycleOwner, action: () -> Unit) {
     editText().setOnFocusChangeListener { _, hasFocus ->
         when (hasFocus) {
-            false -> lifecycleOwner.lifecycleScope.launch { lifecycleOwner.repeatOnLifecycle(Lifecycle.State.RESUMED) { action() } }
+            false -> lifecycleOwner.lifecycleScope.launch { lifecycleOwner.withResumed { action() } }
             else -> {
                 // do nothing
             }

vector/src/main/java/im/vector/app/features/home/room/detail/TimelineFragment.kt

@@ -47,9 +47,8 @@ import androidx.core.view.isInvisible
 import androidx.core.view.isVisible
 import androidx.core.view.updatePadding
 import androidx.fragment.app.setFragmentResultListener
-import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.lifecycleScope
-import androidx.lifecycle.repeatOnLifecycle
+import androidx.lifecycle.withResumed
 import androidx.recyclerview.widget.ItemTouchHelper
 import androidx.recyclerview.widget.LinearLayoutManager
 import androidx.recyclerview.widget.RecyclerView
@@ -1112,29 +1111,31 @@ class TimelineFragment :
     private fun updateJumpToReadMarkerViewVisibility() {
         if (isThreadTimeLine()) return
         viewLifecycleOwner.lifecycleScope.launch {
-            viewLifecycleOwner.repeatOnLifecycle(Lifecycle.State.RESUMED) {
+            withResumed {
-                val state = timelineViewModel.awaitState()
+                viewLifecycleOwner.lifecycleScope.launch {
-                val showJumpToUnreadBanner = when (state.unreadState) {
+                    val state = timelineViewModel.awaitState()
-                    UnreadState.Unknown,
+                    val showJumpToUnreadBanner = when (state.unreadState) {
-                    UnreadState.HasNoUnread -> false
+                        UnreadState.Unknown,
-                    is UnreadState.ReadMarkerNotLoaded -> true
+                        UnreadState.HasNoUnread -> false
-                    is UnreadState.HasUnread -> {
+                        is UnreadState.ReadMarkerNotLoaded -> true
-                        if (state.canShowJumpToReadMarker) {
+                        is UnreadState.HasUnread -> {
-                            val lastVisibleItem = layoutManager.findLastCompletelyVisibleItemPosition()
+                            if (state.canShowJumpToReadMarker) {
-                            val positionOfReadMarker = withContext(Dispatchers.Default) {
+                                val lastVisibleItem = layoutManager.findLastCompletelyVisibleItemPosition()
-                                timelineEventController.getPositionOfReadMarker()
+                                val positionOfReadMarker = withContext(Dispatchers.Default) {
-                            }
+                                    timelineEventController.getPositionOfReadMarker()
-                            if (positionOfReadMarker == null) {
+                                }
-                                false
+                                if (positionOfReadMarker == null) {
+                                    false
+                                } else {
+                                    positionOfReadMarker > lastVisibleItem
+                                }
                             } else {
-                                positionOfReadMarker > lastVisibleItem
+                                false
                             }
-                        } else {
-                            false
                         }
                     }
+                    views.jumpToReadMarkerView.isVisible = showJumpToUnreadBanner
                 }
-                views.jumpToReadMarkerView.isVisible = showJumpToUnreadBanner
             }
         }
     }
@@ -1625,14 +1626,16 @@ class TimelineFragment :
 
     override fun onRoomCreateLinkClicked(url: String) {
         viewLifecycleOwner.lifecycleScope.launch {
-            viewLifecycleOwner.repeatOnLifecycle(Lifecycle.State.RESUMED) {
+            withResumed {
-                permalinkHandler
+                viewLifecycleOwner.lifecycleScope.launch {
-                        .launch(requireActivity(), url, object : NavigationInterceptor {
+                    permalinkHandler
-                            override fun navToRoom(roomId: String?, eventId: String?, deepLink: Uri?, rootThreadEventId: String?): Boolean {
+                            .launch(requireActivity(), url, object : NavigationInterceptor {
-                                requireActivity().finish()
+                                override fun navToRoom(roomId: String?, eventId: String?, deepLink: Uri?, rootThreadEventId: String?): Boolean {
-                                return false
+                                    requireActivity().finish()
-                            }
+                                    return false
-                        })
+                                }
+                            })
+                }
             }
         }
     }

vector/src/main/java/im/vector/app/features/location/LocationSharingFragment.kt

@@ -26,9 +26,7 @@ import androidx.core.content.ContextCompat
 import androidx.core.view.isGone
 import androidx.core.view.isVisible
 import androidx.fragment.app.setFragmentResultListener
-import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.lifecycleScope
-import androidx.lifecycle.repeatOnLifecycle
 import com.airbnb.mvrx.fragmentViewModel
 import com.airbnb.mvrx.withState
 import com.google.android.material.dialog.MaterialAlertDialogBuilder
@@ -101,12 +99,10 @@ class LocationSharingFragment :
         views.mapView.onCreate(savedInstanceState)
 
         viewLifecycleOwner.lifecycleScope.launch {
-            viewLifecycleOwner.repeatOnLifecycle(Lifecycle.State.CREATED) {
+            views.mapView.initialize(
-                views.mapView.initialize(
+                    url = urlMapProvider.getMapUrl(),
-                        url = urlMapProvider.getMapUrl(),
+                    locationTargetChangeListener = this@LocationSharingFragment
-                        locationTargetChangeListener = this@LocationSharingFragment
+            )
-                )
-            }
         }
 
         initLocateButton()

vector/src/main/java/im/vector/app/features/location/preview/LocationPreviewFragment.kt

@@ -22,9 +22,7 @@ import android.view.MenuItem
 import android.view.View
 import android.view.ViewGroup
 import androidx.core.view.isVisible
-import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.lifecycleScope
-import androidx.lifecycle.repeatOnLifecycle
 import com.airbnb.mvrx.args
 import com.airbnb.mvrx.fragmentViewModel
 import com.airbnb.mvrx.withState
@@ -81,9 +79,7 @@ class LocationPreviewFragment :
         views.mapView.onCreate(savedInstanceState)
 
         viewLifecycleOwner.lifecycleScope.launch {
-            viewLifecycleOwner.repeatOnLifecycle(Lifecycle.State.CREATED) {
+            views.mapView.initialize(urlMapProvider.getMapUrl())
-                views.mapView.initialize(urlMapProvider.getMapUrl())
-            }
         }
 
         observeViewEvents()

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

@@ -27,9 +27,8 @@ import android.view.View
 import android.view.ViewGroup
 import androidx.core.content.ContextCompat
 import androidx.core.view.isVisible
-import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.lifecycleScope
-import androidx.lifecycle.repeatOnLifecycle
+import androidx.lifecycle.withResumed
 import androidx.preference.Preference
 import androidx.preference.PreferenceCategory
 import androidx.preference.SwitchPreference
@@ -190,6 +189,10 @@ class VectorSettingsSecurityPrivacyFragment :
                     rawService
                             .getElementWellknown(session.sessionParams)
                             ?.isE2EByDefault() == false
+
+            refreshXSigningStatus()
+            // My device name may have been updated
+            refreshMyDevice()
         }
     }
 
@@ -288,19 +291,6 @@ class VectorSettingsSecurityPrivacyFragment :
             true
         }
 
-        lifecycleScope.launch {
-            repeatOnLifecycle(Lifecycle.State.RESUMED) {
-                refreshXSigningStatus()
-            }
-        }
-
-        lifecycleScope.launch {
-            repeatOnLifecycle(Lifecycle.State.RESUMED) {
-                // My device name may have been updated
-                refreshMyDevice()
-            }
-        }
-
         secureBackupPreference.icon = activity?.let {
             ThemeUtils.tintDrawable(
                     it,
@@ -429,16 +419,18 @@ class VectorSettingsSecurityPrivacyFragment :
 
     private fun openPinCodePreferenceScreen() {
         viewLifecycleOwner.lifecycleScope.launch {
-            viewLifecycleOwner.repeatOnLifecycle(Lifecycle.State.RESUMED) {
+            withResumed {
-                val hasPinCode = pinCodeStore.hasEncodedPin()
+                viewLifecycleOwner.lifecycleScope.launch {
-                if (hasPinCode) {
+                    val hasPinCode = pinCodeStore.hasEncodedPin()
-                    navigator.openPinCode(
+                    if (hasPinCode) {
-                            requireContext(),
+                        navigator.openPinCode(
-                            pinActivityResultLauncher,
+                                requireContext(),
-                            PinMode.AUTH
+                                pinActivityResultLauncher,
-                    )
+                                PinMode.AUTH
-                } else {
+                        )
-                    doOpenPinCodePreferenceScreen()
+                    } else {
+                        doOpenPinCodePreferenceScreen()
+                    }
                 }
             }
         }

vector/src/main/java/im/vector/app/features/userdirectory/UserListFragment.kt

@@ -24,9 +24,8 @@ import android.view.View
 import android.view.ViewGroup
 import android.widget.ScrollView
 import androidx.core.view.isVisible
-import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.lifecycleScope
-import androidx.lifecycle.repeatOnLifecycle
+import androidx.lifecycle.withResumed
 import com.airbnb.mvrx.activityViewModel
 import com.airbnb.mvrx.args
 import com.airbnb.mvrx.withState
@@ -178,7 +177,7 @@ class UserListFragment :
         // Scroll to the bottom when adding chips. When removing chips, do not scroll
         if (newNumberOfChips >= currentNumberOfChips) {
             viewLifecycleOwner.lifecycleScope.launch {
-                viewLifecycleOwner.repeatOnLifecycle(Lifecycle.State.RESUMED) {
+                withResumed {
                     views.chipGroupScrollView.fullScroll(ScrollView.FOCUS_DOWN)
                 }
             }