Opened on 04/18/2016 at 11:12:35 AM
Closed on 09/09/2019 at 03:49:32 PM
#3951 closed defect (invalid)
Duplicate filters can sometimes be added via Block element
Reported by: | Ross | Assignee: | |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | User-Interface | Keywords: | |
Cc: | kzar, sebastian, greiner | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Environment
ABP 1.11.0.1595
Chrome 32 / 42 / 48
Windows 8
How to reproduce
- Navigate to the dev build page: https://adblockplus.org/en/development-builds
- Install the latest ABP Chrome build.
- Open the ABP options page after install.
- Switch back to the dev build page and refresh it.
- Select [ABP Icon] > [Block element] and select the ABP icon in the top left corner of the page.
- Observe two filters are suggested (#logo and .sprite). Add them.
- Select [ABP Icon] > [Block element] and select anything else on the page.
- Manually edit the suggested filter to "adblockplus.org##.sprite" and add it.
- Switch back to the ABP options page > Custom filters.
- Observe the .sprite filter has been added twice / duplicated.
Observed behaviour
Sometimes, but not all the time (reproduced 3/8 attempts), it's possible to add duplicate custom filters via the Block element functionality. Normally duplicates are ignored, both when adding a filter via Block element and in the options page/tab itself.
Expected behaviour
Attempts to add duplicate filters to be ignored.
Attachments (1)
Change History (6)
Changed on 04/18/2016 at 11:14:05 AM by Ross
comment:1 Changed on 04/18/2016 at 11:14:38 AM by Ross
comment:2 Changed on 04/18/2016 at 02:56:46 PM by kzar
- Cc sebastian added
- Component changed from Core to Platform
comment:3 Changed on 04/18/2016 at 02:59:02 PM by sebastian
- Cc greiner added
- Component changed from Platform to User-Interface
Moving to User-Interface, as that is where the responsible message responder is.
comment:4 Changed on 04/18/2016 at 06:49:18 PM by greiner
- Priority changed from Unknown to P3
- Ready set
comment:5 Changed on 09/09/2019 at 03:49:32 PM by greiner
- Resolution set to invalid
- Status changed from new to closed
Closing this ticket because a lot of the code related to UI mentioned here has changed since. Please create a GitLab issue, if you can still reproduce this issue using the latest builds.
I noticed this while testing #3830, although not sure if related.