https://bugzilla.wikimedia.org/show_bug.cgi?id=27321

--- Comment #4 from Brion Vibber <br...@pobox.com> 2011-02-12 01:37:29 UTC ---
Created attachment 8130
  --> https://bugzilla.wikimedia.org/attachment.cgi?id=8130
Provisional fix to mediawiki.load

This fixes it for me, but I'm not sure it's the correct fix.

In debug mode, the loader chunks get replaced with attempts to load the
specific URLs; however only module "names" beginning with 'http:' or 'https:'
actually get loaded this way. Since we were receiving local urls like this:

  mediaWiki.loader.implement("mediawiki.legacy.mwsuggest", function( $, mw ) {
    mediaWiki.loader.load("/trunk/skins/common/mwsuggest.js");

they were getting added to a modules list and then just never work.

Checking also for module "names" that start with "/" and loading them as local
URLs appears to work, but I'm not sure if it's wise. Should full URLs be used
instead?

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to