Completed
Pull Request — master (#1455)
by
unknown
01:43
created

Config   D

Complexity

Total Complexity 59

Size/Duplication

Total Lines 401
Duplicated Lines 20.45 %

Coupling/Cohesion

Components 3
Dependencies 2

Test Coverage

Coverage 100%

Importance

Changes 0
Metric Value
wmc 59
lcom 3
cbo 2
dl 82
loc 401
ccs 141
cts 141
cp 1
rs 4.08
c 0
b 0
f 0

20 Methods

Rating   Name   Duplication   Size   Complexity  
A __construct() 0 5 1
A fromYaml() 13 14 2
A fromJson() 11 12 2
A fromPhp() 18 18 2
A getEnvironments() 0 15 5
A getEnvironment() 0 15 3
A hasEnvironment() 0 4 1
B getDefaultEnvironment() 0 37 7
A getAlias() 0 4 2
A getConfigFilePath() 0 4 1
A getMigrationBaseClassName() 0 6 4
A getTemplateFile() 8 8 2
A getTemplateClass() 8 8 2
A getBootstrapFile() 0 8 2
A replaceTokens() 0 18 3
A recurseArrayForTokens() 0 20 5
A offsetSet() 0 4 1
A offsetGet() 0 8 3
A offsetExists() 0 4 1
A offsetUnset() 0 4 1

How to fix   Duplicated Code    Complexity   

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

Complex Class

 Tip:   Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.

Complex classes like Config often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use Config, and based on these observations, apply Extract Interface, too.

1
<?php
2
/**
3
 * Phinx
4
 *
5
 * (The MIT license)
6
 * Copyright (c) 2015 Rob Morgan
7
 *
8
 * Permission is hereby granted, free of charge, to any person obtaining a copy
9
 * of this software and associated * documentation files (the "Software"), to
10
 * deal in the Software without restriction, including without limitation the
11
 * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or
12
 * sell copies of the Software, and to permit persons to whom the Software is
13
 * furnished to do so, subject to the following conditions:
14
 *
15
 * The above copyright notice and this permission notice shall be included in
16
 * all copies or substantial portions of the Software.
17
 *
18
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
19
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
20
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
21
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
22
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
23
 * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
24
 * IN THE SOFTWARE.
25
 *
26
 * @package    Phinx
27
 * @subpackage Phinx\Config
28
 */
29
namespace Phinx\Config;
30
31
use Symfony\Component\Yaml\Yaml;
32
33
/**
34
 * Phinx configuration class.
35
 *
36
 * @package Phinx
37
 * @author Rob Morgan
38
 */
39
class Config implements ConfigInterface, NamespaceAwareInterface
40
{
41
    use NamespaceAwareTrait;
42
43
    /**
44
     * The value that identifies a version order by creation time.
45
     */
46
    const VERSION_ORDER_CREATION_TIME = 'creation';
47
48
    /**
49
     * The value that identifies a version order by execution time.
50
     */
51
    const VERSION_ORDER_EXECUTION_TIME = 'execution';
52
53
    /**
54
     * @var array
55
     */
56
    private $values = [];
57
58
    /**
59
     * @var string
60
     */
61
    protected $configFilePath;
62
63
    /**
64
     * {@inheritdoc}
65
     */
66 448
    public function __construct(array $configArray, $configFilePath = null)
67
    {
68 448
        $this->configFilePath = $configFilePath;
69 448
        $this->values = $this->replaceTokens($configArray);
70 448
    }
71
72
    /**
73
     * Create a new instance of the config class using a Yaml file path.
74
     *
75
     * @param  string $configFilePath Path to the Yaml File
76
     * @throws \RuntimeException
77
     * @return \Phinx\Config\Config
78
     */
79 2 View Code Duplication
    public static function fromYaml($configFilePath)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
80
    {
81 2
        $configFile = file_get_contents($configFilePath);
82 2
        $configArray = Yaml::parse($configFile);
83
84 2
        if (!is_array($configArray)) {
85 1
            throw new \RuntimeException(sprintf(
86 1
                'File \'%s\' must be valid YAML',
87
                $configFilePath
88 1
            ));
89
        }
90 1
91
        return new static($configArray, $configFilePath);
92
    }
93
94
    /**
95
     * Create a new instance of the config class using a JSON file path.
96
     *
97
     * @param  string $configFilePath Path to the JSON File
98
     * @throws \RuntimeException
99
     * @return \Phinx\Config\Config
100 2
     */
101 View Code Duplication
    public static function fromJson($configFilePath)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
102 2
    {
103 2
        $configArray = json_decode(file_get_contents($configFilePath), true);
104 1
        if (!is_array($configArray)) {
105 1
            throw new \RuntimeException(sprintf(
106
                'File \'%s\' must be valid JSON',
107 1
                $configFilePath
108
            ));
109 1
        }
110
111
        return new static($configArray, $configFilePath);
112
    }
113
114
    /**
115
     * Create a new instance of the config class using a PHP file path.
116
     *
117
     * @param  string $configFilePath Path to the PHP File
118
     * @throws \RuntimeException
119 3
     * @return \Phinx\Config\Config
120
     */
121 3 View Code Duplication
    public static function fromPhp($configFilePath)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
122
    {
123 3
        ob_start();
124
        /** @noinspection PhpIncludeInspection */
125
        $configArray = include($configFilePath);
126 3
127
        // Hide console output
128 3
        ob_end_clean();
129 2
130 2
        if (!is_array($configArray)) {
131
            throw new \RuntimeException(sprintf(
132 2
                'PHP file \'%s\' must return an array',
133
                $configFilePath
134
            ));
135 1
        }
136
137
        return new static($configArray, $configFilePath);
138
    }
139
140
    /**
141 25
     * {@inheritdoc}
142
     */
143 25
    public function getEnvironments()
144 24
    {
145 24
        if (isset($this->values) && isset($this->values['environments'])) {
146 24
            $environments = [];
147 24
            foreach ($this->values['environments'] as $key => $value) {
148 24
                if (is_array($value)) {
149 24
                    $environments[$key] = $value;
150
                }
151 24
            }
152
153
            return $environments;
154 1
        }
155
156
        return null;
157
    }
158
159
    /**
160 25
     * {@inheritdoc}
161
     */
162 25
    public function getEnvironment($name)
163
    {
164 25
        $environments = $this->getEnvironments();
165 21
166 21
        if (isset($environments[$name])) {
167 21
            if (isset($this->values['environments']['default_migration_table'])) {
168 21
                $environments[$name]['default_migration_table'] =
169
                    $this->values['environments']['default_migration_table'];
170 21
            }
171
172
            return $environments[$name];
173 5
        }
174
175
        return null;
176
    }
177
178
    /**
179 9
     * {@inheritdoc}
180
     */
181 9
    public function hasEnvironment($name)
182
    {
183
        return ($this->getEnvironment($name) !== null);
184
    }
185
186
    /**
187 20
     * {@inheritdoc}
188
     */
189
    public function getDefaultEnvironment()
190 20
    {
191 20
        // The $PHINX_ENVIRONMENT variable overrides all other default settings
192 2
        $env = getenv('PHINX_ENVIRONMENT');
193 1
        if (!empty($env)) {
194
            if ($this->hasEnvironment($env)) {
195
                return $env;
196 1
            }
197 1
198
            throw new \RuntimeException(sprintf(
199 1
                'The environment configuration (read from $PHINX_ENVIRONMENT) for \'%s\' is missing',
200
                $env
201
            ));
202
        }
203
204 19
        // if the user has configured a default database then use it,
205 17
        // providing it actually exists!
206 16
        if (isset($this->values['environments']['default_database'])) {
207
            if ($this->getEnvironment($this->values['environments']['default_database'])) {
208
                return $this->values['environments']['default_database'];
209 1
            }
210 1
211 1
            throw new \RuntimeException(sprintf(
212 1
                'The environment configuration for \'%s\' is missing',
213
                $this->values['environments']['default_database']
214
            ));
215
        }
216 2
217 1
        // else default to the first available one
218 1
        if (is_array($this->getEnvironments()) && count($this->getEnvironments()) > 0) {
219
            $names = array_keys($this->getEnvironments());
220
221 1
            return $names[0];
222
        }
223
224
        throw new \RuntimeException('Could not find a default environment');
225
    }
226
227 7
    /**
228
     * {@inheritdoc}
229 7
     */
230
    public function getAlias($alias)
231
    {
232
        return !empty($this->values['aliases'][$alias]) ? $this->values['aliases'][$alias] : null;
233
    }
234
235 448
    /**
236
     * {@inheritdoc}
237 448
     */
238
    public function getConfigFilePath()
239
    {
240
        return $this->configFilePath;
241
    }
242
243 423
    /**
244
     * {@inheritdoc}
245 423
     */
246 1 View Code Duplication
    public function getMigrationPaths()
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
247
    {
248
        if (!isset($this->values['paths']['migrations'])) {
249 422
            throw new \UnexpectedValueException('Migrations path missing from config file');
250 219
        }
251 219
252
        if (is_string($this->values['paths']['migrations'])) {
253 422
            $this->values['paths']['migrations'] = [$this->values['paths']['migrations']];
254
        }
255
256
        return $this->values['paths']['migrations'];
257
    }
258
259
    /**
260
     * Gets the base class name for migrations.
261
     *
262 14
     * @param bool $dropNamespace Return the base migration class name without the namespace.
263
     * @return string
264 14
     */
265
    public function getMigrationBaseClassName($dropNamespace = true)
266 14
    {
267
        $className = !isset($this->values['migration_base_class']) ? 'Phinx\Migration\AbstractMigration' : $this->values['migration_base_class'];
268
269
        return $dropNamespace ? substr(strrchr($className, '\\'), 1) ?: $className : $className;
270
    }
271
272 48
    /**
273
     * {@inheritdoc}
274 48
     */
275 28 View Code Duplication
    public function getSeedPaths()
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
276
    {
277
        if (!isset($this->values['paths']['seeds'])) {
278 20
            throw new \UnexpectedValueException('Seeds path missing from config file');
279 13
        }
280 13
281
        if (is_string($this->values['paths']['seeds'])) {
282 20
            $this->values['paths']['seeds'] = [$this->values['paths']['seeds']];
283
        }
284
285
        return $this->values['paths']['seeds'];
286
    }
287
288
    /**
289
     * Get the template file name.
290 14
     *
291
     * @return string|false
292 14
     */
293 13 View Code Duplication
    public function getTemplateFile()
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
294
    {
295
        if (!isset($this->values['templates']['file'])) {
296 1
            return false;
297
        }
298
299
        return $this->values['templates']['file'];
300
    }
301
302
    /**
303
     * Get the template class name.
304 14
     *
305
     * @return string|false
306 14
     */
307 10 View Code Duplication
    public function getTemplateClass()
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
308
    {
309
        if (!isset($this->values['templates']['class'])) {
310 4
            return false;
311
        }
312
313
        return $this->values['templates']['class'];
314
    }
315
316
    /**
317
     * Get the version order.
318 384
     *
319
     * @return string
320 384
     */
321 162
    public function getVersionOrder()
322
    {
323
        if (!isset($this->values['version_order'])) {
324 222
            return self::VERSION_ORDER_CREATION_TIME;
325
        }
326
327
        return $this->values['version_order'];
328
    }
329
330
    /**
331
     * Is version order creation time?
332 357
     *
333
     * @return bool
334 357
     */
335
    public function isVersionOrderCreationTime()
336 357
    {
337
        $versionOrder = $this->getVersionOrder();
338
339
        return $versionOrder == self::VERSION_ORDER_CREATION_TIME;
340
    }
341
342
    public function getBootstrapFile()
343
    {
344
        if (!isset($this->values['paths']['bootstrap'])) {
345
            return false;
346
        }
347 448
348
        return $this->values['paths']['bootstrap'];
349
    }
350
351 448
    /**
352 448
     * Replace tokens in the specified array.
353 448
     *
354 2
     * @param array $arr Array to replace
355 2
     * @return array
356 448
     */
357
    protected function replaceTokens(array $arr)
358
    {
359 448
        // Get environment variables
360 448
        // $_ENV is empty because variables_order does not include it normally
361
        $tokens = [];
362
        foreach ($_SERVER as $varname => $varvalue) {
363 448
            if (0 === strpos($varname, 'PHINX_')) {
364
                $tokens['%%' . $varname . '%%'] = $varvalue;
365
            }
366
        }
367
368
        // Phinx defined tokens (override env tokens)
369
        $tokens['%%PHINX_CONFIG_PATH%%'] = $this->getConfigFilePath();
370
        $tokens['%%PHINX_CONFIG_DIR%%'] = dirname($this->getConfigFilePath());
371
372
        // Recurse the array and replace tokens
373 448
        return $this->recurseArrayForTokens($arr, $tokens);
374
    }
375 448
376 448
    /**
377 447
     * Recurse an array for the specified tokens and replace them.
378 446
     *
379 446
     * @param array $arr Array to recurse
380
     * @param array $tokens Array of tokens to search for
381 446
     * @return array
382 446
     */
383 446
    protected function recurseArrayForTokens($arr, $tokens)
384 446
    {
385 446
        $out = [];
386 446
        foreach ($arr as $name => $value) {
387
            if (is_array($value)) {
388 43
                $out[$name] = $this->recurseArrayForTokens($value, $tokens);
389 448
                continue;
390 448
            }
391
            if (is_string($value)) {
392
                foreach ($tokens as $token => $tval) {
393
                    $value = str_replace($token, $tval, $value);
394
                }
395
                $out[$name] = $value;
396 213
                continue;
397
            }
398 213
            $out[$name] = $value;
399 213
        }
400
401
        return $out;
402
    }
403
404 2
    /**
405
     * {@inheritdoc}
406 2
     */
407 1
    public function offsetSet($id, $value)
408
    {
409
        $this->values[$id] = $value;
410 1
    }
411
412
    /**
413
     * {@inheritdoc}
414
     */
415
    public function offsetGet($id)
416 1
    {
417
        if (!array_key_exists($id, $this->values)) {
418 1
            throw new \InvalidArgumentException(sprintf('Identifier "%s" is not defined.', $id));
419
        }
420
421
        return $this->values[$id] instanceof \Closure ? $this->values[$id]($this) : $this->values[$id];
422
    }
423
424 1
    /**
425
     * {@inheritdoc}
426 1
     */
427 1
    public function offsetExists($id)
428
    {
429
        return isset($this->values[$id]);
430
    }
431
432
    /**
433
     * {@inheritdoc}
434
     */
435
    public function offsetUnset($id)
436
    {
437
        unset($this->values[$id]);
438
    }
439
}
440