From 9ae9c681c53dfe030f869e54efe7c2e7e23287b2 Mon Sep 17 00:00:00 2001
From: addshore <addshorewiki@gmail.com>
Date: Thu, 21 Dec 2017 21:18:07 +0000
Subject: [PATCH] [MCR] Fix RevisionInsertComplete Revision pass by reference

This was broken in:
6af796f3e0cf3e66cd7d7e59af8445f5712d68fe
Ia4c20a91e98df0b9b14b138eb4825c55e5200384
https://gerrit.wikimedia.org/r/#/c/399174/7

Bug: T183503
Change-Id: I674493507ca4aa6677a3bab7a01d6dc3c2cdbc5b
---
 includes/Revision.php | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/includes/Revision.php b/includes/Revision.php
index ea73a61bbf..c7c708ea63 100644
--- a/includes/Revision.php
+++ b/includes/Revision.php
@@ -979,8 +979,10 @@ class Revision implements IDBAccessObject {
 
 		$this->mRecord = $rec;
 
+		// Avoid PHP 7.1 warning of passing $this by reference
+		$revision = $this;
 		// TODO: hard-deprecate in 1.32 (or even 1.31?)
-		Hooks::run( 'RevisionInsertComplete', [ $this, null, null ] );
+		Hooks::run( 'RevisionInsertComplete', [ &$revision, null, null ] );
 
 		return $rec->getId();
 	}
-- 
2.20.1