jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/367942 )
Change subject: build: Updating mediawiki/mediawiki-codesniffer to 0.10.1 ...................................................................... build: Updating mediawiki/mediawiki-codesniffer to 0.10.1 Change-Id: I36ab81e99ef5d3e01ffa045d47d95cc9c4ab50f6 --- M composer.json M includes/CentralAuthUser.php M includes/CentralAuthUtils.php M phpcs.xml 4 files changed, 11 insertions(+), 13 deletions(-) Approvals: jenkins-bot: Verified Jforrester: Looks good to me, approved diff --git a/composer.json b/composer.json index 4f55859..bf202bf 100644 --- a/composer.json +++ b/composer.json @@ -2,7 +2,7 @@ "require-dev": { "jakub-onderka/php-parallel-lint": "0.9.2", "jakub-onderka/php-console-highlighter": "0.3.2", - "mediawiki/mediawiki-codesniffer": "0.10.0" + "mediawiki/mediawiki-codesniffer": "0.10.1" }, "scripts": { "fix": "phpcbf", diff --git a/includes/CentralAuthUser.php b/includes/CentralAuthUser.php index a09380b..d752899 100644 --- a/includes/CentralAuthUser.php +++ b/includes/CentralAuthUser.php @@ -279,7 +279,7 @@ ); if ( $name !== false ) { - return CentralAuthUser::getInstanceByName( $name ); + return self::getInstanceByName( $name ); } else { return false; } @@ -300,7 +300,7 @@ ); if ( $name !== false ) { - return CentralAuthUser::getMasterInstanceByName( $name ); + return self::getMasterInstanceByName( $name ); } else { return false; } diff --git a/includes/CentralAuthUtils.php b/includes/CentralAuthUtils.php index 57bc6d1..b4d8100 100644 --- a/includes/CentralAuthUtils.php +++ b/includes/CentralAuthUtils.php @@ -178,9 +178,9 @@ * @return array */ public static function getCentralSessionById( $id ) { - $key = CentralAuthUtils::memcKey( 'session', $id ); + $key = self::memcKey( 'session', $id ); $stime = microtime( true ); - $data = CentralAuthUtils::getSessionCache()->get( $key ) ?: []; + $data = self::getSessionCache()->get( $key ) ?: []; $real = microtime( true ) - $stime; MediaWikiServices::getInstance() ->getStatsdDataFactory()->timing( 'centralauth.session.read', $real ); @@ -206,10 +206,10 @@ $id = is_string( $reset ) ? $reset : MWCryptRand::generateHex( 32 ); } $data['sessionId'] = $id; - $key = CentralAuthUtils::memcKey( 'session', $id ); + $key = self::memcKey( 'session', $id ); // Copy certain keys from the existing session, if any (T124821) - $existing = CentralAuthUtils::getSessionCache()->get( $key ); + $existing = self::getSessionCache()->get( $key ); if ( is_array( $existing ) ) { $data += array_intersect_key( $existing, $keepKeys ); } @@ -217,7 +217,7 @@ if ( $data !== $existing || !isset( $data['expiry'] ) || $data['expiry'] < time() + 32100 ) { $data['expiry'] = time() + 86400; $stime = microtime( true ); - CentralAuthUtils::getSessionCache()->set( $key, $data, 86400 ); + self::getSessionCache()->set( $key, $data, 86400 ); $real = microtime( true ) - $stime; MediaWikiServices::getInstance() ->getStatsdDataFactory()->timing( 'centralauth.session.write', $real ); @@ -241,9 +241,9 @@ $id = $session->get( 'CentralAuth::centralSessionId' ); if ( $id !== null ) { - $key = CentralAuthUtils::memcKey( 'session', $id ); + $key = self::memcKey( 'session', $id ); $stime = microtime( true ); - CentralAuthUtils::getSessionCache()->delete( $key ); + self::getSessionCache()->delete( $key ); $real = microtime( true ) - $stime; MediaWikiServices::getInstance() ->getStatsdDataFactory()->timing( "centralauth.session.delete", $real ); diff --git a/phpcs.xml b/phpcs.xml index 25af500..457f48b 100644 --- a/phpcs.xml +++ b/phpcs.xml @@ -6,7 +6,7 @@ <exclude name="MediaWiki.Commenting.FunctionComment"/> </rule> <rule ref="Generic.Files.LineLength"> - <exclude-pattern>CentralAuth.*alias.php</exclude-pattern> + <exclude-pattern>CentralAuth\.*alias\.php</exclude-pattern> </rule> <rule ref="MediaWiki.NamingConventions.PrefixedGlobalFunctions"> <properties> @@ -16,6 +16,4 @@ <file>.</file> <arg name="extensions" value="php,php5,inc"/> <arg name="encoding" value="UTF-8"/> - <exclude-pattern>vendor</exclude-pattern> - <exclude-pattern>node_modules</exclude-pattern> </ruleset> -- To view, visit https://gerrit.wikimedia.org/r/367942 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36ab81e99ef5d3e01ffa045d47d95cc9c4ab50f6 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/CentralAuth Gerrit-Branch: master Gerrit-Owner: Umherirrender <umherirrender_de...@web.de> Gerrit-Reviewer: Jforrester <jforres...@wikimedia.org> Gerrit-Reviewer: Legoktm <lego...@member.fsf.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits