Opened on 05/19/2018 at 05:23:17 PM

Closed on 05/20/2018 at 09:50:38 PM

Last modified on 05/20/2018 at 11:59:32 PM

#6686 closed defect (invalid)

Element cannot be added to storage.js

Reported by: cjsmall Assignee:
Priority: Unknown Milestone:
Module: Unknown Keywords:
Cc: mapx, mjethani, kzar Blocked By:
Blocking: Platform: Firefox
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by mapx)

Environment

Xubuntu 17.10
Firefox 60.0.1
ABP 3.0.4

How to reproduce

Attempts to remove the element:

wired.com##.advertisement__leaderboard

From the wired.com site fails.

Specifically, on the page:

https://www.wired.com/story/congress-latest-move-to-extend-copyright-protection-is-misguided/

This element at the top of the page can be selected and shows up on the filter pop-up. However, selecting Add does NOT add it to the filter list.
I was able to add other elements from this page after this point, but not this one despite repeated attempts.

From the Options page I edited the filter list and pasted it in.
However, it disappears when I do a Save!

A possible related or unrelated issue is that my storage.js file is now 7984288 bytes.
In the old version of ABP, my filter list would grow for a while and then I would notice that it was magically truncated to something around this size, discarding many of my site filters in the process.

I assumed this was fixed in the current version of the program but I'm wondering if I am again bumping up against some hard limit on an internal buffer around 8MB.

I would appreciate someone looking into this to see if there is indeed a problem with the size of custom filters.

Observed behaviour

Cannot add element to filter list.

Attachments (0)

Change History (8)

comment:1 Changed on 05/20/2018 at 06:45:43 AM by mapx

  • Description modified (diff)

comment:2 Changed on 05/20/2018 at 06:59:17 AM by mapx

try these filters

wired.com##[class^=adv]
wired.com##.failsafe-desktop

comment:3 Changed on 05/20/2018 at 06:59:43 AM by mapx

  • Cc mapx mjethani kzar added

comment:4 Changed on 05/20/2018 at 05:37:15 PM by cjsmall

I was able to manually add the two elements you suggested to the filter list and they worked. Thanks.

I'm still unclear why the one identified by ABP can not be added automatically or manually. I see other filters where the element name starts with a period, so that's not it. Do you know the reason? Is this a bug in ABP?

comment:5 Changed on 05/20/2018 at 06:20:06 PM by mapx

Well, on my side no issue with adding that filter (but I don't know why it does not work).
Let's see if the devs could clear this issue.

Last edited on 05/20/2018 at 09:49:17 PM by mapx

comment:6 Changed on 05/20/2018 at 09:49:04 PM by mapx

the mistery is not a ...mistery (courtesy of R.Hill - from ublock origin):

easylist contains this exception filter
wired.com#@#.advertisement__leaderboard which unfortunately is not reported in the logger (ABP dev panel)

Last edited on 05/20/2018 at 09:50:13 PM by mapx

comment:7 Changed on 05/20/2018 at 09:50:38 PM by mapx

  • Resolution set to invalid
  • Status changed from new to closed

comment:8 Changed on 05/20/2018 at 11:59:32 PM by cjsmall

mapx, thanks for the info. I've seen this happen a few times in the past. That will help explain similar things in the future.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.