tfwiki / mediawiki

The MediaWiki code powering the Team Fortress Wiki

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

TypeError: Object [object Object] has no method 'lightBox'

rjackson opened this issue · comments

https://sentry.io/team-fortress-wiki/tfwiki-prod/issues/440812322/

TypeError: Object [object Object] has no method 'lightBox'
  at Function.jQuery.extend.each (/w/load.php:5:235)
  at jQuery.fn.jQuery.each (/w/load.php:2:222)
  at fire (/w/load.php:45:122)
  at Object.self.add [as done] (/w/load.php:45:656)
  at jQuery.fn.ready (/w/load.php:49:63)
...
(1 additional frame(s) were not displayed)

Object [object Object] has no method 'lightBox'

Source: https://wiki.teamfortress.com/wiki/MediaWiki:Common.js

Code beginning // Lightbox stuff

Looks like a case of our MediaWiki:Common.js code firing before MediaWiki's loaded its module's code.

The sole purpose of this code is to apply TF2-style branding to the lightbox, so its non-essential. If we can't stop it clogging our error logs, lets drop it.

Correction: This adds the lightbox functionality to all images... which GeeQuBox should be doing, but isn't. 🤔

Regardless, removed for now. Will re-evaluate once Slimbox is in-place.

Slimbox is also old and terrible.

Wikimedia now use this, which looks pretty solid: https://www.mediawiki.org/wiki/Extension:MultimediaViewer

Replaced GeeQuBox with MultimediaViewer in b48ec88. Wiki should be happier now.