"Catrope" changed the status of MediaWiki.r109254 to "fixme" and commented it. URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/109254#c30047
Old Status: new > New Status: fixme Commit summary for MediaWiki.r109254: AFT5 - short circuit gender thing for the time being, so we can test the layout. Known issue, will fix Catrope's comment: <pre> + .Html::element( 'span', array( + 'class' => $class, + ), wfMessage( $msg, '')->escaped() ) </pre> This'll double-escape the message, because <code>element()</code> already escapes its third argument. OK otherwise. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
