Siebrand has submitted this change and it was merged.

Change subject: Pass by reference is not needed
......................................................................


Pass by reference is not needed

Change-Id: I7f23539c4e648e142c3d61b8b73e1b562f4ef0c4
---
M TranslateHooks.php
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Nikerabbit: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/TranslateHooks.php b/TranslateHooks.php
index dda87a0..10a8af5 100644
--- a/TranslateHooks.php
+++ b/TranslateHooks.php
@@ -609,7 +609,7 @@
         * @param User $newUser
         * @return bool
         */
-       public static function onMergeAccountFromTo( User &$oldUser, User 
&$newUser ) {
+       public static function onMergeAccountFromTo( User $oldUser, User 
$newUser ) {
                $dbw = wfGetDB( DB_MASTER );
 
                // Update the non-duplicate rows, we'll just delete
@@ -635,7 +635,7 @@
         * @param User $oldUser
         * @return bool
         */
-       public static function onDeleteAccount( User &$oldUser ) {
+       public static function onDeleteAccount( User $oldUser ) {
                $dbw = wfGetDB( DB_MASTER );
 
                // Delete any remaining rows that didn't get merged

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7f23539c4e648e142c3d61b8b73e1b562f4ef0c4
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Siebrand <[email protected]>
Gerrit-Reviewer: Nikerabbit <[email protected]>
Gerrit-Reviewer: Siebrand <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to