Opened 6 years ago

Last modified 5 years ago

#690 closed change

Issue reporter: don't consider special subscriptions when complaining about high subscription count — at Version 5

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

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.

Change History (5)

comment:1 Changed 6 years ago 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 6 years ago 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 6 years ago by saroyanm

  • Owner set to saroyanm

comment:4 Changed 6 years ago by saroyanm

  • Status changed from new to reviewing

comment:5 Changed 6 years ago by saroyanm

  • Review URL(s) modified (diff)
Note: See TracTickets for help on using tickets.