From ff5e91853f50349706943bf4f80431bd1d0cab37 Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Mon, 13 Oct 2014 14:52:12 -0700
Subject: #386 Start of adding ability to toggle offline from Tasker

---
 res/layout/edit_play_action.xml                       | 19 +++++++++++++++++++
 res/values/arrays.xml                                 |  6 ++++++
 res/values/strings.xml                                |  2 ++
 .../dsub/activity/EditPlayActionActivity.java         | 18 +++++++++++++-----
 4 files changed, 40 insertions(+), 5 deletions(-)

diff --git a/res/layout/edit_play_action.xml b/res/layout/edit_play_action.xml
index 6eb3b651..1043e2e2 100644
--- a/res/layout/edit_play_action.xml
+++ b/res/layout/edit_play_action.xml
@@ -23,4 +23,23 @@
 			android:longClickable="true"
 			/>
 	</LinearLayout>
+
+	<LinearLayout
+		android:orientation="horizontal"
+		android:layout_width="fill_parent"
+		android:layout_height="wrap_content">
+
+		<TextView
+			android:id="@+id/edit_offline_label"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:layout_marginLeft="4dp"
+			android:textSize="20dp"
+			android:text="@string/tasker.edit_server_offline"/>
+
+		<Spinner
+			android:id="@+id/edit_offline_spinner"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"/>
+	</LinearLayout>
 </LinearLayout>
\ No newline at end of file
diff --git a/res/values/arrays.xml b/res/values/arrays.xml
index 3e85c982..19758562 100644
--- a/res/values/arrays.xml
+++ b/res/values/arrays.xml
@@ -235,4 +235,10 @@
 		<item>@string/settings.replay_gain_type.album</item>
 		<item>@string/settings.replay_gain_type.track</item>
 	</string-array>
+	
+	<string-array name="editServerOptions">
+		<item>@string/tasker.edit_do_nothing</item>
+		<item>@string/main.online</item>
+		<item>@string/main.offline</item>
+	</string-array>
 </resources>
diff --git a/res/values/strings.xml b/res/values/strings.xml
index 0fdb2361..15dc020d 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -559,6 +559,8 @@
 	<string name="tasker.start_playing_shuffled">Start playing in Shuffle Mode</string>
 	<string name="tasker.start_playing_title">Tasker -> Start DSub</string>
 	<string name="tasker.edit_shuffle_mode">Start in shuffle mode: </string>
+	<string name="tasker.edit_server_offline">Toggle offline: </string>
+	<string name="tasker.edit_do_nothing">Do Nothing</string>
 
     <plurals name="select_album_n_songs">
         <item quantity="zero">No songs</item>
diff --git a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
index b7f9de94..c59b679f 100644
--- a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
+++ b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
@@ -22,13 +22,16 @@ import android.support.v4.widget.DrawerLayout;
 import android.view.Menu;
 import android.view.MenuInflater;
 import android.view.MenuItem;
+import android.widget.ArrayAdapter;
 import android.widget.CheckBox;
+import android.widget.Spinner;
 
 import github.daneren2005.dsub.R;
 import github.daneren2005.dsub.util.Constants;
 
 public class EditPlayActionActivity extends SubsonicActivity {
-	private CheckBox checkBox;
+	private CheckBox shuffleCheckbox;
+	private Spinner offlineSpinner;
 
 	@Override
 	public void onCreate(Bundle savedInstanceState) {
@@ -36,11 +39,16 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		setTitle(R.string.tasker_start_playing_title);
 		setContentView(R.layout.edit_play_action);
 
-		checkBox = (CheckBox) findViewById(R.id.edit_shuffle_checkbox);
+		shuffleCheckbox = (CheckBox) findViewById(R.id.edit_shuffle_checkbox);
 		if(getIntent().getBundleExtra(Constants.TASKER_EXTRA_BUNDLE) != null && getIntent().getBundleExtra(Constants.TASKER_EXTRA_BUNDLE).getBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE)) {
-			checkBox.setChecked(true);
+			shuffleCheckbox.setChecked(true);
 		}
 
+		offlineSpinner = (Spinner) findViewById(R.id.edit_offline_spinner);
+		ArrayAdapter<CharSequence> offlineAdapter = ArrayAdapter.createFromResource(this, R.array.editServerOptions, android.R.layout.simple_spinner_item);
+		offlineAdapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
+		offlineSpinner.setAdapter(offlineAdapter);
+
 		drawer.setDrawerLockMode(DrawerLayout.LOCK_MODE_LOCKED_CLOSED);
 	}
 
