wikimedia / mediawiki-extensions-Lingo

src/Tree.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 	 * @param String $definition
 	 * @return Array the tree node the element was stored in
 	 */
-	protected function &addElement( Array &$path, &$term, &$definition ) {
+	protected function &addElement( array &$path, &$term, &$definition ) {
 
 		$tree = &$this->mTree;
 
@@ -142,7 +142,7 @@ 
 		}
 	}
 
-	public function findNextTermNoSkip( Array &$tree, &$lexemes, $index, $countLexemes ) {
+	public function findNextTermNoSkip( array &$tree, &$lexemes, $index, $countLexemes ) {
 
 		if ( $index + 1 < $countLexemes && array_key_exists( $currLex = $lexemes[ $index + 1 ][ 0 ], $tree ) ) {
 			$ret = $this->findNextTermNoSkip( $tree[ $currLex ], $lexemes, $index + 1, $countLexemes );

src/MessageLog.php

Spacing +4 added lines, -4 removed lines

@@ -53,7 +53,7 @@ 
 	 * @param int $severity
 	 */
 	public function addMessage( $message, $severity = self::MESSAGE_NOTICE ) {
-		$this->mMessages[] = array( $message, $severity );
+		$this->mMessages[ ] = array( $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[] = array( $message, self::MESSAGE_ERROR );
+		$this->mMessages[ ] = array( $message, self::MESSAGE_ERROR );
 		wfDebug( "Error: $message\n" );
 	}
 
@@ -75,7 +75,7 @@ 
 	 * @param $message
 	 */
 	public function addWarning( $message ) {
-		$this->mMessages[] = array( $message, self::MESSAGE_WARNING );
+		$this->mMessages[ ] = array( $message, self::MESSAGE_WARNING );
 		wfDebug( "Warning: $message\n" );
 	}
 
@@ -83,7 +83,7 @@ 
 	 * @param $message
 	 */
 	public function addNotice( $message ) {
-		$this->mMessages[] = array( $message, self::MESSAGE_NOTICE );
+		$this->mMessages[ ] = array( $message, self::MESSAGE_NOTICE );
 	}
 
 	/**

src/StashingDOMDocument.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 	 * @param $key
 	 * @return bool
 	 */
-	public function isStashed ( $key ) {
+	public function isStashed( $key ) {
 		return isset( $this->mStash[ $key ] );
 	}
 

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 +2 added lines, -2 removed lines

@@ -116,7 +116,7 @@ 
 				$term = null;
 			}
 
-			$definitions[] = trim( $chunks[ 1 ] );
+			$definitions[ ] = trim( $chunks[ 1 ] );
 		}
 
 		// found a new term?
@@ -136,7 +136,7 @@ 
 		$ret = array();
 
 		foreach ( $definitions as $definition ) {
-			$ret[] = array(
+			$ret[ ] = array(
 				Element::ELEMENT_TERM       => $term,
 				Element::ELEMENT_DEFINITION => $definition,
 				Element::ELEMENT_LINK       => null,

src/Element.php

Doc Comments +2 added lines, -2 removed lines

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

Spacing +9 added lines, -9 removed lines

@@ -47,7 +47,7 @@ 
 	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';
 
@@ -74,7 +74,7 @@ 
 	 * @param $definition
 	 */
 	public function addDefinition( &$definition ) {
-		$this->mDefinitions[] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
+		$this->mDefinitions[ ] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
 	}
 
 	/**
@@ -168,19 +168,19 @@ 
 		$classes = array();
 
 		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 ) {
-			$link->setAttribute( 'class', join(' ', $classes ) );
+		if ( count( $classes ) > 0 ) {
+			$link->setAttribute( 'class', join( ' ', $classes ) );
 		}
 
 		return $link;
@@ -213,7 +213,7 @@ 
 	protected function getFullDefinitionAsTooltip( StashingDOMDocument &$doc, callable $callback = null ) {
 
 		if ( $callback === null ) {
-			$callback = function( $text ){ return htmlentities( $text, ENT_COMPAT, 'UTF-8' ); };
+			$callback = function( $text ) { return htmlentities( $text, ENT_COMPAT, 'UTF-8' ); };
 		}
 
 		// Wrap term and definition in <span> tags
@@ -275,7 +275,7 @@ 
 		if ( $mLinkTemplate === null ) {
 
 			$mLinkTemplate = $doc->createElement( 'a' );
-			$mLinkTemplate->setAttribute( 'class', 'mw-lingo-tooltip-link');
+			$mLinkTemplate->setAttribute( 'class', 'mw-lingo-tooltip-link' );
 
 			$doc->stashSet( $mLinkTemplate, self::LINK_TEMPLATE_ID );
 		}

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( 'ParserAfterParse', array( $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 ) {

src/LingoParser.php

Spacing +2 added lines, -2 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();
 			}
 
 		}