hidev\composer\controllers\ComposerJsonController

Complexity

Total Complexity

19

Size/Duplication

Total Lines	115
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	actionLoad()	0	23	3
A	getType()	0	4	1
A	getName()	0	4	1
A	getFullName()	0	4	1
A	getSupport()	0	10	1
B	getAuthors()	0	17	5
A	getAutoload()	0	13	3
A	getRequire()	0	4	2
A	getRequireDev()	0	4	2

<?php

/*
 * Composer plugin for HiDev
 *
 * @link      https://github.com/hiqdev/hidev-composer
 * @package   hidev-composer
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2016, HiQDev (http://hiqdev.com/)
 */

namespace hidev\composer\controllers;

/**
 * Goal for composer.json.
 */
class ComposerJsonController extends \hidev\controllers\FileController
{
    protected $_file = 'composer.json';

    public function actionLoad()
    {
        parent::actionLoad();
        $sets = [
            'name'        => $this->getName(),

Consider using $this->name. There is an issue with getName() and APC-enabled PHP versions.

            'type'        => $this->getType(),
            'description' => $this->takePackage()->title,
            'keywords'    => $this->takePackage()->keywords,
            'homepage'    => $this->takePackage()->homepage,
            'license'     => $this->takePackage()->license,
            'support'     => $this->support,
            'authors'     => $this->authors,
            'require'     => $this->require,
            'require-dev' => $this->get('require-dev'),
            'autoload'    => $this->autoload,
        ];
        $this->setItems($sets, 'first');
        foreach (['require', 'require-dev'] as $k) {
            if (!$this->get($k)) {
                $this->delete($k);
            }
        }
    }

    /**
     * Converts hidev type to composer type.
     * TODO composer type can be different from package type.
     * @return string
     */
    public function getType()
    {
        return $this->takePackage()->type;
    }

    /**
     * Converts hidev full name to composer name.
     * TODO composer name can be different from package full name.
     * @return string
     */
    public function getName()
    {
        return $this->takePackage()->fullName;
    }

    /**
     * @return string
     */
    public function getFullName()
    {
        return $this->getName();

Consider using $this->name. There is an issue with getName() and APC-enabled PHP versions.

    }

    public function getSupport()
    {
        return array_merge(array_filter([
            'email'  => $this->takeVendor()->email,
            'source' => $this->takePackage()->source,
            'issues' => $this->takePackage()->issues,
            'wiki'   => $this->takePackage()->wiki,
            'forum'  => $this->takePackage()->forum,
        ]), (array) $this->getItem('support'));
    }

    public function getAuthors()
    {
        $res = [];
        if ($this->takePackage()->authors) {
            foreach ($this->takePackage()->authors as $nick => $all_data) {
                $data = [];
                foreach (['name', 'role', 'email', 'homepage'] as $k) {
                    if ($all_data[$k]) {
                        $data[$k] = $all_data[$k];
                    }
                }
                $res[] = $data;
            }
        }

        return $res;
    }

    public function getAutoload()
    {
        $autoload   = $this->rawItem('autoload');
        $psr4       = $autoload['psr-4'] ?: [];
        $namespace  = $this->takePackage()->namespace;
        if (!array_key_exists($namespace, $psr4)) {
            $psr4 = [$namespace . '\\' => $this->takePackage()->src] + $psr4;
            $autoload['psr-4'] = $psr4;
            $this->setItem('autoload', $autoload);
        }

        return $autoload;
    }

    /**
     * @return array
     */
    public function getRequire()
    {
        return $this->getItem('require') ?: [];
    }

    /**
     * @return array
     */
    public function getRequireDev()
    {
        return $this->getItem('require-dev') ?: [];
    }
}