Fixes for ResourceLoaderWikiModule r72776. No serious bugs found, do not merge before...
authorTim Starling <tstarling@users.mediawiki.org>
Tue, 8 Feb 2011 06:34:38 +0000 (06:34 +0000)
committerTim Starling <tstarling@users.mediawiki.org>
Tue, 8 Feb 2011 06:34:38 +0000 (06:34 +0000)
commitfc6f65a85b4096f270c4b02feede86f66b509749
tree08977dbbebcbb117148960817bafc3008329734f
parent9bd321231b982ca20c0f6a59ae934b43d28eb69e
Fixes for ResourceLoaderWikiModule r72776. No serious bugs found, do not merge before deployment.
* Specify page titles as strings instead of split NS/DBK, as suggested by Roan on CR. It seemed sensible to me.
* Pass a Title object to getContent() instead of a string, to avoid unnecessary object construction overhead
* "*" and "/" are valid title characters. Check module input for JS comment end tokens.
* Fixed inappropriate conversion to boolean, when checking result of getContent(). Presumably the idea was to omit empty sections and errors, so that's what I did. Maybe an informative error message would be better in the error case.
* Use LinkBatch for selecting multiple page rows instead of Database::makeWhereFrom2d().
* Fixed assignment expression.
includes/resourceloader/ResourceLoaderSiteModule.php
includes/resourceloader/ResourceLoaderUserModule.php
includes/resourceloader/ResourceLoaderWikiModule.php