fix(plugin): support for spannable string added to dataSelector

srtvprateek · srtvprateek · commit 800de5447a18 · 2024-03-31T00:20:38.000+05:30
diff --git a/pluto-plugins/base/lib/src/main/java/com/pluto/utilities/selector/DataSelector.kt b/pluto-plugins/base/lib/src/main/java/com/pluto/utilities/selector/DataSelector.kt
@@ -41,7 +41,9 @@ class Selector : ViewModel() {
 }
 
 abstract class SelectorOption : ListItem() {
-    abstract fun displayText(): String
+    abstract fun displayText(): CharSequence
+
+    fun displayTextString(): String = displayText().toString()
 }
 
 internal data class SelectorData<T>(val title: String, val list: List<SelectorOption>, val preSelected: T?)
diff --git a/pluto-plugins/base/lib/src/main/java/com/pluto/utilities/selector/ui/DataSelectorDialog.kt b/pluto-plugins/base/lib/src/main/java/com/pluto/utilities/selector/ui/DataSelectorDialog.kt
@@ -80,9 +80,12 @@ class DataSelectorDialog : BottomSheetDialogFragment() {
                 object : DiffAwareAdapter.OnActionListener {
                     override fun onAction(action: String, data: ListItem, holder: DiffAwareHolder) {
                         if (data is SelectorOption) {
-                            val tempSet = tempSelectedOptionLiveData.value?.toHashSet() ?: hashSetOf()
-                            if (!tempSet.add(data)) {
-                                tempSet.remove(data)
+                            var tempSet = tempSelectedOptionLiveData.value?.toHashSet() ?: hashSetOf()
+                            val isOptionAlreadyPresent = tempSet.any { it.displayTextString() == data.displayTextString() }
+                            if (isOptionAlreadyPresent) {
+                                tempSet = tempSet.filter { it.displayTextString() != data.displayTextString() }.toHashSet()
+                            } else {
+                                tempSet.add(data)
                             }
                             tempSelectedOptionLiveData.postValue(tempSet.toList())
                         }
diff --git a/pluto-plugins/base/lib/src/main/java/com/pluto/utilities/selector/ui/MultiSelectorAdapter.kt b/pluto-plugins/base/lib/src/main/java/com/pluto/utilities/selector/ui/MultiSelectorAdapter.kt
@@ -54,7 +54,8 @@ internal class MultiSelectorItemHolder(
         selectedLiveData.removeObserver(selectedChoicesObserver)
     }
 
-    private val selectedChoicesObserver = Observer<List<SelectorOption>> {
-        binding.checkbox.isSelected = it?.contains(item) ?: run { false }
+    private val selectedChoicesObserver = Observer<List<SelectorOption>> { list ->
+        binding.checkbox.isSelected =
+            item is SelectorOption && list?.any { it.displayTextString() == (item as SelectorOption).displayTextString() } ?: run { false }
     }
 }
diff --git a/pluto-plugins/base/lib/src/main/java/com/pluto/utilities/selector/ui/SingleSelectorAdapter.kt b/pluto-plugins/base/lib/src/main/java/com/pluto/utilities/selector/ui/SingleSelectorAdapter.kt
@@ -32,7 +32,7 @@ internal class SingleSelectorItemHolder(
             binding.title.setSpan {
                 append(item.displayText())
             }
-            binding.radio.isSelected = selected?.let { it == item } ?: run { false }
+            binding.radio.isSelected = selected?.let { it.displayTextString() == item.displayTextString() } ?: run { false }
             binding.root.setOnDebounceClickListener {
                 onAction("click")
             }

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,9 @@ class Selector : ViewModel() {`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`abstract class SelectorOption : ListItem() {`
`44`		`- abstract fun displayText(): String`
	`44`	`+ abstract fun displayText(): CharSequence`
	`45`	`+`
	`46`	`+ fun displayTextString(): String = displayText().toString()`
`45`	`47`	`}`
`46`	`48`
`47`	`49`	`internal data class SelectorData<T>(val title: String, val list: List<SelectorOption>, val preSelected: T?)`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,8 @@ internal class MultiSelectorItemHolder(`
`54`	`54`	`selectedLiveData.removeObserver(selectedChoicesObserver)`
`55`	`55`	`}`
`56`	`56`
`57`		`- private val selectedChoicesObserver = Observer<List<SelectorOption>> {`
`58`		`- binding.checkbox.isSelected = it?.contains(item) ?: run { false }`
	`57`	`+ private val selectedChoicesObserver = Observer<List<SelectorOption>> { list ->`
	`58`	`+ binding.checkbox.isSelected =`
	`59`	`+ item is SelectorOption && list?.any { it.displayTextString() == (item as SelectorOption).displayTextString() } ?: run { false }`
`59`	`60`	`}`
`60`	`61`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ internal class SingleSelectorItemHolder(`
`32`	`32`	`binding.title.setSpan {`
`33`	`33`	`append(item.displayText())`
`34`	`34`	`}`
`35`		`- binding.radio.isSelected = selected?.let { it == item } ?: run { false }`
	`35`	`+ binding.radio.isSelected = selected?.let { it.displayTextString() == item.displayTextString() } ?: run { false }`
`36`	`36`	`binding.root.setOnDebounceClickListener {`
`37`	`37`	`onAction("click")`
`38`	`38`	`}`