r38260 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r38259‎ | r38260 | r38261 >
Date:21:43, 30 July 2008
Author:brion
Status:old
Tags:
Comment:
Revert r38257 "should use htmlspecialchars here" -- this *removes* a call to htmlspecialchars(), the opposite of what is described. WTF?
Modified paths:
  • /trunk/extensions/CSS/CSS.php (modified) (history)

Diff [purge]

Index: trunk/extensions/CSS/CSS.php
@@ -13,7 +13,7 @@
1414
1515 if (!defined('MEDIAWIKI')) die('Not an entry point.');
1616
17 -define('CSS_VERSION', '1.0.5, 2008-07-31');
 17+define('CSS_VERSION', '1.0.4, 2008-06-01');
1818
1919 $wgCSSMagic = "css";
2020 $wgExtensionFunctions[] = 'wfSetupCSS';
@@ -41,7 +41,7 @@
4242 if (ereg('\\{', $css)) {
4343
4444 # Inline CSS
45 - $css = trim(Sanitizer::checkCss($css));
 45+ $css = htmlspecialchars(trim(Sanitizer::checkCss($css)));
4646 $parser->mOutput->addHeadItem( <<<EOT
4747 <style type="text/css">
4848 /*<![CDATA[*/

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r38257should use htmlspecialchars herenad21:37, 30 July 2008

Status & tagging log