@@ -70,11 +78,11 @@ public class EditPlayActionActivity extends SubsonicActivity {
 	private void accept() {
 		Intent intent = new Intent();
 
-		String blurb = getResources().getString(checkBox.isChecked() ? R.string.tasker_start_playing_shuffled : R.string.tasker_start_playing);
+		String blurb = getResources().getString(shuffleCheckbox.isChecked() ? R.string.tasker_start_playing_shuffled : R.string.tasker_start_playing);
 		intent.putExtra("com.twofortyfouram.locale.intent.extra.BLURB", blurb);
 
 		Bundle data = new Bundle();
-		data.putBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE, checkBox.isChecked());
+		data.putBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE, shuffleCheckbox.isChecked());
 		intent.putExtra(Constants.TASKER_EXTRA_BUNDLE, data);
 
 		setResult(Activity.RESULT_OK, intent);
-- 
cgit v1.2.3


From 30847da1fd577f08bd02803b788c908df6f8c791 Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Thu, 16 Oct 2014 19:10:04 -0700
Subject: #386 Add shuffle genre to Tasker options

---
 res/layout/edit_play_action.xml                    | 20 ++++++-
 res/values/strings.xml                             |  1 +
 .../dsub/activity/EditPlayActionActivity.java      | 70 ++++++++++++++++++++++
 3 files changed, 89 insertions(+), 2 deletions(-)

diff --git a/res/layout/edit_play_action.xml b/res/layout/edit_play_action.xml
index 1043e2e2..ef5408ee 100644
--- a/res/layout/edit_play_action.xml
+++ b/res/layout/edit_play_action.xml
@@ -10,7 +10,6 @@
 		android:layout_height="wrap_content">
 
 		<TextView
-			android:id="@+id/edit_shuffle"
 			android:layout_width="wrap_content"
 			android:layout_height="wrap_content"
 			android:layout_marginLeft="4dp"
@@ -30,7 +29,24 @@
 		android:layout_height="wrap_content">
 
 		<TextView
-			android:id="@+id/edit_offline_label"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:layout_marginLeft="4dp"
+			android:textSize="20dp"
+			android:text="@string/tasker.edit_shuffle_genre"/>
+
+		<Button
+			android:id="@+id/edit_genre_spinner"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"/>
+	</LinearLayout>
+
+	<LinearLayout
+		android:orientation="horizontal"
+		android:layout_width="fill_parent"
+		android:layout_height="wrap_content">
+
+		<TextView
 			android:layout_width="wrap_content"
 			android:layout_height="wrap_content"
 			android:layout_marginLeft="4dp"
diff --git a/res/values/strings.xml b/res/values/strings.xml
index 15dc020d..8d7cc873 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -559,6 +559,7 @@
 	<string name="tasker.start_playing_shuffled">Start playing in Shuffle Mode</string>
 	<string name="tasker.start_playing_title">Tasker -> Start DSub</string>
 	<string name="tasker.edit_shuffle_mode">Start in shuffle mode: </string>
+	<string name="tasker.edit_shuffle_genre">Shuffle from genre:</string>
 	<string name="tasker.edit_server_offline">Toggle offline: </string>
 	<string name="tasker.edit_do_nothing">Do Nothing</string>
 
diff --git a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
index c59b679f..ef960822 100644
--- a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
+++ b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
@@ -16,21 +16,37 @@
 package github.daneren2005.dsub.activity;
 
 import android.app.Activity;
+import android.app.AlertDialog;
+import android.content.Context;
+import android.content.DialogInterface;
 import android.content.Intent;
 import android.os.Bundle;
 import android.support.v4.widget.DrawerLayout;
 import android.view.Menu;
 import android.view.MenuInflater;
 import android.view.MenuItem;
+import android.view.View;
 import android.widget.ArrayAdapter;
+import android.widget.Button;
 import android.widget.CheckBox;
 import android.widget.Spinner;
 
+import java.util.ArrayList;
+import java.util.List;
+
 import github.daneren2005.dsub.R;
+import github.daneren2005.dsub.domain.Genre;
+import github.daneren2005.dsub.service.MusicService;
+import github.daneren2005.dsub.service.MusicServiceFactory;
+import github.daneren2005.dsub.service.OfflineException;
+import github.daneren2005.dsub.service.ServerTooOldException;
 import github.daneren2005.dsub.util.Constants;
