The noicon parameter for audio files is broken in Mediawiki 1.18.
Version: 1.18.x
Severity: normal
Description
Description
Details
Details
- Reference
- bz31415
| Title | Reference | Author | Source Branch | Dest Branch | |
|---|---|---|---|---|---|
| Continue when errors happen, and send to stderr | repos/commtech/toolforge-docs!49 | samwilson | continue-on-errors | main |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Amire80 | T23778 BiDi issue in RTL wikis at site notice with LTR text ending with Unicode characters with neutral directional | |||
| Resolved | None | T31876 1.18 post-deployment actions (tracking) | |||
| Resolved | None | T33415 Noicon parameter for audio files is broken |
Event Timeline
Comment Actions
What is the noicon parameter? Could you provide a link to what's not working? (along with a clear description of what the expected behavior is)
Comment Actions
all the info there seems to have been hidden in a collapsed comment. :P
It seems the parameter |noicon in the code
[[File:Accordian chords-01.ogg|noicon|right]]
is no longer observed; vide:
noicon
The template {{Listen/core}} issues this parameter and ignoring it changes the appearance of {{Listen}}. What caused this change and how can it be reverted? -- Michael Bednarek (talk) 13:27, 6 October 2011 (UTC)
'noicon' seems to be a special magic word that OggHandler understands which can be used to disable the little 'info' icon that links to the file info page.
Handling still seems to be there, not sure offhand what would have broken it.
Comment Actions
hmm, remember that a while ago I activated input param validation that was already present, to actually do validation, somewhere in a few media handlers ? Perhaps this is one of the keywords that is not in the validator list ?
Comment Actions
Bingo, that looks suspicious -- in r87923 you tweaked OggHandler::validateParam to check width, height, and thumbtime properly (for bug 28886) but it's not checking anything for 'noicon'. Should be easy to add...