Issue 6108 - No filter list is selected after a migration failure

adblockplus/Api.jsm

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 29 matching lines @@
 let {FilterNotifier} = require("filterNotifier");
 let {FilterStorage} = require("filterStorage");
 let {defaultMatcher} = require("matcher");
 let {Prefs} = require("prefs");
 let {Subscription, SpecialSubscription, RegularSubscription, DownloadableSubscription, ExternalSubscription} = require("subscriptionClasses");
 let {Synchronizer} = require("synchronizer");
 let {UI} = require("ui");
 
 const SUBSCRIPTIONS_SAVED_PREF = "subscriptions_saved";
 const USER_REMOVED_BLOCK_SUBS_PREF = "user_removed_block_subscriptions";
+const USER_REMOVED_EXCEPTIONS_SUB_PREF = "user_removed_exception_subscription";
 
 function initFilterListeners()
 {
   FilterNotifier.on("load", onFiltersLoad);
   FilterNotifier.on("save", onFiltersSave);
 }
 
 function onFiltersLoad()
 {
-  let {addonVersion} = require("info");
   let detectedSubscriptionFailure = !hasBlockSubscription() &&
     (!getBoolPref(SUBSCRIPTIONS_SAVED_PREF) || !getBoolPref(USER_REMOVED_BLOCK_SUBS_PREF) || FilterStorage.loadFromDiskFailed);
 
   // We will only try to recover the default subscription settings if the addonVersion hasn't changed,
   // otherwise it will be handled in firstRunActions(), inside ui.js
-  if(Prefs.currentVersion == addonVersion && detectedSubscriptionFailure)

[anton, 2017/12/04 14:00:46]

space is needed here

[diegocarloslima, 2017/12/05 09:44:10]

Acknowledged.

-  {
-    UI.addSubscription(UI.currentWindow, "0.0");
+  let {addonVersion} = require("info");
+  if (Prefs.currentVersion == addonVersion && detectedSubscriptionFailure)
+  {
+    if (getBoolPref(USER_REMOVED_EXCEPTIONS_SUB_PREF))
+    {
+      UI.addSubscription(UI.currentWindow, Prefs.currentVersion);
+    }
+    else
+    {
+      UI.addSubscription(UI.currentWindow, "0.0");
+    }
   }
   Messaging.sendRequest({ type: "Abb:OnFiltersLoad" });
 }
+
 function onFiltersSave()
 {
   if (hasBlockSubscription())
   {
     setBoolPref(SUBSCRIPTIONS_SAVED_PREF, true);
   }
   Messaging.sendRequest({ type: "Abb:OnFiltersSave" });
 }
 
 function getBoolPref(name)
@@ skipping 19 matching lines @@
 function getPrefsBranch()
 {
   let {addonRoot, addonName} = require("info");
   let branchName = "extensions." + addonName + ".";
   return Services.prefs.getBranch(branchName);
 }
 
 function hasBlockSubscription()
 {
   return FilterStorage.subscriptions.some(
-    (subscription) => subscription instanceof DownloadableSubscription && subscription.url != Prefs.subscriptions_exceptionsurl);

[jens, 2017/12/04 11:00:30]

Do we need parentheses when there's only one parameter name?

[diegocarloslima, 2017/12/05 09:44:09]

Acknowledged.

+    subscription => subscription instanceof DownloadableSubscription && subscription.url != Prefs.subscriptions_exceptionsurl);
 }
 
 function getWhitelistingFilter(url)
 {
   let uriObject = Services.io.newURI(url, null, null);
   try
   {
     return defaultMatcher.whitelist.matchesAny(
       uriObject.spec, RegExpFilter.typeMap.DOCUMENT, uriObject.host, false, null, false);
   }
@@ skipping 52 matching lines @@
     FilterStorage.addSubscription(subscriptionToAdd);
     let subscription = FilterStorage.knownSubscriptions[url];
     if (subscription)
     {
       subscription.disabled = false;
       if (!subscription.lastDownload)
       {
         Synchronizer.execute(subscription);
       }
     }
-    if (hasBlockSubscription())
+    if (url == Prefs.subscriptions_exceptionsurl)
+    {
+      setBoolPref(USER_REMOVED_EXCEPTIONS_SUB_PREF, false);
+    }
+    else if (hasBlockSubscription())
     {
       setBoolPref(USER_REMOVED_BLOCK_SUBS_PREF, false);
     }
   },
   removeSubscription: function(url)
   {
     FilterStorage.removeSubscription(FilterStorage.knownSubscriptions[url]);
-    if (!hasBlockSubscription())
+    if (url == Prefs.subscriptions_exceptionsurl)
+    {
+      setBoolPref(USER_REMOVED_EXCEPTIONS_SUB_PREF, true);
+    }
+    else if (!hasBlockSubscription())
     {
       setBoolPref(USER_REMOVED_BLOCK_SUBS_PREF, true);
     }
   },
   getActiveSubscriptions: function()
   {
     let subscriptions = [];
     for (let i = 0; i < FilterStorage.subscriptions.length; i++)
     {
       let subscription = FilterStorage.subscriptions[i];
@@ skipping 111 matching lines @@
           {
             this.whitelistSite(data["url"], data["whitelisted"]);
             return {"success": true};
           }
           break;
       }
       return {"success": false, "error": "malformed request"};
     }).bind(this), "AdblockPlus:Api");
   }
 };