Page MenuHomePhabricator

Timed media handler transcode status often has weird status messages
Closed, ResolvedPublic

Description

See eg. https://commons.wikimedia.org/wiki/File:5-1-15-_White_House_Press_Briefing.webm

Added to Job queue [INVALID] ago
Started [INVALID] ago. comma

Event Timeline

zhuyifei1999 added a project: Commons.

(Self-assigning because I think I can figure this out)

Apparently the patch for the above task fixed the usage of wfMessage( 'timedmedia-in-job-queue' )->params( Message::listParam( [ '1 second' ], 'comma' ) ), but not wfMessage( 'timedmedia-in-job-queue', Message::listParam( [ '1 second' ], 'comma' ) ), is this intended?

Change 328366 had a related patch set uploaded (by Zhuyifei1999):
wfMessage: use Message::params() to handle all the message parameters

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

Looks like wfMessage() has the same bug as in T152603: Invalid parameter for message "api-help-permissions-granted-to".

The easiest fix would be to change wfMessage( $key, ... ) to be the same as ( new Message( $key ) )->params( ... ) instead of trying to duplicate the handling that ->params() already does in order to call new Message( $key, $params ).

Edit: Ha, you beat me to that realization.

Edit: Ha, you beat me to that realization.

lol

Change 328366 merged by jenkins-bot:
wfMessage: use Message::params() to handle all the message parameters

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

matmarex removed a project: Patch-For-Review.
matmarex added a subscriber: matmarex.

This will be deployed to Wikimedia wikis the week of January 2nd, there are no deployments until then due to holidays. I don't think it's a major issue, so it can wait.