From: Timo Tijhof Date: Tue, 22 May 2018 17:58:04 +0000 (+0100) Subject: resourceloader: Move style deprecation warnings to X-Git-Tag: 1.34.0-rc.0~5316^2 X-Git-Url: https://git.cyclocoop.org/%27%20.%20%24this-%3EgetSkin%28%29-%3EescapeSearchLink%28%29%20.%20%27?a=commitdiff_plain;h=2db395abfc5e6fc4035acc359a1fb6e62fad61fb;p=lhc%2Fweb%2Fwiklou.git resourceloader: Move style deprecation warnings to Follows-up If35a106c7. These log messages are not criticial and should not be in the competing with stylesheets and article content. Move them to the end of instead, nearby other low-priority script tags. The getBodyHtml() method from ClientHtml was empty, but has been non-empty in the past. It's fine to repopulate. Also, while ClientHtml::getBodyHtml was empty, there are additional RLQ scripts created by OutputPage that do exist even without this. Namely, there is a ' . "\n" . '' . "\n" . '' . "\n" @@ -304,18 +303,23 @@ Deprecation message.' ] $context = self::makeContext(); $context->getResourceLoader()->register( self::makeSampleModules() ); - $client = new ResourceLoaderClientHtml( $context ); + $client = new ResourceLoaderClientHtml( $context, [ 'nonce' => false ] ); $client->setConfig( [ 'key' => 'value' ] ); $client->setModules( [ 'test', 'test.private.bottom', ] ); + $client->setModuleStyles( [ + 'test.styles.deprecated', + ] ); $client->setModuleScripts( [ 'test.scripts', ] ); - - $expected = ''; - $expected = self::expandVariables( $expected ); + // phpcs:disable Generic.Files.LineLength + $expected = ''; + // phpcs:enable $this->assertEquals( $expected, $client->getBodyHtml() ); }