Page MenuHomePhabricator

MultimediaViewer isn't opening when using Parsoid for Read Views
Closed, ResolvedPublic

Description

The issue seems to be that $pageHasThumbnails = count( $out->getFileSearchOptions() ) > 0; is always false. Parsoid needs to call ContentMetadataCollector::addImage when parsing.

Event Timeline

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

[mediawiki/services/parsoid@master] [WIP] Call ContentMetadataCollector::addImage

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

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

[mediawiki/core@master] Return additional info about files in DataAccess

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

Change 1005627 merged by jenkins-bot:

[mediawiki/core@master] Return additional info about files in DataAccess

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

Arlolra triaged this task as Medium priority.Feb 22 2024, 6:06 PM
Arlolra moved this task from Backlog to Code Review on the Content-Transform-Team-WIP board.

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

[mediawiki/core@master] Support "showmedia" parsertests option

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

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

[mediawiki/services/parsoid@master] Support "showmedia" parsertests option

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

Change 1005628 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Populate image metadata info in parser output

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

Change 1006201 had a related patch set uploaded (by Sbailey; author: Sbailey):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.19.0-a19

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

Change 1006201 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.19.0-a19

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

Arlolra moved this task from To Deploy to To Verify on the Content-Transform-Team-WIP board.

Post-deploy to group0 today, MultimediaViewer is opening on officewiki pages. It sometimes requires a purge.

Change 1005836 merged by jenkins-bot:

[mediawiki/core@master] Support "showmedia" parsertests option

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

Change 1005837 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Support "showmedia" parsertests option

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

Change 1008400 had a related patch set uploaded (by Isabelle Hurbain-Palatin; author: Isabelle Hurbain-Palatin):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.19.0-a20

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

Change 1008400 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.19.0-a20

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