+import github.daneren2005.dsub.util.LoadingTask;
+import github.daneren2005.dsub.util.Util;
 
 public class EditPlayActionActivity extends SubsonicActivity {
 	private CheckBox shuffleCheckbox;
+	private Button genreButton;
 	private Spinner offlineSpinner;
 
 	@Override
@@ -38,12 +54,66 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		super.onCreate(savedInstanceState);
 		setTitle(R.string.tasker_start_playing_title);
 		setContentView(R.layout.edit_play_action);
+		final Activity context = this;
 
 		shuffleCheckbox = (CheckBox) findViewById(R.id.edit_shuffle_checkbox);
 		if(getIntent().getBundleExtra(Constants.TASKER_EXTRA_BUNDLE) != null && getIntent().getBundleExtra(Constants.TASKER_EXTRA_BUNDLE).getBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE)) {
 			shuffleCheckbox.setChecked(true);
 		}
 
+		genreButton = (Button) findViewById(R.id.edit_genre_spinner);
+		genreButton.setOnClickListener(new View.OnClickListener() {
+			public void onClick(View v) {
+				new LoadingTask<List<Genre>>(context, true) {
+					@Override
+					protected List<Genre> doInBackground() throws Throwable {
+						MusicService musicService = MusicServiceFactory.getMusicService(context);
+						return musicService.getGenres(false, context, this);
+					}
+
+					@Override
+					protected void done(final List<Genre> genres) {
+						List<String> names = new ArrayList<String>();
+						String blank = context.getResources().getString(R.string.select_genre_blank);
+						String doNothing = context.getResources().getString(R.string.tasker_edit_do_nothing);
+						names.add(doNothing);
+						names.add(blank);
+						for(Genre genre: genres) {
+							names.add(genre.getName());
+						}
+						final List<String> finalNames = names;
+
+						AlertDialog.Builder builder = new AlertDialog.Builder(context);
+						builder.setTitle(R.string.shuffle_pick_genre)
+								.setItems(names.toArray(new CharSequence[names.size()]), new DialogInterface.OnClickListener() {
+									public void onClick(DialogInterface dialog, int which) {
+										if(which == 1) {
+											genreButton.setText("");
+										} else {
+											genreButton.setText(finalNames.get(which));
+										}
+									}
+								});
+						AlertDialog dialog = builder.create();
+						dialog.show();
+					}
+
+					@Override
+					protected void error(Throwable error) {
+						String msg;
+						if (error instanceof OfflineException || error instanceof ServerTooOldException) {
+							msg = getErrorMessage(error);
+						} else {
+							msg = context.getResources().getString(R.string.playlist_error) + " " + getErrorMessage(error);
+						}
+
+						Util.toast(context, msg, false);
+					}
+				}.execute();
+			}
+		});
+		genreButton.setText("Do Nothing");
+
 		offlineSpinner = (Spinner) findViewById(R.id.edit_offline_spinner);
 		ArrayAdapter<CharSequence> offlineAdapter = ArrayAdapter.createFromResource(this, R.array.editServerOptions, android.R.layout.simple_spinner_item);
 		offlineAdapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
-- 
cgit v1.2.3


From d78618c064085a852a3ebba0064eeb22c83dfc97 Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Thu, 16 Oct 2014 19:26:29 -0700
Subject:  #386 Add shuffle year to Tasker options

---
 res/layout/edit_play_action.xml                    | 57 +++++++++++++++++++++-
 res/values/strings.xml                             |  2 +
 .../dsub/activity/EditPlayActionActivity.java      | 10 ++++
 3 files changed, 68 insertions(+), 1 deletion(-)

diff --git a/res/layout/edit_play_action.xml b/res/layout/edit_play_action.xml
index ef5408ee..346f34c3 100644
--- a/res/layout/edit_play_action.xml
+++ b/res/layout/edit_play_action.xml
@@ -23,6 +23,60 @@
 			/>
 	</LinearLayout>
 
