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

Update Priority Modes segment summary

Browse Source 
* Display active modes regardless of DND status
* Only consider actually automatic rules for the "x rules can activate automatically" text.
* Also removed the duplicate settings observer in favor of ZenSettingsObserver.

Bug: 355615361
Test: atest ZenModesSummaryHelperTest
Flag: android.app.modes_ui
Change-Id: I39acb785e7b27c95ed664261640c42032acca9d1
This commit is contained in:
Matías Hernández
2024-08-06 19:45:57 +02:00
parent 55102216e1
commit f0952bef52
4 changed files with 133 additions and 152 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
res/values/strings.xml
View File

@@ -7984,6 +7984,26 @@
<!-- Priority Modes: Option to choose a calendar-events-based schedule for a mode. [CHAR_LIMIT=40] --> <!-- Priority Modes: Option to choose a calendar-events-based schedule for a mode. [CHAR_LIMIT=40] -->
<string name="zen_mode_select_schedule_calendar">Calendar events</string> <string name="zen_mode_select_schedule_calendar">Calendar events</string>
<!-- Priority Modes: Summary for the modes segment, when at least one mode is active. [CHAR LIMIT=NONE]-->
<string name="zen_modes_summary_some_active">
{count, plural, offset:2
=0 {}
=1 {{mode_1} is active}
=2 {{mode_1} and {mode_2} are active}
=3 {{mode_1}, {mode_2}, and {mode_3} are active}
other {{mode_1}, {mode_2}, and # more are active}
}
</string>
<!-- Priority Modes: Summary for the modes segment, when no modes are active. [CHAR LIMIT=NONE]-->
<string name="zen_modes_summary_none_active">
{count, plural,
=0 {}
=1 {1 mode can turn on automatically}
other {# modes can turn on automatically}
}
</string>
<!-- Priority Modes: Short text that indicates that a mode is currently on (active). [CHAR_LIMIT=10] --> <!-- Priority Modes: Short text that indicates that a mode is currently on (active). [CHAR_LIMIT=10] -->
<string name="zen_mode_active_text">ON</string> <string name="zen_mode_active_text">ON</string>

47
src/com/android/settings/notification/modes/ZenModeSummaryHelper.java
View File

@@ -42,13 +42,10 @@ import static android.service.notification.ZenPolicy.VISUAL_EFFECT_STATUS_BAR;
import android.content.Context; import android.content.Context;
import android.icu.text.MessageFormat; import android.icu.text.MessageFormat;
import android.provider.Settings;
import android.service.notification.ZenDeviceEffects; import android.service.notification.ZenDeviceEffects;
import android.service.notification.ZenModeConfig;
import android.service.notification.ZenPolicy; import android.service.notification.ZenPolicy;
import android.service.notification.ZenPolicy.ConversationSenders; import android.service.notification.ZenPolicy.ConversationSenders;
import android.service.notification.ZenPolicy.PeopleType; import android.service.notification.ZenPolicy.PeopleType;
import android.util.ArrayMap;
import androidx.annotation.NonNull; import androidx.annotation.NonNull;
import androidx.annotation.Nullable; import androidx.annotation.Nullable;
@@ -496,32 +493,36 @@ class ZenModeSummaryHelper {
return msgFormat.format(args); return msgFormat.format(args);
} }
String getSoundSummary(int zenMode, ZenModeConfig config) { String getModesSummary(List<ZenMode> modes) {
if (zenMode != Settings.Global.ZEN_MODE_OFF) { List<ZenMode> activeModes = modes.stream().filter(ZenMode::isActive).toList();
String description = ZenModeConfig.getDescription(mContext, true, config, false);
if (description == null) { if (!activeModes.isEmpty()) {
return mContext.getString(R.string.zen_mode_sound_summary_on); MessageFormat msgFormat = new MessageFormat(
} else { mContext.getString(R.string.zen_modes_summary_some_active),
return mContext.getString(R.string.zen_mode_sound_summary_on_with_info, Locale.getDefault());
description);
} Map<String, Object> args = new HashMap<>();
} else { args.put("count", activeModes.size());
int count = 0; args.put("mode_1", activeModes.get(0).getName());
final ArrayMap<String, ZenModeConfig.ZenRule> ruleMap = config.automaticRules; if (activeModes.size() >= 2) {
if (ruleMap != null) { args.put("mode_2", activeModes.get(1).getName());
for (ZenModeConfig.ZenRule rule : ruleMap.values()) { if (activeModes.size() == 3) {
if (rule != null && rule.enabled) { args.put("mode_3", activeModes.get(2).getName());
count++;
}
} }
} }
return msgFormat.format(args);
} else {
int automaticModeCount = (int) modes.stream()
.filter(m -> !m.isManualDnd() && !m.isCustomManual())
.count();
MessageFormat msgFormat = new MessageFormat( MessageFormat msgFormat = new MessageFormat(
mContext.getString(R.string.modes_sound_summary_off), mContext.getString(R.string.zen_modes_summary_none_active),
Locale.getDefault()); Locale.getDefault());
Map<String, Object> msgArgs = new HashMap<>(); Map<String, Object> msgArgs = Map.of("count", automaticModeCount);
msgArgs.put("count", count);
return msgFormat.format(msgArgs); return msgFormat.format(msgArgs);
} }
} }
} }

