diff options
author | George Lin <giolin@google.com> | 2023-10-17 20:39:16 +0000 |
---|---|---|
committer | George Lin <giolin@google.com> | 2023-10-25 13:25:19 +0000 |
commit | 02ba1a4081bf2e9faae1e9d39b838658c507e957 (patch) | |
tree | 1a2d9828042d3a8064ed1aa29f8cde92982953c0 /src/com | |
parent | 0768fb09137a8068b3326d24521563e93cd9230a (diff) | |
download | ThemePicker-02ba1a4081bf2e9faae1e9d39b838658c507e957.tar.gz |
[WPP logging] Wire logLockScreenNotificationApplied
Test: Manually tested. See bug.
Fixes: 305753308
Flag: NONE
Change-Id: I0be9a454d6cb6f8634dd00f3a25814aeb6323086
Diffstat (limited to 'src/com')
-rw-r--r-- | src/com/android/customization/module/ThemePickerInjector.kt | 5 | ||||
-rw-r--r-- | src/com/android/customization/picker/notifications/ui/viewmodel/NotificationSectionViewModel.kt | 11 |
2 files changed, 12 insertions, 4 deletions
diff --git a/src/com/android/customization/module/ThemePickerInjector.kt b/src/com/android/customization/module/ThemePickerInjector.kt index 8d91a0e8..d2781c23 100644 --- a/src/com/android/customization/module/ThemePickerInjector.kt +++ b/src/com/android/customization/module/ThemePickerInjector.kt @@ -134,9 +134,7 @@ internal constructor( wallpaperColorsRepository = getWallpaperColorsRepository(), ), getKeyguardQuickAffordancePickerViewModelFactory(appContext), - NotificationSectionViewModel.Factory( - interactor = getNotificationsInteractor(appContext), - ), + getNotificationSectionViewModelFactory(appContext), getFlags(), getClockCarouselViewModelFactory( interactor = getClockPickerInteractor(appContext), @@ -286,6 +284,7 @@ internal constructor( return notificationSectionViewModelFactory ?: NotificationSectionViewModel.Factory( interactor = getNotificationsInteractor(context), + logger = getUserEventLogger(context), ) .also { notificationSectionViewModelFactory = it } } diff --git a/src/com/android/customization/picker/notifications/ui/viewmodel/NotificationSectionViewModel.kt b/src/com/android/customization/picker/notifications/ui/viewmodel/NotificationSectionViewModel.kt index 954efa24..1a5254f8 100644 --- a/src/com/android/customization/picker/notifications/ui/viewmodel/NotificationSectionViewModel.kt +++ b/src/com/android/customization/picker/notifications/ui/viewmodel/NotificationSectionViewModel.kt @@ -21,6 +21,7 @@ import androidx.annotation.VisibleForTesting import androidx.lifecycle.ViewModel import androidx.lifecycle.ViewModelProvider import androidx.lifecycle.viewModelScope +import com.android.customization.module.logging.ThemesUserEventLogger import com.android.customization.picker.notifications.domain.interactor.NotificationsInteractor import kotlinx.coroutines.flow.Flow import kotlinx.coroutines.flow.map @@ -31,6 +32,7 @@ class NotificationSectionViewModel @VisibleForTesting constructor( private val interactor: NotificationsInteractor, + private val logger: ThemesUserEventLogger, ) : ViewModel() { /** Whether the switch should be on. */ @@ -39,16 +41,23 @@ constructor( /** Notifies that the section has been clicked. */ fun onClicked() { - viewModelScope.launch { interactor.toggleShowNotificationsOnLockScreenEnabled() } + viewModelScope.launch { + interactor.toggleShowNotificationsOnLockScreenEnabled() + logger.logLockScreenNotificationApplied( + interactor.getSettings().isShowNotificationsOnLockScreenEnabled + ) + } } class Factory( private val interactor: NotificationsInteractor, + private val logger: ThemesUserEventLogger, ) : ViewModelProvider.Factory { @Suppress("UNCHECKED_CAST") override fun <T : ViewModel> create(modelClass: Class<T>): T { return NotificationSectionViewModel( interactor = interactor, + logger = logger, ) as T } |