Fixes #476: Add Batch Mode for Now Playing tab

5 files changed, 65 insertions, 11 deletions

diff --git a/app/src/main/java/github/daneren2005/dsub/adapter/DownloadFileAdapter.java b/app/src/main/java/github/daneren2005/dsub/adapter/DownloadFileAdapter.java
index d4613994..5b3dc289 100644
--- a/app/src/main/java/github/daneren2005/dsub/adapter/DownloadFileAdapter.java
+++ b/app/src/main/java/github/daneren2005/dsub/adapter/DownloadFileAdapter.java
@@ -16,13 +16,18 @@
 package github.daneren2005.dsub.adapter;
 
 import android.content.Context;
+import android.view.Menu;
+import android.view.MenuInflater;
+import android.view.MenuItem;
 import android.view.View;
 import android.view.ViewGroup;
 import android.widget.ArrayAdapter;
 
 import java.util.List;
 
+import github.daneren2005.dsub.R;
 import github.daneren2005.dsub.service.DownloadFile;
+import github.daneren2005.dsub.util.Util;
 import github.daneren2005.dsub.view.FastScroller;
 import github.daneren2005.dsub.view.SongView;
 import github.daneren2005.dsub.view.UpdateView;
@@ -33,6 +38,7 @@ public class DownloadFileAdapter extends SectionAdapter<DownloadFile> implements
 	public DownloadFileAdapter(Context context, List<DownloadFile> entries, OnItemClickedListener onItemClickedListener) {
 		super(context, entries);
 		this.onItemClickedListener = onItemClickedListener;
+		this.checkable = true;
 	}
 
 	@Override
@@ -43,7 +49,7 @@ public class DownloadFileAdapter extends SectionAdapter<DownloadFile> implements
 	@Override
 	public void onBindViewHolder(UpdateView.UpdateViewHolder holder, DownloadFile item, int viewType) {
 		SongView songView = (SongView) holder.getUpdateView();
-		songView.setObject(item.getSong(), false);
+		songView.setObject(item.getSong(), Util.isBatchMode(context));
 		songView.setDownloadFile(item);
 	}
 
@@ -56,4 +62,21 @@ public class DownloadFileAdapter extends SectionAdapter<DownloadFile> implements
 	public String getTextToShowInBubble(int position) {
 		return null;
 	}
+
+	@Override
+	public void onCreateActionModeMenu(Menu menu, MenuInflater menuInflater) {
+		if(Util.isOffline(context)) {
+			menuInflater.inflate(R.menu.multiselect_nowplaying_offline, menu);
+		} else {
+			menuInflater.inflate(R.menu.multiselect_nowplaying, menu);
+		}
+
+		if(!selected.isEmpty()) {
+			MenuItem starItem = menu.findItem(R.id.menu_star);
+			if(starItem != null) {
+				boolean isStarred = selected.get(0).getSong().isStarred();
+				starItem.setTitle(isStarred ? R.string.common_unstar : R.string.common_star);
+			}
+		}
+	}
 }
diff --git a/app/src/main/java/github/daneren2005/dsub/fragments/NowPlayingFragment.java b/app/src/main/java/github/daneren2005/dsub/fragments/NowPlayingFragment.java
index f18b22a0..5de9c595 100644
--- a/app/src/main/java/github/daneren2005/dsub/fragments/NowPlayingFragment.java
+++ b/app/src/main/java/github/daneren2005/dsub/fragments/NowPlayingFragment.java
@@ -457,6 +457,10 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
 				mediaRouteButton.setRouteSelector(downloadService.getRemoteSelector());
 			}
 		}
+
+		if(Util.getPreferences(context).getBoolean(Constants.PREFERENCES_KEY_BATCH_MODE, false)) {
+			menu.findItem(R.id.menu_batch_mode).setChecked(true);
+		}
 	}
 
 	@Override
@@ -474,7 +478,7 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
 			menuInflater.inflate(R.menu.nowplaying_context_offline, menu);
 		} else {
 			menuInflater.inflate(R.menu.nowplaying_context, menu);
-			menu.findItem(R.id.menu_star).setTitle(downloadFile.getSong().isStarred() ? R.string.common_unstar : R.string.common_star);
+			menu.findItem(R.id.song_menu_star).setTitle(downloadFile.getSong().isStarred() ? R.string.common_unstar : R.string.common_star);
 		}
 
 		if (downloadFile.getSong().getParent() == null) {
@@ -620,9 +624,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
 				}
 				createNewPlaylist(entries, true);
 				return true;
-			case R.id.menu_star:
-				UpdateHelper.toggleStarred(context, song.getSong());
-				return true;
 			case R.id.menu_rate:
 				UpdateHelper.setRating(context, song.getSong());
 				return true;
