jenkins-bot has submitted this change and it was merged.

Change subject: Swap method for closure
......................................................................


Swap method for closure

Change-Id: I848f19c34d6e302a71a3d0a845add5fdfea5acd8
---
M wmf-config/CommonSettings.php
1 file changed, 3 insertions(+), 4 deletions(-)

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



diff --git a/wmf-config/CommonSettings.php b/wmf-config/CommonSettings.php
index a69508e..21f2e90 100644
--- a/wmf-config/CommonSettings.php
+++ b/wmf-config/CommonSettings.php
@@ -936,9 +936,9 @@
 $wgNoFollowLinks = true; // In case the MediaWiki default changed, bug 42594
 
 # XFF log for vandal tracking
-function wfLogXFF() {
+$wgExtensionFunctions[] = function() {
        global $wmfUdp2logDest;
-       if ( ( @$_SERVER['REQUEST_METHOD'] ) == 'POST' ) {
+       if ( @$_SERVER['REQUEST_METHOD'] == 'POST' ) {
                $uri = ( $_SERVER['HTTPS'] ? 'https://' : 'http://' ) .
                        $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];
 
@@ -959,8 +959,7 @@
                        );
                }
        }
-}
-$wgExtensionFunctions[] = 'wfLogXFF';
+};
 
 // bug 24313, turn off minordefault on enwiki
 if ( $wgDBname == 'enwiki' ) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I848f19c34d6e302a71a3d0a845add5fdfea5acd8
Gerrit-PatchSet: 3
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: Reedy <[email protected]>
Gerrit-Reviewer: Reedy <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to