Project

General

Profile

Actions

Feature #302

closed
SS SS

Consider refactoring the blocklist helper

Feature #302: Consider refactoring the blocklist helper

Added by Soren Stoutner about 8 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Next Release
Start date:
07/03/2018
Due date:
% Done:

0%

Estimated time:

Description

Currently it returns a boolean on the `isBlocked()` method. It could be refactored to return a String[] on a `checkBlockList()` method. This would unify how whitelist and blacklist entries are handled. But I'm not sure it is the way to go. It might increase the complexity of handling user block lists as described in https://redmine.stoutner.com/issues/181, which will handle whitelist entries differently as whitelist hits will override entries in other blocklists (they are a global whitelist), which they don't do for the standard blocklists (which are local whitelists).

SS Updated by Soren Stoutner about 8 years ago Actions #1

  • Description updated (diff)

SS Updated by Soren Stoutner about 8 years ago Actions #2

  • Priority changed from 2 to 3.x

SS Updated by Soren Stoutner over 7 years ago Actions #3

  • Priority changed from 3.x to Next Release

SS Updated by Soren Stoutner over 7 years ago Actions #4

  • Status changed from New to Closed

I actually did this a while ago without updating the feature request.

Actions

Also available in: PDF Atom