Issue 5810 - Use "specialization" as Subscription display title

libadblockplus-android-settings/src/org/adblockplus/libadblockplus/android/settings/SharedPrefsStorage.java

Index: libadblockplus-android-settings/src/org/adblockplus/libadblockplus/android/settings/SharedPrefsStorage.java
diff --git a/libadblockplus-android-settings/src/org/adblockplus/libadblockplus/android/settings/SharedPrefsStorage.java b/libadblockplus-android-settings/src/org/adblockplus/libadblockplus/android/settings/SharedPrefsStorage.java
index a67707259bda5a0042cba75cf078a7f27fe87e9b..c910d2b662922b5e593100db4a85337d11b8d047 100644
--- a/libadblockplus-android-settings/src/org/adblockplus/libadblockplus/android/settings/SharedPrefsStorage.java
+++ b/libadblockplus-android-settings/src/org/adblockplus/libadblockplus/android/settings/SharedPrefsStorage.java
@@ -34,6 +34,7 @@ public class SharedPrefsStorage extends AdblockSettingsStorage
   private static final String SETTINGS_SUBSCRIPTIONS_KEY = "subscriptions";
   private static final String SETTINGS_SUBSCRIPTION_KEY = "subscription";
   private static final String SETTINGS_SUBSCRIPTION_URL_KEY = "url";
+  private static final String SETTINGS_SUBSCRIPTION_SPECIALIZATION_KEY = "specialization";
   private static final String SETTINGS_SUBSCRIPTION_TITLE_KEY = "title";
   private static final String SETTINGS_WL_DOMAINS_KEY = "whitelisted_domains";
   private static final String SETTINGS_WL_DOMAIN_KEY = "domain";
@@ -115,6 +116,7 @@ public class SharedPrefsStorage extends AdblockSettingsStorage
         Subscription subscription = new Subscription();
         subscription.title = prefs.getString(getSubscriptionTitleKey(i), "");
         subscription.url = prefs.getString(getSubscriptionURLKey(i), "");
+        subscription.specialization = prefs.getString(getSubscriptionSpecializationKey(i), "");
         subscriptions.add(subscription);
       }
       settings.setSubscriptions(subscriptions);
@@ -143,6 +145,11 @@ public class SharedPrefsStorage extends AdblockSettingsStorage
     return getArrayItemKey(index, SETTINGS_SUBSCRIPTION_KEY, SETTINGS_SUBSCRIPTION_URL_KEY);
   }
 
+  private String getSubscriptionSpecializationKey(int index)
+  {
+    return getArrayItemKey(index, SETTINGS_SUBSCRIPTION_KEY, SETTINGS_SUBSCRIPTION_SPECIALIZATION_KEY);
+  }
+
   @Override
   public void save(AdblockSettings settings)
   {
@@ -199,9 +206,10 @@ public class SharedPrefsStorage extends AdblockSettingsStorage
       {
         Subscription eachSubscription = settings.getSubscriptions().get(i);
 
-        // warning: saving `title` and `url` fields only
+        // warning: saving `title`, `url` and `specialization` fields only
         editor.putString(getSubscriptionTitleKey(i), eachSubscription.title);
         editor.putString(getSubscriptionURLKey(i), eachSubscription.url);
+        editor.putString(getSubscriptionSpecializationKey(i), eachSubscription.url);

[diegocarloslima, 2017/09/29 14:27:20]

Shouldn't it be `eachSubscription.specialization` here?

[anton, 2017/09/29 20:39:35]

Yes, sure. Silly mistake.

       }
     }
   }