jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/347886 )
Change subject: formatter: Change log channel of infobox message ...................................................................... formatter: Change log channel of infobox message By default, the minimum log level for Logstash is INFO [0]. The minimum log level for the "mobile" channel, however, is DEBUG [1]. [0] https://github.com/wikimedia/operations-mediawiki-config/blob/8342bcbbd040bab1cd3b140b47a58f7ee08d1a46/wmf-config/InitialiseSettings.php#L4795 [1] https://github.com/wikimedia/operations-mediawiki-config/blob/8342bcbbd040bab1cd3b140b47a58f7ee08d1a46/wmf-config/InitialiseSettings.php#L4868 Bug: T149884 Change-Id: I61d20ea8dcc8cd63e826f56567302248b3c92f61 (cherry picked from commit 9da62ce9bb015427e3043f1aed6980db981a00d1) --- M includes/MobileFormatter.php M tests/phpunit/MobileFormatterTest.php 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Thcipriani: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/MobileFormatter.php b/includes/MobileFormatter.php index 9a88fbb..0686414 100644 --- a/includes/MobileFormatter.php +++ b/includes/MobileFormatter.php @@ -265,7 +265,7 @@ private function logInfoboxesWrappedInContainers( $leadSectionBody, DOMXPath $xPath ) { $infoboxes = $xPath->query( './/table[contains(@class,"infobox")]', $leadSectionBody ); if ( $infoboxes->length > 0 ) { - \MediaWiki\Logger\LoggerFactory::getInstance( 'MobileFrontend' )->debug( + \MediaWiki\Logger\LoggerFactory::getInstance( 'mobile' )->debug( "Found infobox wrapped with container on {$this->title} (rev:{$this->revId})" ); } diff --git a/tests/phpunit/MobileFormatterTest.php b/tests/phpunit/MobileFormatterTest.php index 05773fc..1e2973c 100644 --- a/tests/phpunit/MobileFormatterTest.php +++ b/tests/phpunit/MobileFormatterTest.php @@ -846,7 +846,7 @@ $this->assertContains( '0', $message ); } ) ); - $this->setLogger( 'MobileFrontend', $loggerMock ); + $this->setLogger( 'mobile', $loggerMock ); $formatter->filterContent( false, false, false, true ); } } -- To view, visit https://gerrit.wikimedia.org/r/347886 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I61d20ea8dcc8cd63e826f56567302248b3c92f61 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/MobileFrontend Gerrit-Branch: wmf/1.29.0-wmf.20 Gerrit-Owner: Phuedx <[email protected]> Gerrit-Reviewer: Thcipriani <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
