Random coding style tweaks + documentation clarification

Change-Id: I3d7946507bd489edf248c201877fab580875ed68
This commit is contained in:
Jack Phoenix 2017-01-24 17:25:26 +02:00
parent 453d39b13a
commit dd1cb0dfad

View file

@ -101,8 +101,7 @@ class MonoBookTemplate extends BaseTemplate {
<!-- end content -->
<?php
if ( $this->data['dataAfterContent'] ) {
$this->html( 'dataAfterContent'
);
$this->html( 'dataAfterContent' );
}
?>
<div class="visualClear"></div>
@ -122,8 +121,8 @@ class MonoBookTemplate extends BaseTemplate {
$personalTools = $this->getPersonalTools();
if ( array_key_exists( 'uls', $personalTools ) ) {
echo $this->makeListItem( 'uls', $personalTools[ 'uls' ] );
unset( $personalTools[ 'uls' ] );
echo $this->makeListItem( 'uls', $personalTools['uls'] );
unset( $personalTools['uls'] );
}
if ( !$this->getSkin()->getUser()->isLoggedIn() &&
@ -132,7 +131,6 @@ class MonoBookTemplate extends BaseTemplate {
echo Html::rawElement( 'li', array(
'id' => 'pt-anonuserpage'
), $this->getMsg( 'notloggedin' )->escaped() );
}
foreach ( $personalTools as $key => $item ) { ?>
@ -160,8 +158,8 @@ class MonoBookTemplate extends BaseTemplate {
</div><!-- end of the left (by default at least) column -->
<div class="visualClear"></div>
<?php
$validFooterIcons = $this->getFooterIcons( "icononly" );
$validFooterLinks = $this->getFooterLinks( "flat" ); // Additional footer links
$validFooterIcons = $this->getFooterIcons( 'icononly' );
$validFooterLinks = $this->getFooterLinks( 'flat' ); // Additional footer links
if ( count( $validFooterIcons ) + count( $validFooterLinks ) > 0 ) {
?>
@ -254,20 +252,20 @@ class MonoBookTemplate extends BaseTemplate {
<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" ) ); ?>
<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" )
'go',
array( 'id' => 'searchGoButton', 'class' => 'searchButton' )
);
if ( $this->config->get( 'UseTwoButtonsSearchForm' ) ) {
?>&#160;
<?php echo $this->makeSearchButton(
"fulltext",
array( "id" => "mw-searchButton", "class" => "searchButton" )
'fulltext',
array( 'id' => 'mw-searchButton', 'class' => 'searchButton' )
);
} else {
?>
@ -286,7 +284,7 @@ class MonoBookTemplate extends BaseTemplate {
}
/**
* Prints the cactions bar.
* Prints the content actions (cactions) bar.
* Shared between MonoBook and Modern
*/
function cactions() {
@ -342,8 +340,8 @@ class MonoBookTemplate extends BaseTemplate {
<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
}
@ -378,7 +376,7 @@ class MonoBookTemplate extends BaseTemplate {
?>
<h3><?php echo htmlspecialchars( $msgObj->exists() ? $msgObj->text() : $bar ); ?></h3>
<div class='pBody'>
<div class="pBody">
<?php
if ( is_array( $cont ) ) {
?>