Page MenuHomePhabricator

Centralize URL generation in a single component
Closed, ResolvedPublic

Description

Migrated from: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/373

mmv.js, mmv.ui.reuse.share.js andmmv.ui.reuse.embed.js each need to generate MultimediaViewer-enabled URLs (probably the download tab will too). There should be a single small class focused on that task which can be shared by all components.

Event Timeline

MingleTerminator raised the priority of this task from to Low.Dec 8 2014, 4:58 PM
MingleTerminator added a project: Multimedia.
In mingle on 2014-03-19 at 19:29:57, @Tgr wrote:

See also #371 , #372 .

Mass-removing the Multimedia tag from MediaViewer tasks, as this is now being worked on by the Reading department, not Editing's Multimedia team.

matmarex subscribed.

Looks like this was resolved back in 2014, together with T77300, by https://gerrit.wikimedia.org/r/c/mediawiki/extensions/MultimediaViewer/+/125932. The code has changed since then, but it's still centralized, in 'mmv.head' #getMediaHash method: https://gerrit.wikimedia.org/g/mediawiki/extensions/MultimediaViewer/+/0683f3ecc8a502a0fb0759ac2fe3393126372f48/resources/mmv.head/base.js#46