Page MenuHomePhabricator

Should AbuseFilter identify change tags by id (and not name)?
Open, Needs TriagePublic

Description

Currently, change tag schema in core is being normalised (T185355), so that change tags are identified by a number, not name. Should AbuseFilter improve its schema as well?

The relevant fields are afa_parameters for rows with afa_consequence = 'tag' in abuse_filter_action and afh_actions in abuse_filter_history.

Event Timeline

It'd be good, and should be done as part of the major overhaul of change tags mentioned in several tasks across the workboard.

I think it is a great idea too. We should maintain backward compatibility though, and whenever an old filter is updated, we should replace its references to the new format.

We could also decide whether creating a maintenance script that updates all filters and logs would be appropriate or not. It is best that we list all the places in which something needs to be changed, and assess the overall work.

It'd be good, and should be done as part of the major overhaul of change tags mentioned in several tasks across the workboard.

Is there a task for the major overhaul?