Multiple users have reported that since 10 May 2017, the cat-a-lot gadget fails to appear in their browsers. Appears to affect all platforms with no connection to an external update, say of java or browsers. Possibly something like a css change may have removed the relevant hook.
Description
Description
Details
Details
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | mmodell | T162954 MW-1.30.0-wmf.1 deployment blockers | |||
| Resolved | Krinkle | T165031 Gadgets that use both scripts and styles, but do not specify type=general, are never loaded (JS file not loaded but CSS file is) |
Event Timeline
Comment Actions
Hi @Fae, thanks for taking the time to report this! In general, please provide steps how to reproduce a problem.
I assume in this case:
- enable Cat-a-Lot on https://commons.wikimedia.org/wiki/Special:Preferences#mw-prefsection-gadgets
- go to a category page like https://commons.wikimedia.org/wiki/Category:Oakland_vehicles?debug=true
Expected result:
- See a yellow item in the lower right corner
Actual result:
- Nothing displayed in that corner
Some basic debugging:
Going to https://commons.wikimedia.org/wiki/Category:Oakland_vehicles?debug=true and opening the 'network' tab of the web browser's developer tools, there is a single entry about
https://commons.wikimedia.org/w/load.php?debug=true&lang=en&modules=ext.gadget.Cat-a-lot&only=styles&skin=vector
(loaded as type "stylesheet / css") but I don't see any entry for a Cat-a-lot file with "script / js".
Both a .js and a .css file are listed in https://commons.wikimedia.org/wiki/MediaWiki:Gadgets-definition#Categories
Also, the gadget "Cat-a-lot" styles are loaded twice and should be migrated to type=general (see https://www.mediawiki.org/wiki/RL/MGU#Gadget_type ); not sure if that is related though.
Comment Actions
@Aklapper This is not only cat-a-lot, Navigation popups also doesn't work. Wasn't MediaWiki on Commons updated yesterday? Anyway, without popups reverting vandalisms is a torture…
Comment Actions
The page source includes the following call for me, which is normally used for style-only modules to indicate that their CSS is loaded and should not be loaded again. For some reason 'ext.gadget.Cat-a-lot' is among them, and so it is not being loaded. This obviously makes no sense, and I have no idea what could be causing this.
I suspect setting it to 'type=general' should resolve the problem (since then it won't be loaded as style-only), but we might have other gadgets with this problem on other wikis. There's definitely a MediaWiki bug here.
mw.loader.state({"ext.gadget.Long-Image-Names-in-Categories":"ready","ext.gadget.Cat-a-lot":"ready","ext.globalCssJs.user.styles":"ready","ext.globalCssJs.site.styles":"ready","site.styles":"ready","noscript":"ready","user.styles":"ready","user":"ready","user.options":"loading","user.tokens":"loading","mediawiki.action.view.categoryPage.styles":"ready","ext.categoryTree.css":"ready","mediawiki.page.gallery.styles":"ready","mediawiki.helplink":"ready","wikibase.client.init":"ready","ext.visualEditor.desktopArticleTarget.noscript":"ready","ext.echo.styles.badge":"ready","ext.echo.badgeicons":"ready","ext.uls.pt":"ready","ext.wikimediaBadges":"ready","ext.tmh.thumbnail.styles":"ready","mediawiki.legacy.shared":"ready","mediawiki.legacy.commonPrint":"ready","mediawiki.sectionAnchor":"ready","mediawiki.skinning.interface":"ready","skins.vector.styles":"ready","ext.globalCssJs.user":"loading","ext.globalCssJs.site":"ready"});Comment Actions
I applied this change to fix those important scripts on Commons (Cat-a-lot and Navigation Popups): https://commons.wikimedia.org/w/index.php?title=MediaWiki:Gadgets-definition&diff=243659552&oldid=242815770
There is still a MediaWiki bug here and it should be resolved before this change is deployed further (otherwise we'll have the same problem with all Wikipedias in a couple hours).
Comment Actions
Change 353333 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[mediawiki/extensions/Gadgets@master] Revert "Move gadget styles from main stylesheet request to site request"
Comment Actions
Change 353335 had a related patch set uploaded (by Krinkle; owner: Legoktm):
[mediawiki/extensions/Gadgets@wmf/1.30.0-wmf.1] Revert "Move gadget styles from main stylesheet request to site request"
Comment Actions
Change 353333 merged by jenkins-bot:
[mediawiki/extensions/Gadgets@master] Revert "Move gadget styles from main stylesheet request to site request"
Comment Actions
This is a regression from 86905f8d78afaea2f5 because it added getGroup: 'site' which, I forgot, also has a special meaning in OutputPage::getRlClient. The special meaning being that it is assumed that if a module is loaded with addModuleStyles() and has group=site, it can safely be assumed as fully loaded thereafter.
This avoids bugs with styles loading twice, but also means that gadgets trying to do both don't get their JS parts loaded. This is wrong of course, and if anything, after T92459 is solved, the assumption will be made the other way around (gadgets containing both will be assumed 'general', they are not assumed 'styles' only, which didn't make sense).
The reason styles-only was wrongly assumed is because the Gadgets extension unconditionally loads the module styles, once with addModuleStyles, which itself is also a bug, but we're waiting for community migration to finish before we can fully resolve that.
Comment Actions
Change 353335 merged by jenkins-bot:
[mediawiki/extensions/Gadgets@wmf/1.30.0-wmf.1] Revert "Move gadget styles from main stylesheet request to site request"
Comment Actions
Mentioned in SAL (#wikimedia-operations) [2017-05-11T18:53:05Z] <thcipriani@tin> Synchronized php-1.30.0-wmf.1/extensions/Gadgets/includes/GadgetResourceLoaderModule.php: SWAT: [[gerrit:353335|Revert "Move gadget styles from main stylesheet request to site request"]] T165040 T165031 (duration: 00m 42s)