Last modified: 2010-05-15 14:36:20 UTC
The headscripts variable that Tim Starling added ( http://mail.wikipedia.org/pipermail/mediawiki-cvs/2005-August/010697.html ) is a very useful addition for a plugin I'm working on. However, it doesn't appear to be hooked up in MonoBook (other skins seem to inherit from SkinTemplate.php just fine). This one line patch fixes that. Patch forthcoming. In essense, just adding <?php $this->html('headscripts') ?> to MonoBook.php will do the trick. Hopefully for 1.5 release.
Created attachment 937 [details] Add "headscripts" variable to monobook
Could this please get put into the release branch of MediaWiki? For developers that need to include Javascript as part of extensions and special pages, it is frustrating to release products that force users to modify the MediaWiki code base. The whole idea of the 'headscripts' variable is to allow developers to just use $wgOut->addScript($foo) without having to get into the nitty-gritty of editing skin files. The very presence of the headscript variable and related functions proves that this should be enabled in the skins. It's absence is obviously just an oversight.
Fixed in CVS HEAD. Will check about backporting to 1.5 branch.
are you planning to back port this in the short term? i need it for a 1.5 extension i'm writing - if you're too busy i'll do it myself.
Not until I can be bothered to check, so probably quicker to do it yourself. ;-)