@@ -634,11 +635,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
 					startTimer();
 				}
 				return true;
-			case R.id.menu_add_playlist:
-				songs = new ArrayList<Entry>(1);
-				songs.add(song.getSong());
-				addToPlaylist(songs);
-				return true;
 			case R.id.menu_info:
 				displaySongInfo(song.getSong());
 				return true;
@@ -663,7 +659,18 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
 				// Any failed condition will get here
 				Util.toast(context, "Failed to start equalizer.  Try restarting.");
 				return true;
-			} default:
+			}case R.id.menu_batch_mode:
+				if(Util.isBatchMode(context)) {
+					Util.setBatchMode(context, false);
+					songListAdapter.notifyDataSetChanged();
+				} else {
+					Util.setBatchMode(context, true);
+					songListAdapter.notifyDataSetChanged();
+				}
+				context.supportInvalidateOptionsMenu();
+
+				return true;
+			default:
 				return false;
 		}
 	}
@@ -1369,4 +1376,18 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
 				break;
 		}
 	}
+
+	@Override
+	protected List<Entry> getSelectedEntries() {
+		List<DownloadFile> selected = getCurrentAdapter().getSelected();
+		List<Entry> entries = new ArrayList<>();
+
+		for(DownloadFile downloadFile: selected) {
+			if(downloadFile.getSong() != null) {
+				entries.add(downloadFile.getSong());
+			}
+		}
+
+		return entries;
+	}
 }
diff --git a/app/src/main/java/github/daneren2005/dsub/fragments/SubsonicFragment.java b/app/src/main/java/github/daneren2005/dsub/fragments/SubsonicFragment.java
index bc2418eb..0cb433c2 100644
--- a/app/src/main/java/github/daneren2005/dsub/fragments/SubsonicFragment.java
+++ b/app/src/main/java/github/daneren2005/dsub/fragments/SubsonicFragment.java
@@ -326,6 +326,9 @@ public class SubsonicFragment extends Fragment implements SwipeRefreshLayout.OnR
 	public boolean onContextItemSelected(MenuItem menuItem, Object selectedItem) {
 		Artist artist = selectedItem instanceof Artist ? (Artist) selectedItem : null;
 		Entry entry = selectedItem instanceof Entry ? (Entry) selectedItem : null;
+		if(selectedItem instanceof DownloadFile) {
+			entry = ((DownloadFile) selectedItem).getSong();
+		}
 		List<Entry> songs = new ArrayList<Entry>(1);
 		songs.add(entry);
 
diff --git a/app/src/main/java/github/daneren2005/dsub/util/Constants.java b/app/src/main/java/github/daneren2005/dsub/util/Constants.java
index bfb85492..00682947 100644
--- a/app/src/main/java/github/daneren2005/dsub/util/Constants.java
+++ b/app/src/main/java/github/daneren2005/dsub/util/Constants.java
@@ -169,6 +169,7 @@ public final class Constants {
 	public static final String PREFERENCES_KEY_COLOR_ACTION_BAR = "colorActionBar";
 	public static final String PREFERENCES_KEY_SHUFFLE_BY_ALBUM = "shuffleByAlbum";
 	public static final String PREFERENCES_KEY_RESUME_PLAY_QUEUE_NEVER = "neverResumePlayQueue";
+	public static final String PREFERENCES_KEY_BATCH_MODE = "batchMode";
 	
 	public static final String OFFLINE_SCROBBLE_COUNT = "scrobbleCount";
 	public static final String OFFLINE_SCROBBLE_ID = "scrobbleID";
diff --git a/app/src/main/java/github/daneren2005/dsub/util/Util.java b/app/src/main/java/github/daneren2005/dsub/util/Util.java
index 6975b1bf..98bbe1df 100644
--- a/app/src/main/java/github/daneren2005/dsub/util/Util.java
+++ b/app/src/main/java/github/daneren2005/dsub/util/Util.java
@@ -377,6 +377,12 @@ public final class Util {
         int cacheSize = Integer.parseInt(prefs.getString(Constants.PREFERENCES_KEY_CACHE_SIZE, "-1"));
         return cacheSize == -1 ? Integer.MAX_VALUE : cacheSize;
     }
+	public static boolean isBatchMode(Context context) {
+		return Util.getPreferences(context).getBoolean(Constants.PREFERENCES_KEY_BATCH_MODE, false);
+	}
+	public static void setBatchMode(Context context, boolean batchMode) {
+		Util.getPreferences(context).edit().putBoolean(Constants.PREFERENCES_KEY_BATCH_MODE, batchMode).commit();
+	}
 
     public static String getRestUrl(Context context, String method) {
         return getRestUrl(context, method, true);