+	<LinearLayout
+		android:orientation="horizontal"
+		android:layout_width="fill_parent"
+		android:layout_height="wrap_content">
+
+		<TextView
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:layout_marginLeft="4dp"
+			android:textSize="20dp"
+			android:text="@string/tasker.edit_shuffle_start_year" />
+
+		<CheckBox
+			android:id="@+id/edit_start_year_checkbox"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:longClickable="true"/>
+
+		<EditText
+			android:id="@+id/edit_start_year"
+			android:layout_width="match_parent"
+			android:layout_height="wrap_content"
+			android:inputType="number"
+			android:hint="@string/shuffle.startYear"
+			android:enabled="false"/>
+	</LinearLayout>
+
+	<LinearLayout
+		android:orientation="horizontal"
+		android:layout_width="fill_parent"
+		android:layout_height="wrap_content">
+
+		<TextView
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:layout_marginLeft="4dp"
+			android:textSize="20dp"
+			android:text="@string/tasker.edit_shuffle_end_year" />
+
+		<CheckBox
+			android:id="@+id/edit_end_year_checkbox"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:longClickable="true"/>
+
+		<EditText
+			android:id="@+id/edit_end_year"
+			android:layout_width="match_parent"
+			android:layout_height="wrap_content"
+			android:inputType="number"
+			android:hint="@string/shuffle.endYear"
+			android:enabled="false"/>
+	</LinearLayout>
+
 	<LinearLayout
 		android:orientation="horizontal"
 		android:layout_width="fill_parent"
@@ -38,7 +92,8 @@
 		<Button
 			android:id="@+id/edit_genre_spinner"
 			android:layout_width="wrap_content"
-			android:layout_height="wrap_content"/>
+			android:layout_height="wrap_content"
+			style="?android:attr/spinnerStyle"/>
 	</LinearLayout>
 
 	<LinearLayout
diff --git a/res/values/strings.xml b/res/values/strings.xml
index 8d7cc873..33a40981 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -559,6 +559,8 @@
 	<string name="tasker.start_playing_shuffled">Start playing in Shuffle Mode</string>
 	<string name="tasker.start_playing_title">Tasker -> Start DSub</string>
 	<string name="tasker.edit_shuffle_mode">Start in shuffle mode: </string>
+	<string name="tasker.edit_shuffle_start_year">Shuffle start year:</string>
+	<string name="tasker.edit_shuffle_end_year">Shuffle end year:</string>
 	<string name="tasker.edit_shuffle_genre">Shuffle from genre:</string>
 	<string name="tasker.edit_server_offline">Toggle offline: </string>
 	<string name="tasker.edit_do_nothing">Do Nothing</string>
diff --git a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
index ef960822..2ec0b6e8 100644
--- a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
+++ b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
@@ -29,6 +29,7 @@ import android.view.View;
 import android.widget.ArrayAdapter;
 import android.widget.Button;
 import android.widget.CheckBox;
+import android.widget.EditText;
 import android.widget.Spinner;
 
 import java.util.ArrayList;
