wikimedia / mediawiki-extensions-Lingo

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/BasicBackend.php

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/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/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 ) {

tests/mw-phpunit-runner.php

Spacing +3 added lines, -3 removed lines

@@ -68,11 +68,11 @@
 	return $directory;
 }
 
-$extDirectory = dirname ( getDirectory() );
+$extDirectory = dirname( getDirectory() );
 
 $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' ] ) ) );

tests/phpunit/Util/XmlFileProvider.php

Spacing +3 added lines, -3 removed lines

@@ -78,10 +78,10 @@
 	protected function loadXmlFiles( $path ) {
 
 		$directoryIterator = new \RecursiveDirectoryIterator( $path );
-		$iteratorIterator = new \RecursiveIteratorIterator($directoryIterator);
-		$regexIterator = new \RegexIterator($iteratorIterator, '/^.+\.xml$/i', \RecursiveRegexIterator::GET_MATCH);
+		$iteratorIterator = new \RecursiveIteratorIterator( $directoryIterator );
+		$regexIterator = new \RegexIterator( $iteratorIterator, '/^.+\.xml$/i', \RecursiveRegexIterator::GET_MATCH );
 
-		$files = call_user_func_array('array_merge', iterator_to_array( $regexIterator ) );
+		$files = call_user_func_array( 'array_merge', iterator_to_array( $regexIterator ) );
 
 		return $files;
 	}

tests/phpunit/Integration/ArticleAnnotationTest.php

Spacing +4 added lines, -4 removed lines

@@ -94,7 +94,7 @@ 
 
 	public function provideData() {
 
-		$data = [];
+		$data = [ ];
 
 		$xmlFileProvider = new XmlFileProvider( __DIR__ . '/../Fixture/articleAnnotation' );
 		$files = $xmlFileProvider->getFiles();
@@ -111,7 +111,7 @@ 
 			$testCase = [
 				0 => substr( $file, strlen( __DIR__ . '/../Fixture/articleAnnotation' ) ),
 				1 => trim( $decoded[ 'text' ] ),
-				2 => [],
+				2 => [ ],
 				3 => trim( $decoded[ 'expected' ] ) . "\n",
 			];
 
@@ -120,12 +120,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/phpunit/Unit/LingoI18NTest.php

Spacing +3 added lines, -3 removed lines

@@ -36,9 +36,9 @@ 
  */
 class LingoI18NTest extends \PHPUnit\Framework\TestCase {
 
-	public function testJsonSchemaValidatorExists(){
+	public function testJsonSchemaValidatorExists() {
 		$this->assertTrue( class_exists( '\JsonSchema\Validator' ) );
-		$this->assertTrue( method_exists('\JsonSchema\Validator', 'check' ) );
+		$this->assertTrue( method_exists( '\JsonSchema\Validator', 'check' ) );
 	}
 
 	public function testMagicWordsLoaded() {
@@ -60,7 +60,7 @@ 
 
 		// format error message
 		$errors = implode( '', array_map(
-			function ( $error ) {
+			function( $error ) {
 				return "* [{$error[ 'property' ]}] {$error[ 'message' ]}\n";
 			},
 			$validator->getErrors()