PackagistService

Complexity

Total Complexity

11

Size/Duplication

Total Lines	105
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getComposer()	0	4	1
A	getPackages()	0	19	3
A	getGroupedPackages()	0	16	3
A	getPackageDetails()	0	4	1
A	getPackageVersions()	0	13	2

<?php

use Composer\Factory;
use Composer\IO\NullIO;
use Composer\Package\Loader\ArrayLoader;

/**
 * Interacts with Packagist to retrieve package listings and details.
 */
class PackagistService
{

    /**
     * @var Composer\Composer
     */
    private $composer;

    /**
     * @var Composer\Repository\RepositoryInterface
     */
    private $repository;

    public function __construct()
    {
        $this->composer = Factory::create(new NullIO());
        $this->client = new Packagist\Api\Client();

The property client does not exist. Did you maybe forget to declare it?

    }

    /**
     * @return Composer\Composer
     */
    public function getComposer()
    {
        return $this->composer;
    }

    /**
     * Gets all SilverStripe packages.
     *
     * @return Packagist\Api\Package[]
     */
    public function getPackages()
    {
        $packages = array();
        $loader = new ArrayLoader();

$loader is not used, you could remove the assignment.

        $addonTypes = array(
            'silverstripe-module',
            'silverstripe-vendormodule',
            'silverstripe-theme'
        );
        foreach ($addonTypes as $type) {
            $repositoriesNames = $this->client->all(array('type' => $type));
            foreach ($repositoriesNames as $name) {
                $packages[] = $this->client->get($name);
                echo $name . PHP_EOL; //output to give feedback when running
            }
        }
        return $packages;
    }

    /**
     * Gets all SilverStripe packages, grouped by package name.
     *
     * @return array
     */
    public function getGroupedPackages()
    {
        $grouped = array();

        foreach ($this->getPackages() as $package) {
            $name = $package->getName();

            if (array_key_exists($name, $grouped)) {
                $grouped[$name][] = $package;
            } else {
                $grouped[$name] = array($package);
            }
        }

        return $grouped;
    }

    /**
     * Gets detailed information for a package.
     *
     * @param string $name
     * @return array
     */
    public function getPackageDetails($name)
    {
        return $this->client->get($name);
    }

    /**
     * Gets all versions of a package by name.
     *
     * @param $name
     * @return \Composer\Package\PackageInterface[]
     */
    public function getPackageVersions($name)
    {
        $packages = array();
        $loader = new ArrayLoader();

$loader is not used, you could remove the assignment.

        $package = $this->client->get($name);

        foreach ($package->getVersions() as $repo) {
            $packages[] = $repo;
        }

        return $packages;
    }
}