PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Use string notifications_disabled instead of off" into udc-dev

Browse Source
This commit is contained in:
Kweku Adams
2023-05-01 15:37:22 +00:00
committed by Android (Google) Code Review
parent d7bbff0041 ca615997ea
commit 83c04b32d1
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/com/android/settings/spa/notification/AppNotificationRepository.kt
View File

@@ -123,13 +123,15 @@ class AppNotificationRepository(
} }
override fun getNotificationSummary(app: ApplicationInfo): String { override fun getNotificationSummary(app: ApplicationInfo): String {
if (!isEnabled(app)) return context.getString(R.string.off) if (!isEnabled(app)) return context.getString(R.string.notifications_disabled)
val channelCount = getChannelCount(app) val channelCount = getChannelCount(app)
if (channelCount == 0) { if (channelCount == 0) {
return calculateFrequencySummary(getSentCount(app)) return calculateFrequencySummary(getSentCount(app))
} }
val blockedChannelCount = getBlockedChannelCount(app) val blockedChannelCount = getBlockedChannelCount(app)
if (channelCount == blockedChannelCount) return context.getString(R.string.off) if (channelCount == blockedChannelCount) {
return context.getString(R.string.notifications_disabled)
}
val frequencySummary = calculateFrequencySummary(getSentCount(app)) val frequencySummary = calculateFrequencySummary(getSentCount(app))
if (blockedChannelCount == 0) return frequencySummary if (blockedChannelCount == 0) return frequencySummary
return context.getString( return context.getString(

4
tests/spa_unit/src/com/android/settings/spa/notification/AppNotificationRepositoryTest.kt
View File

@@ -251,7 +251,7 @@ class AppNotificationRepositoryTest {
val summary = repository.getNotificationSummary(APP) val summary = repository.getNotificationSummary(APP)
assertThat(summary).isEqualTo(context.getString(R.string.off)) assertThat(summary).isEqualTo(context.getString(R.string.notifications_disabled))
} }
@Test @Test
@@ -273,7 +273,7 @@ class AppNotificationRepositoryTest {
val summary = repository.getNotificationSummary(APP) val summary = repository.getNotificationSummary(APP)
assertThat(summary).isEqualTo(context.getString(R.string.off)) assertThat(summary).isEqualTo(context.getString(R.string.notifications_disabled))
} }
@Test @Test