Page MenuHomePhabricator
Create Task
Maniphest T66525

MediaViewer fails in Internet Explorer 11 with error about localStorage.getItem being null
Closed, ResolvedPublic
Actions

Description

Internet Explorer 11, not logged in:

  1. visit https://www.mediawiki.org/wiki/Help:Multimedia/Media_Viewer/de?uselang=de&debug=true
  2. click on first image on the right
  3. black lightbox comes up but then fails and closes
  4. right side automatic disappearing floating errorbox notifcation with:

Error loading MediaViewer: Die Eigenschaft "getItem" eines undefinierten oder Nullverweises kann nicht abgerufen werden. TypeError: Die Eigenschaft "getItem" eines undefinierten oder Nullverweises kann nicht abgerufen werden. at MetadataPanel (https://bits.wikimedia.org/static-1.24wmf2/extensions/MultimediaViewer/resources/mmv/ui/mmv.ui.metadataPanel.js:40:3) at init (https://bits.wikimedia.org/static-1.24wmf2/extensions/MultimediaViewer/resources/mmv/mmv.lightboxinterface.js:91:3) at LightboxInterface (https://bits.wikimedia.org/static-1.24wmf2/extensions/MultimediaViewer/resources/mmv/mmv.lightboxinterface.js:35:3) at MultimediaViewer (https://bits.wikimedia.org/static-1.24wmf2/extensions/MultimediaViewer/resources/mmv/mmv.js:112:3) at getViewer (https://bits.wikimedia.org/static-1.24wmf2/extensions/MultimediaViewer/resources/mmv/mmv.bootstrap.js:316:4) at isCSSReady (https://bits.wikimedia.org/static-1.24wmf2/extensions/MultimediaViewer/resources/mmv/mmv.bootstrap.js:106:5) at Anonymous function (https://bits.wikimedia.org/static-1.24wmf2/extensions/MultimediaViewer/resources/mmv/mmv.bootstrap.js:70:4) at fire (https://bits.wikimedia.org/www.mediawiki.org/load.php?debug=true&lang=de&modules=jquery%2Cmediawiki%2CSpinner%7Cjquery.triggerQueueCallback%2CloadingSpinner%2CmwEmbedUtil%7Cmw.MwEmbedSupport&only=scripts&skin=vector&version=20140424T184325Z:974:5) at fireWith (https://bits.wikimedia.org/www.mediawiki.org/load.php?debug=true&lang=de&modules=jquery%2Cmediawiki%2CSpinner%7Cjquery.triggerQueueCallback%2CloadingSpinner%2CmwEmbedUtil%7Cmw.MwEmbedSupport&only=scripts&skin=vector&version=20140424T184325Z:1084:7) at fire (https://bits.wikimedia.org/www.mediawiki.org/load.php?debug=true&lang=de&modules=jquery%2Cmediawiki%2CSpinner%7Cjquery.triggerQueueCallback%2CloadingSpinner%2CmwEmbedUtil%7Cmw.MwEmbedSupport&only=scripts&skin=vector&version=20140424T184325Z:1091:5)

  1. can be repeated by clicking on the thumb again

Expected Behaviour:
*) It loads and works or
*) Isn't loaded at all and old behaviour (if unsupported browser; see bug 63801 for degrade gracefully)

Version: unspecified
Severity: major

Details

Reference
bz64525

Related Objects

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:10 AM
• bzimport added projects: Browser-Support-Internet-Explorer, MediaViewer.
• bzimport set Reference to bz64525.
• bzimport added a subscriber: Unknown Object (MLST).
Se4598 created this task.Apr 28 2014, 6:04 AM
Se4598 added a comment.Apr 28 2014, 6:15 AM

could it be that there only is a "window." before "localStorage.getItem" missing on line 40?

"this.hasAnimatedMetadata = window.localStorage === undefined ||

			localStorage.getItem( 'mmv.hasOpenedMetadata' );"
Tgr added a comment.Apr 28 2014, 6:33 AM

That would be weird. If there is no localStorage variable in the current scope, the JS engine should walk up the scope chain until it reaches the window object. At any rate, opened bug 64527 for handling errors more gracefully.

Tgr added a comment.Apr 28 2014, 7:45 AM

Apparently when localStorage is disabled (about:config dom.storage.enabled in Firefox, don't know about IE), it is null, not undefined, so the test fails:
https://github.com/openstreetmap/iD/issues/591
We should just for truthiness instead.

Tracked as https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/521

Umherirrender added a comment.Apr 28 2014, 4:20 PM

Looks similar to gerrit 129245 (for CentralAuth)

gerritbot added a comment.Apr 28 2014, 10:27 PM

Change 130230 had a related patch set uploaded by Gergő Tisza:
Handle window.localStorage === null case correctly

https://gerrit.wikimedia.org/r/130230

gerritbot added a comment.Apr 29 2014, 7:10 AM

Change 130230 merged by jenkins-bot:
Handle window.localStorage === null case correctly.

https://gerrit.wikimedia.org/r/130230

gerritbot added a comment.Apr 29 2014, 9:15 PM

Change 130493 had a related patch set uploaded by Gergő Tisza:
Handle window.localStorage === null case correctly.

https://gerrit.wikimedia.org/r/130493

gerritbot added a comment.Apr 29 2014, 9:15 PM

Change 130494 had a related patch set uploaded by Gergő Tisza:
Handle window.localStorage === null case correctly.

https://gerrit.wikimedia.org/r/130494

gerritbot added a comment.Apr 29 2014, 11:23 PM

Change 130493 merged by jenkins-bot:
Handle window.localStorage === null case correctly.

https://gerrit.wikimedia.org/r/130493

gerritbot added a comment.Apr 29 2014, 11:32 PM

Change 130494 merged by jenkins-bot:
Handle window.localStorage === null case correctly.

https://gerrit.wikimedia.org/r/130494

• Gilles added a project: Multimedia.Dec 4 2014, 9:26 AM
• Gilles triaged this task as Unbreak Now! priority.Dec 4 2014, 10:11 AM
• Gilles moved this task from Untriaged to Done on the Multimedia board.
• Gilles lowered the priority of this task from Unbreak Now! to Needs Triage.Dec 4 2014, 11:23 AM
Tgr mentioned this in T66527: Click handler should be removed if MediaViewer fails to load.Dec 31 2014, 2:27 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL