https://www.mediawiki.org/wiki/Special:LintErrors/bogus-image-options
These parameters are added by TimedMediaHandler. Maybe related to the a/v work @Arlolra has been doing?
https://www.mediawiki.org/wiki/Special:LintErrors/bogus-image-options
These parameters are added by TimedMediaHandler. Maybe related to the a/v work @Arlolra has been doing?
Ya, I expect this will be fixed as part of the A/V patches -- but the deploy my not happen till sometime next month. So, for now, if you want, you could add a hardcoded hack to exclude these from linter reports.
Change 343080 had a related patch set uploaded (by Subramanya Sastry):
[mediawiki/services/parsoid] T160599: Ignore thumbtime linter errors temporarily
Change 343080 merged by jenkins-bot:
[mediawiki/services/parsoid] T160599: Ignore thumbtime linter errors temporarily
Mentioned in SAL (#wikimedia-operations) [2017-03-27T20:29:26Z] <arlolra> Updated Parsoid to 6eaad376 (T160599, T161178, T133267)
Ya, I expect this will be fixed as part of the A/V patches
It will.
Meanwhile, we just deployed this patch but I'm not sure it's working.
I made this page,
https://test.wikipedia.org/w/index.php?title=LintVid&type=revision&diff=311890&oldid=311889
and it's listed in the bogus options,
https://test.wikipedia.org/wiki/Special:LintErrors/bogus-image-options
@ssastry did you test that patch?
mysql:wikiadmin@db1078 [testwiki]> select * from linter where linter_id=418; +-----------+-------------+------------+--------------+------------+----------------+ | linter_id | linter_page | linter_cat | linter_start | linter_end | linter_params | +-----------+-------------+------------+--------------+------------+----------------+ | 418 | 94868 | 3 | 0 | 41 | {"items":[""]} | +-----------+-------------+------------+--------------+------------+----------------+ 1 row in set (0.00 sec)
That's weird...
The wikitext is [[File:Hernia de hiato.ogv||thumbtime=5]], which has ||. So, this was just my typo.
Hmm .. https://wikimania2017.wikimedia.org/wiki/Special:LintErrors/bogus-image-options
[[File:Wikipedia Science Conference 2015 summary video.webm|frameless|right|thumbtime=2:22|Summary video of the [https://wikimedia.org.uk/wiki/Wikipedia_Science_Conference Wikipedia Science Conference 2015] that brought researchers and Wikimedians together]]
That wikitext looks right to me.
start=1:40, end=2:05 valid as well .. but https://nl.wikipedia.org/w/index.php?title=Grutto&action=edit&lintid=3849 ..
Change 348106 had a related patch set uploaded (by Subramanya Sastry):
[mediawiki/services/parsoid@master] Tweak linter regexp for handling more T160599 scenarios
Change 348106 abandoned by Arlolra:
Tweak linter regexp for handling more T160599 scenarios
Reason:
In favour of https://gerrit.wikimedia.org/r/#/c/335956/