Skip to content

Commit f35a5a5

Browse files
committed
+ Adds: a check for the selected site null state
1 parent 5f832ec commit f35a5a5

1 file changed

Lines changed: 10 additions & 3 deletions

File tree

WordPress/src/main/java/org/wordpress/android/ui/mysite/personalization/PersonalizationViewModel.kt

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package org.wordpress.android.ui.mysite.personalization
22

3+
import androidx.lifecycle.LiveData
4+
import androidx.lifecycle.MutableLiveData
35
import androidx.lifecycle.viewModelScope
46
import dagger.hilt.android.lifecycle.HiltViewModel
57
import kotlinx.coroutines.CoroutineDispatcher
@@ -27,9 +29,14 @@ class PersonalizationViewModel @Inject constructor(
2729
}
2830

2931
fun start() {
30-
val siteId = selectedSiteRepository.getSelectedSite()!!.siteId
31-
dashboardCardPersonalizationViewModelSlice.start(siteId)
32-
shortcutsPersonalizationViewModelSlice.start(selectedSiteRepository.getSelectedSite()!!)
32+
val site = selectedSiteRepository.getSelectedSite()
33+
if (site == null) {
34+
//todo pass a state to the ui
35+
return
36+
}
37+
38+
dashboardCardPersonalizationViewModelSlice.start(site.siteId)
39+
shortcutsPersonalizationViewModelSlice.start(site)
3340
}
3441

3542
fun onCardToggled(cardType: CardType, enabled: Boolean) {

0 commit comments

Comments
 (0)