From efef58276ec90749f91d0bf5e69ab8e9fbc599ea Mon Sep 17 00:00:00 2001
From: Christopher Eby <kreed@kreed.org>
Date: Fri, 2 Apr 2010 10:51:23 -0500
Subject: [PATCH] Allow the notification to never be shown

---
 res/values/arrays.xml                      | 10 +++++
 res/values/strings.xml                     |  4 +-
 res/xml/preferences.xml                    | 12 +++---
 src/org/kreed/vanilla/PlaybackService.java | 45 ++++++++++++----------
 4 files changed, 43 insertions(+), 28 deletions(-)

diff --git a/res/values/arrays.xml b/res/values/arrays.xml
index a92e7f43..ae6dc45b 100644
--- a/res/values/arrays.xml
+++ b/res/values/arrays.xml
@@ -5,4 +5,14 @@
 		<item>Enqueue</item>
 		<item>Last Used Action</item>
 	</string-array>
+	<string-array name="notification_mode_entries">
+		<item>Never Show</item>
+		<item>Show When Playing</item>
+		<item>Always Show</item>
+	</string-array>
+	<string-array name="entry_values">
+		<item>0</item>
+		<item>1</item>
+		<item>2</item>
+	</string-array>
 </resources>
\ No newline at end of file
diff --git a/res/values/strings.xml b/res/values/strings.xml
index b40ce0f4..267f016c 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -35,8 +35,8 @@
 	<string name="remote_player_summary_on">Clicking the notification will open a mini-player dialog</string>
 	<string name="remote_player_summary_off">Clicking the notification will open a the full player activity</string>
 
-	<string name="notify_while_paused_title">Persistent Notification</string>
-	<string name="notify_while_paused_summary">Always display the notification</string>
+	<string name="notification_mode_title">Notification Mode</string>
+	<string name="notification_mode_summary">When to show the notification</string>
 
 	<string name="phone_input_title">Use Phone Dialer Input</string>
 	<string name="phone_input_summary">Enter filter text using buttons 1-9, where 1 matches 1, a, b or c, etc</string>
diff --git a/res/xml/preferences.xml b/res/xml/preferences.xml
index 1024eb02..4e5f1b89 100644
--- a/res/xml/preferences.xml
+++ b/res/xml/preferences.xml
@@ -15,17 +15,19 @@
 			android:summary="@string/headset_pause_summary" />
 	</PreferenceCategory>
 	<PreferenceCategory android:title="@string/pref_notifications">
+		<ListPreference
+			android:key="notification_mode"
+			android:title="@string/notification_mode_title"
+			android:summary="@string/notification_mode_summary"
+			android:entries="@array/notification_mode_entries"
+			android:entryValues="@array/entry_values"
+			android:defaultValue="1" />
 		<CheckBoxPreference
 			android:key="remote_player"
 			android:title="@string/remote_player_title"
 			android:defaultValue="true"
 			android:summaryOn="@string/remote_player_summary_on"
 			android:summaryOff="@string/remote_player_summary_off" />
-		<CheckBoxPreference
-			android:key="notify_while_paused"
-			android:title="@string/notify_while_paused_title"
-			android:defaultValue="true"
-			android:summary="@string/notify_while_paused_summary" />
 	</PreferenceCategory>
 	<PreferenceCategory android:title="@string/pref_song_selector">
 		<CheckBoxPreference
diff --git a/src/org/kreed/vanilla/PlaybackService.java b/src/org/kreed/vanilla/PlaybackService.java
index d6884391..a9deb95f 100644
--- a/src/org/kreed/vanilla/PlaybackService.java
+++ b/src/org/kreed/vanilla/PlaybackService.java
@@ -64,6 +64,10 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On
 	public static final int FLAG_NO_MEDIA = 0x2;
 	public static final int FLAG_PLAYING = 0x1;
 
+	public static final int NEVER = 0;
+	public static final int WHEN_PLAYING = 1;
+	public static final int ALWAYS = 2;
+
 	private int mPendingGo = -1;
 
 	public IPlaybackService.Stub mBinder = new IPlaybackService.Stub() {
@@ -122,6 +126,7 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On
 			}
 		}
 	};
+
 	@Override
 	public IBinder onBind(Intent intent)
 	{
@@ -224,6 +229,9 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On
 				Log.w("VanillaMusic", e);
 			}
 		}
