diff options
Diffstat (limited to 'skins/MonoBook.php')
-rw-r--r-- | skins/MonoBook.php | 120 |
1 files changed, 61 insertions, 59 deletions
diff --git a/skins/MonoBook.php b/skins/MonoBook.php index f03d31b8..6d66cac3 100644 --- a/skins/MonoBook.php +++ b/skins/MonoBook.php @@ -25,8 +25,9 @@ * @ingroup Skins */ -if( !defined( 'MEDIAWIKI' ) ) +if ( !defined( 'MEDIAWIKI' ) ) { die( -1 ); +} /** * Inherit main code from SkinTemplate, set the CSS and template filter. @@ -42,17 +43,10 @@ class SkinMonoBook extends SkinTemplate { * @param $out OutputPage */ function setupSkinUserCss( OutputPage $out ) { - global $wgHandheldStyle; parent::setupSkinUserCss( $out ); $out->addModuleStyles( 'skins.monobook' ); - // Ugh. Can't do this properly because $wgHandheldStyle may be a URL - if( $wgHandheldStyle ) { - // Currently in testing... try 'chick/main.css' - $out->addStyle( $wgHandheldStyle, 'handheld' ); - } - // TODO: Migrate all of these $out->addStyle( 'monobook/IE60Fixes.css', 'screen', 'IE 6' ); $out->addStyle( 'monobook/IE70Fixes.css', 'screen', 'IE 7' ); @@ -82,39 +76,39 @@ class MonoBookTemplate extends BaseTemplate { ?><div id="globalWrapper"> <div id="column-content"><div id="content" class="mw-body-primary" role="main"> <a id="top"></a> - <?php if($this->data['sitenotice']) { ?><div id="siteNotice"><?php $this->html('sitenotice') ?></div><?php } ?> + <?php if ( $this->data['sitenotice'] ) { ?><div id="siteNotice"><?php $this->html( 'sitenotice' ) ?></div><?php } ?> <h1 id="firstHeading" class="firstHeading" lang="<?php - $this->data['pageLanguage'] = $this->getSkin()->getTitle()->getPageViewLanguage()->getCode(); - $this->html( 'pageLanguage' ); - ?>"><span dir="auto"><?php $this->html('title') ?></span></h1> + $this->data['pageLanguage'] = $this->getSkin()->getTitle()->getPageViewLanguage()->getHtmlCode(); + $this->text( 'pageLanguage' ); + ?>"><span dir="auto"><?php $this->html( 'title' ) ?></span></h1> <div id="bodyContent" class="mw-body"> - <div id="siteSub"><?php $this->msg('tagline') ?></div> - <div id="contentSub"<?php $this->html('userlangattributes') ?>><?php $this->html('subtitle') ?></div> -<?php if($this->data['undelete']) { ?> - <div id="contentSub2"><?php $this->html('undelete') ?></div> -<?php } ?><?php if($this->data['newtalk'] ) { ?> - <div class="usermessage"><?php $this->html('newtalk') ?></div> -<?php } ?><?php if($this->data['showjumplinks']) { ?> - <div id="jump-to-nav" class="mw-jump"><?php $this->msg('jumpto') ?> <a href="#column-one"><?php $this->msg('jumptonavigation') ?></a><?php $this->msg( 'comma-separator' ) ?><a href="#searchInput"><?php $this->msg('jumptosearch') ?></a></div> + <div id="siteSub"><?php $this->msg( 'tagline' ) ?></div> + <div id="contentSub"<?php $this->html( 'userlangattributes' ) ?>><?php $this->html( 'subtitle' ) ?></div> +<?php if ( $this->data['undelete'] ) { ?> + <div id="contentSub2"><?php $this->html( 'undelete' ) ?></div> +<?php } ?><?php if ( $this->data['newtalk'] ) { ?> + <div class="usermessage"><?php $this->html( 'newtalk' ) ?></div> <?php } ?> + <div id="jump-to-nav" class="mw-jump"><?php $this->msg( 'jumpto' ) ?> <a href="#column-one"><?php $this->msg( 'jumptonavigation' ) ?></a><?php $this->msg( 'comma-separator' ) ?><a href="#searchInput"><?php $this->msg( 'jumptosearch' ) ?></a></div> + <!-- start content --> -<?php $this->html('bodytext') ?> - <?php if($this->data['catlinks']) { $this->html('catlinks'); } ?> +<?php $this->html( 'bodytext' ) ?> + <?php if ( $this->data['catlinks'] ) { $this->html( 'catlinks' ); } ?> <!-- end content --> - <?php if($this->data['dataAfterContent']) { $this->html ('dataAfterContent'); } ?> + <?php if ( $this->data['dataAfterContent'] ) { $this->html( 'dataAfterContent' ); } ?> <div class="visualClear"></div> </div> </div></div> -<div id="column-one"<?php $this->html('userlangattributes') ?>> +<div id="column-one"<?php $this->html( 'userlangattributes' ) ?>> <h2><?php $this->msg( 'navigation-heading' ) ?></h2> <?php $this->cactions(); ?> <div class="portlet" id="p-personal" role="navigation"> - <h3><?php $this->msg('personaltools') ?></h3> + <h3><?php $this->msg( 'personaltools' ) ?></h3> <div class="pBody"> - <ul<?php $this->html('userlangattributes') ?>> -<?php foreach($this->getPersonalTools() as $key => $item) { ?> - <?php echo $this->makeListItem($key, $item); ?> + <ul<?php $this->html( 'userlangattributes' ) ?>> +<?php foreach ( $this->getPersonalTools() as $key => $item ) { ?> + <?php echo $this->makeListItem( $key, $item ); ?> <?php } ?> </ul> @@ -125,7 +119,7 @@ class MonoBookTemplate extends BaseTemplate { echo Html::element( 'a', array( 'href' => $this->data['nav_urls']['mainpage']['href'], 'style' => "background-image: url({$this->data['logopath']});" ) - + Linker::tooltipAndAccesskeyAttribs('p-logo') ); ?> + + Linker::tooltipAndAccesskeyAttribs( 'p-logo' ) ); ?> </div> <?php @@ -138,14 +132,14 @@ class MonoBookTemplate extends BaseTemplate { $validFooterLinks = $this->getFooterLinks( "flat" ); // Additional footer links if ( count( $validFooterIcons ) + count( $validFooterLinks ) > 0 ) { ?> -<div id="footer" role="contentinfo"<?php $this->html('userlangattributes') ?>> +<div id="footer" role="contentinfo"<?php $this->html( 'userlangattributes' ) ?>> <?php $footerEnd = '</div>'; } else { $footerEnd = ''; } foreach ( $validFooterIcons as $blockName => $footerIcons ) { ?> - <div id="f-<?php echo htmlspecialchars($blockName); ?>ico"> + <div id="f-<?php echo htmlspecialchars( $blockName ); ?>ico"> <?php foreach ( $footerIcons as $icon ) { ?> <?php echo $this->getSkin()->makeFooterIcon( $icon ); ?> @@ -157,8 +151,8 @@ class MonoBookTemplate extends BaseTemplate { if ( count( $validFooterLinks ) > 0 ) { ?> <ul id="f-list"> <?php - foreach( $validFooterLinks as $aLink ) { ?> - <li id="<?php echo $aLink ?>"><?php $this->html($aLink) ?></li> + foreach ( $validFooterLinks as $aLink ) { ?> + <li id="<?php echo $aLink ?>"><?php $this->html( $aLink ) ?></li> <?php } ?> @@ -181,13 +175,20 @@ echo $footerEnd; * @param $sidebar array */ protected function renderPortals( $sidebar ) { - if ( !isset( $sidebar['SEARCH'] ) ) $sidebar['SEARCH'] = true; - if ( !isset( $sidebar['TOOLBOX'] ) ) $sidebar['TOOLBOX'] = true; - if ( !isset( $sidebar['LANGUAGES'] ) ) $sidebar['LANGUAGES'] = true; + if ( !isset( $sidebar['SEARCH'] ) ) { + $sidebar['SEARCH'] = true; + } + if ( !isset( $sidebar['TOOLBOX'] ) ) { + $sidebar['TOOLBOX'] = true; + } + if ( !isset( $sidebar['LANGUAGES'] ) ) { + $sidebar['LANGUAGES'] = true; + } - foreach( $sidebar as $boxName => $content ) { - if ( $content === false ) + foreach ( $sidebar as $boxName => $content ) { + if ( $content === false ) { continue; + } if ( $boxName == 'SEARCH' ) { $this->searchBox(); @@ -205,19 +206,19 @@ echo $footerEnd; global $wgUseTwoButtonsSearchForm; ?> <div id="p-search" class="portlet" role="search"> - <h3><label for="searchInput"><?php $this->msg('search') ?></label></h3> + <h3><label for="searchInput"><?php $this->msg( 'search' ) ?></label></h3> <div id="searchBody" class="pBody"> - <form action="<?php $this->text('wgScript') ?>" id="searchform"> - <input type='hidden' name="title" value="<?php $this->text('searchtitle') ?>"/> - <?php echo $this->makeSearchInput(array( "id" => "searchInput" )); ?> + <form action="<?php $this->text( 'wgScript' ) ?>" id="searchform"> + <input type='hidden' name="title" value="<?php $this->text( 'searchtitle' ) ?>"/> + <?php echo $this->makeSearchInput( array( "id" => "searchInput" ) ); ?> - <?php echo $this->makeSearchButton("go", array( "id" => "searchGoButton", "class" => "searchButton" )); - if ($wgUseTwoButtonsSearchForm): ?>  - <?php echo $this->makeSearchButton("fulltext", array( "id" => "mw-searchButton", "class" => "searchButton" )); - else: ?> + <?php echo $this->makeSearchButton( "go", array( "id" => "searchGoButton", "class" => "searchButton" ) ); + if ( $wgUseTwoButtonsSearchForm ) { ?>  + <?php echo $this->makeSearchButton( "fulltext", array( "id" => "mw-searchButton", "class" => "searchButton" ) ); + } else { ?> - <div><a href="<?php $this->text('searchaction') ?>" rel="search"><?php $this->msg('powersearch-legend') ?></a></div><?php - endif; ?> + <div><a href="<?php $this->text( 'searchaction' ) ?>" rel="search"><?php $this->msg( 'powersearch-legend' ) ?></a></div><?php + } ?> </form> </div> @@ -232,10 +233,10 @@ echo $footerEnd; function cactions() { ?> <div id="p-cactions" class="portlet" role="navigation"> - <h3><?php $this->msg('views') ?></h3> + <h3><?php $this->msg( 'views' ) ?></h3> <div class="pBody"> <ul><?php - foreach($this->data['content_actions'] as $key => $tab) { + foreach ( $this->data['content_actions'] as $key => $tab ) { echo ' ' . $this->makeListItem( $key, $tab ); } ?> @@ -249,12 +250,12 @@ echo $footerEnd; function toolbox() { ?> <div class="portlet" id="p-tb" role="navigation"> - <h3><?php $this->msg('toolbox') ?></h3> + <h3><?php $this->msg( 'toolbox' ) ?></h3> <div class="pBody"> <ul> <?php foreach ( $this->getToolbox() as $key => $tbitem ) { ?> - <?php echo $this->makeListItem($key, $tbitem); ?> + <?php echo $this->makeListItem( $key, $tbitem ); ?> <?php } @@ -269,14 +270,14 @@ echo $footerEnd; /*************************************************************************************************/ function languageBox() { - if( $this->data['language_urls'] ) { + if ( $this->data['language_urls'] ) { ?> <div id="p-lang" class="portlet" role="navigation"> - <h3<?php $this->html('userlangattributes') ?>><?php $this->msg('otherlanguages') ?></h3> + <h3<?php $this->html( 'userlangattributes' ) ?>><?php $this->msg( 'otherlanguages' ) ?></h3> <div class="pBody"> <ul> -<?php foreach($this->data['language_urls'] as $key => $langlink) { ?> - <?php echo $this->makeListItem($key, $langlink); ?> +<?php foreach ( $this->data['language_urls'] as $key => $langlink ) { ?> + <?php echo $this->makeListItem( $key, $langlink ); ?> <?php } ?> </ul> @@ -298,14 +299,15 @@ echo $footerEnd; $portletAttribs['title'] = $tooltip; } echo ' ' . Html::openElement( 'div', $portletAttribs ); + $msgObj = wfMessage( $bar ); ?> - <h3><?php $msg = wfMessage( $bar ); echo htmlspecialchars( $msg->exists() ? $msg->text() : $bar ); ?></h3> + <h3><?php echo htmlspecialchars( $msgObj->exists() ? $msgObj->text() : $bar ); ?></h3> <div class='pBody'> <?php if ( is_array( $cont ) ) { ?> <ul> -<?php foreach($cont as $key => $val) { ?> - <?php echo $this->makeListItem($key, $val); ?> +<?php foreach ( $cont as $key => $val ) { ?> + <?php echo $this->makeListItem( $key, $val ); ?> <?php } ?> </ul> |