jenkins-bot has submitted this change and it was merged.
Change subject: Leverage revision object for parent
......................................................................
Leverage revision object for parent
The old revision (parent) is an attribute of the Revision obj, so use that
instead of looking it up via the Title.
Bug: T116786
Change-Id: I47535ee226f546f6acaabb4107f751a0b1383d8e
---
M EventBus.hooks.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Aaron Schulz: Looks good to me, approved
jenkins-bot: Verified
diff --git a/EventBus.hooks.php b/EventBus.hooks.php
index bacadc1..fcc714b 100644
--- a/EventBus.hooks.php
+++ b/EventBus.hooks.php
@@ -190,7 +190,7 @@
$attrs['old_title'] = $title->getText();
$attrs['page_id'] = $oldid;
$attrs['new_revision_id'] = $newRevision->getId();
- $attrs['old_revision_id'] = $newtitle->getLatestRevId();
+ $attrs['old_revision_id'] = $newRevision->getParentId();
$attrs['user_id'] = $user->getId();
$attrs['user_text'] = $user->getName();
$attrs['summary'] = $reason;
--
To view, visit https://gerrit.wikimedia.org/r/259790
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I47535ee226f546f6acaabb4107f751a0b1383d8e
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/EventBus
Gerrit-Branch: master
Gerrit-Owner: Eevans <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: GWicke <[email protected]>
Gerrit-Reviewer: Hashar <[email protected]>
Gerrit-Reviewer: Krinkle <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Mobrovac <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
