Page MenuHomePhabricator

Use RevisionRecordInserted hook for EventBus revision-create records
Closed, ResolvedPublic

Description

From @Stas:

I've noticed that RevisionInsertComplete hook is now deprecated in favor
of RevisionRecordInserted. However, EventBus still uses
RevisionInsertComplete. Is this going to change soon? If so, will the
underlying event/topic change too? I couldn't find anything in
Phabricator about this - is there plan to change it or still use old
hook for now and foreseeable future?

I guess we should!

Event Timeline

Change 407863 had a related patch set uploaded (by Ppchelko; owner: Ppchelko):
[mediawiki/extensions/EventBus@master] Use RevisionRecordInserted hook for EventBus revision-create records

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

Change 407863 merged by jenkins-bot:
[mediawiki/extensions/EventBus@master] Use RevisionRecordInserted hook for EventBus revision-create records

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