Last modified: 2011-10-15 22:03:14 UTC
The noicon parameter for audio files is broken in Mediawiki 1.18.
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)
Here: http://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28technical%29#.5B.5BFile:foo.ogg.7Cnoicon.5D.5D:_noicon_ignored
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.
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 ?
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...
Fixed on trunk in r99167.
tagging bugs for Marcus to look at
