fix(NPE): https://sentry-mobile.infomaniak.com/organizations/sentry/i…

…ssues/18587/?project=7
Infomaniak · Dec 6, 2023 · 7f477f6 · 7f477f6
1 parent 74f10b8
commit 7f477f6
Showing 1 changed file with 25 additions and 7 deletions.
diff --git a/app/src/main/java/com/infomaniak/drive/ui/addFiles/CreateFolderFragment.kt b/app/src/main/java/com/infomaniak/drive/ui/addFiles/CreateFolderFragment.kt
@@ -18,10 +18,11 @@
 package com.infomaniak.drive.ui.addFiles
 
 import android.os.Bundle
+import android.text.Editable
+import android.text.TextWatcher
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
-import androidx.core.widget.doOnTextChanged
 import androidx.fragment.app.Fragment
 import androidx.fragment.app.activityViewModels
 import androidx.navigation.fragment.findNavController
@@ -38,21 +39,34 @@ import com.infomaniak.drive.ui.MainViewModel
 import com.infomaniak.drive.ui.fileList.fileShare.PermissionsAdapter
 import com.infomaniak.drive.utils.AccountUtils
 import com.infomaniak.drive.utils.showSnackbar
-import com.infomaniak.lib.core.utils.*
 import com.infomaniak.lib.core.utils.ApiErrorCode.Companion.translateError
+import com.infomaniak.lib.core.utils.hideKeyboard
+import com.infomaniak.lib.core.utils.hideProgress
+import com.infomaniak.lib.core.utils.initProgress
+import com.infomaniak.lib.core.utils.showProgress
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.runBlocking
 
 open class CreateFolderFragment : Fragment() {
 
-    protected var binding: FragmentCreateFolderBinding by safeBinding()
+    private var _binding: FragmentCreateFolderBinding? = null
+    protected val binding get() = _binding!! // This property is only valid between onCreateView and onDestroyView
 
     protected val newFolderViewModel: NewFolderViewModel by navGraphViewModels(R.id.newFolderFragment)
     protected val mainViewModel: MainViewModel by activityViewModels()
+
     protected lateinit var adapter: PermissionsAdapter
 
+    private val folderNameTextWatcher by lazy {
+        object : TextWatcher {
+            override fun beforeTextChanged(s: CharSequence?, start: Int, count: Int, after: Int) = Unit
+            override fun onTextChanged(s: CharSequence?, start: Int, before: Int, count: Int) = toggleCreateFolderButton()
+            override fun afterTextChanged(s: Editable?) = Unit
+        }
+    }
+
     override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View {
-        return FragmentCreateFolderBinding.inflate(inflater, container, false).also { binding = it }.root
+        return FragmentCreateFolderBinding.inflate(inflater, container, false).also { _binding = it }.root
     }
 
     override fun onViewCreated(view: View, savedInstanceState: Bundle?): Unit = with(binding) {
@@ -66,9 +80,13 @@ open class CreateFolderFragment : Fragment() {
             findNavController().popBackStack()
         }
 
-        folderNameValueInput.doOnTextChanged { _, _, _, _ ->
-            toggleCreateFolderButton()
-        }
+        folderNameValueInput.addTextChangedListener(folderNameTextWatcher)
+    }
+
+    override fun onDestroyView() {
+        binding.folderNameValueInput.removeTextChangedListener(folderNameTextWatcher)
+        super.onDestroyView()
+        _binding = null
     }
 
     protected fun canInherit(userList: ArrayList<UserFileAccess>, teamList: ArrayList<Team>): Boolean {