Hirak\Prestissimo\Plugin

Complexity

Total Complexity

15

Size/Duplication

Total Lines	117
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes	18
Bugs	1	Features	3

7 Methods

Rating	Name	Duplication	Size	Complexity
A	activate()	0	18	3
A	getSubscribedEvents()	0	8	1
A	onPostDependenciesSolving()	0	14	3
A	filterPackages()	0	17	4
A	setPluginConfig()	0	8	2
A	disable()	0	4	1
A	isDisabled()	0	4	1

<?php
/*
 * hirak/prestissimo
 * @author Hiraku NAKANO
 * @license MIT https://github.com/hirak/prestissimo
 */
namespace Hirak\Prestissimo;

use Composer\Composer;
use Composer\IO;
use Composer\Plugin as CPlugin;
use Composer\EventDispatcher;
use Composer\Package;
use Composer\Installer;
use Composer\DependencyResolver;

class Plugin implements
    CPlugin\PluginInterface,
    EventDispatcher\EventSubscriberInterface
{
    /** @var IO\IOInterface */
    private $io;

    /** @var Composer\Config */
    private $config;

    /** @var Config */
    private $pluginConfig;

    /** @var boolean */
    private $disabled = false;

    private static $pluginClasses = array(
        'GitHubRequest',
        'GitLabRequest',
        'HttpGetRequest',
        'HttpGetResponse',
        'Config',
        'CurlMulti',
        'Factory',
        'FileDownloaderDummy',
        'OutputFile',
        'ParallelDownloader',
        'Plugin',
    );

    public function activate(Composer $composer, IO\IOInterface $io)
    {
        // @codeCoverageIgnoreStart
        // guard for self-update problem
        if (__CLASS__ !== 'Hirak\Prestissimo\Plugin') {
            return $this->disable();
        }
        // @codeCoverageIgnoreEnd

        // load all classes
        foreach (self::$pluginClasses as $class) {
            class_exists(__NAMESPACE__ . '\\' . $class);
        }

        $this->config = $composer->getConfig();

It seems like $composer->getConfig() of type object<Composer\Config> is incompatible with the declared type object<Composer\Composer\Config> of property $config.

        $this->io = $io;
        $this->pluginConfig = $this->setPluginConfig();
    }

    public static function getSubscribedEvents()
    {
        return array(
            Installer\InstallerEvents::POST_DEPENDENCIES_SOLVING => array(
                array('onPostDependenciesSolving', PHP_INT_MAX),
            ),
        );
    }

    /**
     * pre-fetch parallel by curl_multi
     */
    public function onPostDependenciesSolving(Installer\InstallerEvent $ev)
    {
        if ($this->disabled) {
            return;
        }
        $ops = $ev->getOperations();
        $packages = $this->filterPackages($ops);
        $pluginConfig = $this->pluginConfig->get();

        if ($packages) {

The expression $packages of type Composer\Package\PackageInterface[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $downloader = new ParallelDownloader($this->io, $this->config);
            $downloader->download($packages, $pluginConfig);
        }
    }

    /**
     * @param DependencyResolver\Operation\OperationInterface[]
     * @return Package\PackageInterface[]
     */
    private static function filterPackages(array $operations)
    {
        $packs = array();
        foreach ($operations as $op) {
            $type = $op->getJobType();
            if ('install' === $type) {
                $packs[] = $op->getPackage();
                continue;
            }

            if ('update' === $type) {
                $packs[] = $op->getTargetPackage();
                continue;
            }
        }
        return $packs;
    }

    private function setPluginConfig()
    {
        $config = $this->config->get('prestissimo');
        if (!is_array($config)) {
            $config = array();
        }
        return new Config($config);
    }

    public function disable()
    {
        $this->disabled = true;
    }

    public function isDisabled()
    {
        return $this->disabled;
    }
}