Last modified: 2014-03-11 14:31:59 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T36853, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 34853 - mw.loader.load should (continue to) load other modules even if one of them doesn't make it
mw.loader.load should (continue to) load other modules even if one of them do...
Status: ASSIGNED
Product: MediaWiki
Classification: Unclassified
ResourceLoader (Other open bugs)
1.19
All All
: Normal normal (vote)
: Future release
Assigned To: Roan Kattouw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-01 14:21 UTC by Helder
Modified: 2014-03-11 14:31 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Helder 2012-03-01 14:21:46 UTC
After some reports of broken gadgets on Portuguese Wikipedia, I noticed that they were failing because one of the default gadgets was defined by
* Topicon[ResourceLoader|default|dependencies=jquery.mwPrototypes]|Topicon.js|Topicon.css
which worked fine on MW 1.18, and then stoped working on MW 1.19 because the module was renamed to "jquery.mwExtension" on r94227. This was fixed by updating the definition:
https://pt.wikipedia.org/w/index.php?title=MediaWiki%3AGadgets-definition&diff=29150779&oldid=29083316

But the point is: the unknown dependency was breaking not only that (default) script, but other gadgets as well.

You can check this by defining a dummy gadget by means of a line such as
* bug-test[ResourceLoader|dependencies=foo.bar]|bug-test.js
https://pt.wikibooks.org/w/index.php?title=MediaWiki%3AGadgets-definition&action=historysubmit&diff=233968&oldid=233722
and enable it together with some other gadget (e.g. the UTCLiveClock, which is working fine). There will be an error like this:
> Uncaught Error: Unknown dependency: foo.bar
and the other gadget won't work.
Comment 1 Krinkle 2012-03-02 22:30:51 UTC
Rephrasing bug and moving to MediaWiki core > ResourceLoader.

The gadgets (and other modules as well), are loaded from the html page with mw.loader.load(), passing it an array of module names. Among them gadget.Topicon and what not.

By default modules loaded in the same batch are related (e.g. a dependency resolution, or a call to mw.loader.using() ). But calls to mw.loader.load() shouldn't be treated like that.
Comment 2 Rob Lanphier 2012-03-14 17:06:03 UTC
Not a lot of gadgets on third party wikis, so I don't think this is going to be a major blocker for 1.19 tarball release.  Adjusting milestone.
Comment 3 Mark A. Hershberger 2012-09-30 18:09:12 UTC
I have seen this in 1.19, but, yes, not a huge issue. Bumping to nebulous future.
Comment 4 Andre Klapper 2014-03-11 14:31:59 UTC
Two years ago, Krinkle assigned this ticket to Roan and set the status to ASSIGNED.

Roan: Are you still working (or still plan to work) on this issue? In case you do not plan to work on this issue anymore, should the assignee be set back to default and the bug status changed from ASSIGNED to NEW/UNCONFIRMED? Thanks.

(In reply to Mark A. Hershberger from comment #3)
> not a huge issue.

-> Changing priority from high to normal.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links