81
src/com/android/settings/notification/modes/ZenModesLinkPreferenceController.java
View File

@@ -17,35 +17,31 @@
package com.android.settings.notification.modes; package com.android.settings.notification.modes;
import android.app.Flags; import android.app.Flags;
import android.app.NotificationManager;
import android.content.ContentResolver;
import android.content.Context; import android.content.Context;
import android.database.ContentObserver;
import android.net.Uri;
import android.os.Handler;
import android.os.UserHandle;
import android.provider.Settings;
import androidx.preference.Preference; import androidx.preference.Preference;
import androidx.preference.PreferenceScreen; import androidx.preference.PreferenceScreen;
import com.android.settings.R;
import com.android.settings.core.BasePreferenceController; import com.android.settings.core.BasePreferenceController;
import com.android.settingslib.core.lifecycle.LifecycleObserver; import com.android.settingslib.core.lifecycle.LifecycleObserver;
import com.android.settingslib.core.lifecycle.events.OnPause; import com.android.settingslib.core.lifecycle.events.OnStart;
import com.android.settingslib.core.lifecycle.events.OnResume; import com.android.settingslib.core.lifecycle.events.OnStop;
import com.android.settingslib.notification.modes.ZenModesBackend;
public class ZenModesLinkPreferenceController extends BasePreferenceController public class ZenModesLinkPreferenceController extends BasePreferenceController
implements LifecycleObserver, OnResume, OnPause { implements LifecycleObserver, OnStart, OnStop {
private SettingObserver mSettingObserver; private final ZenModesBackend mBackend;
private ZenModeSummaryHelper mSummaryBuilder; private final ZenSettingsObserver mSettingObserver;
private NotificationManager mNm; private final ZenModeSummaryHelper mSummaryBuilder;
private Preference mPreference;
public ZenModesLinkPreferenceController(Context context, String key) { public ZenModesLinkPreferenceController(Context context, String key) {
super(context, key); super(context, key);
mBackend = ZenModesBackend.getInstance(context);
mSummaryBuilder = new ZenModeSummaryHelper(context, ZenHelperBackend.getInstance(context)); mSummaryBuilder = new ZenModeSummaryHelper(context, ZenHelperBackend.getInstance(context));
mNm = mContext.getSystemService(NotificationManager.class); mSettingObserver = new ZenSettingsObserver(context, this::onZenSettingsChanged);
} }
@Override @Override
@@ -57,64 +53,31 @@ public class ZenModesLinkPreferenceController extends BasePreferenceController
@Override @Override
public void displayPreference(PreferenceScreen screen) { public void displayPreference(PreferenceScreen screen) {
super.displayPreference(screen); super.displayPreference(screen);
Preference preference = screen.findPreference(getPreferenceKey()); mPreference = screen.findPreference(getPreferenceKey());
mSettingObserver = new SettingObserver(preference);
} }
@Override @Override
public void onResume() { public void onStart() {
if (mSettingObserver != null) { if (mSettingObserver != null) {
mSettingObserver.register(mContext.getContentResolver()); mSettingObserver.register();
} }
} }
@Override private void onZenSettingsChanged() {
public void onPause() { if (mPreference != null) {
if (mSettingObserver != null) { updateState(mPreference);
mSettingObserver.unregister(mContext.getContentResolver());
} }
} }
@Override @Override
public void updateState(Preference preference) { public void updateState(Preference preference) {
preference.setSummary(mSummaryBuilder.getSoundSummary( preference.setSummary(mSummaryBuilder.getModesSummary(mBackend.getModes()));
Settings.Global.getInt(mContext.getContentResolver(),
Settings.Global.ZEN_MODE,
Settings.Global.ZEN_MODE_OFF),
mNm.getZenModeConfig()));
} }
class SettingObserver extends ContentObserver { @Override
private final Uri ZEN_MODE_URI = Settings.Global.getUriFor(Settings.Global.ZEN_MODE); public void onStop() {
private final Uri ZEN_MODE_CONFIG_ETAG_URI = Settings.Global.getUriFor( if (mSettingObserver != null) {
Settings.Global.ZEN_MODE_CONFIG_ETAG); mSettingObserver.unregister();
private final Preference mPreference;
public SettingObserver(Preference preference) {
super(new Handler());
mPreference = preference;
}
public void register(ContentResolver cr) {
cr.registerContentObserver(ZEN_MODE_URI, false, this, UserHandle.USER_ALL);
cr.registerContentObserver(ZEN_MODE_CONFIG_ETAG_URI, false, this, UserHandle.USER_ALL);
}
public void unregister(ContentResolver cr) {
cr.unregisterContentObserver(this);
}
@Override
public void onChange(boolean selfChange, Uri uri) {
super.onChange(selfChange, uri);
if (ZEN_MODE_URI.equals(uri)) {
updateState(mPreference);
}
if (ZEN_MODE_CONFIG_ETAG_URI.equals(uri)) {
updateState(mPreference);
}
} }
} }
} }

