From a046e9fa0aa0ec0567b02f083642d48482108977 Mon Sep 17 00:00:00 2001
From: DjBushido <bspeice.nc@gmail.com>
Date: Thu, 8 May 2014 21:39:29 -0400
Subject: [PATCH] Refactor download prefs, when to d/l over internet

---
 MinimalBible/AndroidManifest.xml              |  1 +
 .../minimalbible/MinimalBibleConstants.java   | 10 ----
 .../activities/ActivityModules.java           |  2 +-
 .../ActivityDownloaderModule.java             | 19 +++++--
 .../downloader/BookListFragment.java          | 28 +++-------
 .../downloader/DownloadPrefsManager.java      | 51 +++++++++++++++++++
 .../downloader/manager/BookRefreshTask.java   | 50 ++++++++++++------
 7 files changed, 110 insertions(+), 51 deletions(-)
 delete mode 100644 MinimalBible/src/org/bspeice/minimalbible/MinimalBibleConstants.java
 rename MinimalBible/src/org/bspeice/minimalbible/activities/downloader/{manager => }/ActivityDownloaderModule.java (56%)
 create mode 100644 MinimalBible/src/org/bspeice/minimalbible/activities/downloader/DownloadPrefsManager.java

diff --git a/MinimalBible/AndroidManifest.xml b/MinimalBible/AndroidManifest.xml
index 12288b4..cd934cb 100644
--- a/MinimalBible/AndroidManifest.xml
+++ b/MinimalBible/AndroidManifest.xml
@@ -8,6 +8,7 @@
         android:minSdkVersion="8"
         android:targetSdkVersion="19" />
     <uses-permission android:name="android.permission.INTERNET"/>
+    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
 
     <application
         android:allowBackup="true"
diff --git a/MinimalBible/src/org/bspeice/minimalbible/MinimalBibleConstants.java b/MinimalBible/src/org/bspeice/minimalbible/MinimalBibleConstants.java
deleted file mode 100644
index c1cb2d7..0000000
--- a/MinimalBible/src/org/bspeice/minimalbible/MinimalBibleConstants.java
+++ /dev/null
@@ -1,10 +0,0 @@
-package org.bspeice.minimalbible;
-
-public class MinimalBibleConstants {
-	
-	public static final String DOWNLOAD_PREFS_FILE = "DOWNLOADER_PREFERENCES";
-	
-	public static final String KEY_DOWNLOAD_ENABLED = "HAS_ENABLED_DOWNLOAD";
-	public static final String KEY_SHOWED_DOWNLOAD_DIALOG = "SHOWED_DOWNLOAD_DIALOG";
-
-}
diff --git a/MinimalBible/src/org/bspeice/minimalbible/activities/ActivityModules.java b/MinimalBible/src/org/bspeice/minimalbible/activities/ActivityModules.java
index 80e56fa..de3a0d9 100644
--- a/MinimalBible/src/org/bspeice/minimalbible/activities/ActivityModules.java
+++ b/MinimalBible/src/org/bspeice/minimalbible/activities/ActivityModules.java
@@ -1,6 +1,6 @@
 package org.bspeice.minimalbible.activities;
 
-import org.bspeice.minimalbible.activities.downloader.manager.ActivityDownloaderModule;
+import org.bspeice.minimalbible.activities.downloader.ActivityDownloaderModule;
 
 import dagger.Module;
 
diff --git a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/manager/ActivityDownloaderModule.java b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/ActivityDownloaderModule.java
similarity index 56%
rename from MinimalBible/src/org/bspeice/minimalbible/activities/downloader/manager/ActivityDownloaderModule.java
rename to MinimalBible/src/org/bspeice/minimalbible/activities/downloader/ActivityDownloaderModule.java
index 7f420f1..21322a4 100644
--- a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/manager/ActivityDownloaderModule.java
+++ b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/ActivityDownloaderModule.java
@@ -1,6 +1,11 @@
-package org.bspeice.minimalbible.activities.downloader.manager;
+package org.bspeice.minimalbible.activities.downloader;
 
-import org.bspeice.minimalbible.activities.downloader.BookListFragment;
+import android.app.Activity;
+import android.content.Context;
+
+import org.bspeice.minimalbible.MinimalBible;
+import org.bspeice.minimalbible.activities.downloader.manager.BookRefreshTask;
+import org.bspeice.minimalbible.activities.downloader.manager.DownloadManager;
 
 import javax.inject.Singleton;
 
