Completed
Pull Request — master (#114)
by Fabian
21:27
created

ExtraPackage   C

Complexity

Total Complexity 61

Size/Duplication

Total Lines 605
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 13

Test Coverage

Coverage 100%

Importance

Changes 21
Bugs 2 Features 5
Metric Value
wmc 61
c 21
b 2
f 5
lcom 1
cbo 13
dl 0
loc 605
ccs 231
cts 231
cp 1
rs 5.9771

21 Methods

Rating   Name   Duplication   Size   Complexity  
A __construct() 0 9 1
A getIncludes() 0 5 2
A getRequires() 0 5 2
A readPackageJson() 0 13 3
A loadPackage() 0 14 2
A mergeInto() 0 23 3
B addRepositories() 0 27 4
B mergeRequires() 0 29 2
B mergeOrDefer() 0 22 4
A mergeAutoload() 0 16 2
A fixRelativePaths() 0 13 2
A mergeStabilityFlags() 0 13 1
A mergePackageLinks() 0 23 3
A mergeSuggests() 0 11 2
B mergeExtra() 0 32 6
A arrayMerge() 0 8 2
C arrayMergeDeep() 0 24 7
B replaceSelfVersionDependencies() 0 41 3
A unwrapIfNeeded() 0 14 3
A mergeReferences() 0 17 3
A extractReferences() 0 16 4

How to fix   Complexity   

Complex Class

Complex classes like ExtraPackage 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 ExtraPackage, and based on these observations, apply Extract Interface, too.

1
<?php
2
/**
3
 * This file is part of the Composer Merge plugin.
4
 *
5
 * Copyright (C) 2015 Bryan Davis, Wikimedia Foundation, and contributors
6
 *
7
 * This software may be modified and distributed under the terms of the MIT
8
 * license. See the LICENSE file for details.
9
 */
10
11
namespace Wikimedia\Composer\Merge;
12
13
use Wikimedia\Composer\Logger;
14
15
use Composer\Composer;
16
use Composer\Json\JsonFile;
17
use Composer\Package\BasePackage;
18
use Composer\Package\CompletePackage;
19
use Composer\Package\Link;
20
use Composer\Package\Loader\ArrayLoader;
21
use Composer\Package\RootAliasPackage;
22
use Composer\Package\RootPackage;
23
use Composer\Package\RootPackageInterface;
24
use Composer\Package\Version\VersionParser;
25
use UnexpectedValueException;
26
27
/**
28
 * Processing for a composer.json file that will be merged into
29
 * a RootPackageInterface
30
 *
31
 * @author Bryan Davis <[email protected]>
32
 */
33
class ExtraPackage
34
{
35
36
    /**
37
     * @var Composer $composer
38
     */
39
    protected $composer;
40
41
    /**
42
     * @var Logger $logger
43
     */
44
    protected $logger;
45
46
    /**
47
     * @var string $path
48
     */
49
    protected $path;
50
51
    /**
52
     * @var array $json
53
     */
54
    protected $json;
55
56
    /**
57
     * @var CompletePackage $package
58
     */
59
    protected $package;
60
61
    /**
62
     * @var VersionParser $versionParser
63
     */
64
    protected $versionParser;
65
66
    /**
67
     * @param string $path Path to composer.json file
68
     * @param Composer $composer
69
     * @param Logger $logger
70
     */
71 95
    public function __construct($path, Composer $composer, Logger $logger)
72
    {
73 95
        $this->path = $path;
74 95
        $this->composer = $composer;
75 95
        $this->logger = $logger;
76 95
        $this->json = $this->readPackageJson($path);
77 95
        $this->package = $this->loadPackage($this->json);
0 ignored issues
show
Documentation introduced by
$this->json is of type array, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
78 95
        $this->versionParser = new VersionParser();
79 95
    }
80
81
    /**
82
     * Get list of additional packages to include if precessing recursively.
83
     *
84
     * @return array
85
     */
86 90
    public function getIncludes()
87
    {
88 90
        return isset($this->json['extra']['merge-plugin']['include']) ?
89 90
            $this->json['extra']['merge-plugin']['include'] : array();
90
    }
91
92
    /**
93
     * Get list of additional packages to require if precessing recursively.
94
     *
95
     * @return array
96
     */
97 90
    public function getRequires()
98
    {
99 90
        return isset($this->json['extra']['merge-plugin']['require']) ?
100 90
            $this->json['extra']['merge-plugin']['require'] : array();
101
    }
102
103
    /**
104
     * Read the contents of a composer.json style file into an array.
105
     *
106
     * The package contents are fixed up to be usable to create a Package
107
     * object by providing dummy "name" and "version" values if they have not
108
     * been provided in the file. This is consistent with the default root
109
     * package loading behavior of Composer.
110
     *
111
     * @param string $path
112
     * @return array
113
     */
114 95
    protected function readPackageJson($path)
115
    {
116 95
        $file = new JsonFile($path);
117 95
        $json = $file->read();
118 95
        if (!isset($json['name'])) {
119 90
            $json['name'] = 'merge-plugin/' .
120 90
                strtr($path, DIRECTORY_SEPARATOR, '-');
121 90
        }
122 95
        if (!isset($json['version'])) {
123 95
            $json['version'] = '1.0.0';
124 95
        }
125 95
        return $json;
126
    }
127
128
    /**
129
     * @param string $json
130
     * @return CompletePackage
131
     */
132 95
    protected function loadPackage($json)
133
    {
134 95
        $loader = new ArrayLoader();
135 95
        $package = $loader->load($json);
0 ignored issues
show
Documentation introduced by
$json is of type string, but the function expects a array.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
136
        // @codeCoverageIgnoreStart
137
        if (!$package instanceof CompletePackage) {
138
            throw new UnexpectedValueException(
139
                'Expected instance of CompletePackage, got ' .
140
                get_class($package)
141
            );
142
        }
143
        // @codeCoverageIgnoreEnd
144 95
        return $package;
145
    }
146
147
    /**
148
     * Merge this package into a RootPackageInterface
149
     *
150
     * @param RootPackageInterface $root
151
     * @param PluginState $state
152
     */
153 95
    public function mergeInto(RootPackageInterface $root, PluginState $state)
154
    {
155 95
        $this->addRepositories($root);
156
157 95
        $this->mergeRequires('require', $root, $state);
158 95
        if ($state->isDevMode()) {
159 90
            $this->mergeRequires('require-dev', $root, $state);
160 90
        }
161
162 95
        $this->mergePackageLinks('conflict', $root);
163 95
        $this->mergePackageLinks('replace', $root);
164 95
        $this->mergePackageLinks('provide', $root);
165
166 95
        $this->mergeSuggests($root);
167
168 95
        $this->mergeAutoload('autoload', $root);
169 95
        if ($state->isDevMode()) {
170 90
            $this->mergeAutoload('devAutoload', $root);
171 90
        }
172
173 95
        $this->mergeExtra($root, $state);
174 95
        $this->mergeReferences($root);
175 95
    }
176
177
    /**
178
     * Add a collection of repositories described by the given configuration
179
     * to the given package and the global repository manager.
180
     *
181
     * @param RootPackageInterface $root
182
     */
183 95
    protected function addRepositories(RootPackageInterface $root)
184
    {
185 95
        if (!isset($this->json['repositories'])) {
186 85
            return;
187
        }
188 10
        $repoManager = $this->composer->getRepositoryManager();
189 10
        $newRepos = array();
190
191 10
        foreach ($this->json['repositories'] as $repoJson) {
192 10
            if (!isset($repoJson['type'])) {
193 10
                continue;
194
            }
195 10
            $this->logger->info("Adding {$repoJson['type']} repository");
196 10
            $repo = $repoManager->createRepository(
197 10
                $repoJson['type'],
198
                $repoJson
199 10
            );
200 10
            $repoManager->addRepository($repo);
201 10
            $newRepos[] = $repo;
202 10
        }
203
204 10
        $unwrapped = self::unwrapIfNeeded($root, 'setRepositories');
205 10
        $unwrapped->setRepositories(array_merge(
206 10
            $newRepos,
207 10
            $root->getRepositories()
208 10
        ));
209 10
    }
210
211
    /**
212
     * Merge require or require-dev into a RootPackageInterface
213
     *
214
     * @param string $type 'require' or 'require-dev'
215
     * @param RootPackageInterface $root
216
     * @param PluginState $state
217
     */
218 95
    protected function mergeRequires(
219
        $type,
220
        RootPackageInterface $root,
221
        PluginState $state
222
    ) {
223 95
        $linkType = BasePackage::$supportedLinkTypes[$type];
224 95
        $getter = 'get' . ucfirst($linkType['method']);
225 95
        $setter = 'set' . ucfirst($linkType['method']);
226
227 95
        $requires = $this->package->{$getter}();
228 95
        if (empty($requires)) {
229 75
            return;
230
        }
231
232 65
        $this->mergeStabilityFlags($root, $requires);
233
234 65
        $requires = $this->replaceSelfVersionDependencies(
235 65
            $type,
236 65
            $requires,
237
            $root
238 65
        );
239
240 65
        $root->{$setter}($this->mergeOrDefer(
241 65
            $type,
242 65
            $root->{$getter}(),
243 65
            $requires,
244
            $state
245 65
        ));
246 65
    }
247
248
    /**
249
     * Merge two collections of package links and collect duplicates for
250
     * subsequent processing.
251
     *
252
     * @param string $type 'require' or 'require-dev'
253
     * @param array $origin Primary collection
254
     * @param array $merge Additional collection
255
     * @param PluginState $state
256
     * @return array Merged collection
257
     */
258 65
    protected function mergeOrDefer(
259
        $type,
260
        array $origin,
261
        array $merge,
262
        $state
263
    ) {
264 65
        $dups = array();
265 65
        foreach ($merge as $name => $link) {
266 65
            if (!isset($origin[$name]) || $state->replaceDuplicateLinks()) {
267 65
                $this->logger->info("Merging <comment>{$name}</comment>");
268 65
                $origin[$name] = $link;
269 65
            } else {
270
                // Defer to solver.
271 15
                $this->logger->info(
272 15
                    "Deferring duplicate <comment>{$name}</comment>"
273 15
                );
274 15
                $dups[] = $link;
275
            }
276 65
        }
277 65
        $state->addDuplicateLinks($type, $dups);
278 65
        return $origin;
279
    }
280
281
    /**
282
     * Merge autoload or autoload-dev into a RootPackageInterface
283
     *
284
     * @param string $type 'autoload' or 'devAutoload'
285
     * @param RootPackageInterface $root
286
     */
287 95
    protected function mergeAutoload($type, RootPackageInterface $root)
288
    {
289 95
        $getter = 'get' . ucfirst($type);
290 95
        $setter = 'set' . ucfirst($type);
291
292 95
        $autoload = $this->package->{$getter}();
293 95
        if (empty($autoload)) {
294 90
            return;
295
        }
296
297 10
        $unwrapped = self::unwrapIfNeeded($root, $setter);
298 10
        $unwrapped->{$setter}(array_merge_recursive(
299 10
            $root->{$getter}(),
300 10
            $this->fixRelativePaths($autoload)
301 10
        ));
302 10
    }
303
304
    /**
305
     * Fix a collection of paths that are relative to this package to be
306
     * relative to the base package.
307
     *
308
     * @param array $paths
309
     * @return array
310
     */
311 10
    protected function fixRelativePaths(array $paths)
312
    {
313 10
        $base = dirname($this->path);
314 10
        $base = ($base === '.') ? '' : "{$base}/";
315
316 10
        array_walk_recursive(
317 10
            $paths,
318
            function (&$path) use ($base) {
319 10
                $path = "{$base}{$path}";
320 10
            }
321 10
        );
322 10
        return $paths;
323
    }
324
325
    /**
326
     * Extract and merge stability flags from the given collection of
327
     * requires and merge them into a RootPackageInterface
328
     *
329
     * @param RootPackageInterface $root
330
     * @param array $requires
331
     */
332 65
    protected function mergeStabilityFlags(
333
        RootPackageInterface $root,
334
        array $requires
335
    ) {
336 65
        $flags = $root->getStabilityFlags();
337 65
        $sf = new StabilityFlags($flags, $root->getMinimumStability());
338
339 65
        $unwrapped = self::unwrapIfNeeded($root, 'setStabilityFlags');
340 65
        $unwrapped->setStabilityFlags(array_merge(
341 65
            $flags,
342 65
            $sf->extractAll($requires)
343 65
        ));
344 65
    }
345
346
    /**
347
     * Merge package links of the given type  into a RootPackageInterface
348
     *
349
     * @param string $type 'conflict', 'replace' or 'provide'
350
     * @param RootPackageInterface $root
351
     */
352 95
    protected function mergePackageLinks($type, RootPackageInterface $root)
353
    {
354 95
        $linkType = BasePackage::$supportedLinkTypes[$type];
355 95
        $getter = 'get' . ucfirst($linkType['method']);
356 95
        $setter = 'set' . ucfirst($linkType['method']);
357
358 95
        $links = $this->package->{$getter}();
359 95
        if (!empty($links)) {
360 20
            $unwrapped = self::unwrapIfNeeded($root, $setter);
361
            // @codeCoverageIgnoreStart
362
            if ($root !== $unwrapped) {
363
                $this->logger->warning(
364
                    'This Composer version does not support ' .
365
                    "'{$type}' merging for aliased packages."
366
                );
367
            }
368
            // @codeCoverageIgnoreEnd
369 20
            $unwrapped->{$setter}(array_merge(
370 20
                $root->{$getter}(),
371 20
                $this->replaceSelfVersionDependencies($type, $links, $root)
372 20
            ));
373 20
        }
374 95
    }
375
376
    /**
377
     * Merge suggested packages into a RootPackageInterface
378
     *
379
     * @param RootPackageInterface $root
380
     */
381 95
    protected function mergeSuggests(RootPackageInterface $root)
382
    {
383 95
        $suggests = $this->package->getSuggests();
384 95
        if (!empty($suggests)) {
385 10
            $unwrapped = self::unwrapIfNeeded($root, 'setSuggests');
386 10
            $unwrapped->setSuggests(array_merge(
387 10
                $root->getSuggests(),
388
                $suggests
389 10
            ));
390 10
        }
391 95
    }
392
393
    /**
394
     * Merge extra config into a RootPackageInterface
395
     *
396
     * @param RootPackageInterface $root
397
     * @param PluginState $state
398
     */
399 95
    public function mergeExtra(RootPackageInterface $root, PluginState $state)
400
    {
401 95
        $extra = $this->package->getExtra();
402 95
        unset($extra['merge-plugin']);
403 95
        if (!$state->shouldMergeExtra() || empty($extra)) {
404 80
            return;
405
        }
406
407 15
        $rootExtra = $root->getExtra();
408 15
        $unwrapped = self::unwrapIfNeeded($root, 'setExtra');
409
410 15
        if ($state->replaceDuplicateLinks()) {
411 5
            $unwrapped->setExtra(
412 5
                $this->arrayMerge($state->shouldMergeDeep(), $rootExtra, $extra)
413 5
            );
414
        } else {
415 5
            if (!$state->shouldMergeDeep()) {
416 10
                foreach (array_intersect(
417 10
                    array_keys($extra),
418 10
                    array_keys($rootExtra)
419 10
                ) as $key) {
420 5
                    $this->logger->info(
421 5
                        "Ignoring duplicate <comment>{$key}</comment> in ".
422 5
                        "<comment>{$this->path}</comment> extra config."
423 5
                    );
424 10
                }
425 10
            }
426 10
            $unwrapped->setExtra(
427 10
                $this->arrayMerge($state->shouldMergeDeep(), $extra, $rootExtra)
428
            );
429 15
        }
430
    }
431
432
    /**
433
     * Merges two arrays either via arrayMergeDeep or via array_merge.
434
     *
435
     * @param bool $mergeDeep
436
     * @param array $array1
437
     * @param array $array2
438
     * @return array
439
     */
440 75
    protected function arrayMerge($mergeDeep, $array1, $array2)
441
    {
442
        if ($mergeDeep) {
443
            return $this->arrayMergeDeep($array1, $array2);
444
        }
445 75
446 75
        return array_merge($array1, $array2);
447 75
    }
448 75
449
    /**
450 75
     * Merges multiple arrays, recursively, and returns the merged array.
451 75
     *
452
     * This function is similar to PHP's array_merge_recursive() function, but it
453 75
     * handles non-array values differently. When merging values that are not both
454 75
     * arrays, the latter value replaces the former rather than merging with it.
455 75
     *
456 10
     * Example:
457
     * @code
458 5
     * $link_options_1 = array('fragment' => 'x', 'attributes' => array('title' => t('X'), 'class' => array('a', 'b')));
459 5
     * $link_options_2 = array('fragment' => 'y', 'attributes' => array('title' => t('Y'), 'class' => array('c', 'd')));
460 5
     *
461 5
     * // This results in array(
462 5
     * //     'fragment' => array('x', 'y'),
463 5
     * //     'attributes' => array('title' => array(t('X'), t('Y')), 'class' => array('a', 'b', 'c', 'd'))
464 5
     * // ).
465 5
     * $incorrect = array_merge_recursive($link_options_1, $link_options_2);
466
     *
467
     * // This results in array(
468 5
     * //     'fragment' => 'y',
469 5
     * //     'attributes' => array('title' => t('Y'), 'class' => array('a', 'b', 'c', 'd'))
470 5
     * // ).
471 5
     * $correct = $this->arrayMergeDeep($link_options_1, $link_options_2);
472 5
     * @endcode
473
     *
474 5
     * Note: This function was derived from Drupal's drupal_array_merge_deep().
475
     *
476 75
     * @param array ...
477 75
     *   Arrays to merge.
478
     *
479 75
     * @return array
480
     *   The merged array.
481
     */
482
    protected function arrayMergeDeep()
483
    {
484
        $arrays = func_get_args();
485
        $result = array();
486
487
        foreach ($arrays as $array) {
488
            foreach ($array as $key => $value) {
489
                // Renumber integer keys as array_merge_recursive() does. Note that PHP
490
                // automatically converts array keys that are integer strings (e.g., '1')
491
                // to integers.
492
                if (is_integer($key)) {
493
                    $result[] = $value;
494
                } elseif (isset($result[$key]) && is_array($result[$key]) && is_array($value)) {
495
                    // Recurse when both values are arrays.
496
                    $result[$key] = $this->arrayMergeDeep($result[$key], $value);
497
                } else {
498
                    // Otherwise, use the latter value, overriding any previous value.
499 95
                    $result[$key] = $value;
500
                }
501
            }
502
        }
503
504
        return $result;
505
    }
506
507
    /**
508
     * Update Links with a 'self.version' constraint with the root package's
509
     * version.
510
     *
511 95
     * @param string $type Link type
512
     * @param array $links
513
     * @param RootPackageInterface $root
514
     * @return array
515
     */
516
    protected function replaceSelfVersionDependencies(
517
        $type,
518
        array $links,
519 95
        RootPackageInterface $root
520
    ) {
521
        $linkType = BasePackage::$supportedLinkTypes[$type];
522
        $version = $root->getVersion();
523 95
        $prettyVersion = $root->getPrettyVersion();
524 95
        $vp = $this->versionParser;
525 95
526 95
        $method = 'get' . ucfirst($linkType['method']);
527 95
        $packages = $root->$method();
528 95
529 95
        return array_map(
530 35
            function ($link) use ($linkType, $version, $prettyVersion, $vp, $packages) {
531 95
                if ('self.version' === $link->getPrettyConstraint()) {
532 95
                    if (isset($packages[$link->getSource()])) {
533 95
                        /** @var Link $package */
534 95
                        $package = $packages[$link->getSource()];
535 95
                        return new Link(
536
                            $link->getSource(),
537
                            $link->getTarget(),
538
                            $vp->parseConstraints($package->getConstraint()->getPrettyString()),
539
                            $linkType['description'],
540
                            $package->getPrettyConstraint()
541
                        );
542
                    }
543
544
                    return new Link(
545 95
                        $link->getSource(),
546
                        $link->getTarget(),
547 95
                        $vp->parseConstraints($version),
548 35
                        $linkType['description'],
549 35
                        $prettyVersion
550
                    );
551 35
                }
552
                return $link;
553 35
            },
554 5
            $links
555 5
        );
556 5
    }
557 95
558
    /**
559 95
     * Get a full featured Package from a RootPackageInterface.
560
     *
561
     * In Composer versions before 599ad77 the RootPackageInterface only
562
     * defines a sub-set of operations needed by composer-merge-plugin and
563
     * RootAliasPackage only implemented those methods defined by the
564
     * interface. Most of the unimplemented methods in RootAliasPackage can be
565
     * worked around because the getter methods that are implemented proxy to
566
     * the aliased package which we can modify by unwrapping. The exception
567
     * being modifying the 'conflicts', 'provides' and 'replaces' collections.
568
     * We have no way to actually modify those collections unfortunately in
569
     * older versions of Composer.
570
     *
571
     * @param RootPackageInterface $root
572
     * @param string $method Method needed
573
     * @return RootPackageInterface|RootPackage
574
     */
575
    public static function unwrapIfNeeded(
576
        RootPackageInterface $root,
577
        $method = 'setExtra'
578
    ) {
579
        // @codeCoverageIgnoreStart
580
        if ($root instanceof RootAliasPackage &&
581
            !method_exists($root, $method)
582
        ) {
583
            // Unwrap and return the aliased RootPackage.
584
            $root = $root->getAliasOf();
585
        }
586
        // @codeCoverageIgnoreEnd
587
        return $root;
588
    }
589
590
    /**
591
     * Update the root packages reference information.
592
     *
593
     * @param RootPackageInterface $root
594
     */
595
    protected function mergeReferences(RootPackageInterface $root)
596
    {
597
        // Merge source reference information for merged packages.
598
        // @see RootPackageLoader::load
599
        $references = array();
600
        $unwrapped = $this->unwrapIfNeeded($root, 'setReferences');
601
        foreach (array('require', 'require-dev') as $linkType) {
602
            $linkInfo = BasePackage::$supportedLinkTypes[$linkType];
603
            $method = 'get'.ucfirst($linkInfo['method']);
604
            $links = array();
605
            foreach ($unwrapped->$method() as $link) {
606
                $links[$link->getTarget()] = $link->getConstraint()->getPrettyString();
607
            }
608
            $references = $this->extractReferences($links, $references);
609
        }
610
        $unwrapped->setReferences($references);
611
    }
612
613
    /**
614
     * Extract vcs revision from version constraint (dev-master#abc123.
615
     *
616
     * @param array $requires
617
     * @param array $references
618
     * @return array
619
     * @see RootPackageLoader::extractReferences()
620
     */
621
    protected function extractReferences(array $requires, array $references)
622
    {
623
        foreach ($requires as $reqName => $reqVersion) {
624
            $reqVersion = preg_replace('{^([^,\s@]+) as .+$}', '$1', $reqVersion);
625
            $stabilityName = VersionParser::parseStability($reqVersion);
626
            if (
627
                preg_match('{^[^,\s@]+?#([a-f0-9]+)$}', $reqVersion, $match) &&
628
                $stabilityName === 'dev'
629
            ) {
630
                $name = strtolower($reqName);
631
                $references[$name] = $match[1];
632
            }
633
        }
634
635
        return $references;
636
    }
637
}
638
// vim:sw=4:ts=4:sts=4:et:
639