137
tests/robotests/src/com/android/settings/notification/modes/ZenModesSummaryHelperTest.java
View File

@@ -16,10 +16,6 @@
package com.android.settings.notification.modes; package com.android.settings.notification.modes;
import static android.provider.Settings.Global.ZEN_MODE_IMPORTANT_INTERRUPTIONS;
import static android.provider.Settings.Global.ZEN_MODE_OFF;
import static android.service.notification.Condition.SOURCE_UNKNOWN;
import static android.service.notification.Condition.STATE_TRUE;
import static android.service.notification.ZenPolicy.CONVERSATION_SENDERS_ANYONE; import static android.service.notification.ZenPolicy.CONVERSATION_SENDERS_ANYONE;
import static android.service.notification.ZenPolicy.CONVERSATION_SENDERS_IMPORTANT; import static android.service.notification.ZenPolicy.CONVERSATION_SENDERS_IMPORTANT;
import static android.service.notification.ZenPolicy.PEOPLE_TYPE_ANYONE; import static android.service.notification.ZenPolicy.PEOPLE_TYPE_ANYONE;
@@ -33,17 +29,16 @@ import static com.google.common.truth.Truth.assertThat;
import static org.robolectric.Shadows.shadowOf; import static org.robolectric.Shadows.shadowOf;
import android.app.AutomaticZenRule;
import android.app.Flags; import android.app.Flags;
import android.content.ComponentName;
import android.content.Context; import android.content.Context;
import android.content.pm.ApplicationInfo; import android.content.pm.ApplicationInfo;
import android.content.pm.UserInfo; import android.content.pm.UserInfo;
import android.net.Uri;
import android.os.UserHandle; import android.os.UserHandle;
import android.os.UserManager; import android.os.UserManager;
import android.platform.test.annotations.EnableFlags; import android.platform.test.annotations.EnableFlags;
import android.platform.test.flag.junit.SetFlagsRule; import android.platform.test.flag.junit.SetFlagsRule;
import android.service.notification.Condition; import android.service.notification.SystemZenRules;
import android.service.notification.ZenDeviceEffects; import android.service.notification.ZenDeviceEffects;
import android.service.notification.ZenModeConfig; import android.service.notification.ZenModeConfig;
import android.service.notification.ZenPolicy; import android.service.notification.ZenPolicy;
@@ -65,6 +60,7 @@ import org.robolectric.RuntimeEnvironment;
import java.util.Random; import java.util.Random;
@RunWith(RobolectricTestRunner.class) @RunWith(RobolectricTestRunner.class)
@EnableFlags(Flags.FLAG_MODES_UI)
public class ZenModesSummaryHelperTest { public class ZenModesSummaryHelperTest {
private static final int WORK_PROFILE_ID = 3; private static final int WORK_PROFILE_ID = 3;
@@ -480,85 +476,86 @@ public class ZenModesSummaryHelperTest {
} }
@Test @Test
@EnableFlags(Flags.FLAG_MODES_UI) public void getModesSummary_noRules_noSummary() {
public void getSoundSummary_off_noRules() { String summary = mSummaryHelper.getModesSummary(ImmutableList.of());
ZenModeConfig config = new ZenModeConfig(); assertThat(summary).isEmpty();
assertThat(mSummaryHelper.getSoundSummary(ZEN_MODE_OFF, config)).isEqualTo("Off");
} }
@Test @Test
@EnableFlags(Flags.FLAG_MODES_UI) public void getModesSummary_onlyDndAndNotActive_noSummary() {
public void getSoundSummary_off_oneRule() { ImmutableList<ZenMode> modes = ImmutableList.of(TestModeBuilder.MANUAL_DND_INACTIVE);
ZenModeConfig config = new ZenModeConfig(); String summary = mSummaryHelper.getModesSummary(modes);
ZenModeConfig.ZenRule rule = new ZenModeConfig.ZenRule(); assertThat(summary).isEmpty();
rule.enabled = true;
config.automaticRules.put("key", rule);
assertThat(mSummaryHelper.getSoundSummary(ZEN_MODE_OFF, config))
.isEqualTo("Off / 1 mode can turn on automatically");
} }
@Test @Test
@EnableFlags(Flags.FLAG_MODES_UI) public void getModesSummary_noRulesActive_countsOnlyAutomaticModes() {
public void getSoundSummary_off_twoRules() { ImmutableList<ZenMode> modes = ImmutableList.of(
ZenModeConfig config = new ZenModeConfig(); TestModeBuilder.MANUAL_DND_INACTIVE, // Not automatic
ZenModeConfig.ZenRule rule = new ZenModeConfig.ZenRule(); new TestModeBuilder().setName("Auto 1").build(), // App provided automatic
rule.enabled = true; new TestModeBuilder()
ZenModeConfig.ZenRule rule2 = new ZenModeConfig.ZenRule(); .setName("Custom manual 1")
rule2.enabled = true; .setPackage(SystemZenRules.PACKAGE_ANDROID)
config.automaticRules.put("key", rule); .setType(AutomaticZenRule.TYPE_OTHER)
config.automaticRules.put("key2", rule2); .setConditionId(ZenModeConfig.toCustomManualConditionId())
.build(), // Custom manual, not automatic
new TestModeBuilder()
.setName("Sleep")
.setPackage(SystemZenRules.PACKAGE_ANDROID)
.setType(AutomaticZenRule.TYPE_SCHEDULE_TIME)
.build() // Time based, automatic.
);
assertThat(mSummaryHelper.getSoundSummary(ZEN_MODE_OFF, config)) String summary = mSummaryHelper.getModesSummary(modes);
.isEqualTo("Off / 2 modes can turn on automatically"); assertThat(summary).isEqualTo("2 modes can turn on automatically");
} }
@Test @Test
@EnableFlags(Flags.FLAG_MODES_UI) public void getModesSummary_oneModeActive_listsMode() {
public void getSoundSummary_on_noDescription() { ImmutableList<ZenMode> modes = ImmutableList.of(
ZenModeConfig config = new ZenModeConfig(); TestModeBuilder.MANUAL_DND_ACTIVE,
config.manualRule.conditionId = Uri.EMPTY; new TestModeBuilder().setName("Inactive").setActive(false).build());
config.manualRule.pkg = "android";
config.manualRule.zenMode = ZEN_MODE_IMPORTANT_INTERRUPTIONS; String summary = mSummaryHelper.getModesSummary(modes);
config.manualRule.condition = new Condition(Uri.EMPTY, "", STATE_TRUE, SOURCE_UNKNOWN); assertThat(summary).isEqualTo("Do Not Disturb is active");
assertThat(mSummaryHelper.getSoundSummary(ZEN_MODE_IMPORTANT_INTERRUPTIONS, config))
.isEqualTo("On");
} }
@Test @Test
@EnableFlags(Flags.FLAG_MODES_UI) public void getModesSummary_twoModesActive_listsModes() {
public void getSoundSummary_on_manualDescription() { ImmutableList<ZenMode> modes = ImmutableList.of(
ZenModeConfig config = new ZenModeConfig(); TestModeBuilder.MANUAL_DND_ACTIVE,
config.manualRule.conditionId = ZenModeConfig.toCountdownConditionId( new TestModeBuilder().setName("Inactive").setActive(false).build(),
System.currentTimeMillis() + 10000, false); new TestModeBuilder().setName("Active #1").setActive(true).build());
config.manualRule.pkg = "android";
config.manualRule.zenMode = ZEN_MODE_IMPORTANT_INTERRUPTIONS; String summary = mSummaryHelper.getModesSummary(modes);
config.manualRule.condition = new Condition(Uri.EMPTY, "", STATE_TRUE, SOURCE_UNKNOWN); assertThat(summary).isEqualTo("Do Not Disturb and Active #1 are active");
assertThat(mSummaryHelper.getSoundSummary(ZEN_MODE_IMPORTANT_INTERRUPTIONS, config))
.startsWith("On /");
} }
@Test @Test
@EnableFlags(Flags.FLAG_MODES_UI) public void getModesSummary_threeModesActive_listsModes() {
public void getSoundSummary_on_automatic() { ImmutableList<ZenMode> modes = ImmutableList.of(
ZenModeConfig config = new ZenModeConfig(); TestModeBuilder.MANUAL_DND_INACTIVE,
ZenModeConfig.ZenRule rule = new ZenModeConfig.ZenRule(); new TestModeBuilder().setName("Inactive #1").setActive(false).build(),
rule.configurationActivity = new ComponentName("a", "a"); new TestModeBuilder().setName("Active #1").setActive(true).build(),
rule.component = new ComponentName("b", "b"); new TestModeBuilder().setName("Active #2").setActive(true).build(),
rule.conditionId = new Uri.Builder().scheme("hello").build(); new TestModeBuilder().setName("Inactive #2").setActive(false).build(),
rule.condition = new Condition(rule.conditionId, "", STATE_TRUE); new TestModeBuilder().setName("Active #3").setActive(true).build());
rule.enabled = true;
rule.creationTime = 123;
rule.id = "id";
rule.zenMode = ZEN_MODE_IMPORTANT_INTERRUPTIONS;
rule.modified = true;
rule.name = "name";
rule.snoozing = false;
rule.pkg = "b";
config.automaticRules.put("key", rule);
assertThat(mSummaryHelper.getSoundSummary(ZEN_MODE_IMPORTANT_INTERRUPTIONS, config)) String summary = mSummaryHelper.getModesSummary(modes);
.startsWith("On /"); assertThat(summary).isEqualTo("Active #1, Active #2, and Active #3 are active");
}
@Test
public void getModesSummary_manyModesActive_listsACouple() {
ImmutableList<ZenMode> modes = ImmutableList.of(
TestModeBuilder.MANUAL_DND_ACTIVE,
new TestModeBuilder().setName("Inactive #1").setActive(false).build(),
new TestModeBuilder().setName("Active #1").setActive(true).build(),
new TestModeBuilder().setName("Active #2").setActive(true).build(),
new TestModeBuilder().setName("Inactive #2").setActive(false).build(),
new TestModeBuilder().setName("Active #3").setActive(true).build());
String summary = mSummaryHelper.getModesSummary(modes);
assertThat(summary).isEqualTo("Do Not Disturb, Active #1, and 2 more are active");
} }
} }