Ori.livneh has submitted this change and it was merged.

Change subject: Remove ampersands from onTitleMoveComplete()
......................................................................


Remove ampersands from onTitleMoveComplete()

Change-Id: If7c8cb56deee794f59da5abf26af9b19530c48a8
(cherry picked from commit d1426101747f47ab8074685bdcd080c51eea61fe)
---
M includes/Hooks.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Ori.livneh: Verified; Looks good to me, approved



diff --git a/includes/Hooks.php b/includes/Hooks.php
index e006e2d..09b6cb0 100644
--- a/includes/Hooks.php
+++ b/includes/Hooks.php
@@ -583,7 +583,7 @@
         * @param int $oldId The page id of the old page.
         * @return bool
         */
-       public static function onTitleMoveComplete( Title &$title, Title 
&$newTitle, &$user, $oldId ) {
+       public static function onTitleMoveComplete( Title $title, Title 
$newTitle, $user, $oldId ) {
                // When a page is moved the update and delete hooks are good 
enough to catch
                // almost everything.  The only thing they miss is if a page 
moves from one
                // index to another.  That only happens if it switches 
namespace.

-- 
To view, visit https://gerrit.wikimedia.org/r/249598
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If7c8cb56deee794f59da5abf26af9b19530c48a8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CirrusSearch
Gerrit-Branch: wmf/1.27.0-wmf.4
Gerrit-Owner: Ori.livneh <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to