wikimedia / mediawiki-extensions-Lingo

src/BasicBackend.php

Doc Comments +2 added lines, -2 removed lines

@@ -128,8 +128,8 @@
 	}
 
 	/**
-	 * @param $definitions
-	 * @param $term
+	 * @param string[] $definitions
+	 * @param string $term
 	 * @return array
 	 */
 	protected function queueDefinitions( $definitions, $term ) {

Spacing +6 added lines, -6 removed lines

@@ -75,8 +75,8 @@ 
 	public function next() {
 
 		static $term = null;
-		static $definitions = [];
-		static $ret = [];
+		static $definitions = [ ];
+		static $ret = [ ];
 
 		$this->collectDictionaryLines();
 
@@ -113,11 +113,11 @@ 
 
 			// wipe the data if it's a totally new term definition
 			if ( !empty( $term ) && count( $definitions ) > 0 ) {
-				$definitions = [];
+				$definitions = [ ];
 				$term = null;
 			}
 
-			$definitions[] = trim( $chunks[ 1 ] );
+			$definitions[ ] = trim( $chunks[ 1 ] );
 		}
 
 		// found a new term?
@@ -134,10 +134,10 @@ 
 	 * @return array
 	 */
 	protected function queueDefinitions( $definitions, $term ) {
-		$ret = [];
+		$ret = [ ];
 
 		foreach ( $definitions as $definition ) {
-			$ret[] = [
+			$ret[ ] = [
 				Element::ELEMENT_TERM       => $term,
 				Element::ELEMENT_DEFINITION => $definition,
 				Element::ELEMENT_LINK       => null,

src/Element.php

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 	/**
 	 * Lingo\Element constructor.
 	 *
-	 * @param $term
+	 * @param string $term
 	 * @param $definition
 	 */
 	public function __construct( &$term, &$definition = null ) {

Spacing +16 added lines, -16 removed lines

@@ -48,14 +48,14 @@ 
 	const ELEMENT_LINK = 3;
 	const ELEMENT_STYLE = 4;
 
-	const ELEMENT_FIELDCOUNT = 5;  // number of fields stored for each element; (last field's index) + 1
+	const ELEMENT_FIELDCOUNT = 5; // number of fields stored for each element; (last field's index) + 1
 
 	const LINK_TEMPLATE_ID = 'LingoLink';
 
 	private $formattedTerm = null;
 	private $formattedDefinitions = null;
 
-	private $mDefinitions = [];
+	private $mDefinitions = [ ];
 	private $mTerm = null;
 
 	private $hasBeenDisplayed = false;
@@ -79,7 +79,7 @@ 
 	 * @param array $definition
 	 */
 	public function addDefinition( &$definition ) {
-		$this->mDefinitions[] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
+		$this->mDefinitions[ ] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
 	}
 
 	/**
@@ -185,18 +185,18 @@ 
 		// part here.
 
 		// set style
-		$classes = [];
+		$classes = [ ];
 
 		if ( $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ] !== null ) {
-			$classes[] = $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ];
+			$classes[ ] = $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ];
 		}
 
 		if ( !$target->isKnown() ) {
-			$classes[] = 'new';
+			$classes[ ] = 'new';
 		}
 
 		if ( $target->isExternal() ) {
-			$classes[] = 'extiw';
+			$classes[ ] = 'extiw';
 		}
 
 		if ( count( $classes ) > 0 ) {
@@ -243,16 +243,16 @@ 
 	protected function buildFormattedDefinitions() {
 
 		// Wrap definition in a <div> tag
-		$divDefinitions = [];
-		$divDefinitions[] = '<div class="mw-lingo-tooltip ' . $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ] . '" id="' . $this->getId() . '" >';
+		$divDefinitions = [ ];
+		$divDefinitions[ ] = '<div class="mw-lingo-tooltip ' . $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ] . '" id="' . $this->getId() . '" >';
 
 		foreach ( $this->mDefinitions as $definition ) {
 
-			$divDefinitions[] = '<div class="mw-lingo-definition">';
+			$divDefinitions[ ] = '<div class="mw-lingo-definition">';
 
-			$divDefinitions[] = '<div class="mw-lingo-definition-text ' . $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ] . "\">\n";
-			$divDefinitions[] = $definition[ self::ELEMENT_DEFINITION ];
-			$divDefinitions[] = "\n" . '</div>';
+			$divDefinitions[ ] = '<div class="mw-lingo-definition-text ' . $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ] . "\">\n";
+			$divDefinitions[ ] = $definition[ self::ELEMENT_DEFINITION ];
+			$divDefinitions[ ] = "\n" . '</div>';
 
 			if ( $definition[ self::ELEMENT_LINK ] ) {
 
@@ -263,14 +263,14 @@ 
 				}
 
 				if ( $url !== null ) {
-					$divDefinitions[] = '<div class="mw-lingo-definition-link">[' . $url . ' <nowiki/>]</div>';
+					$divDefinitions[ ] = '<div class="mw-lingo-definition-link">[' . $url . ' <nowiki/>]</div>';
 				}
 			}
 
-			$divDefinitions[] = '</div>';
+			$divDefinitions[ ] = '</div>';
 		}
 
-		$divDefinitions[] = "\n" . '</div>';
+		$divDefinitions[ ] = "\n" . '</div>';
 
 		$this->formattedDefinitions = join( $divDefinitions );
 	}

src/Tree.php

Spacing +4 added lines, -4 removed lines

@@ -52,8 +52,8 @@ 
 
 	const TREE_VERSION = 2;
 
-	private $mTree = [];
-	private $mList = [];
+	private $mTree = [ ];
+	private $mList = [ ];
 	private $mMinLength = 1000;
 
 	/**
@@ -71,7 +71,7 @@ 
 			$this->mList[ $term ]->addDefinition( $definition );
 		} else {
 
-			$matches = [];
+			$matches = [ ];
 			preg_match_all( LingoParser::getInstance()->regex, $term, $matches );
 
 			$element = $this->addElement( $matches[ 0 ], $term, $definition );
@@ -97,7 +97,7 @@ 
 		while ( ( $step = array_shift( $path ) ) !== null ) {
 
 			if ( !isset( $tree[ $step ] ) ) {
-				$tree[ $step ] = [];
+				$tree[ $step ] = [ ];
 			}
 
 			$tree = &$tree[ $step ];

src/MessageLog.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
  */
 class MessageLog {
 
-	private $mMessages = [];
+	private $mMessages = [ ];
 	private $mParser = null;
 
 	const MESSAGE_ERROR = 1;
@@ -53,7 +53,7 @@ 
 	 * @param int $severity
 	 */
 	public function addMessage( $message, $severity = self::MESSAGE_NOTICE ) {
-		$this->mMessages[] = [ $message, $severity ];
+		$this->mMessages[ ] = [ $message, $severity ];
 
 		// log errors and warnings in debug log
 		if ( $severity == self::MESSAGE_WARNING ||
@@ -67,7 +67,7 @@ 
 	 * @param $message
 	 */
 	public function addError( $message ) {
-		$this->mMessages[] = [ $message, self::MESSAGE_ERROR ];
+		$this->mMessages[ ] = [ $message, self::MESSAGE_ERROR ];
 		wfDebug( "Error: $message\n" );
 	}
 
@@ -75,7 +75,7 @@ 
 	 * @param $message
 	 */
 	public function addWarning( $message ) {
-		$this->mMessages[] = [ $message, self::MESSAGE_WARNING ];
+		$this->mMessages[ ] = [ $message, self::MESSAGE_WARNING ];
 		wfDebug( "Warning: $message\n" );
 	}
 
@@ -83,7 +83,7 @@ 
 	 * @param $message
 	 */
 	public function addNotice( $message ) {
-		$this->mMessages[] = [ $message, self::MESSAGE_NOTICE ];
+		$this->mMessages[ ] = [ $message, self::MESSAGE_NOTICE ];
 	}
 
 	/**

src/Lingo.i18n.magic.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * @ingroup Extensions
  */
 
-$magicWords = [];
+$magicWords = [ ];
 
 /** English (English) */
 $magicWords[ 'en' ] = [

src/LingoParser.php

Spacing +10 added lines, -10 removed lines

@@ -160,7 +160,7 @@ 
 				} else {
 
 					wfDebug( "Cache miss: Lingo tree not found in cache.\n" );
-					$this->mLingoTree =& $this->buildLingo();
+					$this->mLingoTree = & $this->buildLingo();
 					wfDebug( "Cached lingo tree.\n" );
 				}
 
@@ -171,7 +171,7 @@ 
 
 			} else {
 				wfDebug( "Caching of lingo tree disabled.\n" );
-				$this->mLingoTree =& $this->buildLingo();
+				$this->mLingoTree = & $this->buildLingo();
 			}
 
 		}
@@ -236,7 +236,7 @@ 
 		// Iterate all HTML text matches
 		$numberOfTextElements = $textElements->length;
 
-		$definitions = [];
+		$definitions = [ ];
 
 		for ( $textElementIndex = 0; $textElementIndex < $numberOfTextElements; $textElementIndex++ ) {
 			$textElement = $textElements->item( $textElementIndex );
@@ -245,7 +245,7 @@ 
 				continue;
 			}
 
-			$matches = [];
+			$matches = [ ];
 			preg_match_all(
 				$this->regex,
 				$textElement->nodeValue,
@@ -257,7 +257,7 @@ 
 				continue;
 			}
 
-			$wordDescriptors = &$matches[ 0 ];  // See __construct() for definition of "word"
+			$wordDescriptors = &$matches[ 0 ]; // See __construct() for definition of "word"
 			$numberOfWordDescriptors = count( $wordDescriptors );
 
 			$parentNode = &$textElement->parentNode;
@@ -275,12 +275,12 @@ 
 
 					if ( $skippedWords > 0 ) { // skipped some text, insert it as is
 
-						$start = $wordDescriptors[$wordDescriptorIndex][self::WORD_OFFSET];
-						$length = $wordDescriptors[$wordDescriptorIndex + $skippedWords][self::WORD_OFFSET] - $start;
+						$start = $wordDescriptors[ $wordDescriptorIndex ][ self::WORD_OFFSET ];
+						$length = $wordDescriptors[ $wordDescriptorIndex + $skippedWords ][ self::WORD_OFFSET ] - $start;
 
 						$parentNode->insertBefore(
 							$doc->createTextNode(
-								substr( $textElement->nodeValue, $start, $length)
+								substr( $textElement->nodeValue, $start, $length )
 							),
 							$textElement
 						);
@@ -301,11 +301,11 @@ 
 					// Only change element if found term before
 					if ( $changedElem === true ) {
 
-						$start = $wordDescriptors[$wordDescriptorIndex][self::WORD_OFFSET];
+						$start = $wordDescriptors[ $wordDescriptorIndex ][ self::WORD_OFFSET ];
 
 						$parentNode->insertBefore(
 							$doc->createTextNode(
-								substr( $textElement->nodeValue, $start)
+								substr( $textElement->nodeValue, $start )
 							),
 							$textElement
 						);

src/Lingo.php

Spacing +6 added lines, -6 removed lines

@@ -43,7 +43,7 @@ 
 	 */
 	public static function initExtension() {
 
-		$GLOBALS[ 'wgExtensionFunctions' ][] = function () {
+		$GLOBALS[ 'wgExtensionFunctions' ][ ] = function() {
 
 			$parser = LingoParser::getInstance();
 
@@ -53,14 +53,14 @@ 
 
 			\Hooks::register( 'ContentAlterParserOutput', [ $parser, 'parse' ] );
 
-			\Hooks::register('GetDoubleUnderscoreIDs', function ( array &$doubleUnderscoreIDs ) {
-				$doubleUnderscoreIDs[] = 'noglossary';
+			\Hooks::register( 'GetDoubleUnderscoreIDs', function( array &$doubleUnderscoreIDs ) {
+				$doubleUnderscoreIDs[ ] = 'noglossary';
 				return true;
 			} );
 
-			\Hooks::register( 'ParserFirstCallInit', function ( \Parser $parser ) {
+			\Hooks::register( 'ParserFirstCallInit', function( \Parser $parser ) {
 
-				$parser->setHook( 'noglossary', function ( $input, array $args, \Parser $parser, \PPFrame $frame ) {
+				$parser->setHook( 'noglossary', function( $input, array $args, \Parser $parser, \PPFrame $frame ) {
 					$output = $parser->recursiveTagParse( $input, $frame );
 					return '<span class="noglossary">' . $output . '</span>';
 				} );
@@ -68,7 +68,7 @@ 
 				return true;
 			} );
 
-			\Hooks::register( 'SpecialPageBeforeExecute', function ( \SpecialPage $specialPage, $subPage ) {
+			\Hooks::register( 'SpecialPageBeforeExecute', function( \SpecialPage $specialPage, $subPage ) {
 
 				if ( $specialPage instanceof \SpecialVersion ) {
 					foreach ( $GLOBALS[ 'wgExtensionCredits' ][ 'parserhook' ] as $index => $description ) {