Unified Diff: lib/filterStorage.js
Issue 29853570:
Issue 6908 - Pass updated subscription's old filters as an event argument (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set:
Created Aug. 11, 2018, 2:52 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« lib/filterListener.js ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/filterStorage.js |
| =================================================================== |
| --- a/lib/filterStorage.js |
| +++ b/lib/filterStorage.js |
| @@ -186,21 +186,21 @@ |
| /** |
| * Replaces the list of filters in a subscription by a new list |
| * @param {Subscription} subscription filter subscription to be updated |
| * @param {Filter[]} filters new filter list |
| */ |
| updateSubscriptionFilters(subscription, filters) |
| { |
| removeSubscriptionFilters(subscription); |
| - subscription.oldFilters = subscription.filters; |
| + let oldFilters = subscription.filters; |
| subscription.filters = filters; |
| addSubscriptionFilters(subscription); |
| - FilterNotifier.triggerListeners("subscription.updated", subscription); |
| - delete subscription.oldFilters; |
| + FilterNotifier.triggerListeners("subscription.updated", subscription, |
| + oldFilters); |
| }, |
| /** |
| * Adds a user-defined filter to the list |
| * @param {Filter} filter |
| * @param {SpecialSubscription} [subscription] |
| * particular group that the filter should be added to |
| * @param {number} [position] |
« no previous file with comments
|
« lib/filterListener.js ('k')
|
no next file »
|
no next file with comments »