@@ -46,6 +47,10 @@ import github.daneren2005.dsub.util.Util;
 
 public class EditPlayActionActivity extends SubsonicActivity {
 	private CheckBox shuffleCheckbox;
+	private CheckBox startYearCheckbox;
+	private EditText startYearBox;
+	private CheckBox endYearCheckbox;
+	private EditText endYearBox;
 	private Button genreButton;
 	private Spinner offlineSpinner;
 
@@ -61,6 +66,11 @@ public class EditPlayActionActivity extends SubsonicActivity {
 			shuffleCheckbox.setChecked(true);
 		}
 
+		startYearCheckbox = (CheckBox) findViewById(R.id.edit_start_year_checkbox);
+		startYearBox = (EditText) findViewById(R.id.edit_start_year);
+		endYearCheckbox = (CheckBox) findViewById(R.id.edit_end_year_checkbox);
+		endYearBox = (EditText) findViewById(R.id.edit_end_year);
+
 		genreButton = (Button) findViewById(R.id.edit_genre_spinner);
 		genreButton.setOnClickListener(new View.OnClickListener() {
 			public void onClick(View v) {
-- 
cgit v1.2.3


From 6802f91bfea4d378c8e27690430a86e1d7efb503 Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Fri, 17 Oct 2014 14:28:11 -0700
Subject: #386 Add new Tasker's options restore logic

---
 .../dsub/activity/EditPlayActionActivity.java      | 60 ++++++++++++++++++++--
 1 file changed, 55 insertions(+), 5 deletions(-)

diff --git a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
index 2ec0b6e8..ab17c087 100644
--- a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
+++ b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
@@ -53,6 +53,8 @@ public class EditPlayActionActivity extends SubsonicActivity {
 	private EditText endYearBox;
 	private Button genreButton;
 	private Spinner offlineSpinner;
+	
+	private String doNothing;
 
 	@Override
 	public void onCreate(Bundle savedInstanceState) {
@@ -60,16 +62,36 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		setTitle(R.string.tasker_start_playing_title);
 		setContentView(R.layout.edit_play_action);
 		final Activity context = this;
+		doNothing = context.getResources().getString(R.string.tasker_edit_do_nothing);
 
 		shuffleCheckbox = (CheckBox) findViewById(R.id.edit_shuffle_checkbox);
-		if(getIntent().getBundleExtra(Constants.TASKER_EXTRA_BUNDLE) != null && getIntent().getBundleExtra(Constants.TASKER_EXTRA_BUNDLE).getBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE)) {
-			shuffleCheckbox.setChecked(true);
-		}
+		shuffleCheckbox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
+			@Override
+			public void onCheckedChange(CompoundView view, boolean isChecked) {
+				startYearCheckbox.setEnabled(isChecked);
+				endYearCheckbox.setEnabled(isChecked);
+				genreButton.setEnabled(isChecked);
+			}
+		});
 
 		startYearCheckbox = (CheckBox) findViewById(R.id.edit_start_year_checkbox);
 		startYearBox = (EditText) findViewById(R.id.edit_start_year);
+		// Disable/enable number box if checked
+		startYearCheckbox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
+			@Override
+			public void onCheckedChange(CompoundView view, boolean isChecked) {
+				startYearBox.setEnabled(isChecked);
+			}
+		});
+		
 		endYearCheckbox = (CheckBox) findViewById(R.id.edit_end_year_checkbox);
 		endYearBox = (EditText) findViewById(R.id.edit_end_year);
+		endYearCheckbox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
+			@Override
+			public void onCheckedChange(CompoundView view, boolean isChecked) {
+				endYearBox.setEnabled(isChecked);
+			}
+		});
 
 		genreButton = (Button) findViewById(R.id.edit_genre_spinner);
 		genreButton.setOnClickListener(new View.OnClickListener() {
@@ -85,7 +107,6 @@ public class EditPlayActionActivity extends SubsonicActivity {
 					protected void done(final List<Genre> genres) {
 						List<String> names = new ArrayList<String>();
 						String blank = context.getResources().getString(R.string.select_genre_blank);
-						String doNothing = context.getResources().getString(R.string.tasker_edit_do_nothing);
 						names.add(doNothing);
 						names.add(blank);
 						for(Genre genre: genres) {
@@ -122,12 +143,41 @@ public class EditPlayActionActivity extends SubsonicActivity {
 				}.execute();
 			}
 		});
-		genreButton.setText("Do Nothing");
+		genreButton.setText(doNothing);
 
 		offlineSpinner = (Spinner) findViewById(R.id.edit_offline_spinner);
 		ArrayAdapter<CharSequence> offlineAdapter = ArrayAdapter.createFromResource(this, R.array.editServerOptions, android.R.layout.simple_spinner_item);
 		offlineAdapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
 		offlineSpinner.setAdapter(offlineAdapter);
+		
+		// Setup default for everything
+		Bundle extras = getIntent().getBundleExtra(Constants.TASKER_EXTRA_BUNDLE);
+		if(extras != null) {
+			if(extras.getBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE)) {
+				shuffleCheckbox.setChecked(true);
+			}
+			
+			String startYear = extras.getString(Constants.PREFERENCES_KEY_START_YEAR, null);
+			if(startYear != null) {
+				startYearCheckbox.setEnabled(true);
+				startYearBox.setText(startYear);
+			}
+			String endYear = extras.getString(Constants.PREFERENCES_KEY_END_YEAR, null);
+			if(endYear != null) {
+				endYearCheckbox.setEnabled(true);
+				endYearBox.setText(endYear);
+			}
+			
+			String genre = extras.getString(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, doNothing);
+			if(genre != null)
+				genreButton.setText(genre);
+			}
+			
+			short offline = extras.getShort(Constants.PREFERENCES_KEY_OFFLINE, 0);
+			if(offline != 0) {
+				offline.setSelection((int) offline);
+			}
+		}
 
 		drawer.setDrawerLockMode(DrawerLayout.LOCK_MODE_LOCKED_CLOSED);
 	}
-- 
cgit v1.2.3


From 1059cdbc0d7e65130efbc49f9fb4f3f90fe84efe Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Fri, 17 Oct 2014 14:50:23 -0700
Subject: #386 Save Tasker options on accept()

---
 .../dsub/activity/EditPlayActionActivity.java      | 26 +++++++++++++++++++---
 1 file changed, 23 insertions(+), 3 deletions(-)

