Page MenuHomePhabricator

Fix talk page message behavior when ORES extension is enabled
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Have AutoModerator enabled
  • Have bot flag enabled or disabled
  • Have ORES extension enabled

What happens?:
When the bot flag is enabled, no messages are sent.
When the bot flag is disabled, duplicate messages are sent.

What should have happened instead?:
One message sent ber rollback.

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):
1.44.0-wmf.16

Other information (browser name/version, screenshots, etc.):
See patch here for more info: T387095: SPIKE: AutoModerator sends repetitive notifications on user's talkpage on Chinese Wikipedia [8hrs]

Event Timeline

Kgraessle triaged this task as High priority.
Kgraessle moved this task from Ready to In Progress on the Moderator-Tools-Team (Kanban) board.

Change #1125254 had a related patch set uploaded (by Kgraessle; author: Kgraessle):

[mediawiki/extensions/AutoModerator@master] AutoModerator sends repetitive notifications on user's talkpage on Chinese Wikipedia - Refactored the talk page message sending logic to an onRollbackComplete hook - This was done because when ORES was enabled and the bot flag was disabled, we were sending multiple messages - The other bug this fixes is that when ORES was enabled and the bot flag was enabled, we were not sending any messags

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

Change #1125254 merged by jenkins-bot:

[mediawiki/extensions/AutoModerator@master] AutoModerator sends repetitive notifications on user's talkpage on Chinese Wikipedia - Refactored the talk page message sending logic to an onRollbackComplete hook - This was done because when ORES was enabled and the bot flag was disabled, we were sending multiple messages - The other bug this fixes is that when ORES was enabled and the bot flag was enabled, we were not sending any messags

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

We'll be able to QA this tomorrow after the train has rolled out to group 1 wikis.

This should be fixed in production. I see that the latest AutoModerator revert only sent one talk page message. See here.