Merge pull request #1889 from Infomaniak/mutable-parameter

Remove mutable parameter.
Infomaniak · Jun 4, 2024 · df00f52 · df00f52
2 parents 16cd037 + 5fa2e68
commit df00f52
Showing 1 changed file with 6 additions and 9 deletions.
diff --git a/app/src/main/java/com/infomaniak/mail/ui/newMessage/ContactAdapter.kt b/app/src/main/java/com/infomaniak/mail/ui/newMessage/ContactAdapter.kt
@@ -43,7 +43,7 @@ class ContactAdapter(
 ) : Adapter<ContactViewHolder>() {
 
     private var allContacts: List<MergedContact> = emptyList()
-    private var matchedContacts = mutableListOf<MatchedContact>()
+    private var matchedContacts = listOf<MatchedContact>()
 
     private var displayAddUnknownContactButton = true
     private var searchQuery = ""
@@ -99,12 +99,13 @@ class ContactAdapter(
     }
 
     fun clear() {
-        matchedContacts.clear()
+        matchedContacts = listOf()
         notifyDataSetChanged()
     }
 
     fun searchContacts(text: CharSequence) {
-        fun performFiltering(constraint: CharSequence): MutableList<MatchedContact> {
+
+        fun performFiltering(constraint: CharSequence): List<MatchedContact> {
             val searchTerm = constraint.standardize()
 
             val finalUserList = mutableListOf<MatchedContact>()
@@ -126,13 +127,9 @@ class ContactAdapter(
             return finalUserList
         }
 
-        fun publishResults(results: MutableList<MatchedContact>) {
-            matchedContacts = results
-            notifyDataSetChanged()
-        }
-
         searchQuery = text.toString()
-        publishResults(performFiltering(text))
+        matchedContacts = performFiltering(text)
+        notifyDataSetChanged()
     }
 
     fun removeUsedEmail(email: String): Boolean {