AccountEditionFragment: avoid using TabLayoutMediator before adapter is set

Change-Id: Ide37d61b7b4d911093e54fb8d1d24aa46f367ff1

jami-android/app/src/main/java/cx/ring/account/AccountEditionFragment.kt

@@ -25,6 +25,8 @@ import androidx.annotation.StringRes
 import androidx.fragment.app.*
 import androidx.recyclerview.widget.RecyclerView
 import androidx.viewpager2.adapter.FragmentStateAdapter
+import androidx.viewpager2.widget.ViewPager2
+import com.google.android.material.tabs.TabLayout
 import com.google.android.material.tabs.TabLayoutMediator
 import cx.ring.R
 import cx.ring.contactrequests.BlockListFragment
@@ -72,17 +74,21 @@ class AccountEditionFragment : BaseSupportFragment<AccountEditionPresenter, Acco
         super.onViewCreated(view, savedInstanceState)
 
         mAccountId = requireArguments().getString(ACCOUNT_ID_KEY)
-        mBinding!!.fragmentContainer.viewTreeObserver.addOnScrollChangedListener(this)
         presenter.init(mAccountId!!)
 
         mBinding?.apply {
-            TabLayoutMediator(slidingTabs, pager){ tab, position ->
+            fragmentContainer.viewTreeObserver.addOnScrollChangedListener(this@AccountEditionFragment)
+        }
+    }
+
+    private fun updateAdapter(tabLayout: TabLayout, viewPager: ViewPager2, accountId: String, isJami: Boolean) {
+        viewPager.adapter = PreferencesPagerAdapter(this@AccountEditionFragment, accountId, isJami)
+        TabLayoutMediator(tabLayout, viewPager){ tab, position ->
             tab.text = context?.getString(
                 if (mAccountIsJami) getJamiPanelTitle(position)
                 else getSIPPanelTitle(position))
         }.attach()
     }
-    }
 
     override fun displaySummary(accountId: String) {
         toggleView(accountId, true)
@@ -111,7 +117,7 @@ class AccountEditionFragment : BaseSupportFragment<AccountEditionPresenter, Acco
     override fun initViewPager(accountId: String, isJami: Boolean) {
         mBinding?.apply {
             pager.offscreenPageLimit = 4
-            pager.adapter = PreferencesPagerAdapter(this@AccountEditionFragment, accountId, isJami)
+            updateAdapter(slidingTabs, pager, accountId, isJami)
         }
         val existingFragment = childFragmentManager.findFragmentByTag(BlockListFragment.TAG) as BlockListFragment?
         if (existingFragment != null) {

jami-android/app/src/main/java/cx/ring/share/ScanFragment.kt

@@ -24,6 +24,7 @@ import android.content.pm.PackageManager
 import android.view.LayoutInflater
 import android.view.ViewGroup
 import android.os.Bundle
+import android.util.Log
 import cx.ring.R
 import androidx.annotation.StringRes
 import com.journeyapps.barcodescanner.DefaultDecoderFactory
@@ -129,7 +130,11 @@ class ScanFragment : Fragment() {
     }
 
     private fun goToConversation(conversationUri: String) {
+        try {
             (requireActivity() as HomeActivity).startConversation(conversationUri)
+        } catch (e: Exception) {
+            Log.w(TAG, "Error while starting conversation", e)
+        }
     }
 
     private fun checkPermission(): Boolean {