wikimedia / mediawiki-extensions-Lingo

tests/phpunit/Unit/LingoI18NTest.php

Spacing +2 added lines, -2 removed lines

@@ -54,12 +54,12 @@
 		$data = json_decode( json_encode( $defined_vars[ 'magicWords' ] ) );
 
 		$validator = new \JsonSchema\Validator();
-		$validator->check( $data, (object)[ '$ref' =>
+		$validator->check( $data, (object) [ '$ref' =>
 			'file://' . realpath( __DIR__ . '/../Fixture/magicWordsSchema.json' ) ] );
 
 		// format error message
 		$errors = implode( '', array_map(
-			function ( $error ) {
+			function( $error ) {
 				return "* [{$error[ 'property' ]}] {$error[ 'message' ]}\n";
 			},
 			$validator->getErrors()

tests/phpunit/Unit/ElementTest.php

Spacing +6 added lines, -6 removed lines

@@ -53,7 +53,7 @@ 
 	 */
 	public function testCanConstruct() {
 		$term = 'someTerm';
-		$definition = [];
+		$definition = [ ];
 		$element = new Element( $term, $definition );
 
 		$this->assertInstanceOf( '\Lingo\Element', $element );
@@ -100,7 +100,7 @@ 
 	public function testGetFormattedTerm_2() {
 		// Setup
 		$term = 'someTerm';
-		$definition = [];
+		$definition = [ ];
 		$element = new Element( $term, $definition );
 
 		$GLOBALS[ 'wgexLingoDisplayOnce' ] = true;
@@ -142,7 +142,7 @@ 
 
 		$element = new Element( $term, $definition );
 
-		$expectedAttributes = [ 'class' => [ 'mw-lingo-term', 'new' ],  'title' => wfMessage( 'red-link-title', $title )->text() ];
+		$expectedAttributes = [ 'class' => [ 'mw-lingo-term', 'new' ], 'title' => wfMessage( 'red-link-title', $title )->text() ];
 
 		$GLOBALS[ 'wgexLingoDisplayOnce' ] = false;
 
@@ -178,7 +178,7 @@ 
 
 		$element = new Element( $term, $definition );
 
-		$expectedAttributes = [ 'class' => [ 'mw-lingo-term' ],  'title' => $title ];
+		$expectedAttributes = [ 'class' => [ 'mw-lingo-term' ], 'title' => $title ];
 
 		$GLOBALS[ 'wgexLingoDisplayOnce' ] = false;
 
@@ -476,7 +476,7 @@ 
 	 * @param string[] $expectedAttributes
 	 * @param array $unexpectedAttributes
 	 */
-	protected function checkTermIsDomElement( $node, $tagName, $text, $expectedAttributes = [], $unexpectedAttributes = [] ) {
+	protected function checkTermIsDomElement( $node, $tagName, $text, $expectedAttributes = [ ], $unexpectedAttributes = [ ] ) {
 		$nodeText = $this->doc->saveHTML( $node );
 
 		$this->assertInstanceOf( 'DOMElement', $node );
@@ -487,7 +487,7 @@ 
 
 			$classes = array_flip( array_filter( explode( ' ', $node->getAttribute( 'class' ) ) ) );
 
-			foreach ( (array)$expectedAttributes[ 'class' ] as $expectedClass ) {
+			foreach ( (array) $expectedAttributes[ 'class' ] as $expectedClass ) {
 				$this->assertTrue( array_key_exists( $expectedClass, $classes ) );
 			}
 

tests/phpunit/Unit/BasicBackendTest.php

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

@@ -185,7 +185,7 @@ 
 <<<'TESTTEXT'
 ;CIP:Common image point
 ;CMP:Common midpoint
-TESTTEXT
+testtext
 			,
 				[
 					[ 'CMP', 'Common midpoint', null, null ],
@@ -200,7 +200,7 @@ 
 :Common image point
 ;CMP
 :Common midpoint
-TESTTEXT
+testtext
 			,
 				[
 					[ 'CMP', 'Common midpoint', null, null ],
@@ -214,7 +214,7 @@ 
 ;CIP
 ;CMP
 :Common midpoint
-TESTTEXT
+testtext
 			,
 				[
 					[ 'CMP', 'Common midpoint', null, null ],
@@ -228,7 +228,7 @@ 
 ;CIP
 :Common image point
 :Common midpoint
-TESTTEXT
+testtext
 			,
 				[
 					[ 'CIP', 'Common image point', null, null ],
@@ -243,7 +243,7 @@ 
 ;CMP
 :Common image point
 :Common midpoint
-TESTTEXT
+testtext
 			,
 				[
 					[ 'CMP', 'Common image point', null, null ],
@@ -267,7 +267,7 @@ 
 Sed ut perspiciatis unde; omnis iste natus error: sit voluptatem accusantium...
 
 ;NMO:Normal move-out
-TESTTEXT
+testtext
 			,
 				[
 					[ 'NMO', 'Normal move-out', null, null ],

tests/phpunit/Integration/ArticleAnnotationTest.php

Spacing +4 added lines, -4 removed lines

@@ -88,7 +88,7 @@ 
 	}
 
 	public function provideData() {
-		$data = [];
+		$data = [ ];
 
 		$xmlFileProvider = new XmlFileProvider( __DIR__ . '/../Fixture/articleAnnotation' );
 		$files = $xmlFileProvider->getFiles();
@@ -105,7 +105,7 @@ 
 			$testCase = [
 				0 => substr( $file, strlen( __DIR__ . '/../Fixture/articleAnnotation' ) ),
 				1 => trim( $decoded[ 'text' ] ),
-				2 => [],
+				2 => [ ],
 				3 => trim( $decoded[ 'expected' ] ) . "\n",
 			];
 
@@ -114,12 +114,12 @@ 
 			}
 
 			foreach ( $decoded[ 'glossary-entry' ] as $entry ) {
-				$testCase[ 2 ][] = [ $entry[ 'term' ], $entry[ 'definition' ], $entry[ 'link' ], $entry[ 'style' ] ];
+				$testCase[ 2 ][ ] = [ $entry[ 'term' ], $entry[ 'definition' ], $entry[ 'link' ], $entry[ 'style' ] ];
 			}
 
 			\MediaWiki\restoreWarnings();
 
-			$data[] = $testCase;
+			$data[ ] = $testCase;
 		}
 
 		return $data;

tests/mw-phpunit-runner.php

Spacing +2 added lines, -2 removed lines

@@ -70,6 +70,6 @@
 $config = isReadablePath( "$extDirectory/phpunit.xml.dist" );
 $mw = isReadablePath( dirname( dirname( $extDirectory ) ) . "/tests/phpunit/phpunit.php" );
 
-echo "php {$mw} -c {$config} " . implode( ' ', addArguments( $GLOBALS['argv'] ) ) . "\n\n";
+echo "php {$mw} -c {$config} " . implode( ' ', addArguments( $GLOBALS[ 'argv' ] ) ) . "\n\n";
 
-passthru( "php {$mw} -c {$config} " . implode( ' ', addArguments( $GLOBALS['argv'] ) ) );
+passthru( "php {$mw} -c {$config} " . implode( ' ', addArguments( $GLOBALS[ 'argv' ] ) ) );

src/BasicBackend.php

Spacing +6 added lines, -6 removed lines

@@ -72,8 +72,8 @@ 
 	 */
 	public function next() {
 		static $term = null;
-		static $definitions = [];
-		static $ret = [];
+		static $definitions = [ ];
+		static $ret = [ ];
 
 		$this->collectDictionaryLines();
 
@@ -109,11 +109,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?
@@ -130,10 +130,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

Spacing +10 added lines, -10 removed lines

@@ -47,14 +47,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;
@@ -74,7 +74,7 @@ 
 	 * @param array $definition
 	 */
 	public function addDefinition( &$definition ) {
-		$this->mDefinitions[] = $definition + array_fill( 0, self::ELEMENT_FIELDCOUNT, null );
+		$this->mDefinitions[ ] = $definition + array_fill( 0, self::ELEMENT_FIELDCOUNT, null );
 	}
 
 	/**
@@ -129,7 +129,7 @@ 
 		$descriptor = $this->getDescriptorFromLinkTarget( $linkTarget );
 
 		if ( $descriptor === null ) {
-			$this->mDefinitions = [];
+			$this->mDefinitions = [ ];
 			$this->addErrorMessageForInvalidLink( $linkTarget );
 			return $this->buildFormattedTermAsTooltip( $doc );
 		}
@@ -177,20 +177,20 @@ 
 		// set style
 		$classes = [ 'mw-lingo-term' ];
 
-		$classes[] = $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ];
+		$classes[ ] = $this->mDefinitions[ 0 ][ self::ELEMENT_STYLE ];
 
 		if ( array_key_exists( 'title', $descriptor ) && $descriptor[ 'title' ] instanceof Title ) {
 
-			if ( !$descriptor['title']->isKnown() ) {
-				$classes[] = 'new';
+			if ( !$descriptor[ 'title' ]->isKnown() ) {
+				$classes[ ] = 'new';
 			}
 
-			if ( $descriptor['title']->isExternal() ) {
-				$classes[] = 'extiw';
+			if ( $descriptor[ 'title' ]->isExternal() ) {
+				$classes[ ] = 'extiw';
 			}
 
 		} else {
-			$classes[] = 'ext';
+			$classes[ ] = 'ext';
 		}
 
 		return array_filter( $classes );

src/Lingo.php

Spacing +6 added lines, -6 removed lines

@@ -42,7 +42,7 @@ 
 	 * @since 2.0.2
 	 */
 	public static function initExtension() {
-		$GLOBALS[ 'wgExtensionFunctions' ][] = function () {
+		$GLOBALS[ 'wgExtensionFunctions' ][ ] = function() {
 			$parser = LingoParser::getInstance();
 
 			$backend = new $GLOBALS[ 'wgexLingoBackend' ]();
@@ -51,13 +51,13 @@ 
 
 			\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 ) {
-				$parser->setHook( 'noglossary', function ( $input, array $args, \Parser $parser, \PPFrame $frame ) {
+			\Hooks::register( 'ParserFirstCallInit', function( \Parser $parser ) {
+				$parser->setHook( 'noglossary', function( $input, array $args, \Parser $parser, \PPFrame $frame ) {
 					$output = $parser->recursiveTagParse( $input, $frame );
 					return '<span class="noglossary">' . $output . '</span>';
 				} );
@@ -65,7 +65,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 ) {
 						if ( $GLOBALS[ 'wgExtensionCredits' ][ 'parserhook' ][ $index ][ 'name' ] === 'Lingo' ) {