wikimedia / mediawiki-extensions-Lingo

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 +13 added lines, -13 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,14 +74,14 @@ 
 	 * @param $definition
 	 */
 	public function addDefinition( &$definition ) {
-		$this->mDefinitions[] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
+		$this->mDefinitions[ ] = array_pad( $definition, self::ELEMENT_FIELDCOUNT, null );
 	}
 
 	/**
 	 * @param StashingDOMDocument $doc
 	 * @return DOMNode|DOMText
 	 */
-	public function getFullDefinition( StashingDOMDocument &$doc ) {
+	public function getFullDefinition( StashingDOMDocument & $doc ) {
 
 		global $wgexLingoDisplayOnce;
 
@@ -98,7 +98,7 @@ 
 	/**
 	 * @param StashingDOMDocument $doc
 	 */
-	private function buildFullDefinition( StashingDOMDocument &$doc ) {
+	private function buildFullDefinition( StashingDOMDocument & $doc ) {
 
 		// only create if not yet created
 		if ( $this->mFullDefinition === null || $this->mFullDefinition->ownerDocument !== $doc ) {
@@ -125,7 +125,7 @@ 
 	 * @return DOMElement
 	 * @throws \MWException
 	 */
-	protected function getFullDefinitionAsLink( StashingDOMDocument &$doc ) {
+	protected function getFullDefinitionAsLink( StashingDOMDocument & $doc ) {
 
 		// create Title object for target page
 		$target = Title::newFromText( $this->mDefinitions[ 0 ][ self::ELEMENT_LINK ] );
@@ -166,19 +166,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;
@@ -208,7 +208,7 @@ 
 	 * @return DOMElement
 	 * @throws \MWException
 	 */
-	protected function getFullDefinitionAsTooltip( StashingDOMDocument &$doc ) {
+	protected function getFullDefinitionAsTooltip( StashingDOMDocument & $doc ) {
 
 		// Wrap term and definition in <span> tags
 		$span = $doc->createElement( 'span' );
@@ -253,7 +253,7 @@ 
 	 * @param StashingDOMDocument $doc
 	 * @return DOMNode
 	 */
-	private function getLinkTemplate( StashingDOMDocument &$doc ) {
+	private function getLinkTemplate( StashingDOMDocument & $doc ) {
 
 		$mLinkTemplate = $doc->stashGet( self::LINK_TEMPLATE_ID );
 
@@ -261,7 +261,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 +5 added lines, -5 removed lines

@@ -43,7 +43,7 @@ 
 	 */
 	public static function initExtension() {
 
-		$GLOBALS[ 'wgExtensionFunctions' ][] = function () {
+		$GLOBALS[ 'wgExtensionFunctions' ][ ] = function() {
 
 			$parser = LingoParser::getInstance();
 
@@ -53,9 +53,9 @@ 
 
 			\Hooks::register( 'ParserAfterParse', array( $parser, 'parse' ) );
 
-			\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>';
 				} );
@@ -63,7 +63,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 ) {
@@ -83,6 +83,6 @@ 
 	 * @param array $doubleUnderscoreIDs
 	 */
 	public static function onGetDoubleUnderscoreIDs( array &$doubleUnderscoreIDs ) {
-		$doubleUnderscoreIDs[] = 'noglossary';
+		$doubleUnderscoreIDs[ ] = 'noglossary';
 	}
 }