[#3482] Set and Get navigation results according to destination's bac…

…kstacks (#3483)
Oztechan · May 31, 2024 · 7578901 · 7578901
1 parent 1e83765
commit 7578901
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 9 deletions.
diff --git a/...c/main/kotlin/com/oztechan/ccc/android/ui/mobile/content/calculator/CalculatorFragment.kt b/...c/main/kotlin/com/oztechan/ccc/android/ui/mobile/content/calculator/CalculatorFragment.kt
@@ -78,7 +78,7 @@ class CalculatorFragment : BaseVBFragment<FragmentCalculatorBinding>() {
     private fun observeNavigationResults() = getNavigationResult<String>(
         CHANGE_BASE_EVENT,
         R.id.calculatorFragment
-    )?.observe(viewLifecycleOwner) {
+    ).observe(viewLifecycleOwner) {
         Logger.i { "CalculatorFragment observeNavigationResults $it" }
         calculatorViewModel.event.onBaseChange(it)
     }

diff --git a/android/ui/mobile/src/main/kotlin/com/oztechan/ccc/android/ui/mobile/util/ViewExtensions.kt b/android/ui/mobile/src/main/kotlin/com/oztechan/ccc/android/ui/mobile/util/ViewExtensions.kt
@@ -22,7 +22,6 @@ import androidx.core.view.isVisible
 import androidx.fragment.app.Fragment
 import androidx.navigation.fragment.findNavController
 import com.github.submob.scopemob.castTo
-import com.github.submob.scopemob.whether
 import com.oztechan.ccc.android.core.ad.AdManager
 import com.oztechan.ccc.android.core.ad.BannerAdView
 import com.oztechan.ccc.android.ui.mobile.R
@@ -77,19 +76,18 @@ fun <T> Fragment.getNavigationResult(
     key: String,
     destinationId: Int
 ) = findNavController()
-    .currentBackStackEntry
-    ?.whether { it.destination.id == destinationId }
-    ?.savedStateHandle
-    ?.getLiveData<T>(key)
+    .getBackStackEntry(destinationId)
+    .savedStateHandle
+    .getLiveData<T>(key)
 
 fun <T> Fragment.setNavigationResult(
     destinationId: Int,
     result: T,
     key: String
 ) = findNavController()
-    .previousBackStackEntry
-    ?.whether { it.destination.id == destinationId }
-    ?.savedStateHandle?.set(key, result)
+    .getBackStackEntry(destinationId)
+    .savedStateHandle
+    .set(key, result)
 
 fun View?.visibleIf(visible: Boolean, bringFront: Boolean = false) = this?.apply {
     if (visible) {