Page MenuHomePhabricator

Diff of image pages don't have js execute properly even though needed for file history
Closed, ResolvedPublic

Description

Example: https://commons.wikimedia.org/w/index.php?title=File:Highly-Sensitive-Quantitative-Imaging-for-Monitoring-Single-Cancer-Cell-Growth-Kinetics-and-Drug-pone.0089000.s004.ogv&oldid=prev&diff=117375667

View a diff on an image page for an ogv or webm. Go to the file history section. Play button is too far right, and clicking links to video instead of playing it.

Probably some of TMH JS (but not all) is not being loaded properly


Version: unspecified
Severity: normal

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:55 AM
bzimport set Reference to bz61923.
Gilles added a subscriber: Gilles.

https://gerrit.wikimedia.org/r/#/c/172421/ and friends might help with this. I haven't tested yet, but i'll put it on my list.

It seems that an image page is not copying it's modules into the destination page. An imageoutput is not just parseroutput of course, but also the file, the file history etc...., Categories and File pages are special in that way.... And that is why the player is not loading.

Actually, only mw.PopUpMediaTransform seems to be missing. Which is strange, since all of that is added at the same time... Perhaps it's a parser cache issue with old pages ?

Ah no, that's on the 'old version'. The new version (as on beta labs) seems to have none of the modules in this case. Probably because the main file is missing, and the history table is calling transform directly.

TheDJ raised the priority of this task from Low to High.Oct 26 2015, 11:22 AM

Seems this has actually become worse now that T74675 was closed.

Change 248998 had a related patch set uploaded (by TheDJ):
Add module dependencies for files in the file history

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

Change 248998 merged by jenkins-bot:
Add module dependencies for files in the file history

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

TheDJ claimed this task.
TheDJ removed a project: Patch-For-Review.
TheDJ set Security to None.