Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/subscriptionInit.js

Issue 29907589: Issue 7054 - Update the adblockpluscore dependency to 5cb695da5a40, adblockplusui to f86abf2efdfd (Closed)
Patch Set: Created Nov. 29, 2018, 4:52 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/subscriptionInit.js
===================================================================
--- a/lib/subscriptionInit.js
+++ b/lib/subscriptionInit.js
@@ -44,17 +44,17 @@
* first run, but something went wrong.
*
* This function detects the first run, and makes sure that the user
* gets notified (on the first run page) if the data appears incomplete
* and therefore will be reinitialized.
*/
function detectFirstRun()
{
- firstRun = FilterStorage.subscriptions.length == 0;
+ firstRun = FilterStorage.subscriptionCount == 0;
if (firstRun && (!FilterStorage.firstRun || Prefs.currentVersion))
reinitialized = true;
Prefs.currentVersion = info.addonVersion;
}
/**
@@ -66,17 +66,17 @@
* is no data and therefore no subscriptions. But it also causes the
* default ad blocking subscriptions to be added again after some
* data corruption or misconfiguration.
*
* @return {boolean}
*/
function shouldAddDefaultSubscriptions()
{
- for (let subscription of FilterStorage.subscriptions)
+ for (let subscription of FilterStorage.subscriptions())
{
if (subscription instanceof DownloadableSubscription &&
subscription.url != Prefs.subscriptions_exceptionsurl &&
subscription.url != Prefs.subscriptions_antiadblockurl &&
subscription.type != "circumvention")
return false;
if (subscription instanceof SpecialSubscription &&
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld