Update providing of SharedPreferences

app/src/main/java/com/nutomic/syncthingandroid/SyncthingApp.java

@@ -4,6 +4,9 @@
 import android.os.StrictMode;
 
 import com.google.android.material.color.DynamicColors;
+import com.nutomic.syncthingandroid.di.DaggerComponent;
+import com.nutomic.syncthingandroid.di.DaggerDaggerComponent;
+import com.nutomic.syncthingandroid.di.SyncthingModule;
 import com.nutomic.syncthingandroid.util.Languages;
 
 import javax.inject.Inject;

app/src/main/java/com/nutomic/syncthingandroid/activities/FirstStartActivity.java

@@ -38,6 +38,7 @@
 import com.nutomic.syncthingandroid.R;
 import com.nutomic.syncthingandroid.SyncthingApp;
 import com.nutomic.syncthingandroid.databinding.ActivityFirstStartBinding;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.service.Constants;
 import com.nutomic.syncthingandroid.util.PermissionUtil;
 import com.nutomic.syncthingandroid.util.Util;
@@ -75,6 +76,7 @@ private enum Slide {
     private ActivityFirstStartBinding binding;
 
     @Inject
+    @DefaultSharedPreferences
     SharedPreferences mPreferences;
 
     @SuppressLint("ClickableViewAccessibility")

app/src/main/java/com/nutomic/syncthingandroid/activities/MainActivity.java

@@ -49,6 +49,7 @@
 import com.annimon.stream.function.Consumer;
 import com.nutomic.syncthingandroid.R;
 import com.nutomic.syncthingandroid.SyncthingApp;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.fragments.DeviceListFragment;
 import com.nutomic.syncthingandroid.fragments.DrawerFragment;
 import com.nutomic.syncthingandroid.fragments.FolderListFragment;
@@ -102,7 +103,10 @@ public class MainActivity extends StateDialogActivity
 
     private ActionBarDrawerToggle mDrawerToggle;
     private DrawerLayout          mDrawerLayout;
-    @Inject SharedPreferences mPreferences;
+
+    @Inject
+    @DefaultSharedPreferences
+    SharedPreferences mPreferences;
 
     /**
      * Handles various dialogs based on current state.

app/src/main/java/com/nutomic/syncthingandroid/activities/SettingsActivity.java

@@ -27,6 +27,7 @@
 import com.google.common.collect.Iterables;
 import com.nutomic.syncthingandroid.R;
 import com.nutomic.syncthingandroid.SyncthingApp;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.model.Config;
 import com.nutomic.syncthingandroid.model.Device;
 import com.nutomic.syncthingandroid.model.Options;
@@ -103,7 +104,10 @@ public static class SettingsFragment extends PreferenceFragment
         private static final String KEY_ST_RESET_DELTAS = "st_reset_deltas";
 
         @Inject NotificationHandler mNotificationHandler;
-        @Inject SharedPreferences mPreferences;
+
+        @Inject
+        @DefaultSharedPreferences
+        SharedPreferences mPreferences;
 
         private PreferenceGroup    mCategoryRunConditions;
         private CheckBoxPreference mRunConditions;

app/src/main/java/com/nutomic/syncthingandroid/DaggerComponent.java → app/src/main/java/com/nutomic/syncthingandroid/di/DaggerComponent.java

@@ -1,5 +1,6 @@
-package com.nutomic.syncthingandroid;
+package com.nutomic.syncthingandroid.di;
 
+import com.nutomic.syncthingandroid.SyncthingApp;
 import com.nutomic.syncthingandroid.activities.FirstStartActivity;
 import com.nutomic.syncthingandroid.activities.FolderPickerActivity;
 import com.nutomic.syncthingandroid.activities.MainActivity;

app/src/main/java/com/nutomic/syncthingandroid/di/DefaultSharedPreferences.java

@@ -0,0 +1,13 @@
+package com.nutomic.syncthingandroid.di;
+
+import static java.lang.annotation.RetentionPolicy.RUNTIME;
+
+import java.lang.annotation.Retention;
+
+import javax.inject.Qualifier;
+
+@Qualifier
+@Retention(RUNTIME)
+public @interface DefaultSharedPreferences {
+
+}

app/src/main/java/com/nutomic/syncthingandroid/SyncthingModule.java → app/src/main/java/com/nutomic/syncthingandroid/di/SyncthingModule.java

@@ -1,8 +1,9 @@
-package com.nutomic.syncthingandroid;
+package com.nutomic.syncthingandroid.di;
 
 import android.content.SharedPreferences;
 import android.preference.PreferenceManager;
 
+import com.nutomic.syncthingandroid.SyncthingApp;
 import com.nutomic.syncthingandroid.service.NotificationHandler;
 
 import javax.inject.Singleton;
@@ -21,6 +22,7 @@ public SyncthingModule(SyncthingApp app) {
 
     @Provides
     @Singleton
+    @DefaultSharedPreferences
     public SharedPreferences getPreferences() {
         return PreferenceManager.getDefaultSharedPreferences(mApp);
     }

app/src/main/java/com/nutomic/syncthingandroid/service/EventProcessor.java

@@ -21,6 +21,7 @@
 import com.nutomic.syncthingandroid.SyncthingApp;
 import com.nutomic.syncthingandroid.activities.DeviceActivity;
 import com.nutomic.syncthingandroid.activities.FolderActivity;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.model.CompletionInfo;
 import com.nutomic.syncthingandroid.model.Device;
 import com.nutomic.syncthingandroid.model.Event;
@@ -60,9 +61,12 @@ public class EventProcessor implements  Runnable, RestApi.OnReceiveEventListener
 
     private final Context mContext;
     private final RestApi mApi;
-    @Inject SharedPreferences mPreferences;
     @Inject NotificationHandler mNotificationHandler;
 
+    @Inject
+    @DefaultSharedPreferences
+    SharedPreferences mPreferences;
+
     public EventProcessor(Context context, RestApi api) {
         ((SyncthingApp) context.getApplicationContext()).component().inject(this);
         mContext = context;

app/src/main/java/com/nutomic/syncthingandroid/service/NotificationHandler.java

@@ -18,6 +18,7 @@
 import com.nutomic.syncthingandroid.activities.FirstStartActivity;
 import com.nutomic.syncthingandroid.activities.LogActivity;
 import com.nutomic.syncthingandroid.activities.MainActivity;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.service.Constants;
 import com.nutomic.syncthingandroid.service.SyncthingService.State;
 
@@ -37,7 +38,6 @@ public class NotificationHandler {
     private static final String CHANNEL_PERSISTENT_WAITING = "03_syncthing_persistent_waiting";
 
     private final Context mContext;
-    @Inject SharedPreferences mPreferences;
     private final NotificationManager mNotificationManager;
     private final NotificationChannel mPersistentChannel;
     private final NotificationChannel mPersistentChannelWaiting;
@@ -46,6 +46,10 @@ public class NotificationHandler {
     private Boolean lastStartForegroundService = false;
     private Boolean appShutdownInProgress = false;
 
+    @Inject
+    @DefaultSharedPreferences
+    SharedPreferences mPreferences;
+
     public NotificationHandler(Context context) {
         ((SyncthingApp) context.getApplicationContext()).component().inject(this);
         mContext = context;

app/src/main/java/com/nutomic/syncthingandroid/service/RunConditionMonitor.java

@@ -20,6 +20,7 @@
 import android.util.Log;
 
 import com.nutomic.syncthingandroid.SyncthingApp;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.model.RunConditionCheckResult;
 
 import java.util.ArrayList;
@@ -59,9 +60,12 @@ public interface OnRunConditionChangedListener {
     }
 
     private final Context mContext;
-    @Inject SharedPreferences mPreferences;
     private ReceiverManager mReceiverManager;
 
+    @Inject
+    @DefaultSharedPreferences
+    SharedPreferences mPreferences;
+
     /**
      * Sending callback notifications through {@link OnRunConditionChangedListener} is enabled if not null.
      */

app/src/main/java/com/nutomic/syncthingandroid/service/SyncthingRunnable.java

@@ -14,6 +14,7 @@
 import com.google.common.io.Files;
 import com.nutomic.syncthingandroid.R;
 import com.nutomic.syncthingandroid.SyncthingApp;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.service.Constants;
 import com.nutomic.syncthingandroid.util.Util;
 
@@ -56,10 +57,13 @@ public class SyncthingRunnable implements Runnable {
     private final File mSyncthingBinary;
     private String[] mCommand;
     private final File mLogFile;
-    @Inject SharedPreferences mPreferences;
     private final boolean mUseRoot;
     @Inject NotificationHandler mNotificationHandler;
 
+    @Inject
+    @DefaultSharedPreferences
+    SharedPreferences mPreferences;
+
     public enum Command {
         deviceid,           // Output the device ID to the command line.
         generate,           // Generate keys, a config file and immediately exit.

app/src/main/java/com/nutomic/syncthingandroid/service/SyncthingService.java

@@ -12,6 +12,7 @@
 import com.google.common.io.Files;
 import com.nutomic.syncthingandroid.R;
 import com.nutomic.syncthingandroid.SyncthingApp;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.http.PollWebGuiAvailableTask;
 import com.nutomic.syncthingandroid.model.RunConditionCheckResult;
 import com.nutomic.syncthingandroid.util.ConfigXml;
@@ -159,7 +160,10 @@ public enum State {
     private final SyncthingServiceBinder mBinder = new SyncthingServiceBinder(this);
 
     @Inject NotificationHandler mNotificationHandler;
-    @Inject SharedPreferences mPreferences;
+
+    @Inject
+    @DefaultSharedPreferences
+    SharedPreferences mPreferences;
 
     /**
      * Object that must be locked upon accessing mCurrentState

app/src/main/java/com/nutomic/syncthingandroid/util/ConfigXml.java

@@ -9,6 +9,7 @@
 import android.util.Log;
 
 import com.nutomic.syncthingandroid.R;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 import com.nutomic.syncthingandroid.service.Constants;
 import com.nutomic.syncthingandroid.service.SyncthingRunnable;
 
@@ -54,7 +55,10 @@ public class OpenConfigException extends RuntimeException {
     private static final int FOLDER_ID_APPENDIX_LENGTH = 4;
 
     private final Context mContext;
-    @Inject SharedPreferences mPreferences;
+
+    @Inject
+    @DefaultSharedPreferences
+    SharedPreferences mPreferences;
 
     private final File mConfigFile;
 

app/src/main/java/com/nutomic/syncthingandroid/util/Languages.java

@@ -13,6 +13,7 @@
 
 import com.nutomic.syncthingandroid.R;
 import com.nutomic.syncthingandroid.SyncthingApp;
+import com.nutomic.syncthingandroid.di.DefaultSharedPreferences;
 
 import java.util.Arrays;
 import java.util.Collections;
@@ -34,7 +35,10 @@ public final class Languages {
     private static final Locale DEFAULT_LOCALE;
     public static final String PREFERENCE_LANGUAGE = "pref_current_language";
 
-    @Inject SharedPreferences mPreferences;
+    @Inject
+    @DefaultSharedPreferences
+    SharedPreferences mPreferences;
+
     private static Map<String, String> mAvailableLanguages;
 
     static {