X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2Fskins%2FBaseTemplate.php;h=3408db3487467f230e32e212711380fb65803235;hb=6f5751aa3a3a2ec981d7aebd45791a18dbd8b4e3;hp=143b6214a88729bbb04363b01c47265b4c8b9de2;hpb=fe2ef20c92361561be1801a6cfee2e6daa624b09;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/skins/BaseTemplate.php b/includes/skins/BaseTemplate.php index 143b6214a8..3408db3487 100644 --- a/includes/skins/BaseTemplate.php +++ b/includes/skins/BaseTemplate.php @@ -56,7 +56,7 @@ abstract class BaseTemplate extends QuickTemplate { */ function getToolbox() { - $toolbox = array(); + $toolbox = []; if ( isset( $this->data['nav_urls']['whatlinkshere'] ) && $this->data['nav_urls']['whatlinkshere'] ) { @@ -72,7 +72,7 @@ abstract class BaseTemplate extends QuickTemplate { } if ( isset( $this->data['feeds'] ) && $this->data['feeds'] ) { $toolbox['feeds']['id'] = 'feedlinks'; - $toolbox['feeds']['links'] = array(); + $toolbox['feeds']['links'] = []; foreach ( $this->data['feeds'] as $key => $feed ) { $toolbox['feeds']['links'][$key] = $feed; $toolbox['feeds']['links'][$key]['id'] = "feed-$key"; @@ -81,8 +81,8 @@ abstract class BaseTemplate extends QuickTemplate { $toolbox['feeds']['links'][$key]['class'] = 'feedlink'; } } - foreach ( array( 'contributions', 'log', 'blockip', 'emailuser', - 'userrights', 'upload', 'specialpages' ) as $special + foreach ( [ 'contributions', 'log', 'blockip', 'emailuser', + 'userrights', 'upload', 'specialpages' ] as $special ) { if ( isset( $this->data['nav_urls'][$special] ) && $this->data['nav_urls'][$special] ) { $toolbox[$special] = $this->data['nav_urls'][$special]; @@ -111,7 +111,7 @@ abstract class BaseTemplate extends QuickTemplate { $toolbox['info']['id'] = 't-info'; } - Hooks::run( 'BaseTemplateToolbox', array( &$this, &$toolbox ) ); + Hooks::run( 'BaseTemplateToolbox', [ &$this, &$toolbox ] ); return $toolbox; } @@ -126,21 +126,21 @@ abstract class BaseTemplate extends QuickTemplate { * @return array */ function getPersonalTools() { - $personal_tools = array(); + $personal_tools = []; foreach ( $this->get( 'personal_urls' ) as $key => $plink ) { # The class on a personal_urls item is meant to go on the instead # of the
  • so we have to use a single item "links" array instead # of using most of the personal_url's keys directly. - $ptool = array( - 'links' => array( - array( 'single-id' => "pt-$key" ), - ), + $ptool = [ + 'links' => [ + [ 'single-id' => "pt-$key" ], + ], 'id' => "pt-$key", - ); + ]; if ( isset( $plink['active'] ) ) { $ptool['active'] = $plink['active']; } - foreach ( array( 'href', 'class', 'text', 'dir' ) as $k ) { + foreach ( [ 'href', 'class', 'text', 'dir' ] as $k ) { if ( isset( $plink[$k] ) ) { $ptool['links'][0][$k] = $plink[$k]; } @@ -150,7 +150,7 @@ abstract class BaseTemplate extends QuickTemplate { return $personal_tools; } - function getSidebar( $options = array() ) { + function getSidebar( $options = [] ) { // Force the rendering of the following portals $sidebar = $this->data['sidebar']; if ( !isset( $sidebar['SEARCH'] ) ) { @@ -173,7 +173,7 @@ abstract class BaseTemplate extends QuickTemplate { unset( $sidebar['LANGUAGES'] ); } - $boxes = array(); + $boxes = []; foreach ( $sidebar as $boxName => $content ) { if ( $content === false ) { continue; @@ -181,41 +181,41 @@ abstract class BaseTemplate extends QuickTemplate { switch ( $boxName ) { case 'SEARCH': // Search is a special case, skins should custom implement this - $boxes[$boxName] = array( + $boxes[$boxName] = [ 'id' => 'p-search', 'header' => $this->getMsg( 'search' )->text(), 'generated' => false, 'content' => true, - ); + ]; break; case 'TOOLBOX': $msgObj = $this->getMsg( 'toolbox' ); - $boxes[$boxName] = array( + $boxes[$boxName] = [ 'id' => 'p-tb', 'header' => $msgObj->exists() ? $msgObj->text() : 'toolbox', 'generated' => false, 'content' => $this->getToolbox(), - ); + ]; break; case 'LANGUAGES': if ( $this->data['language_urls'] ) { $msgObj = $this->getMsg( 'otherlanguages' ); - $boxes[$boxName] = array( + $boxes[$boxName] = [ 'id' => 'p-lang', 'header' => $msgObj->exists() ? $msgObj->text() : 'otherlanguages', 'generated' => false, 'content' => $this->data['language_urls'], - ); + ]; } break; default: $msgObj = $this->getMsg( $boxName ); - $boxes[$boxName] = array( + $boxes[$boxName] = [ 'id' => "p-$boxName", 'header' => $msgObj->exists() ? $msgObj->text() : $boxName, 'generated' => true, 'content' => $content, - ); + ]; break; } } @@ -226,7 +226,7 @@ abstract class BaseTemplate extends QuickTemplate { ob_start(); // We pass an extra 'true' at the end so extensions using BaseTemplateToolbox // can abort and avoid outputting double toolbox links - Hooks::run( 'SkinTemplateToolboxEnd', array( &$this, true ) ); + Hooks::run( 'SkinTemplateToolboxEnd', [ &$this, true ] ); $hookContents = ob_get_contents(); ob_end_clean(); if ( !trim( $hookContents ) ) { @@ -253,14 +253,14 @@ abstract class BaseTemplate extends QuickTemplate { } } else { if ( $hookContents ) { - $boxes['TOOLBOXEND'] = array( + $boxes['TOOLBOXEND'] = [ 'id' => 'p-toolboxend', 'header' => $boxes['TOOLBOX']['header'], 'generated' => false, 'content' => "", - ); + ]; // HACK: Make sure that TOOLBOXEND is sorted next to TOOLBOX - $boxes2 = array(); + $boxes2 = []; foreach ( $boxes as $key => $box ) { if ( $key === 'TOOLBOXEND' ) { continue; @@ -283,7 +283,7 @@ abstract class BaseTemplate extends QuickTemplate { */ protected function renderAfterPortlet( $name ) { $content = ''; - Hooks::run( 'BaseTemplateAfterPortlet', array( $this, $name, &$content ) ); + Hooks::run( 'BaseTemplateAfterPortlet', [ $this, $name, &$content ] ); if ( $content !== '' ) { echo "
    $content
    "; @@ -334,7 +334,7 @@ abstract class BaseTemplate extends QuickTemplate { * * @return string */ - function makeLink( $key, $item, $options = array() ) { + function makeLink( $key, $item, $options = [] ) { if ( isset( $item['text'] ) ) { $text = $item['text']; } else { @@ -346,7 +346,7 @@ abstract class BaseTemplate extends QuickTemplate { if ( isset( $options['text-wrapper'] ) ) { $wrapper = $options['text-wrapper']; if ( isset( $wrapper['tag'] ) ) { - $wrapper = array( $wrapper ); + $wrapper = [ $wrapper ]; } while ( count( $wrapper ) > 0 ) { $element = array_pop( $wrapper ); @@ -358,8 +358,8 @@ abstract class BaseTemplate extends QuickTemplate { if ( isset( $item['href'] ) || isset( $options['link-fallback'] ) ) { $attrs = $item; - foreach ( array( 'single-id', 'text', 'msg', 'tooltiponly', 'context', 'primary', - 'tooltip-params' ) as $k ) { + foreach ( [ 'single-id', 'text', 'msg', 'tooltiponly', 'context', 'primary', + 'tooltip-params' ] as $k ) { unset( $attrs[$k] ); } @@ -367,7 +367,7 @@ abstract class BaseTemplate extends QuickTemplate { $item['single-id'] = $item['id']; } - $tooltipParams = array(); + $tooltipParams = []; if ( isset( $item['tooltip-params'] ) ) { $tooltipParams = $item['tooltip-params']; } @@ -431,9 +431,9 @@ abstract class BaseTemplate extends QuickTemplate { * * @return string */ - function makeListItem( $key, $item, $options = array() ) { + function makeListItem( $key, $item, $options = [] ) { if ( isset( $item['links'] ) ) { - $links = array(); + $links = []; foreach ( $item['links'] as $linkKey => $link ) { $links[] = $this->makeLink( $linkKey, $link, $options ); } @@ -441,7 +441,7 @@ abstract class BaseTemplate extends QuickTemplate { } else { $link = $item; // These keys are used by makeListItem and shouldn't be passed on to the link - foreach ( array( 'id', 'class', 'active', 'tag', 'itemtitle' ) as $k ) { + foreach ( [ 'id', 'class', 'active', 'tag', 'itemtitle' ] as $k ) { unset( $link[$k] ); } if ( isset( $item['id'] ) && !isset( $item['single-id'] ) ) { @@ -453,8 +453,8 @@ abstract class BaseTemplate extends QuickTemplate { $html = $this->makeLink( $key, $link, $options ); } - $attrs = array(); - foreach ( array( 'id', 'class' ) as $attr ) { + $attrs = []; + foreach ( [ 'id', 'class' ] as $attr ) { if ( isset( $item[$attr] ) ) { $attrs[$attr] = $item[$attr]; } @@ -472,27 +472,27 @@ abstract class BaseTemplate extends QuickTemplate { return Html::rawElement( isset( $options['tag'] ) ? $options['tag'] : 'li', $attrs, $html ); } - function makeSearchInput( $attrs = array() ) { - $realAttrs = array( + function makeSearchInput( $attrs = [] ) { + $realAttrs = [ 'type' => 'search', 'name' => 'search', 'placeholder' => wfMessage( 'searchsuggest-search' )->text(), 'value' => $this->get( 'search', '' ), - ); + ]; $realAttrs = array_merge( $realAttrs, Linker::tooltipAndAccesskeyAttribs( 'search' ), $attrs ); return Html::element( 'input', $realAttrs ); } - function makeSearchButton( $mode, $attrs = array() ) { + function makeSearchButton( $mode, $attrs = [] ) { switch ( $mode ) { case 'go': case 'fulltext': - $realAttrs = array( + $realAttrs = [ 'type' => 'submit', 'name' => $mode, 'value' => $this->translator->translate( $mode == 'go' ? 'searcharticle' : 'searchbutton' ), - ); + ]; $realAttrs = array_merge( $realAttrs, Linker::tooltipAndAccesskeyAttribs( "search-$mode" ), @@ -500,10 +500,10 @@ abstract class BaseTemplate extends QuickTemplate { ); return Html::element( 'input', $realAttrs ); case 'image': - $buttonAttrs = array( + $buttonAttrs = [ 'type' => 'submit', 'name' => 'button', - ); + ]; $buttonAttrs = array_merge( $buttonAttrs, Linker::tooltipAndAccesskeyAttribs( 'search-fulltext' ), @@ -513,14 +513,14 @@ abstract class BaseTemplate extends QuickTemplate { unset( $buttonAttrs['alt'] ); unset( $buttonAttrs['width'] ); unset( $buttonAttrs['height'] ); - $imgAttrs = array( + $imgAttrs = [ 'src' => $attrs['src'], 'alt' => isset( $attrs['alt'] ) ? $attrs['alt'] : $this->translator->translate( 'searchbutton' ), 'width' => isset( $attrs['width'] ) ? $attrs['width'] : null, 'height' => isset( $attrs['height'] ) ? $attrs['height'] : null, - ); + ]; return Html::rawElement( 'button', $buttonAttrs, Html::element( 'img', $imgAttrs ) ); default: throw new MWException( 'Unknown mode passed to BaseTemplate::makeSearchButton' ); @@ -540,9 +540,9 @@ abstract class BaseTemplate extends QuickTemplate { $footerlinks = $this->get( 'footerlinks' ); // Reduce footer links down to only those which are being used - $validFooterLinks = array(); + $validFooterLinks = []; foreach ( $footerlinks as $category => $links ) { - $validFooterLinks[$category] = array(); + $validFooterLinks[$category] = []; foreach ( $links as $link ) { if ( isset( $this->data[$link] ) && $this->data[$link] ) { $validFooterLinks[$category][] = $link; @@ -574,7 +574,7 @@ abstract class BaseTemplate extends QuickTemplate { * display the text from footericons instead of the images and don't want a * duplicate copyright statement because footerlinks already rendered one. * @param string $option - * @return string + * @return array */ function getFooterIcons( $option = null ) { // Generate additional footer icons @@ -625,10 +625,10 @@ abstract class BaseTemplate extends QuickTemplate { foreach ( $this->data['indicators'] as $id => $content ) { $out .= Html::rawElement( 'div', - array( + [ 'id' => Sanitizer::escapeId( "mw-indicator-$id" ), 'class' => 'mw-indicator', - ), + ], $content ) . "\n"; }