From b6e9bef76a72b8033a4f4658d1d47c55293dd5ab Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Sat, 12 Sep 2015 15:24:33 -0700
Subject: #536 Auto skip non-downloaded songs when Wifi mobile

---
 .../daneren2005/dsub/service/DownloadService.java  | 38 +++++++++++++++++++---
 .../java/github/daneren2005/dsub/util/Util.java    |  3 ++
 2 files changed, 37 insertions(+), 4 deletions(-)

diff --git a/app/src/main/java/github/daneren2005/dsub/service/DownloadService.java b/app/src/main/java/github/daneren2005/dsub/service/DownloadService.java
index 0db091ad..5673c427 100644
--- a/app/src/main/java/github/daneren2005/dsub/service/DownloadService.java
+++ b/app/src/main/java/github/daneren2005/dsub/service/DownloadService.java
@@ -841,7 +841,8 @@ public class DownloadService extends Service {
 	private int getNextPlayingIndex() {
 		int index = getCurrentPlayingIndex();
 		if (index != -1) {
-			switch (getRepeatMode()) {
+			RepeatMode repeatMode = getRepeatMode();
+			switch (repeatMode) {
 				case OFF:
 					index = index + 1;
 					break;
@@ -853,7 +854,32 @@ public class DownloadService extends Service {
 				default:
 					break;
 			}
+
+			index = checkNextIndexValid(index, repeatMode);
+		}
+		return index;
+	}
+	private int checkNextIndexValid(int index, RepeatMode repeatMode) {
+		int size = size();
+		if(index < size && index != -1) {
+			if(!Util.isAllowedToDownload(this)){
+				DownloadFile next = downloadList.get(index);
+				while(!next.isCompleteFileAvailable()) {
+					index++;
+
+					if (index >= size) {
+						if(repeatMode == RepeatMode.ALL) {
+							index = 0;
+						} else {
+							return -1;
+						}
+					}
+
+					next = downloadList.get(index);
+				}
+			}
 		}
+
 		return index;
 	}
 
@@ -1588,10 +1614,14 @@ public class DownloadService extends Service {
 	}
 	private synchronized void bufferAndPlay(int position, boolean start) {
 		if(!currentPlaying.isCompleteFileAvailable()) {
-			reset();
+			if(Util.isAllowedToDownload(this)) {
+				reset();
 
-			bufferTask = new BufferTask(currentPlaying, position, start);
-			bufferTask.execute();
+				bufferTask = new BufferTask(currentPlaying, position, start);
+				bufferTask.execute();
+			} else {
+				next(false, start);
+			}
 		} else {
 			doPlay(currentPlaying, position, start);
 		}
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 24895226..99926716 100644
--- a/app/src/main/java/github/daneren2005/dsub/util/Util.java
+++ b/app/src/main/java/github/daneren2005/dsub/util/Util.java
@@ -1104,6 +1104,9 @@ public final class Util {
         return Environment.MEDIA_MOUNTED.equals(Environment.getExternalStorageState());
     }
 
+	public static boolean isAllowedToDownload(Context context) {
+		return !isWifiRequiredForDownload(context) || isWifiConnected(context);
+	}
     public static boolean isWifiRequiredForDownload(Context context) {
         SharedPreferences prefs = getPreferences(context);
         return prefs.getBoolean(Constants.PREFERENCES_KEY_WIFI_REQUIRED_FOR_DOWNLOAD, false);
-- 
cgit v1.2.3