diff --git a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
index ab17c087..609f1939 100644
--- a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
+++ b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
@@ -157,12 +157,12 @@ public class EditPlayActionActivity extends SubsonicActivity {
 				shuffleCheckbox.setChecked(true);
 			}
 			
-			String startYear = extras.getString(Constants.PREFERENCES_KEY_START_YEAR, null);
+			String startYear = extras.getString(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, null);
 			if(startYear != null) {
 				startYearCheckbox.setEnabled(true);
 				startYearBox.setText(startYear);
 			}
-			String endYear = extras.getString(Constants.PREFERENCES_KEY_END_YEAR, null);
+			String endYear = extras.getString(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, null);
 			if(endYear != null) {
 				endYearCheckbox.setEnabled(true);
 				endYearBox.setText(endYear);
@@ -211,8 +211,28 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		String blurb = getResources().getString(shuffleCheckbox.isChecked() ? R.string.tasker_start_playing_shuffled : R.string.tasker_start_playing);
 		intent.putExtra("com.twofortyfouram.locale.intent.extra.BLURB", blurb);
 
+		// Get settings user specified
 		Bundle data = new Bundle();
-		data.putBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE, shuffleCheckbox.isChecked());
+		boolean shuffle = shuffleCheckbox.isChecked();
+		data.putBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE, shuffle);
+		if(shuffle) {
+			if(startYearCheckbox.isChecked()) {
+				data.putString(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, startYearBox.getText());
+			}
+			if(endYearCheckbox.isChecked()) {
+				data.putString(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, endYearBox.getText());
+			}
+			String genre = genreButton.getText();
+			if(!genre.equals(doNothing)) {
+				data.putString(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, genre);
+			}
+		}
+		
+		int offline = offlineSpinner.getSelectedItemPosition();
+		if(offline != 0) {
+			data.putShort(Constants.PREFERENCES_KEY_OFFLINE, (short) offline);
+		}
+		
 		intent.putExtra(Constants.TASKER_EXTRA_BUNDLE, data);
 
 		setResult(Activity.RESULT_OK, intent);
-- 
cgit v1.2.3


From 07b90a9663d4becac54efef2463f87d22e8a57c3 Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Fri, 17 Oct 2014 18:26:26 -0700
Subject: #386 Fix saving options

---
 res/layout/edit_play_action.xml                       |  9 ++++++---
 .../dsub/activity/EditPlayActionActivity.java         | 19 ++++++++++---------
 2 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/res/layout/edit_play_action.xml b/res/layout/edit_play_action.xml
index 346f34c3..a1115da6 100644
--- a/res/layout/edit_play_action.xml
+++ b/res/layout/edit_play_action.xml
@@ -39,7 +39,8 @@
 			android:id="@+id/edit_start_year_checkbox"
 			android:layout_width="wrap_content"
 			android:layout_height="wrap_content"
-			android:longClickable="true"/>
+			android:longClickable="true"
+			android:enabled="false"/>
 
 		<EditText
 			android:id="@+id/edit_start_year"
@@ -66,7 +67,8 @@
 			android:id="@+id/edit_end_year_checkbox"
 			android:layout_width="wrap_content"
 			android:layout_height="wrap_content"
-			android:longClickable="true"/>
+			android:longClickable="true"
+			android:enabled="false"/>
 
 		<EditText
 			android:id="@+id/edit_end_year"
@@ -93,7 +95,8 @@
 			android:id="@+id/edit_genre_spinner"
 			android:layout_width="wrap_content"
 			android:layout_height="wrap_content"
-			style="?android:attr/spinnerStyle"/>
+			style="?android:attr/spinnerStyle"
+			android:enabled="false"/>
 	</LinearLayout>
 
 	<LinearLayout
diff --git a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
index 609f1939..64439289 100644
--- a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
+++ b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
@@ -29,6 +29,7 @@ import android.view.View;
 import android.widget.ArrayAdapter;
 import android.widget.Button;
 import android.widget.CheckBox;
+import android.widget.CompoundButton;
 import android.widget.EditText;
 import android.widget.Spinner;
 
@@ -67,7 +68,7 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		shuffleCheckbox = (CheckBox) findViewById(R.id.edit_shuffle_checkbox);
 		shuffleCheckbox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
 			@Override
