Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Preload ResourceLoaderWikiModule::getTitleInfo in OutputPage

Browse source 
This avoids a separate query for each module.

Bug: T46362
Change-Id: Ie109a8776cbdcd5928cbb59351f2cf94088c0c95
This commit is contained in:
Aaron Schulz 2016-09-08 01:38:26 -07:00 committed by Timo Tijhof
parent a8c44660c1
commit 6f8dc27ca2
1 changed files with 15 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
includes/OutputPage.php
View file

@ -2679,16 +2679,29 @@ class OutputPage extends ContextSource {
// Prepare exempt modules for buildExemptModules()
$exemptGroups = [ 'site' => [], 'noscript' => [], 'private' => [], 'user' => [] ];
$exemptStates = [];
$moduleStyles = array_filter( $this->getModuleStyles( /*filter*/ true ),
$moduleStyles = $this->getModuleStyles( /*filter*/ true );
// Batch preload getTitleInfo for isKnownEmpty() calls below
$exemptModules = array_filter( $moduleStyles,
function ( $name ) use ( $rl, &$exemptGroups ) {
$module = $rl->getModule( $name );
return $module && isset( $exemptGroups[ $module->getGroup() ] );
}
);
ResourceLoaderWikiModule::preloadTitleInfo(
$context, wfGetDB( DB_REPLICA ), $exemptModules );
// Filter out modules handled by buildExemptModules()
$moduleStyles = array_filter( $moduleStyles,
function ( $name ) use ( $rl, $context, &$exemptGroups, &$exemptStates ) {
$module = $rl->getModule( $name );
if ( $module ) {
$group = $module->getGroup();
if ( $name === 'user.styles' && $this->isUserCssPreview() ) {
$exemptStates[$name] = 'ready';
// Special case in buildExemptModules()
return false;
}
$group = $module->getGroup();
if ( isset( $exemptGroups[$group] ) ) {
$exemptStates[$name] = 'ready';
if ( !$module->isKnownEmpty( $context ) ) {