Last modified: 2012-05-03 02:42:44 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 T36147, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 34147 - importScript, importScriptURI, and addPortletLink errors
importScript, importScriptURI, and addPortletLink errors
Status: RESOLVED WORKSFORME
Product: MediaWiki
Classification: Unclassified
ResourceLoader (Other open bugs)
1.20.x
All All
: High normal with 1 vote (vote)
: 1.19.0 release
Assigned To: Roan Kattouw
:
Depends on:
Blocks: 31217
  Show dependency treegraph
 
Reported: 2012-02-02 01:55 UTC by Mark A. Hershberger
Modified: 2012-05-03 02:42 UTC (History)
8 users (show)

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


Attachments

Description Mark A. Hershberger 2012-02-02 01:55:25 UTC
While testing gadgets tonight on commonswiki.beta I tested enabling all gadgets and visiting <http://commons.wikimedia.beta.wmflabs.org/wiki/File:20080406165033!V-22_Osprey_refueling_edit1.jpg?debug=true>.

I had to disable some gadgets that referenced "addPortletLink" but still ended up with importScript and importScriptURI errors.  I thought these calls were supposed to be part of core, so I'm reporting them here:

  importScript is not defined
  http://commons.wikimedia.beta.wmflabs.org/w/index.php?title=MediaWiki:Gadget-
  GalleryDetails.js&action=raw&ctype=text/javascript&26218
  Line 21

  importScriptURI is not defined
  http://commons.wikimedia.beta.wmflabs.org/w/index.php?title=MediaWiki:Gadget-
  RegexMenuFramework.js&action=raw&ctype=text/javascript&26217
  Line 9
Comment 1 Mark A. Hershberger 2012-02-02 01:56:06 UTC
adding Saibo so he knows what I found.
Comment 2 Bawolff (Brian Wolff) 2012-02-02 16:00:38 UTC
>I had to disable some gadgets that referenced "addPortletLink"

Surely addPortletLink should also (Still) be in core?
Comment 3 Krinkle 2012-02-02 19:44:35 UTC
All legacy wikibits functions are still in core and preloaded from a blocking state on all pages. If they are unavailable at any point in execution, that is a bug. 

This also includes the various import*  functions and addPortletLink.
Comment 4 Mark A. Hershberger 2012-02-02 20:53:04 UTC
Assigning to krinkle since he has the most clue about this, AFAIK
Comment 5 Krinkle 2012-02-07 18:20:57 UTC
Marking INVALID. These are part of legacy wikibits, available by default unconditionally. If they are not working then there is something heavily broken on beta.
Comment 6 Mark A. Hershberger 2012-02-07 18:42:35 UTC
I want to get this confirmed (and, hopefully, fixed) as a beta problem before we deploy the code.
Comment 7 Trevor Parscal 2012-02-07 18:45:55 UTC
I also can not reproduce and suspect it's been fixed between the time it was reported and now.
Comment 8 Krinkle 2012-02-07 18:47:25 UTC
Fixed by r110592 and related revisions. The "blocking" top queue wasn't blocking properly in debug mode.
Comment 10 Saibo 2012-02-08 02:04:24 UTC
Still got two errors when accessing http://commons.wikimedia.beta.wmflabs.org/wiki/User:Saibo (logged in with "my" gadgets enabled")

Fehler: addPortletLink is not defined
Quelldatei: http://commons.wikimedia.beta.wmflabs.org/w/index.php?title=MediaWiki:Gadget-Geotoolbox.js&action=raw&ctype=text/javascript&1601
Zeile: 27

Fehler: importScript is not defined
Quelldatei: http://commons.wikimedia.beta.wmflabs.org/w/index.php?title=MediaWiki:Gadget-GalleryDetails.js&action=raw&ctype=text/javascript&26324
Zeile: 21
Comment 11 Mark A. Hershberger 2012-02-08 16:22:03 UTC
yep, still seeing errors on commons.
Comment 12 Roan Kattouw 2012-02-08 16:49:32 UTC
(In reply to comment #10)
> Still got two errors when accessing
> http://commons.wikimedia.beta.wmflabs.org/wiki/User:Saibo (logged in with "my"
> gadgets enabled")
> 
Is this still the case? I created an account, enabled Geotoolbox and GalleryDetails, then visited User:Saibo, but got no errors.
Comment 13 Helder 2012-02-08 17:01:53 UTC
(In reply to comment #12)
> (In reply to comment #10)
> > Still got two errors when accessing
> > http://commons.wikimedia.beta.wmflabs.org/wiki/User:Saibo (logged in with "my"
> > gadgets enabled")
> > 
> Is this still the case? I created an account, enabled Geotoolbox and
> GalleryDetails, then visited User:Saibo, but got no errors.

Yep! I enabled the same two gadgets (keeping also enabled the default ones) and then opened
http://commons.wikimedia.beta.wmflabs.org/wiki/Special:MyPage
When I SHIFT+Reloaded the page, the errors happened.
Comment 14 Roan Kattouw 2012-02-08 17:02:32 UTC
Never mind, I've got it reproduced now, just needed to visit the main page instead.
Comment 15 Roan Kattouw 2012-02-08 19:02:57 UTC
I messed around a bit with introducing debugging code into mediawiki.js and removing it again, and the issue seems to have magically gone away now. Maybe it's due to my purging the startup module cache a few times.

I can't reproduce this anymore, can anyone else?
Comment 16 Mark A. Hershberger 2012-02-08 19:49:58 UTC
(In reply to comment #15)
> I can't reproduce this anymore, can anyone else?

Looks fixed to me, too.
Comment 17 Saibo 2012-02-09 17:07:02 UTC
fails completely for me... On every page:

Fehler: addPortletLink is not defined
Quelldatei: http://commons.wikimedia.beta.wmflabs.org/w/index.php?title=MediaWiki:Gadget-Geotoolbox.js&action=raw&ctype=text/javascript&1601
Zeile: 27

Fehler: importScript is not defined
Quelldatei: http://commons.wikimedia.beta.wmflabs.org/w/load.php?debug=false&lang=de&modules=user&only=scripts&skin=monobook&user=Saibo&version=20120131T165649Z&*
Zeile: 1

Fehler: importScript is not defined
Quelldatei: http://commons.wikimedia.beta.wmflabs.org/w/index.php?title=MediaWiki:Gadget-GalleryDetails.js&action=raw&ctype=text/javascript&26324
Zeile: 21

Apparently no gadgets are loaded now due to that bugs.
Comment 18 Mark A. Hershberger 2012-02-09 18:56:03 UTC
In looking at this bug yesterday (and similar RL issues on the beta cluster), I think this showed a problem with caching.  Whether it was a RL issue or a beta configuration issue, I don't know, but I think this is more of a symptom than the actual problem.
Comment 19 Mark A. Hershberger 2012-02-09 19:17:32 UTC
Changing priority and assignment since Krinkle is only available half time and we will make sure to test this thoroughly on test2.
Comment 20 Rob Lanphier 2012-02-16 03:06:07 UTC
I'm assuming based on our conversation earlier today that this one won't repro in production

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


Navigation
Links