hidev\controllers\GithubController

Complexity

Total Complexity

27

Size/Duplication

Total Lines	165
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	9
Bugs	1	Features	1

17 Methods

Rating	Name	Duplication	Size	Complexity
A	setName()	0	7	3
A	getName()	0	8	2
A	setVendor()	0	4	1
A	getVendor()	0	8	2
A	setPackage()	0	4	1
A	getPackage()	0	8	2
A	setDescription()	0	4	1
A	getDescription()	0	8	2
A	actionCreate()	0	7	1
A	actionClone()	0	4	1
A	actionExists()	0	4	2
A	exists()	0	4	1
A	actionRelease()	0	17	2
A	actionWaitPush()	0	6	1
A	request()	0	6	1
A	findToken()	0	4	2
A	getToken()	0	8	2

<?php

/*
 * Task runner, code generator and build tool for easier continuos integration
 *
 * @link      https://github.com/hiqdev/hidev
 * @package   hidev
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2016, HiQDev (http://hiqdev.com/)
 */

namespace hidev\controllers;

use yii\helpers\Json;

/**
 * Goal for GitHub.
 */
class GithubController extends CommonController
{
    protected $_name;
    protected $_vendor;
    protected $_package;
    protected $_description;

    /**
     * @var string GitHub OAuth access token
     */
    protected $_token;

    public function setName($value)
    {
        list($vendor, $package) = explode('/', $value, 2);
        $this->_name    = $value;
        $this->_vendor  = $vendor ?: $package;
        $this->_package = $package ?: $vendor;
    }

    public function getName()
    {
        if ($this->_name === null) {
            $this->setName($this->takePackage()->fullName);
        }

        return $this->_name;
    }

    public function setVendor($value)
    {
        $this->_vendor = $value;
    }

    public function getVendor()
    {
        if ($this->_vendor === null) {
            $this->_vendor = $this->takeVendor()->name;
        }

        return $this->_vendor;
    }

    public function setPackage($value)
    {
        $this->_package = $value;
    }

    public function getPackage()
    {
        if ($this->_package === null) {
            $this->_package = $this->takePackage()->name;
        }

        return $this->_package;
    }

    public function setDescription($value)
    {
        $this->_description = $value;
    }

    public function getDescription()
    {
        if ($this->_description === null) {
            $this->_description = $this->takePackage()->getTitle();
        }

        return $this->_description;
    }

    /**
     * Create the repo on GitHub.
     * @return int exit code
     */
    public function actionCreate()
    {
        return $this->request('POST', '/orgs/' . $this->getVendor() . '/repos', [
            'name'        => $this->getPackage(),
            'description' => $this->getDescription(),
        ]);
    }

    /**
     * Clone repo from github.
     * TODO this action must be run without `start`.
     * @param string $repo full name vendor/package
     * @return int exit code
     */
    public function actionClone($repo)
    {
        return $this->passthru('git', ['clone', 'git@github.com:' . $repo]);

array('clone', 'git@github.com:' . $repo) is of type array<integer,string,{"0":"string","1":"string"}>, but the function expects a string.

    }

    /**
     * Action to check if repo exists.
     * @param string $repo full name vendor/package defaults to this repo name
     * @return int exit code
     */
    public function actionExists($repo = null)
    {
        return static::exists($repo ?: $this->getName());
    }

    /**
     * Check if repo exists.
     * @param string $repo
     * @return bool
     */
    static public function exists($repo)

As per PSR2, the static declaration should come after the visibility declaration.

    {
        return !static::exec('git', ['ls-remote', 'git@github.com:' . $repo], true);

array('ls-remote', 'git@github.com:' . $repo) is of type array<integer,string,{"0":"string","1":"string"}>, but the function expects a string.

    }

    public function actionRelease($version = null)
    {
        $this->runRequest('CHANGELOG.md');
        $changelog = $this->takeGoal('CHANGELOG.md');
        $notes = reset($changelog->getFile()->getHandler()->releaseNotes);
        $version = $this->takeGoal('bump')->getVersion($version);
        $wait = $this->actionWaitPush();
        if ($wait) {
            return $wait;
        }

        return $this->request('POST', '/repos/' . $this->getName() . '/releases', [
            'tag_name'  => $version,
            'name'      => $version,
            'body'      => $notes,
        ]);
    }

    /**
     * Waits until push is actually finished.
     * TODO Check github if it really has latest local commit.
     * @return int 0 - success, 1 - failed
     */
    public function actionWaitPush()
    {
        sleep(7);

        return 0;
    }

    public function request($method, $path, $data)
    {
        $url = 'https://api.github.com' . $path;

        return $this->passthru('curl', ['-X', $method, '-H', 'Authorization: token ' . $this->getToken(), '--data', Json::encode($data), $url]);

array('-X', $method, '-H', 'Authorization: token ' . $this->getToken(), '--data', \yii\helpers\Json::encode($data), $url) is of type array<integer,?,{"0":"string","1":"?","2":"string","3":"string","4":"string","5":"string","6":"string"}>, but the function expects a string.

    }

    public function findToken()

findToken uses the super-global variable $_SERVER which is generally not recommended.

    {
        return $_SERVER['GITHUB_TOKEN'] ?: $this->readpassword('GitHub token:');
    }

    public function getToken()
    {
        if ($this->_token === null) {
            $this->_token = $this->findToken();
        }

        return $this->_token;
    }
}