Opened on 06/20/2014 at 01:15:29 PM

Closed on 07/30/2014 at 11:02:21 AM

Last modified on 05/20/2015 at 02:22:39 PM

#690 closed change (fixed)

Issue reporter: don't consider special subscriptions when complaining about high subscription count

Reported by: trev Assignee: saroyanm
Priority: P3 Milestone: Adblock-Plus-2.6.5-for-Firefox
Module: Adblock-Plus-for-Firefox Keywords:
Cc: Ede_123 Blocked By:
Blocking: Platform: Firefox
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/5410618009976832/

Description (last modified by trev)

Background

Issue reporter currently disallows reports listing more than five filter subscription to discourage reports where people subscribed to everything on our list and to give people the feedback that this is a good idea. However, this check was implemented before acceptable ads and anti-adblock were introduced, these "special" subscriptions also count into the number.

What to change

Don't count acceptable ads and anti-adblock for the purpose of this warning.

Testing notes

This is about Firefox desktop only:

  1. In the Adblock Plus filter preferences, make sure that you have four filter subscriptions in addition to "Adblock Warning Removal List", add more if necessary. Also make sure that "Adblock Warning Removal List" and "Allow some non-intrusive advertising" are checked.
  2. Go to some webpage, click the ABP icon and select "Report issue on this page." Choose "Adblock Plus doesn't block an advertisement" as issue type and proceed - you should get to the screenshot page.
  3. Now add one more filter subscription in Adblock Plus filter preferences so that you have five now, not counting "Adblock Warning Removal List."
  4. Repeat step 2, this time you should get a message telling you to remove some filter lists.

Attachments (0)

Change History (8)

comment:1 Changed on 06/24/2014 at 08:12:43 AM by philll

  • Cc Ede_123 added

by Ede_123:
(I think my last comment was eaten by the SPAM filter...?)

While changing this behaviour you should also consider

  • Some lists should probably be grouped into "families" and only counted once.

E.g. I have "EasyList", "EasyList Germany" and "EasyPrivacy" installed. At least "EasyList" and the localized counterpart "EasyList Germany" should probably only count as one list as they are meant to be used together.
(Actually these lists even exist combined into a single list, but I like the modular approach of putting together only the needed list, which after all is the reason for the limitation described in this bug - otherwise one could just use a huge "all-in-one" list and still pass the test)

  • To solve the above in a more appropriate way, you might want to consider the number of filters instead of the number of lists. E.g. five tiny lists are probably better than one huge list.

comment:2 Changed on 07/01/2014 at 12:55:32 PM by trev

  • Platform set to Unknown

Five tiny lists still have five different maintainers. This limitation is about figuring out who is responsible, not about sifting through the list of filters.

comment:3 Changed on 07/28/2014 at 08:04:14 AM by saroyanm

  • Owner set to saroyanm

comment:4 Changed on 07/28/2014 at 08:04:26 AM by saroyanm

  • Status changed from new to reviewing

comment:5 Changed on 07/28/2014 at 08:05:03 AM by saroyanm

  • Review URL(s) modified (diff)

comment:6 Changed on 07/30/2014 at 11:02:21 AM by saroyanm

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:7 Changed on 07/30/2014 at 12:40:06 PM by trev

  • Description modified (diff)
  • Milestone set to Adblock-Plus-for-Firefox-next
  • Platform changed from Unknown to Firefox/Firefox Mobile

comment:8 Changed on 05/20/2015 at 02:22:39 PM by philll

  • Platform changed from Firefox/Firefox Mobile to Firefox

Made Firefox and Firefox mobile available as seperate platforms.

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 saroyanm.
 
Note: See TracTickets for help on using tickets.