-			public void onCheckedChange(CompoundView view, boolean isChecked) {
+			public void onCheckedChanged(CompoundButton view, boolean isChecked) {
 				startYearCheckbox.setEnabled(isChecked);
 				endYearCheckbox.setEnabled(isChecked);
 				genreButton.setEnabled(isChecked);
@@ -79,7 +80,7 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		// Disable/enable number box if checked
 		startYearCheckbox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
 			@Override
-			public void onCheckedChange(CompoundView view, boolean isChecked) {
+			public void onCheckedChanged(CompoundButton view, boolean isChecked) {
 				startYearBox.setEnabled(isChecked);
 			}
 		});
@@ -88,7 +89,7 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		endYearBox = (EditText) findViewById(R.id.edit_end_year);
 		endYearCheckbox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
 			@Override
-			public void onCheckedChange(CompoundView view, boolean isChecked) {
+			public void onCheckedChanged(CompoundButton view, boolean isChecked) {
 				endYearBox.setEnabled(isChecked);
 			}
 		});
@@ -169,13 +170,13 @@ public class EditPlayActionActivity extends SubsonicActivity {
 			}
 			
 			String genre = extras.getString(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, doNothing);
-			if(genre != null)
+			if(genre != null) {
 				genreButton.setText(genre);
 			}
 			
-			short offline = extras.getShort(Constants.PREFERENCES_KEY_OFFLINE, 0);
+			short offline = extras.getShort(Constants.PREFERENCES_KEY_OFFLINE, (short) 0);
 			if(offline != 0) {
-				offline.setSelection((int) offline);
+				offlineSpinner.setSelection((int) offline);
 			}
 		}
 
@@ -217,12 +218,12 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		data.putBoolean(Constants.INTENT_EXTRA_NAME_SHUFFLE, shuffle);
 		if(shuffle) {
 			if(startYearCheckbox.isChecked()) {
-				data.putString(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, startYearBox.getText());
+				data.putString(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, startYearBox.getText().toString());
 			}
 			if(endYearCheckbox.isChecked()) {
-				data.putString(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, endYearBox.getText());
+				data.putString(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, endYearBox.getText().toString());
 			}
-			String genre = genreButton.getText();
+			String genre = genreButton.getText().toString();
 			if(!genre.equals(doNothing)) {
 				data.putString(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, genre);
 			}
-- 
cgit v1.2.3


From b943aabed7828afb92162f52f44788d732d14811 Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Mon, 20 Oct 2014 16:22:46 -0700
Subject: #386 Add logic to actually change online status, shuffle options

---
 .../service/DownloadServiceLifecycleSupport.java   | 25 ++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/src/github/daneren2005/dsub/service/DownloadServiceLifecycleSupport.java b/src/github/daneren2005/dsub/service/DownloadServiceLifecycleSupport.java
index 13c3cea7..32919aff 100644
--- a/src/github/daneren2005/dsub/service/DownloadServiceLifecycleSupport.java
+++ b/src/github/daneren2005/dsub/service/DownloadServiceLifecycleSupport.java
@@ -167,8 +167,33 @@ public class DownloadServiceLifecycleSupport {
 							lock.lock();
 							lock.unlock();
 						}
+						
+						short offline = intent.getShortExtra(Constants.INTENT_EXTRA_OFFLINE, 0);
+						if(offline != 0) {
+							boolean offline = (offline == 2);
+							Util.setOffline(context, offline);
+							downloadService.setOnline(!offline);
+						}
 
 						if(intent.getBooleanExtra(Constants.INTENT_EXTRA_NAME_SHUFFLE, false)) {
+							// Add shuffle parameters
+							SharedPreferences.Editor editor = Util.getPreferences(context).edit();
+							String startYear = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, null);
+							if(startYear != null) {
+								editor.putString(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, startYear);
+							}
+							
+							String endYear = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, null);
+							if(endYear != null) {
+								editor.putString(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, endYear);
+							}
+							
+							String genre = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, null);
+							if(genre != null) {
+								editor.putString(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, genre);
+							}
+							editor.commit();
+							
 							downloadService.setShufflePlayEnabled(true);
 						} else {
 							downloadService.start();
-- 
cgit v1.2.3


From 08c7e436bdd25e72a386b45e2bdbb719ffde8f3c Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Mon, 20 Oct 2014 20:18:55 -0700
Subject:  #386 Finish up tasker additions

---
 .../dsub/activity/EditPlayActionActivity.java      |  6 +++---
 .../dsub/receiver/PlayActionReceiver.java          |  4 ++++
 .../service/DownloadServiceLifecycleSupport.java   | 22 +++++++++++++---------
 3 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
index 64439289..e1f2cad3 100644
--- a/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
+++ b/src/github/daneren2005/dsub/activity/EditPlayActionActivity.java
@@ -174,9 +174,9 @@ public class EditPlayActionActivity extends SubsonicActivity {
 				genreButton.setText(genre);
 			}
 			
-			short offline = extras.getShort(Constants.PREFERENCES_KEY_OFFLINE, (short) 0);
+			int offline = extras.getInt(Constants.PREFERENCES_KEY_OFFLINE, 0);
 			if(offline != 0) {
-				offlineSpinner.setSelection((int) offline);
+				offlineSpinner.setSelection(offline);
 			}
 		}
 
