Opened on 11/09/2017 at 01:12:17 PM
Closed on 11/21/2018 at 11:02:32 AM
Last modified on 01/30/2019 at 11:42:46 AM
#6017 closed change (fixed)
Update subscriptions.xml
Reported by: | jwangenheim | Assignee: | |
---|---|---|---|
Priority: | P2 | Milestone: | Adblock-Plus-for-Samsung-Browser-next |
Module: | Adblock-Plus-for-Samsung-Browser | Keywords: | |
Cc: | mario, diegocarloslima, fhd, rjeschke, vickyyu | Blocked By: | |
Blocking: | Platform: | Android | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
https://gitlab.com/eyeo/adblockplus/adblockplussbrowser/merge_requests/2 |
Description (last modified by mario)
Background
The subscriptions the user can select are parsed from subscriptions.xml file. We need to update the file to the latest version and make sure that the format of the xml can be handled by the parser.
What to change
Update subscriptions.xml to the latest version
Hint for tesers
This can be tested by checking whether the list of available filter lists is the same as in our official subscription.xml
Please make sure to also test for data migration:
- If the removed filter lists aren't subscribed to, they're supposed to be removed from the subscription list in ABP.
- If the removed filter lists are subscribed to, they're supposed to stay in the subscription list in ABP and marked as active.
Attachments (0)
Change History (13)
comment:1 Changed on 11/09/2017 at 01:13:06 PM by jwangenheim
- Cc mario diegocarloslima fhd added
comment:2 Changed on 11/09/2017 at 01:18:06 PM by jwangenheim
- Cc rjeschke added
comment:3 Changed on 11/10/2017 at 10:32:08 AM by jwangenheim
- Component changed from Adblock-Plus-for-Internet-Explorer to Adblock-Plus-for-Samsung-Browser
comment:4 Changed on 11/10/2017 at 11:02:33 AM by jwangenheim
comment:5 Changed on 04/24/2018 at 12:23:26 PM by vickyyu
- Cc vickyyu added
comment:6 Changed on 11/06/2018 at 03:45:09 PM by jwangenheim
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:7 Changed on 11/21/2018 at 11:02:32 AM by jwangenheim
- Resolution set to fixed
- Status changed from reviewing to closed
comment:9 Changed on 11/21/2018 at 11:07:17 AM by jwangenheim
For the automatic updates I created a separate ticket in GitLab: https://gitlab.com/eyeo/adblockplus/adblockplussbrowser/issues/4
comment:10 Changed on 12/20/2018 at 01:55:48 PM by mario
- Milestone set to Adblock-Plus-for-Samsung-Browser-next
comment:11 Changed on 12/20/2018 at 02:02:46 PM by mario
- Description modified (diff)
comment:12 Changed on 12/20/2018 at 02:03:07 PM by mario
- Description modified (diff)
comment:13 Changed on 01/30/2019 at 11:42:46 AM by mario
- Description modified (diff)
I checked the latest version of adblockpluscore:chrome/content/ui/subscriptions.xml and compared it to the version we currently use in ABP for Samsung internet. The two lists have a different structure, it looks like we use an old format in ABP for Samsung internet:
<--- Currently used in ABP for Samsung Internet --->
<--- Latest version in adblockpluscore --->
The latest version in core also only contains subscriptions of type "ads". If we would start using this list, we loose all 4 lists of type "other" (Anti-Adblocker-Blocker, Tracking, Social Media, Malware Domains).
So before we decide how to keep subscriptions.xml up to date, we need to adjust the code in ABP for Samsung Internet in order to work with the new structure.
@fhd @rene
Whats your opinion on that?