| Révision | 585d4e43e87b3cc95763bd4342a3997acb55de8b (tree) |
|---|---|
| l'heure | 2010-03-09 00:27:52 |
| Auteur | Patrick Scott <phanna@andr...> |
| Commiter | Patrick Scott |
Enable the alarm when preferences change and not when hitting done.
The original intent was to enable the alarm when hitting done after making edits
but this caused the alarm to always turn on even if the user turned the alarm
off and hit done.
Bug: 2441762
Change-Id: I9f37ec2477ddced26df1c1fb4444cd5e5121e7d5
src/com/android/deskclock/SetAlarm.java (diff)| @@ -144,8 +144,6 @@ public class SetAlarm extends PreferenceActivity | ||
| 144 | 144 | Button b = (Button) findViewById(R.id.alarm_save); |
| 145 | 145 | b.setOnClickListener(new View.OnClickListener() { |
| 146 | 146 | public void onClick(View v) { |
| 147 | - // Enable the alarm when clicking "Done" | |
| 148 | - mEnabledPref.setChecked(true); | |
| 149 | 147 | saveAlarm(); |
| 150 | 148 | finish(); |
| 151 | 149 | } |
| @@ -187,11 +185,15 @@ public class SetAlarm extends PreferenceActivity | ||
| 187 | 185 | // Used to post runnables asynchronously. |
| 188 | 186 | private static final Handler sHandler = new Handler(); |
| 189 | 187 | |
| 190 | - public boolean onPreferenceChange(Preference p, Object newValue) { | |
| 188 | + public boolean onPreferenceChange(final Preference p, Object newValue) { | |
| 191 | 189 | // Asynchronously save the alarm since this method is called _before_ |
| 192 | 190 | // the value of the preference has changed. |
| 193 | 191 | sHandler.post(new Runnable() { |
| 194 | 192 | public void run() { |
| 193 | + // Editing any preference (except enable) enables the alarm. | |
| 194 | + if (p != mEnabledPref) { | |
| 195 | + mEnabledPref.setChecked(true); | |
| 196 | + } | |
| 195 | 197 | saveAlarmAndEnableRevert(); |
| 196 | 198 | } |
| 197 | 199 | }); |
Android-x86
Fork