Page MenuHomePhabricator

Thanks should run a hook before logging
Open, Needs TriagePublic

Description

In ApiThank, as part of logThanks, a hook should be run, so that other extensions can modify the log entry before it is added to the database

Use case:
AbuseFilter should be able to add tags to the log entry (see T235873)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 561114 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/Thanks@master] Add ApiThanksBeforeLogging hook

https://gerrit.wikimedia.org/r/561114

I'm sorry that I haven't handled this task. I recently returned from a long bout of unexpected inactivity, and while I plan to resume my contributions here on Phabricator its unfair to claim tasks that I might not work on when others may be interested in handling them. I'm removing myself as the assignee in a batch-action, but if someone feels that I really should be the one to handle this task feel free to re-assign me and I'll take a look.