@@ -49,7 +49,7 @@ discard block |
||
49 | 49 | * |
50 | 50 | * @param mixed[] $configuration |
51 | 51 | */ |
52 | - public function __construct( array $configuration ) { |
|
52 | + public function __construct(array $configuration) { |
|
53 | 53 | $this->configuration = $configuration; |
54 | 54 | } |
55 | 55 | |
@@ -68,8 +68,8 @@ discard block |
||
68 | 68 | $this->assertAcceptableConfiguration(); |
69 | 69 | |
70 | 70 | $this->registerResourceLoaderModules( |
71 | - $this->isReadablePath( $this->configuration['localBasePath'] ), |
|
72 | - $this->configuration[ 'remoteBasePath' ] |
|
71 | + $this->isReadablePath($this->configuration['localBasePath']), |
|
72 | + $this->configuration['remoteBasePath'] |
|
73 | 73 | ); |
74 | 74 | |
75 | 75 | $this->registerCacheTriggers(); |
@@ -83,9 +83,9 @@ discard block |
||
83 | 83 | * @param string $localBasePath |
84 | 84 | * @param string $remoteBasePath |
85 | 85 | */ |
86 | - protected function registerResourceLoaderModules( $localBasePath, $remoteBasePath ) { |
|
86 | + protected function registerResourceLoaderModules($localBasePath, $remoteBasePath) { |
|
87 | 87 | |
88 | - $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ] = array_replace_recursive( |
|
88 | + $GLOBALS['wgResourceModules']['ext.bootstrap.styles'] = array_replace_recursive( |
|
89 | 89 | [ |
90 | 90 | 'localBasePath' => $localBasePath . '/scss', |
91 | 91 | 'remoteBasePath' => $remoteBasePath . '/scss', |
@@ -99,20 +99,20 @@ discard block |
||
99 | 99 | 'composer.lock' => null, |
100 | 100 | ], |
101 | 101 | ], |
102 | - array_key_exists( 'ext.bootstrap.styles', $GLOBALS[ 'wgResourceModules' ] ) ? $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ] : [] |
|
102 | + array_key_exists('ext.bootstrap.styles', $GLOBALS['wgResourceModules']) ? $GLOBALS['wgResourceModules']['ext.bootstrap.styles'] : [] |
|
103 | 103 | ); |
104 | 104 | |
105 | - $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.scripts' ] = array_replace_recursive( |
|
105 | + $GLOBALS['wgResourceModules']['ext.bootstrap.scripts'] = array_replace_recursive( |
|
106 | 106 | [ |
107 | 107 | 'localBasePath' => $localBasePath . '/js', |
108 | 108 | 'remoteBasePath' => $remoteBasePath . '/js', |
109 | 109 | 'scripts' => [], |
110 | 110 | ], |
111 | - array_key_exists( 'ext.bootstrap.scripts', $GLOBALS[ 'wgResourceModules' ] ) ? $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.scripts' ] : [] |
|
111 | + array_key_exists('ext.bootstrap.scripts', $GLOBALS['wgResourceModules']) ? $GLOBALS['wgResourceModules']['ext.bootstrap.scripts'] : [] |
|
112 | 112 | ); |
113 | 113 | |
114 | - $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap' ] = [ |
|
115 | - 'dependencies' => [ 'ext.bootstrap.styles', 'ext.bootstrap.scripts' ], |
|
114 | + $GLOBALS['wgResourceModules']['ext.bootstrap'] = [ |
|
115 | + 'dependencies' => ['ext.bootstrap.styles', 'ext.bootstrap.scripts'], |
|
116 | 116 | ]; |
117 | 117 | } |
118 | 118 | |
@@ -120,8 +120,8 @@ discard block |
||
120 | 120 | * @param string $id |
121 | 121 | * @return bool |
122 | 122 | */ |
123 | - protected function hasConfiguration( $id ) { |
|
124 | - return isset( $this->configuration[ $id ] ); |
|
123 | + protected function hasConfiguration($id) { |
|
124 | + return isset($this->configuration[$id]); |
|
125 | 125 | } |
126 | 126 | |
127 | 127 | /** |
@@ -129,28 +129,28 @@ discard block |
||
129 | 129 | * @return string |
130 | 130 | * @throws RuntimeException |
131 | 131 | */ |
132 | - protected function isReadablePath( $localBasePath ) { |
|
132 | + protected function isReadablePath($localBasePath) { |
|
133 | 133 | |
134 | - $localBasePath = str_replace( [ '\\', '/' ], DIRECTORY_SEPARATOR, $localBasePath ); |
|
134 | + $localBasePath = str_replace(['\\', '/'], DIRECTORY_SEPARATOR, $localBasePath); |
|
135 | 135 | |
136 | - if ( is_readable( $localBasePath ) ) { |
|
136 | + if (is_readable($localBasePath)) { |
|
137 | 137 | return $localBasePath; |
138 | 138 | } |
139 | 139 | |
140 | - throw new RuntimeException( "Expected an accessible {$localBasePath} path" ); |
|
140 | + throw new RuntimeException("Expected an accessible {$localBasePath} path"); |
|
141 | 141 | } |
142 | 142 | |
143 | 143 | protected function registerCacheTriggers() { |
144 | 144 | |
145 | 145 | $defaultRecacheTriggers = [ |
146 | - 'LocalSettings.php' => $this->configuration[ 'IP' ] . '/LocalSettings.php', |
|
147 | - 'composer.lock' => $this->configuration[ 'IP' ] . '/composer.lock', |
|
146 | + 'LocalSettings.php' => $this->configuration['IP'] . '/LocalSettings.php', |
|
147 | + 'composer.lock' => $this->configuration['IP'] . '/composer.lock', |
|
148 | 148 | ]; |
149 | 149 | |
150 | - foreach ( $defaultRecacheTriggers as $key => $filename ) { |
|
151 | - if ( array_key_exists( $key, $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ][ 'cacheTriggers' ] ) && |
|
152 | - $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ][ 'cacheTriggers' ][ $key ] === null ) { |
|
153 | - $GLOBALS[ 'wgResourceModules' ][ 'ext.bootstrap.styles' ][ 'cacheTriggers' ][ $key ] = $filename; |
|
150 | + foreach ($defaultRecacheTriggers as $key => $filename) { |
|
151 | + if (array_key_exists($key, $GLOBALS['wgResourceModules']['ext.bootstrap.styles']['cacheTriggers']) && |
|
152 | + $GLOBALS['wgResourceModules']['ext.bootstrap.styles']['cacheTriggers'][$key] === null) { |
|
153 | + $GLOBALS['wgResourceModules']['ext.bootstrap.styles']['cacheTriggers'][$key] = $filename; |
|
154 | 154 | } |
155 | 155 | } |
156 | 156 | } |
@@ -163,9 +163,9 @@ discard block |
||
163 | 163 | 'IP' => 'Full path to working directory ($IP) not found.', |
164 | 164 | ]; |
165 | 165 | |
166 | - foreach ( $configElements as $key => $errorMessage ) { |
|
167 | - if ( !$this->hasConfiguration( $key ) ) { |
|
168 | - throw new InvalidArgumentException( $errorMessage ); |
|
166 | + foreach ($configElements as $key => $errorMessage) { |
|
167 | + if (!$this->hasConfiguration($key)) { |
|
168 | + throw new InvalidArgumentException($errorMessage); |
|
169 | 169 | } |
170 | 170 | } |
171 | 171 | } |