@@ -231,7 +231,7 @@ public class EditPlayActionActivity extends SubsonicActivity {
 		
 		int offline = offlineSpinner.getSelectedItemPosition();
 		if(offline != 0) {
-			data.putShort(Constants.PREFERENCES_KEY_OFFLINE, (short) offline);
+			data.putInt(Constants.PREFERENCES_KEY_OFFLINE, offline);
 		}
 		
 		intent.putExtra(Constants.TASKER_EXTRA_BUNDLE, data);
diff --git a/src/github/daneren2005/dsub/receiver/PlayActionReceiver.java b/src/github/daneren2005/dsub/receiver/PlayActionReceiver.java
index 60814cc6..2c04d829 100644
--- a/src/github/daneren2005/dsub/receiver/PlayActionReceiver.java
+++ b/src/github/daneren2005/dsub/receiver/PlayActionReceiver.java
@@ -36,6 +36,10 @@ public class PlayActionReceiver extends BroadcastReceiver {
 			Intent start = new Intent(context, DownloadService.class);
 			start.setAction(DownloadService.START_PLAY);
 			start.putExtra(Constants.INTENT_EXTRA_NAME_SHUFFLE, startShuffled);
+			start.putExtra(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, data.getString(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR));
+			start.putExtra(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, data.getString(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR));
+			start.putExtra(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, data.getString(Constants.PREFERENCES_KEY_SHUFFLE_GENRE));
+			start.putExtra(Constants.PREFERENCES_KEY_OFFLINE, data.getInt(Constants.PREFERENCES_KEY_OFFLINE));
 			context.startService(start);
 		}
 	}
diff --git a/src/github/daneren2005/dsub/service/DownloadServiceLifecycleSupport.java b/src/github/daneren2005/dsub/service/DownloadServiceLifecycleSupport.java
index 32919aff..0a1e2cdd 100644
--- a/src/github/daneren2005/dsub/service/DownloadServiceLifecycleSupport.java
+++ b/src/github/daneren2005/dsub/service/DownloadServiceLifecycleSupport.java
@@ -168,27 +168,31 @@ public class DownloadServiceLifecycleSupport {
 							lock.unlock();
 						}
 						
-						short offline = intent.getShortExtra(Constants.INTENT_EXTRA_OFFLINE, 0);
-						if(offline != 0) {
-							boolean offline = (offline == 2);
-							Util.setOffline(context, offline);
-							downloadService.setOnline(!offline);
+						int offlinePref = intent.getIntExtra(Constants.PREFERENCES_KEY_OFFLINE, 0);
+						if(offlinePref != 0) {
+							boolean offline = (offlinePref == 2);
+							Util.setOffline(downloadService, offline);
+							if (offline) {
+								downloadService.clearIncomplete();
+							} else {
+								downloadService.checkDownloads();
+							}
 						}
 
 						if(intent.getBooleanExtra(Constants.INTENT_EXTRA_NAME_SHUFFLE, false)) {
 							// Add shuffle parameters
-							SharedPreferences.Editor editor = Util.getPreferences(context).edit();
-							String startYear = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, null);
+							SharedPreferences.Editor editor = Util.getPreferences(downloadService).edit();
+							String startYear = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR);
 							if(startYear != null) {
 								editor.putString(Constants.PREFERENCES_KEY_SHUFFLE_START_YEAR, startYear);
 							}
 							
-							String endYear = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, null);
+							String endYear = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR);
 							if(endYear != null) {
 								editor.putString(Constants.PREFERENCES_KEY_SHUFFLE_END_YEAR, endYear);
 							}
 							
-							String genre = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, null);
+							String genre = intent.getStringExtra(Constants.PREFERENCES_KEY_SHUFFLE_GENRE);
 							if(genre != null) {
 								editor.putString(Constants.PREFERENCES_KEY_SHUFFLE_GENRE, genre);
 							}
-- 
cgit v1.2.3