@@ -14,11 +19,14 @@ import de.greenrobot.event.EventBus;
 @Module(
         injects = {
             BookListFragment.class,
-            DownloadManager.class
+            DownloadManager.class,
+            BookRefreshTask.class
         }
 )
 public class ActivityDownloaderModule {
 
+    private final Context ctx = MinimalBible.getAppContext();
+
     /**
      * Provide a Singleton DownloadManager for injection
      * Note that we need to annotate Singleton here, only annotating on the
@@ -34,4 +42,9 @@ public class ActivityDownloaderModule {
     EventBus provideBus() {
         return new EventBus();
     }
+
+    @Provides @Singleton
+    DownloadPrefsManager providePrefsManager() {
+        return new DownloadPrefsManager(ctx);
+    }
 }
diff --git a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/BookListFragment.java b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/BookListFragment.java
index d9ba9f3..5d4a73f 100644
--- a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/BookListFragment.java
+++ b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/BookListFragment.java
@@ -3,9 +3,7 @@ package org.bspeice.minimalbible.activities.downloader;
 import android.app.Activity;
 import android.app.AlertDialog;
 import android.app.ProgressDialog;
-import android.content.Context;
 import android.content.DialogInterface;
-import android.content.SharedPreferences;
 import android.os.Build;
 import android.os.Bundle;
 import android.support.v4.app.Fragment;
@@ -19,7 +17,6 @@ import android.widget.Toast;
 import com.readystatesoftware.systembartint.SystemBarTintManager;
 
 import org.bspeice.minimalbible.MinimalBible;
-import org.bspeice.minimalbible.MinimalBibleConstants;
 import org.bspeice.minimalbible.R;
 import org.bspeice.minimalbible.activities.downloader.manager.DownloadManager;
 import org.bspeice.minimalbible.activities.downloader.manager.EventBookList;
@@ -50,8 +47,8 @@ public class BookListFragment extends Fragment {
     @InjectView(R.id.lst_download_available)
     ListView downloadsAvailable;
 
-    @Inject
-    DownloadManager downloadManager;
+    @Inject DownloadManager downloadManager;
+    @Inject DownloadPrefsManager prefsManager;
 
 	private ProgressDialog refreshDialog;
 
@@ -92,12 +89,7 @@ public class BookListFragment extends Fragment {
     }
 
  	public void displayModules() {
-		SharedPreferences prefs = getActivity()
-				.getSharedPreferences(
-						MinimalBibleConstants.DOWNLOAD_PREFS_FILE,
-						Context.MODE_PRIVATE);
-		boolean dialogDisplayed = prefs.getBoolean(
-				MinimalBibleConstants.KEY_SHOWED_DOWNLOAD_DIALOG, false);
+		boolean dialogDisplayed = prefsManager.getShowedDownloadDialog();
 		
 		if (!dialogDisplayed) {
 			AlertDialog.Builder builder = new AlertDialog.Builder(getActivity());
@@ -162,18 +154,12 @@ public class BookListFragment extends Fragment {
 			DialogInterface.OnClickListener {
 		@Override
 		public void onClick(DialogInterface dialog, int which) {
-			SharedPreferences prefs = getActivity().getSharedPreferences(
-					MinimalBibleConstants.DOWNLOAD_PREFS_FILE,
-					Context.MODE_PRIVATE);
-			prefs.edit().putBoolean(MinimalBibleConstants.KEY_SHOWED_DOWNLOAD_DIALOG, true)
-				.commit();
+			prefsManager.setShowedDownloadDialog(true);
 
 			switch (which) {
 			case DialogInterface.BUTTON_POSITIVE:
 				// Clicked ready to continue - allow downloading in the future
-				prefs.edit()
-						.putBoolean(MinimalBibleConstants.KEY_DOWNLOAD_ENABLED,
-								true).commit();
+				prefsManager.setDownloadEnabled(true);
 
 				// And warn them that it has been enabled in the future.
 				Toast.makeText(getActivity(),
@@ -184,9 +170,7 @@ public class BookListFragment extends Fragment {
 
 			case DialogInterface.BUTTON_NEGATIVE:
 				// Clicked to not download - Permanently disable downloading
-				prefs.edit()
-						.putBoolean(MinimalBibleConstants.KEY_DOWNLOAD_ENABLED,
-								false).commit();
+				prefsManager.setDownloadEnabled(false);
 				Toast.makeText(getActivity(),
 						"Disabling downloading. Re-enable it in settings.",
 						Toast.LENGTH_SHORT).show();
diff --git a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/DownloadPrefsManager.java b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/DownloadPrefsManager.java
new file mode 100644
index 0000000..8374741
--- /dev/null
+++ b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/DownloadPrefsManager.java
@@ -0,0 +1,51 @@
+package org.bspeice.minimalbible.activities.downloader;
+
+import android.content.Context;
+import android.content.SharedPreferences;
+
+import java.util.Date;
+
+import javax.inject.Singleton;
+
+/**
+ * Created by Bradlee Speice on 5/8/2014.
+ */
+@Singleton
+public class DownloadPrefsManager {
+    private final SharedPreferences prefs;
+
+    public static final String DOWNLOAD_PREFS_FILE = "DOWNLOADER_PREFERENCES";
+
+    public static final String KEY_DOWNLOAD_ENABLED = "HAS_ENABLED_DOWNLOAD";
+    public static final String KEY_SHOWED_DOWNLOAD_DIALOG = "SHOWED_DOWNLOAD_DIALOG";
+    public static final String KEY_DOWNLOAD_REFRESHED_ON = "DOWNLOAD_REFRESHED_ON";
+
+    public DownloadPrefsManager(Context ctx) {
+        prefs = ctx.getSharedPreferences(DOWNLOAD_PREFS_FILE, Context.MODE_PRIVATE);
+    }
+
+    public boolean getDownloadEnabled() {
+        return prefs.getBoolean(KEY_DOWNLOAD_ENABLED, false);
+    }
+
+    public void setDownloadEnabled(boolean val) {
+        prefs.edit().putBoolean(KEY_DOWNLOAD_ENABLED, val).commit();
+    }
+
+    public boolean getShowedDownloadDialog() {
+        return prefs.getBoolean(KEY_SHOWED_DOWNLOAD_DIALOG, false);
+    }
+
+    public void setShowedDownloadDialog(boolean val) {
+        prefs.edit().putBoolean(KEY_SHOWED_DOWNLOAD_DIALOG, val).commit();
+    }
+
+    public Date getDownloadRefreshedOn() {
+        return new Date(prefs.getLong(KEY_DOWNLOAD_REFRESHED_ON, 0));
+    }
+
+    public void setDownloadRefreshedOn(Date d) {
+        prefs.edit().putLong(KEY_DOWNLOAD_REFRESHED_ON, d.getTime()).commit();
+    }
+
+}
diff --git a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/manager/BookRefreshTask.java b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/manager/BookRefreshTask.java
index b1cb78a..dbf53e9 100644
--- a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/manager/BookRefreshTask.java
+++ b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/manager/BookRefreshTask.java
@@ -1,35 +1,46 @@
 package org.bspeice.minimalbible.activities.downloader.manager;
 
-import java.util.LinkedList;
-import java.util.List;
+import android.content.Context;
+import android.net.ConnectivityManager;
+import android.net.NetworkInfo;
+import android.os.AsyncTask;
+import android.util.Log;
 
 import org.bspeice.minimalbible.MinimalBible;
-import org.bspeice.minimalbible.MinimalBibleConstants;
+import org.bspeice.minimalbible.activities.downloader.DownloadPrefsManager;
 import org.crosswire.jsword.book.Book;
 import org.crosswire.jsword.book.BookFilter;
 import org.crosswire.jsword.book.install.InstallException;
 import org.crosswire.jsword.book.install.Installer;
 
-import de.greenrobot.event.EventBus;
+import java.util.Date;
+import java.util.LinkedList;
+import java.util.List;
 
-import android.content.Context;
-import android.content.SharedPreferences;
-import android.os.AsyncTask;
-import android.util.Log;
+import javax.inject.Inject;
+
+import de.greenrobot.event.EventBus;
 
 public class BookRefreshTask extends AsyncTask<Installer, Integer, List<Book>> {
 	private static final String TAG = "EventBookRefreshTask";
 
+    // Refresh if last refresh date is after time below
+    private final Date refreshBefore = new Date(System.currentTimeMillis() -  604800000L); // 1 Week in millis
+
+    @Inject protected DownloadPrefsManager prefsManager;
+
 	private EventBus downloadBus;
 	private BookFilter filter;
 
 	public BookRefreshTask(EventBus downloadBus) {
 		this.downloadBus = downloadBus;
+        MinimalBible.getApplication().inject(this);
 	}
 
 	public BookRefreshTask(EventBus downloadBus, BookFilter f) {
 		this.downloadBus = downloadBus;
 		this.filter = f;
+        MinimalBible.getApplication().inject(this);
 	}
 
 	@Override
@@ -41,6 +52,7 @@ public class BookRefreshTask extends AsyncTask<Installer, Integer, List<Book>> {
 			if (doRefresh()) {
 				try {
 					i.reloadBookList();
+                    prefsManager.setDownloadRefreshedOn(new Date(System.currentTimeMillis()));
 				} catch (InstallException e) {
 					Log.e(TAG,
 							"Error downloading books from installer: "
@@ -66,12 +78,20 @@ public class BookRefreshTask extends AsyncTask<Installer, Integer, List<Book>> {
 		// copy - likely something time-based, also check network state.
 		// Fun fact - jSword handles the caching for us.
 
-		SharedPreferences prefs = MinimalBible.getAppContext()
-				.getSharedPreferences(
-						MinimalBibleConstants.DOWNLOAD_PREFS_FILE,
-						Context.MODE_PRIVATE);
-
-		// Refresh if download enabled
-		return prefs.getBoolean(MinimalBibleConstants.KEY_DOWNLOAD_ENABLED, false);
+        return (isWifi() && downloadEnabled() && needsRefresh());
 	}
+
+    private boolean isWifi() {
+        ConnectivityManager mgr = (ConnectivityManager)MinimalBible.getAppContext().getSystemService(Context.CONNECTIVITY_SERVICE);
+        NetworkInfo networkInfo = mgr.getNetworkInfo(ConnectivityManager.TYPE_WIFI);
+        return networkInfo.isConnected();
+    }
+
+    private boolean downloadEnabled() {
+        return prefsManager.getDownloadEnabled();
+    }
+
+    private boolean needsRefresh() {
+        return (prefsManager.getDownloadRefreshedOn().before(refreshBefore));
+    }
 }