Completed
Pull Request — master (#114)
by Fabian
25:33
created

ExtraPackage   C

Complexity

Total Complexity 59

Size/Duplication

Total Lines 591
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 13

Test Coverage

Coverage 100%

Importance

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

20 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 35 6
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
                array_merge($rootExtra, $extra)
413 5
            );
414
415 5
        } elseif ($state->shouldMergeDeep()) {
416 10
            $unwrapped->setExtra(
417 10
                $this->arrayMergeDeep($extra, $rootExtra)
418 10
            );
419 10
        } else {
420 5
            foreach (array_intersect(
421 5
                array_keys($extra),
422 5
                array_keys($rootExtra)
423 5
            ) as $key) {
424 10
                $this->logger->info(
425 10
                    "Ignoring duplicate <comment>{$key}</comment> in ".
426 10
                    "<comment>{$this->path}</comment> extra config."
427 10
                );
428
            }
429 15
            $unwrapped->setExtra(
430
                array_merge($extra, $rootExtra)
431
            );
432
        }
433
    }
434
435
    /**
436
     * Merges multiple arrays, recursively, and returns the merged array.
437
     *
438
     * This function is similar to PHP's array_merge_recursive() function, but it
439
     * handles non-array values differently. When merging values that are not both
440 75
     * arrays, the latter value replaces the former rather than merging with it.
441
     *
442
     * Example:
443
     * @code
444
     * $link_options_1 = array('fragment' => 'x', 'attributes' => array('title' => t('X'), 'class' => array('a', 'b')));
445 75
     * $link_options_2 = array('fragment' => 'y', 'attributes' => array('title' => t('Y'), 'class' => array('c', 'd')));
446 75
     *
447 75
     * // This results in array(
448 75
     * //     'fragment' => array('x', 'y'),
449
     * //     'attributes' => array('title' => array(t('X'), t('Y')), 'class' => array('a', 'b', 'c', 'd'))
450 75
     * // ).
451 75
     * $incorrect = array_merge_recursive($link_options_1, $link_options_2);
452
     *
453 75
     * // This results in array(
454 75
     * //     'fragment' => 'y',
455 75
     * //     'attributes' => array('title' => t('Y'), 'class' => array('a', 'b', 'c', 'd'))
456 10
     * // ).
457
     * $correct = $this->arrayMergeDeep($link_options_1, $link_options_2);
458 5
     * @endcode
459 5
     *
460 5
     * Note: This function was derived from Drupal's drupal_array_merge_deep().
461 5
     *
462 5
     * @param array ...
463 5
     *   Arrays to merge.
464 5
     *
465 5
     * @return array
466
     *   The merged array.
467
     */
468 5
    protected function arrayMergeDeep()
469 5
    {
470 5
        $arrays = func_get_args();
471 5
        $result = array();
472 5
473
        foreach ($arrays as $array) {
474 5
            foreach ($array as $key => $value) {
475
                // Renumber integer keys as array_merge_recursive() does. Note that PHP
476 75
                // automatically converts array keys that are integer strings (e.g., '1')
477 75
                // to integers.
478
                if (is_integer($key)) {
479 75
                    $result[] = $value;
480
                } elseif (isset($result[$key]) && is_array($result[$key]) && is_array($value)) {
481
                    // Recurse when both values are arrays.
482
                    $result[$key] = $this->arrayMergeDeep($result[$key], $value);
483
                } else {
484
                    // Otherwise, use the latter value, overriding any previous value.
485
                    $result[$key] = $value;
486
                }
487
            }
488
        }
489
490
        return $result;
491
    }
492
493
    /**
494
     * Update Links with a 'self.version' constraint with the root package's
495
     * version.
496
     *
497
     * @param string $type Link type
498
     * @param array $links
499 95
     * @param RootPackageInterface $root
500
     * @return array
501
     */