+
+		if (cancelNotification && mNotificationManager != null)
+			mNotificationManager.cancel(NOTIFICATION_ID);
 	}
 
 	private BroadcastReceiver mReceiver = new BroadcastReceiver() {
@@ -272,8 +280,8 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On
 
 	private boolean mHeadsetPause;
 	private boolean mHeadsetOnly;
-	private boolean mNotifyWhilePaused;
 	private boolean mScrobble;
+	private int mNotificationMode;
 
 	private Handler mHandler;
 	private MediaPlayer mMediaPlayer;
@@ -362,7 +370,7 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On
 		mSettings.registerOnSharedPreferenceChangeListener(this);
 		mHeadsetPause = mSettings.getBoolean("headset_pause", true);
 		mHeadsetOnly = mSettings.getBoolean("headset_only", false);
-		mNotifyWhilePaused = mSettings.getBoolean("notify_while_paused", true);
+		mNotificationMode = Integer.parseInt(mSettings.getString("notification_mode", "1"));
 		mScrobble = mSettings.getBoolean("scrobble", false);
 
 		PowerManager powerManager = (PowerManager)getSystemService(POWER_SERVICE);
@@ -397,12 +405,11 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On
 			if (mHeadsetOnly && isSpeakerOn())
 				unsetFlag(FLAG_PLAYING);
 		} else if ("remote_player".equals(key)) {
+			// the preference is loaded in SongNotification class
 			updateNotification(getSong(0));
-		} else if ("notify_while_paused".equals(key)){
-			mNotifyWhilePaused = mSettings.getBoolean(key, true);
+		} else if ("notification_mode".equals(key)){
+			mNotificationMode = Integer.parseInt(mSettings.getString("notification_mode", "1"));
 			updateNotification(getSong(0));
-			if (!mNotifyWhilePaused && (mState & FLAG_PLAYING) == 0)
-				stopForegroundCompat(true);
 		} else if ("scrobble".equals(key)) {
 			mScrobble = mSettings.getBoolean("scrobble", false);
 		}
@@ -454,21 +461,19 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On
 		Song lastBroadcast = mLastSongBroadcast;
 		broadcastChange(oldState, state, song);
 
-		boolean cancelNotification;
 		if (state != oldState || song != lastBroadcast)
-			cancelNotification = updateNotification(song);
-		else
-			cancelNotification = false;
+			updateNotification(song);
 
 		if ((state & FLAG_PLAYING) != 0 && (oldState & FLAG_PLAYING) == 0) {
-			startForegroundCompat(NOTIFICATION_ID, mNotification);
+			if (mNotificationMode != NEVER)
+				startForegroundCompat(NOTIFICATION_ID, mNotification);
 			if (mMediaPlayerInitialized) {
 				synchronized (mMediaPlayer) {
 					mMediaPlayer.start();
 				}
 			}
 		} else if ((state & FLAG_PLAYING) == 0 && (oldState & FLAG_PLAYING) != 0) {
-			stopForegroundCompat(cancelNotification);
+			stopForegroundCompat(false);
 			if (mMediaPlayerInitialized) {
 				synchronized (mMediaPlayer) {
 					mMediaPlayer.pause();
@@ -481,17 +486,15 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On
 		return true;
 	}
 
-	private boolean updateNotification(Song song)
+	private void updateNotification(Song song)
 	{
-		if (song == null || !mNotifyWhilePaused && (mState & FLAG_PLAYING) == 0) {
-			if (mNotificationManager != null)
-				mNotificationManager.cancel(NOTIFICATION_ID);
-			return true;
+		boolean shouldNotify = mNotificationMode == ALWAYS || mNotificationMode == WHEN_PLAYING && (mState & FLAG_PLAYING) != 0;
+		if (song != null && shouldNotify) {
+			mNotification = new SongNotification(song, (mState & FLAG_PLAYING) != 0);
+			mNotificationManager.notify(NOTIFICATION_ID, mNotification);
+		} else {
+			stopForegroundCompat(true);
 		}
-
-		mNotification = new SongNotification(song, (mState & FLAG_PLAYING) != 0);
-		mNotificationManager.notify(NOTIFICATION_ID, mNotification);
-		return false;
 	}
 
 	private boolean isSpeakerOn()