Page MenuHomePhabricator

Thumbtime for TimedMedia no longer working
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Choose any video file, for example: Ikusgela - Matsuo Bashō.webm
  • Set up the desired thumbtime, for example: 16
  • The wikicode should be [[File:Ikusgela - Matsuo Bashō.webm|960px|center|thumbtime=16|Bideoa]]

What happens?: The thumb doesn't match the thumbtime. Changing the number gives the same thumbnail, so it seems that is not respecting the thumbtime anymore.

What should have happened instead?: The thumb should change accordingly.

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

Event Timeline

Aklapper renamed this task from Thumtime for TimedMedia no longer working to Thumbtime for TimedMedia no longer working.Apr 14 2026, 11:43 AM
Aklapper edited projects, added: Regression; removed: Thumbor.
Aklapper updated the task description. (Show Details)
TheDJ subscribed.

Most limely broke due to the recent thumbnail standardization

I'm trying to see what's going on. https://test.wikipedia.org/w/index.php?title=Ma%E2%80%8Bin_Page&oldid=737973 looks correct. The URL is this //upload.wikimedia.org/wikipedia/commons/thumb/6/66/Ikusgela_-_Matsuo_Bash%C5%8D.webm/960px-seek%3D16-Ikusgela_-_Matsuo_Bash%C5%8D.webm.jpg?utm_source=test.wikipedia.org&utm_campaign=parser which includes seek and the 16 in it as well.

grafik.png (959×646 px, 535 KB)

I changed it to 30s and it showed a different thumbnail.

I'm looking at the patches we made for thumb steps in TMH and I really can't see how https://gerrit.wikimedia.org/r/c/mediawiki/extensions/TimedMediaHandler/+/1265416/3/includes/TimedMediaHandler.php could cause thumbtime to stop working. It's from somewhere else.

Maybe it was broken in the previous branch which is deployed to euwiki but now got fixed in the new version which is deployed on testwiki?

I observed the problem with thumbtime too, but only with Parsoid.

Example: the migration tool for https://de.wikipedia.org/wiki/Benutzer:Kallichore/Parsoid_thumbtime shows a starting rocket on the left (t=36) for the legacy parser, while Parsoid shows the default thumb on the right.

@Ladsgroup https://test.wikipedia.org/wiki/Ma%E2%80%8Bin_Page?useparsoid=1 shows the problem. So the problem is clearly not fixed on testwiki.

A comparison of the thumb URLs shows that "seek%3D36" is missing when using Parsoid:

Legacy parser : https://upload.wikimedia.org/wikipedia/commons/thumb/d/d7/STS-114-launch-Jul26-2005.ogv/250px-seek%3D36-STS-114-launch-Jul26-2005.ogv.jpg

Parsoid : https://upload.wikimedia.org/wikipedia/commons/thumb/d/d7/STS-114-launch-Jul26-2005.ogv/250px--STS-114-launch-Jul26-2005.ogv.jpg

I seems that Parsoid does not pass the thumbtime parameter to the thumbnail URL. I wonder if this is a regression or if it was never implemented for Parsoid.

A comparison of the thumb URLs shows that "seek%3D36" is missing when using Parsoid:

Legacy parser : https://upload.wikimedia.org/wikipedia/commons/thumb/d/d7/STS-114-launch-Jul26-2005.ogv/250px-seek%3D36-STS-114-launch-Jul26-2005.ogv.jpg

Parsoid : https://upload.wikimedia.org/wikipedia/commons/thumb/d/d7/STS-114-launch-Jul26-2005.ogv/250px--STS-114-launch-Jul26-2005.ogv.jpg

I seems that Parsoid does not pass the thumbtime parameter to the thumbnail URL. I wonder if this is a regression or if it was never implemented for Parsoid.

It seems a regression as the parserTests testcases of TimedMediaHandler do show seek as a thumb url parameter for parsoid.

Change #1278540 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/core@master] [parsoid] Cast thumbtime to a string

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

Change #1278544 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/services/parsoid@master] Cast thumbtime to a string

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

Change #1278540 merged by jenkins-bot:

[mediawiki/core@master] [parsoid] Cast thumbtime to a string

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

Change #1278629 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/extensions/TimedMediaHandler@master] Unbreak passing seek to ApiQueryVideoInfo

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

Change #1278544 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Cast thumbtime to a string

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

Change #1278688 had a related patch set uploaded (by Zabe; author: Arlolra):

[mediawiki/core@REL1_46] [parsoid] Cast thumbtime to a string

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

Change #1278688 merged by jenkins-bot:

[mediawiki/core@REL1_46] [parsoid] Cast thumbtime to a string

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

Change #1278629 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Unbreak passing seek to ApiQueryVideoInfo

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

Change #1279374 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/extensions/TimedMediaHandler@REL1_46] Unbreak passing seek to ApiQueryVideoInfo

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

Change #1279374 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@REL1_46] Unbreak passing seek to ApiQueryVideoInfo

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

Change #1284813 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.24.0-a2

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

Change #1284813 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.24.0-a2

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

Change #1284832 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):

[mediawiki/vendor@wmf/1.47.0-wmf.1] Bump wikimedia/parsoid to 0.24.0-a2

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

Change #1284832 merged by jenkins-bot:

[mediawiki/vendor@wmf/1.47.0-wmf.1] Bump wikimedia/parsoid to 0.24.0-a2

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

Mentioned in SAL (#wikimedia-operations) [2026-05-07T21:15:34Z] <cscott@deploy1003> Started scap sync-world: Backport for [[gerrit:1284828|Upgrading webonyx/graphql-php (v15.31.5 => v15.32.3)]], [[gerrit:1284834|composer.json: Update webonyx/graphql-php to ^15.32.3]], [[gerrit:1284832|Bump wikimedia/parsoid to 0.24.0-a2 (T319058 T368724 T373384 T420336 T423241 T423701 T424446 T424773 T425008 T425056 T425107 T425731)]], [[gerrit:1284837|Bump wikimedia/parsoid to 0.24.0-a2 (T425731)]

Mentioned in SAL (#wikimedia-operations) [2026-05-07T21:17:28Z] <cscott@deploy1003> cscott: Backport for [[gerrit:1284828|Upgrading webonyx/graphql-php (v15.31.5 => v15.32.3)]], [[gerrit:1284834|composer.json: Update webonyx/graphql-php to ^15.32.3]], [[gerrit:1284832|Bump wikimedia/parsoid to 0.24.0-a2 (T319058 T368724 T373384 T420336 T423241 T423701 T424446 T424773 T425008 T425056 T425107 T425731)]], [[gerrit:1284837|Bump wikimedia/parsoid to 0.24.0-a2 (T425731)]] synced to the t

Mentioned in SAL (#wikimedia-operations) [2026-05-07T21:27:35Z] <cscott@deploy1003> Finished scap sync-world: Backport for [[gerrit:1284828|Upgrading webonyx/graphql-php (v15.31.5 => v15.32.3)]], [[gerrit:1284834|composer.json: Update webonyx/graphql-php to ^15.32.3]], [[gerrit:1284832|Bump wikimedia/parsoid to 0.24.0-a2 (T319058 T368724 T373384 T420336 T423241 T423701 T424446 T424773 T425008 T425056 T425107 T425731)]], [[gerrit:1284837|Bump wikimedia/parsoid to 0.24.0-a2 (T425731)