ProfessionalWiki / Bootstrap

src/Definition/ModuleDefinition.php

Spacing +1 added lines, -1 removed lines

@@ -38,6 +38,6 @@
 	 *
 	 * @return array
 	 */
-	public function get( $key );
+	public function get($key);
 
 }

src/Definition/V3ModuleDefinition.php

Spacing +49 added lines, -49 removed lines

@@ -32,58 +32,58 @@ 
 class V3ModuleDefinition implements ModuleDefinition {
 
 	static private $moduleDescriptions = array(
-		'variables'            => array( 'styles' => 'variables' ),
-		'mixins'               => array( 'styles' => 'mixins' ),
-		'normalize'            => array( 'styles' => 'normalize' ),
-		'print'                => array( 'styles' => 'print' ),
-		'scaffolding'          => array( 'styles' => 'scaffolding' ),
-		'type'                 => array( 'styles' => 'type' ),
-		'code'                 => array( 'styles' => 'code' ),
-		'grid'                 => array( 'styles' => 'grid' ),
-		'tables'               => array( 'styles' => 'tables' ),
-		'forms'                => array( 'styles' => 'forms' ),
-		'buttons'              => array( 'styles' => 'buttons' ),
-		'component-animations' => array( 'styles' => 'component-animations' ),
-		'glyphicons'           => array( 'styles' => 'glyphicons' ),
-		'dropdowns'            => array( 'styles' => 'dropdowns' ),
-		'button-groups'        => array( 'styles' => 'button-groups' ),
-		'input-groups'         => array( 'styles' => 'input-groups' ),
-		'navs'                 => array( 'styles' => 'navs' ),
-		'navbar'               => array( 'styles' => 'navbar' ),
-		'breadcrumbs'          => array( 'styles' => 'breadcrumbs' ),
-		'pagination'           => array( 'styles' => 'pagination' ),
-		'pager'                => array( 'styles' => 'pager' ),
-		'labels'               => array( 'styles' => 'labels' ),
-		'badges'               => array( 'styles' => 'badges' ),
-		'jumbotron'            => array( 'styles' => 'jumbotron' ),
-		'thumbnails'           => array( 'styles' => 'thumbnails' ),
-		'alerts'               => array( 'styles' => 'alerts' ),
-		'progress-bars'        => array( 'styles' => 'progress-bars' ),
-		'media'                => array( 'styles' => 'media' ),
-		'list-group'           => array( 'styles' => 'list-group' ),
-		'panels'               => array( 'styles' => 'panels' ),
-		'wells'                => array( 'styles' => 'wells' ),
-		'close'                => array( 'styles' => 'close' ),
+		'variables'            => array('styles' => 'variables'),
+		'mixins'               => array('styles' => 'mixins'),
+		'normalize'            => array('styles' => 'normalize'),
+		'print'                => array('styles' => 'print'),
+		'scaffolding'          => array('styles' => 'scaffolding'),
+		'type'                 => array('styles' => 'type'),
+		'code'                 => array('styles' => 'code'),
+		'grid'                 => array('styles' => 'grid'),
+		'tables'               => array('styles' => 'tables'),
+		'forms'                => array('styles' => 'forms'),
+		'buttons'              => array('styles' => 'buttons'),
+		'component-animations' => array('styles' => 'component-animations'),
+		'glyphicons'           => array('styles' => 'glyphicons'),
+		'dropdowns'            => array('styles' => 'dropdowns'),
+		'button-groups'        => array('styles' => 'button-groups'),
+		'input-groups'         => array('styles' => 'input-groups'),
+		'navs'                 => array('styles' => 'navs'),
+		'navbar'               => array('styles' => 'navbar'),
+		'breadcrumbs'          => array('styles' => 'breadcrumbs'),
+		'pagination'           => array('styles' => 'pagination'),
+		'pager'                => array('styles' => 'pager'),
+		'labels'               => array('styles' => 'labels'),
+		'badges'               => array('styles' => 'badges'),
+		'jumbotron'            => array('styles' => 'jumbotron'),
+		'thumbnails'           => array('styles' => 'thumbnails'),
+		'alerts'               => array('styles' => 'alerts'),
+		'progress-bars'        => array('styles' => 'progress-bars'),
+		'media'                => array('styles' => 'media'),
+		'list-group'           => array('styles' => 'list-group'),
+		'panels'               => array('styles' => 'panels'),
+		'wells'                => array('styles' => 'wells'),
+		'close'                => array('styles' => 'close'),
 
 		// Components w/ JavaScript
-		'modals'               => array( 'styles' => 'modals', 'scripts' => 'modal' ),
-		'tooltip'              => array( 'styles' => 'tooltip', 'scripts' => 'tooltip' ),
-		'popovers'             => array( 'styles' => 'popovers', 'scripts' => 'popover', 'dependencies' => 'tooltip' ),
-		'carousel'             => array( 'styles' => 'carousel', 'scripts' => 'carousel' ),
+		'modals'               => array('styles' => 'modals', 'scripts' => 'modal'),
+		'tooltip'              => array('styles' => 'tooltip', 'scripts' => 'tooltip'),
+		'popovers'             => array('styles' => 'popovers', 'scripts' => 'popover', 'dependencies' => 'tooltip'),
+		'carousel'             => array('styles' => 'carousel', 'scripts' => 'carousel'),
 
 		// Utility classes
-		'utilities'            => array( 'styles' => 'utilities' ),
-		'responsive-utilities' => array( 'styles' => 'responsive-utilities' ),
+		'utilities'            => array('styles' => 'utilities'),
+		'responsive-utilities' => array('styles' => 'responsive-utilities'),
 
 		// JS-only components
-		'affix'                => array( 'scripts' => 'affix' ),
-		'alert'                => array( 'scripts' => 'alert' ),
-		'button'               => array( 'scripts' => 'button' ),
-		'collapse'             => array( 'scripts' => 'collapse' ),
-		'dropdown'             => array( 'scripts' => 'dropdown' ),
-		'scrollspy'            => array( 'scripts' => 'scrollspy' ),
-		'tab'                  => array( 'scripts' => 'tab' ),
-		'transition'           => array( 'scripts' => 'transition' ),
+		'affix'                => array('scripts' => 'affix'),
+		'alert'                => array('scripts' => 'alert'),
+		'button'               => array('scripts' => 'button'),
+		'collapse'             => array('scripts' => 'collapse'),
+		'dropdown'             => array('scripts' => 'dropdown'),
+		'scrollspy'            => array('scripts' => 'scrollspy'),
+		'tab'                  => array('scripts' => 'tab'),
+		'transition'           => array('scripts' => 'transition'),
 
 	);
 
@@ -110,9 +110,9 @@ 
 	 * @return array
 	 * @throws InvalidArgumentException
 	 */
-	public function get( $key ) {
+	public function get($key) {
 
-		switch ( $key ) {
+		switch ($key) {
 			case 'core':
 				return self::$coreModules;
 			case 'optional':
@@ -121,7 +121,7 @@ 
 				return self::$moduleDescriptions;
 		}
 
-		throw new InvalidArgumentException( 'Expected a valid key' );
+		throw new InvalidArgumentException('Expected a valid key');
 	}
 
 }

src/BootstrapManager.php

Spacing +33 added lines, -33 removed lines

@@ -47,7 +47,7 @@ 
 	 *
 	 * @param ModuleDefinition $moduleDefinition
 	 */
-	public function __construct( ModuleDefinition $moduleDefinition ) {
+	public function __construct(ModuleDefinition $moduleDefinition) {
 		$this->moduleDefinition = $moduleDefinition;
 		$this->initCoreModules();
 	}
@@ -61,8 +61,8 @@ 
 	 */
 	public static function getInstance() {
 
-		if ( self::$instance === null ) {
-			self::$instance = new self( new V3ModuleDefinition );
+		if (self::$instance === null) {
+			self::$instance = new self(new V3ModuleDefinition);
 		}
 
 		return self::$instance;
@@ -82,29 +82,29 @@ 
 	 *
 	 * @param string|string[] $modules
 	 */
-	public function addBootstrapModule( $modules ) {
+	public function addBootstrapModule($modules) {
 
 		$modules = (array) $modules;
 
-		foreach ( $modules as $module ) {
+		foreach ($modules as $module) {
 
 			// if the module is known
-			if ( isset( $this->mModuleDescriptions[ $module ] ) ) {
+			if (isset($this->mModuleDescriptions[$module])) {
 
-				$description = $this->mModuleDescriptions[ $module ];
+				$description = $this->mModuleDescriptions[$module];
 
 				// prevent adding this module again; this also prevents infinite recursion in case
 				// of dependency resolution
-				unset( $this->mModuleDescriptions[ $module ] );
+				unset($this->mModuleDescriptions[$module]);
 
 				// first add any dependencies recursively, so they are available when the styles and
 				// scripts of $module are loaded
-				if ( isset( $description[ 'dependencies' ] ) ) {
-					$this->addBootstrapModule( $description[ 'dependencies' ] );
+				if (isset($description['dependencies'])) {
+					$this->addBootstrapModule($description['dependencies']);
 				}
 
-				$this->addFilesToGlobalResourceModules( 'styles', $description, '.less' );
-				$this->addFilesToGlobalResourceModules( 'scripts', $description, '.js' );
+				$this->addFilesToGlobalResourceModules('styles', $description, '.less');
+				$this->addFilesToGlobalResourceModules('scripts', $description, '.js');
 
 			}
 		}
@@ -116,18 +116,18 @@ 
 	 * @param mixed[]      $description
 	 * @param              $fileExt
 	 */
-	protected function addFilesToGlobalResourceModules ( $filetype, $description, $fileExt ) {
+	protected function addFilesToGlobalResourceModules($filetype, $description, $fileExt) {
 
-		if ( isset( $description[ $filetype ] ) ) {
+		if (isset($description[$filetype])) {
 
 			$files = array_map(
-				function ( $filename ) use ( $fileExt ) {
+				function($filename) use ($fileExt) {
 					return $filename . $fileExt;
 				},
-				(array) $description[ $filetype ]
+				(array) $description[$filetype]
 			);
 
-			$this->adjustArrayElementOfResourceModuleDescription( $filetype, $files, $filetype );
+			$this->adjustArrayElementOfResourceModuleDescription($filetype, $files, $filetype);
 
 		}
 	}
@@ -138,7 +138,7 @@ 
 	 * @since  1.0
 	 */
 	public function addAllBootstrapModules() {
-		$this->addBootstrapModule( $this->moduleDefinition->get( 'optional' ) );
+		$this->addBootstrapModule($this->moduleDefinition->get('optional'));
 	}
 
 	/**
@@ -149,8 +149,8 @@ 
 	 *
 	 * @internal param string $path
 	 */
-	public function addExternalModule( $file, $remotePath = '' ) {
-		$this->adjustArrayElementOfResourceModuleDescription( 'external styles', array( $file => $remotePath ) );
+	public function addExternalModule($file, $remotePath = '') {
+		$this->adjustArrayElementOfResourceModuleDescription('external styles', array($file => $remotePath));
 	}
 
 	/**
@@ -159,8 +159,8 @@ 
 	 * @param string $key   the LESS variable name
 	 * @param string $value the value to assign to the variable
 	 */
-	public function setLessVariable( $key, $value ) {
-		$this->setLessVariables( array( $key => $value ) );
+	public function setLessVariable($key, $value) {
+		$this->setLessVariables(array($key => $value));
 	}
 
 	/**
@@ -168,21 +168,21 @@ 
 	 *
 	 * @param mixed[] $variables
 	 */
-	public function setLessVariables( $variables ) {
-		$this->adjustArrayElementOfResourceModuleDescription( 'variables', $variables );
+	public function setLessVariables($variables) {
+		$this->adjustArrayElementOfResourceModuleDescription('variables', $variables);
 	}
 
 	/**
 	 * @since 1.1
 	 * @param string|string[] $files
 	 */
-	public function addCacheTriggerFile( $files ){
-		$this->adjustArrayElementOfResourceModuleDescription( 'cachetriggers', $files );
+	public function addCacheTriggerFile($files) {
+		$this->adjustArrayElementOfResourceModuleDescription('cachetriggers', $files);
 	}
 
 	protected function initCoreModules() {
-		$this->mModuleDescriptions = $this->moduleDefinition->get( 'descriptions' );
-		$this->addBootstrapModule( $this->moduleDefinition->get( 'core' ) );
+		$this->mModuleDescriptions = $this->moduleDefinition->get('descriptions');
+		$this->addBootstrapModule($this->moduleDefinition->get('core'));
 	}
 
 	/**
@@ -190,14 +190,14 @@ 
 	 * @param mixed  $value
 	 * @param string $filetype 'styles'|'scripts'
 	 */
-	protected function adjustArrayElementOfResourceModuleDescription( $key, $value, $filetype = 'styles' ) {
+	protected function adjustArrayElementOfResourceModuleDescription($key, $value, $filetype = 'styles') {
 
-		if (!isset($GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.' . $filetype ][ $key ])) {
-			$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.' . $filetype ][ $key ] = $value;
+		if (!isset($GLOBALS['wgResourceModules']['ext.bootstrap.' . $filetype][$key])) {
+			$GLOBALS['wgResourceModules']['ext.bootstrap.' . $filetype][$key] = $value;
 		} else {
-			$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.' . $filetype ][ $key ] =
+			$GLOBALS['wgResourceModules']['ext.bootstrap.' . $filetype][$key] =
 				array_merge(
-					$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.' . $filetype ][ $key ],
+					$GLOBALS['wgResourceModules']['ext.bootstrap.' . $filetype][$key],
 					(array) $value
 				);
 		}

src/Hooks/SetupAfterCache.php

Spacing +31 added lines, -31 removed lines

@@ -48,7 +48,7 @@ 
 	 *
 	 * @param array $configuration
 	 */
-	public function __construct( array $configuration ) {
+	public function __construct(array $configuration) {
 		$this->configuration = $configuration;
 	}
 
@@ -65,8 +65,8 @@ 
 		$this->removeLegacyLessCompilerFromComposerAutoloader();
 
 		$this->registerBootstrapResourcePaths(
-			$this->isReadablePath( $this->configuration['localBasePath'] ),
-			$this->configuration[ 'remoteBasePath' ]
+			$this->isReadablePath($this->configuration['localBasePath']),
+			$this->configuration['remoteBasePath']
 		);
 
 		$this->registerCacheTriggers();
@@ -79,23 +79,23 @@ 
 	 * @param string $localBasePath
 	 * @param string $remoteBasePath
 	 */
-	protected function registerBootstrapResourcePaths( $localBasePath, $remoteBasePath ) {
+	protected function registerBootstrapResourcePaths($localBasePath, $remoteBasePath) {
 
-		$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ] = array_replace_recursive( array(
+		$GLOBALS['wgResourceModules']['ext.bootstrap.styles'] = array_replace_recursive(array(
 				'localBasePath'  => $localBasePath . '/less',
 				'remoteBasePath' => $remoteBasePath . '/less',
 				'variables'      => array(
 					'icon-font-path' => "\"$remoteBasePath/fonts/\"",
 				),
 			),
-			$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ]
+			$GLOBALS['wgResourceModules']['ext.bootstrap.styles']
 		);
 
-		$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.scripts' ] = array_replace_recursive( array(
+		$GLOBALS['wgResourceModules']['ext.bootstrap.scripts'] = array_replace_recursive(array(
 				'localBasePath'  => $localBasePath . '/js',
 				'remoteBasePath' => $remoteBasePath . '/js',
 			),
-			$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.scripts' ]
+			$GLOBALS['wgResourceModules']['ext.bootstrap.scripts']
 		);
 	}
 
@@ -114,23 +114,23 @@ 
 
 		$autoloadFunctions = spl_autoload_functions();
 
-		foreach ( $autoloadFunctions as $autoloadFunction ) {
+		foreach ($autoloadFunctions as $autoloadFunction) {
 
-			if ( is_object( $autoloadFunction ) && ( $autoloadFunction instanceof Closure ) ) {
+			if (is_object($autoloadFunction) && ($autoloadFunction instanceof Closure)) {
 				continue;
 			}
 
-			$classLoader = $autoloadFunction[ 0 ];
+			$classLoader = $autoloadFunction[0];
 
-			if ( is_a( $classLoader, '\Composer\Autoload\ClassLoader' ) ) {
+			if (is_a($classLoader, '\Composer\Autoload\ClassLoader')) {
 
 				$classMap = $classLoader->getClassMap();
 
-				if ( !is_array( $classMap ) ||
-					!array_key_exists( 'lessc', $classMap ) ||
-					strpos( $classMap[ 'lessc' ], '/less.php/less.php/lessc.inc.php') !== false ) {
+				if (!is_array($classMap) ||
+					!array_key_exists('lessc', $classMap) ||
+					strpos($classMap['lessc'], '/less.php/less.php/lessc.inc.php') !== false) {
 
-					$classLoader->addClassMap( array( 'lessc' => null ) );
+					$classLoader->addClassMap(array('lessc' => null));
 				}
 				break;
 			}
@@ -142,8 +142,8 @@ 
 	 * @param string $id
 	 * @return bool
 	 */
-	protected function hasConfiguration( $id ) {
-		return isset( $this->configuration[ $id ] );
+	protected function hasConfiguration($id) {
+		return isset($this->configuration[$id]);
 	}
 
 	/**
@@ -151,28 +151,28 @@ 
 	 * @return string
 	 * @throws RuntimeException
 	 */
-	protected function isReadablePath( $localBasePath ) {
+	protected function isReadablePath($localBasePath) {
 
-		$localBasePath = str_replace( array( '\\', '/' ), DIRECTORY_SEPARATOR, $localBasePath );
+		$localBasePath = str_replace(array('\\', '/'), DIRECTORY_SEPARATOR, $localBasePath);
 
-		if ( is_readable( $localBasePath ) ) {
+		if (is_readable($localBasePath)) {
 			return $localBasePath;
 		}
 
-		throw new RuntimeException( "Expected an accessible {$localBasePath} path" );
+		throw new RuntimeException("Expected an accessible {$localBasePath} path");
 	}
 
 	protected function registerCacheTriggers() {
 
 		$defaultRecacheTriggers = array(
-			'LocalSettings.php' => $this->configuration[ 'IP' ] . '/LocalSettings.php',
-			'composer.lock' => $this->configuration[ 'IP' ] . '/composer.lock',
+			'LocalSettings.php' => $this->configuration['IP'] . '/LocalSettings.php',
+			'composer.lock' => $this->configuration['IP'] . '/composer.lock',
 		);
 
-		foreach ( $defaultRecacheTriggers as $key => $filename ) {
-			if ( array_key_exists( $key, $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ][ 'cachetriggers' ] ) &&
-				$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ][ 'cachetriggers' ][ $key ] === null ) {
-				$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ][ 'cachetriggers' ][ $key ] = $filename;
+		foreach ($defaultRecacheTriggers as $key => $filename) {
+			if (array_key_exists($key, $GLOBALS['wgResourceModules']['ext.bootstrap.styles']['cachetriggers']) &&
+				$GLOBALS['wgResourceModules']['ext.bootstrap.styles']['cachetriggers'][$key] === null) {
+				$GLOBALS['wgResourceModules']['ext.bootstrap.styles']['cachetriggers'][$key] = $filename;
 			}
 		}
 	}
@@ -185,9 +185,9 @@ 
 			'IP' => 'Full path to working directory ($IP) not found.',
 		);
 
-		foreach ( $configElements as $key => $errorMessage ) {
-			if ( !$this->hasConfiguration( $key ) ) {
-				throw new InvalidArgumentException( $errorMessage );
+		foreach ($configElements as $key => $errorMessage) {
+			if (!$this->hasConfiguration($key)) {
+				throw new InvalidArgumentException($errorMessage);
 			}
 		}
 	}

src/ResourceLoaderBootstrapModule.php

Spacing +46 added lines, -46 removed lines

@@ -60,12 +60,12 @@ 
 
 	protected $styleText = null;
 
-	public function __construct( $options = array(), $localBasePath = null, $remoteBasePath = null
+	public function __construct($options = array(), $localBasePath = null, $remoteBasePath = null
 	) {
 
-		parent::__construct( $options, $localBasePath, $remoteBasePath );
+		parent::__construct($options, $localBasePath, $remoteBasePath);
 
-		$this->applyOptions( $options );
+		$this->applyOptions($options);
 	}
 
 	/**
@@ -75,18 +75,18 @@ 
 	 *
 	 * @return array
 	 */
-	public function getStyles( ResourceLoaderContext $context ) {
+	public function getStyles(ResourceLoaderContext $context) {
 
-		if ( $this->styleText === null ) {
+		if ($this->styleText === null) {
 
-			$this->retrieveStylesFromCache( $context );
+			$this->retrieveStylesFromCache($context);
 
-			if ( $this->styleText === null ) {
-				$this->compileStyles( $context );
+			if ($this->styleText === null) {
+				$this->compileStyles($context);
 			}
 		}
 
-		return array( 'all' => $this->styleText );
+		return array('all' => $this->styleText);
 	}
 
 	/**
@@ -103,79 +103,79 @@ 
 	 *
 	 * @param BagOStuff $cache
 	 */
-	public function setCache( BagOStuff $cache ) {
+	public function setCache(BagOStuff $cache) {
 		$this->cache = $cache;
 	}
 
 	protected function getCache() {
 
-		if ( $this->cache === null ) {
-			$this->cache = wfGetCache( CACHE_ANYTHING );
+		if ($this->cache === null) {
+			$this->cache = wfGetCache(CACHE_ANYTHING);
 		}
 
 		return $this->cache;
 	}
 
-	protected function getCacheKey( ResourceLoaderContext $context ) {
-		return wfMemcKey( 'ext', 'bootstrap', $context->getHash() );
+	protected function getCacheKey(ResourceLoaderContext $context) {
+		return wfMemcKey('ext', 'bootstrap', $context->getHash());
 	}
 
-	protected function retrieveStylesFromCache( ResourceLoaderContext $context ) {
+	protected function retrieveStylesFromCache(ResourceLoaderContext $context) {
 
 		// Try for cache hit
-		$cacheResult = $this->getCache()->get( $this->getCacheKey( $context ) );
+		$cacheResult = $this->getCache()->get($this->getCacheKey($context));
 
-		if ( is_array( $cacheResult ) ) {
+		if (is_array($cacheResult)) {
 
-			if ( $this->isCacheOutdated( $cacheResult[ 'storetime' ] ) ) {
-				wfDebug( __METHOD__ . " ext.bootstrap: Cache miss: Cache outdated.\n" );
+			if ($this->isCacheOutdated($cacheResult['storetime'])) {
+				wfDebug(__METHOD__ . " ext.bootstrap: Cache miss: Cache outdated.\n");
 			} else {
-				$this->styleText = $cacheResult[ 'styles' ];
-				wfDebug( __METHOD__ . " ext.bootstrap: Cache hit: Got styles from cache.\n" );
+				$this->styleText = $cacheResult['styles'];
+				wfDebug(__METHOD__ . " ext.bootstrap: Cache hit: Got styles from cache.\n");
 			}
 
 		} else {
-			wfDebug( __METHOD__ .  " ext.bootstrap: Cache miss: Styles not found in cache.\n" );
+			wfDebug(__METHOD__ . " ext.bootstrap: Cache miss: Styles not found in cache.\n");
 		}
 	}
 
-	protected function updateCache( ResourceLoaderContext $context ) {
+	protected function updateCache(ResourceLoaderContext $context) {
 
 		$this->getCache()->set(
-			$this->getCacheKey( $context ),
-			array( 'styles' => $this->styleText, 'storetime' => time() )
+			$this->getCacheKey($context),
+			array('styles' => $this->styleText, 'storetime' => time())
 		);
 	}
 
-	protected function purgeCache( ResourceLoaderContext $context ) {
-		$this->getCache()->delete( $this->getCacheKey( $context ) );
+	protected function purgeCache(ResourceLoaderContext $context) {
+		$this->getCache()->delete($this->getCacheKey($context));
 	}
 
-	protected function compileStyles( ResourceLoaderContext $context ) {
+	protected function compileStyles(ResourceLoaderContext $context) {
 
 		$lessParser = new Less_Parser();
-		$remotePath = $this->getRemotePath( '' );
+		$remotePath = $this->getRemotePath('');
 
 		try {
 
-			foreach ( $this->styles as $style ) {
-				$lessParser->parseFile( $this->getLocalPath( $style ), $remotePath );
+			foreach ($this->styles as $style) {
+				$lessParser->parseFile($this->getLocalPath($style), $remotePath);
 			}
 
-			foreach ( $this->extStyles as $stylefile => $remotePath ) {
-				$lessParser->parseFile( $stylefile, $remotePath );
+			foreach ($this->extStyles as $stylefile => $remotePath) {
+				$lessParser->parseFile($stylefile, $remotePath);
 			}
 
-			$lessParser->ModifyVars( $this->variables );
+			$lessParser->ModifyVars($this->variables);
 
 			$this->styleText = $lessParser->getCss();
 
-			$this->updateCache( $context );
+			$this->updateCache($context);
 
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 
-			$this->purgeCache( $context );
-			wfDebug( $e->getMessage() );
+			$this->purgeCache($context);
+			wfDebug($e->getMessage());
 			$this->styleText = '/* LESS compile error: ' . $e->getMessage() . '*/';
 		}
 
@@ -184,17 +184,17 @@ 
 	/**
 	 * @param mixed[] $options
 	 */
-	protected function applyOptions( $options ) {
-		$mapConfigToLocalVar = array (
+	protected function applyOptions($options) {
+		$mapConfigToLocalVar = array(
 			'variables'       => 'variables',
 			'paths'           => 'paths',
 			'external styles' => 'extStyles',
 			'cachetriggers'   => 'cacheTriggers',
 		);
 
-		foreach ( $mapConfigToLocalVar as $config => $local ) {
-			if ( isset( $options[ $config ] ) ) {
-				$this->$local = $options[ $config ];
+		foreach ($mapConfigToLocalVar as $config => $local) {
+			if (isset($options[$config])) {
+				$this->$local = $options[$config];
 			}
 		}
 	}
@@ -204,10 +204,10 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function isCacheOutdated( $cacheStoreTime ) {
+	protected function isCacheOutdated($cacheStoreTime) {
 
-		foreach ( $this->cacheTriggers as $triggerFile ) {
-			if ( $triggerFile !== null && $cacheStoreTime < filemtime( $triggerFile ) ) {
+		foreach ($this->cacheTriggers as $triggerFile) {
+			if ($triggerFile !== null && $cacheStoreTime < filemtime($triggerFile)) {
 				return true;
 			}
 		}

Bootstrap.i18n.php

Spacing +12 added lines, -12 removed lines

@@ -11,22 +11,22 @@
  * This shim maintains compatibility back to MediaWiki 1.17.
  */
 $messages = array();
-if ( !function_exists( 'wfJsonI18nShim8d1eb754050adb9a' ) ) {
-	function wfJsonI18nShim8d1eb754050adb9a( $cache, $code, &$cachedData ) {
-		$codeSequence = array_merge( array( $code ), $cachedData['fallbackSequence'] );
-		foreach ( $codeSequence as $csCode ) {
-			$fileName = dirname( __FILE__ ) . "/i18n/$csCode.json";
-			if ( is_readable( $fileName ) ) {
-				$data = FormatJson::decode( file_get_contents( $fileName ), true );
-				foreach ( array_keys( $data ) as $key ) {
-					if ( $key === '' || $key[0] === '@' ) {
-						unset( $data[$key] );
+if (!function_exists('wfJsonI18nShim8d1eb754050adb9a')) {
+	function wfJsonI18nShim8d1eb754050adb9a($cache, $code, &$cachedData) {
+		$codeSequence = array_merge(array($code), $cachedData['fallbackSequence']);
+		foreach ($codeSequence as $csCode) {
+			$fileName = dirname(__FILE__) . "/i18n/$csCode.json";
+			if (is_readable($fileName)) {
+				$data = FormatJson::decode(file_get_contents($fileName), true);
+				foreach (array_keys($data) as $key) {
+					if ($key === '' || $key[0] === '@') {
+						unset($data[$key]);
 					}
 				}
-				$cachedData['messages'] = array_merge( $data, $cachedData['messages'] );
+				$cachedData['messages'] = array_merge($data, $cachedData['messages']);
 			}
 
-			$cachedData['deps'][] = new FileDependency( $fileName );
+			$cachedData['deps'][] = new FileDependency($fileName);
 		}
 		return true;
 	}

Bootstrap.php

Spacing +26 added lines, -26 removed lines

@@ -35,26 +35,26 @@ 
  *
  * @codeCoverageIgnore
  */
-call_user_func( function () {
+call_user_func(function() {
 
-	if ( !defined( 'MEDIAWIKI' ) ) {
-		die( 'This file is part of the MediaWiki extension Bootstrap, it is not a valid entry point.' );
+	if (!defined('MEDIAWIKI')) {
+		die('This file is part of the MediaWiki extension Bootstrap, it is not a valid entry point.');
 	}
 
-	if ( version_compare( $GLOBALS[ 'wgVersion' ], '1.22', 'lt' ) ) {
-		die( '<b>Error:</b> This version of <a href="https://www.mediawiki.org/wiki/Extension:Bootstrap">Bootstrap</a> is only compatible with MediaWiki 1.22 or above. You need to upgrade MediaWiki first.' );
+	if (version_compare($GLOBALS['wgVersion'], '1.22', 'lt')) {
+		die('<b>Error:</b> This version of <a href="https://www.mediawiki.org/wiki/Extension:Bootstrap">Bootstrap</a> is only compatible with MediaWiki 1.22 or above. You need to upgrade MediaWiki first.');
 	}
 
 	/**
 	 * The extension version
 	 */
-	define( 'BS_VERSION', '1.3.0' );
+	define('BS_VERSION', '1.3.0');
 
 	// register the extension
-	$GLOBALS[ 'wgExtensionCredits' ][ 'other' ][ ] = array(
+	$GLOBALS['wgExtensionCredits']['other'][] = array(
 		'path'           => __FILE__,
 		'name'           => 'Bootstrap',
-		'author' => array( '[https://www.mediawiki.org/wiki/User:F.trott Stephan Gambke]', 'James Hong Kong' ),
+		'author' => array('[https://www.mediawiki.org/wiki/User:F.trott Stephan Gambke]', 'James Hong Kong'),
 		'url'            => 'https://www.mediawiki.org/wiki/Extension:Bootstrap',
 		'descriptionmsg' => 'bootstrap-desc',
 		'version'        => BS_VERSION,
@@ -62,35 +62,35 @@ 
 	);
 
 	// register message files
-	$GLOBALS[ 'wgMessagesDirs' ][ 'Bootstrap' ] = __DIR__ . '/i18n';
-	$GLOBALS[ 'wgExtensionMessagesFiles' ][ 'Bootstrap' ] = __DIR__ . '/Bootstrap.i18n.php';
+	$GLOBALS['wgMessagesDirs']['Bootstrap'] = __DIR__ . '/i18n';
+	$GLOBALS['wgExtensionMessagesFiles']['Bootstrap'] = __DIR__ . '/Bootstrap.i18n.php';
 
 	// register classes
-	$GLOBALS[ 'wgAutoloadClasses' ][ 'Bootstrap\ResourceLoaderBootstrapModule' ] = __DIR__ . '/src/ResourceLoaderBootstrapModule.php';
-	$GLOBALS[ 'wgAutoloadClasses' ][ 'Bootstrap\BootstrapManager' ]      = __DIR__ . '/src/BootstrapManager.php';
-	$GLOBALS[ 'wgAutoloadClasses' ][ 'Bootstrap\Hooks\SetupAfterCache' ] = __DIR__ . '/src/Hooks/SetupAfterCache.php';
-	$GLOBALS[ 'wgAutoloadClasses' ][ 'Bootstrap\Definition\ModuleDefinition' ]   = __DIR__ . '/src/Definition/ModuleDefinition.php';
-	$GLOBALS[ 'wgAutoloadClasses' ][ 'Bootstrap\Definition\V3ModuleDefinition' ] = __DIR__ . '/src/Definition/V3ModuleDefinition.php';
+	$GLOBALS['wgAutoloadClasses']['Bootstrap\ResourceLoaderBootstrapModule'] = __DIR__ . '/src/ResourceLoaderBootstrapModule.php';
+	$GLOBALS['wgAutoloadClasses']['Bootstrap\BootstrapManager']      = __DIR__ . '/src/BootstrapManager.php';
+	$GLOBALS['wgAutoloadClasses']['Bootstrap\Hooks\SetupAfterCache'] = __DIR__ . '/src/Hooks/SetupAfterCache.php';
+	$GLOBALS['wgAutoloadClasses']['Bootstrap\Definition\ModuleDefinition']   = __DIR__ . '/src/Definition/ModuleDefinition.php';
+	$GLOBALS['wgAutoloadClasses']['Bootstrap\Definition\V3ModuleDefinition'] = __DIR__ . '/src/Definition/V3ModuleDefinition.php';
 
-	$GLOBALS[ 'wgHooks' ][ 'SetupAfterCache' ][ ] = function() {
+	$GLOBALS['wgHooks']['SetupAfterCache'][] = function() {
 
 		$configuration = array();
-		$configuration[ 'IP' ] = $GLOBALS[ 'IP' ];
-		$configuration[ 'remoteBasePath' ] = $GLOBALS[ 'wgExtensionAssetsPath' ] . '/Bootstrap/resources/bootstrap';
+		$configuration['IP'] = $GLOBALS['IP'];
+		$configuration['remoteBasePath'] = $GLOBALS['wgExtensionAssetsPath'] . '/Bootstrap/resources/bootstrap';
 
-		if ( isset( $GLOBALS[ 'wgExtensionDirectory' ] ) ) { // MW >= 1.25
-			$configuration[ 'localBasePath' ] = $GLOBALS[ 'wgExtensionDirectory' ] . '/Bootstrap/resources/bootstrap';
+		if (isset($GLOBALS['wgExtensionDirectory'])) { // MW >= 1.25
+			$configuration['localBasePath'] = $GLOBALS['wgExtensionDirectory'] . '/Bootstrap/resources/bootstrap';
 		} else {
-			$configuration[ 'localBasePath' ] = __DIR__ . '/resources/bootstrap';
+			$configuration['localBasePath'] = __DIR__ . '/resources/bootstrap';
 		}
 
-		$setupAfterCache = new \Bootstrap\Hooks\SetupAfterCache( $configuration );
+		$setupAfterCache = new \Bootstrap\Hooks\SetupAfterCache($configuration);
 		$setupAfterCache->process();
 	};
 
 	// register skeleton resource module with the Resource Loader
 	// do not add paths, globals are not set yet
-	$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ] = array(
+	$GLOBALS['wgResourceModules']['ext.bootstrap.styles'] = array(
 		'class'          => 'Bootstrap\ResourceLoaderBootstrapModule',
 		'position'       => 'top',
 		'styles'         => array(),
@@ -102,12 +102,12 @@ 
 		),
 	);
 
-	$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.scripts' ] = array(
+	$GLOBALS['wgResourceModules']['ext.bootstrap.scripts'] = array(
 		'scripts'        => array(),
 	);
 
-	$GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap' ] = array(
-		'dependencies' => array( 'ext.bootstrap.styles', 'ext.bootstrap.scripts' ),
+	$GLOBALS['wgResourceModules']['ext.bootstrap'] = array(
+		'dependencies' => array('ext.bootstrap.styles', 'ext.bootstrap.scripts'),
 	);
 
 } );