MediaWiki r7497 - Code Review

Jump to: navigation, search
Repository:MediaWiki
Revision:r7496‎ | r7497 (on ViewVC)‎ | r7498 >
Date:15:44, 20 February 2005
Author:gabrielwicke
Status:old
Tags:
Comment:
correct include order, user stylesheet needs to be last
Modified paths:

Diff [purge]

Index: branches/REL1_4/phase3/includes/Skin.php
@@ -241,6 +241,8 @@
242242 $action = $wgRequest->getText('action');
243243 $s = "@import \"$wgStylePath/$sheet\";\n";
244244 if($wgContLang->isRTL()) $s .= "@import \"$wgStylePath/common/common_rtl.css\";\n";
 245+ $csspage = $wgContLang->getNsText( NS_MEDIAWIKI ) . ':' . $this->getSkinName() . '.css';
 246+ $s .= '@import "'.$this->makeUrl($csspage, 'action=raw&ctype=text/css')."\";\n";
245247 if( $wgAllowUserCss && $wgUser->getID() != 0 ) { # logged in
246248 if($wgTitle->isCssSubpage() && $this->userCanPreview( $action ) ) {
247249 $s .= $wgRequest->getText('wpTextbox1');
@@ -277,9 +279,7 @@
278280 function doGetUserStyles() {
279281 global $wgUser, $wgContLang;
280282
281 - $csspage = $wgContLang->getNsText( NS_MEDIAWIKI ) . ':' . $this->getSkinName() . '.css';
282 - $s = '@import "'.$this->makeUrl($csspage, 'action=raw&ctype=text/css')."\";\n";
283 -
 283+ $s = '';
284284 if ( 1 == $wgUser->getOption( 'underline' ) ) {
285285 # Don't override browser settings
286286 } else {

Status & tagging log

  • 15:03, 12 September 2011 Meno25 (Talk | contribs) changed the status of r7497 [removed: ok added: old]
  • 14:06, 1 July 2009 ^demon (Talk | contribs) changed the status of r7497 [removed: new added: ok]
Personal tools
Namespaces

Variants
Views
Actions
Navigation
Support
Download
Development
Communication
Toolbox