502
    protected function replaceSelfVersionDependencies(
503
        $type,
504
        array $links,
505
        RootPackageInterface $root
506
    ) {
507
        $linkType = BasePackage::$supportedLinkTypes[$type];
508
        $version = $root->getVersion();
509
        $prettyVersion = $root->getPrettyVersion();
510
        $vp = $this->versionParser;
511 95
512
        $method = 'get' . ucfirst($linkType['method']);
513
        $packages = $root->$method();
514
515
        return array_map(
516
            function ($link) use ($linkType, $version, $prettyVersion, $vp, $packages) {
517
                if ('self.version' === $link->getPrettyConstraint()) {
518
                    if (isset($packages[$link->getSource()])) {
519 95
                        /** @var Link $package */
520
                        $package = $packages[$link->getSource()];
521
                        return new Link(
522
                            $link->getSource(),
523 95
                            $link->getTarget(),
524 95
                            $vp->parseConstraints($package->getConstraint()->getPrettyString()),
525 95
                            $linkType['description'],
526 95
                            $package->getPrettyConstraint()
527 95
                        );
528 95
                    }
529 95
530 35
                    return new Link(
531 95
                        $link->getSource(),
532 95
                        $link->getTarget(),
533 95
                        $vp->parseConstraints($version),
534 95
                        $linkType['description'],
535 95
                        $prettyVersion
536
                    );
537
                }
538
                return $link;
539
            },
540
            $links
541
        );
542
    }
543
544
    /**
545 95
     * Get a full featured Package from a RootPackageInterface.
546
     *
547 95
     * In Composer versions before 599ad77 the RootPackageInterface only
548 35
     * defines a sub-set of operations needed by composer-merge-plugin and
549 35
     * RootAliasPackage only implemented those methods defined by the
550
     * interface. Most of the unimplemented methods in RootAliasPackage can be
551 35
     * worked around because the getter methods that are implemented proxy to
552
     * the aliased package which we can modify by unwrapping. The exception
553 35
     * being modifying the 'conflicts', 'provides' and 'replaces' collections.
554 5
     * We have no way to actually modify those collections unfortunately in
555 5
     * older versions of Composer.
556 5
     *
557 95
     * @param RootPackageInterface $root
558
     * @param string $method Method needed
559 95
     * @return RootPackageInterface|RootPackage
560
     */
561
    public static function unwrapIfNeeded(
562
        RootPackageInterface $root,
563
        $method = 'setExtra'
564
    ) {
565
        // @codeCoverageIgnoreStart
566
        if ($root instanceof RootAliasPackage &&
567
            !method_exists($root, $method)
568
        ) {
569
            // Unwrap and return the aliased RootPackage.
570
            $root = $root->getAliasOf();
571
        }
572
        // @codeCoverageIgnoreEnd
573
        return $root;
574
    }
575
576
    /**
577
     * Update the root packages reference information.
578
     *
579
     * @param RootPackageInterface $root
580
     */
581
    protected function mergeReferences(RootPackageInterface $root)
582
    {
583
        // Merge source reference information for merged packages.
584
        // @see RootPackageLoader::load
585
        $references = array();
586
        $unwrapped = $this->unwrapIfNeeded($root, 'setReferences');
587
        foreach (array('require', 'require-dev') as $linkType) {
588
            $linkInfo = BasePackage::$supportedLinkTypes[$linkType];
589
            $method = 'get'.ucfirst($linkInfo['method']);
590
            $links = array();
591
            foreach ($unwrapped->$method() as $link) {
592
                $links[$link->getTarget()] = $link->getConstraint()->getPrettyString();
593
            }
594
            $references = $this->extractReferences($links, $references);
595
        }
596
        $unwrapped->setReferences($references);
597
    }
598
599
    /**
600
     * Extract vcs revision from version constraint (dev-master#abc123.
601
     *
602
     * @param array $requires
603
     * @param array $references
604
     * @return array
605
     * @see RootPackageLoader::extractReferences()
606
     */
607
    protected function extractReferences(array $requires, array $references)
608
    {
609
        foreach ($requires as $reqName => $reqVersion) {
610
            $reqVersion = preg_replace('{^([^,\s@]+) as .+$}', '$1', $reqVersion);
611
            $stabilityName = VersionParser::parseStability($reqVersion);
612
            if (
613
                preg_match('{^[^,\s@]+?#([a-f0-9]+)$}', $reqVersion, $match) &&
614
                $stabilityName === 'dev'
615
            ) {
616
                $name = strtolower($reqName);
617
                $references[$name] = $match[1];
618
            }
619
        }
620
621
        return $references;
622
    }
623
}
624
// vim:sw=4:ts=4:sts=4:et:
625