GithubController::setDescription() - Code Metrics - Inspection of "minor" - hiqdev/hidev - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 61af37...142460 )

by Andrii

created 2016-02-23 17:18 UTC

GithubController::setDescription() A

↳ Parent: GithubController

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	4
ccs	0
cts	4
cp	0
rs	10
cc	1
eloc	2
nc	1
nop	1
crap	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', '[email protected]:' . $repo]);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    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]);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    public function getToken()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if ($this->_token === null) {
            $this->_token = $_SERVER['GITHUB_TOKEN'];
        }

        return $this->_token;
    }
}


1			<?php
2
3			/*
4			* Task runner, code generator and build tool for easier continuos integration
5			*
6			* @link https://github.com/hiqdev/hidev
7			* @package hidev
8			* @license BSD-3-Clause
9			* @copyright Copyright (c) 2015-2016, HiQDev (http://hiqdev.com/)
10			*/
11
12			namespace hidev\controllers;
13
14			use yii\helpers\Json;
15
16			/**
17			* Goal for GitHub.
18			*/
19			class GithubController extends CommonController
20			{
21			protected $_name;
22			protected $_vendor;
23			protected $_package;
24			protected $_description;
25
26			/**
27			* @var string GitHub OAuth access token
28			*/
29			protected $_token;
30
31			public function setName($value)
32			{
33			list($vendor, $package) = explode('/', $value, 2);
34			$this->_name = $value;
35			$this->_vendor = $vendor ?: $package;
36			$this->_package = $package ?: $vendor;
37			}
38
39			public function getName()
40			{
41			if ($this->_name === null) {
42			$this->setName($this->takePackage()->fullName);
43			}
44
45			return $this->_name;
46			}
47
48			public function setVendor($value)
49			{
50			$this->_vendor = $value;
51			}
52
53			public function getVendor()
54			{
55			if ($this->_vendor === null) {
56			$this->_vendor = $this->takeVendor()->name;
57			}
58
59			return $this->_vendor;
60			}
61
62			public function setPackage($value)
63			{
64			$this->_package = $value;
65			}
66
67			public function getPackage()
68			{
69			if ($this->_package === null) {
70			$this->_package = $this->takePackage()->name;
71			}
72
73			return $this->_package;
74			}
75
76			public function setDescription($value)
77			{
78			$this->_description = $value;
79			}
80
81			public function getDescription()
82			{
83			if ($this->_description === null) {
84			$this->_description = $this->takePackage()->getTitle();
85			}
86
87			return $this->_description;
88			}
89
90			/**
91			* Create the repo on GitHub.
92			* @return int exit code
93			*/
94			public function actionCreate()
95			{
96			return $this->request('POST', '/orgs/' . $this->getVendor() . '/repos', [
97			'name' => $this->getPackage(),
98			'description' => $this->getDescription(),
99			]);
100			}
101
102			/**
103			* Clone repo from github.
104			* TODO this action must be run without `start`.
105			* @param string $repo full name vendor/package
106			* @return int exit code
107			*/
108			public function actionClone($repo)
109			{
110			return $this->passthru('git', ['clone', '[email protected]:' . $repo]);
			0 ignored issues – show Documentation introduced 2016-01-13 13:30 UTC by Report Bug Copy Issue Report `array('clone', '[email protected]:' . $repo)` is of type `array<integer,string,{"0":"string","1":"string"}>`, 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...
111			}
112
113			public function actionRelease($version = null)
114			{
115			$this->runRequest('CHANGELOG.md');
116			$changelog = $this->takeGoal('CHANGELOG.md');
117			$notes = reset($changelog->getFile()->getHandler()->releaseNotes);
118			$version = $this->takeGoal('bump')->getVersion($version);
119			$wait = $this->actionWaitPush();
120			if ($wait) {
121			return $wait;
122			}
123
124			return $this->request('POST', '/repos/' . $this->getName() . '/releases', [
125			'tag_name' => $version,
126			'name' => $version,
127			'body' => $notes,
128			]);
129			}
130
131			/**
132			* Waits until push is actually finished.
133			* TODO Check github if it really has latest local commit.
134			* @return int 0 - success, 1 - failed
135			*/
136			public function actionWaitPush()
137			{
138			sleep(7);
139
140			return 0;
141			}
142
143			public function request($method, $path, $data)
144			{
145			$url = 'https://api.github.com' . $path;
146
147			return $this->passthru('curl', ['-X', $method, '-H', 'Authorization: token ' . $this->getToken(), '--data', Json::encode($data), $url]);
			0 ignored issues – show Documentation introduced 2016-01-26 12:27 UTC by Report Bug Copy Issue Report `array('-X', $method, '-H...n::encode($data), $url)` is of type `array<integer,?,{"0":"st..."string","6":"string"}>`, 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...
148			}
149
150			public function getToken()
			0 ignored issues – show Coding Style introduced 2016-01-26 11:42 UTC by Report Bug Copy Issue Report `getToken` uses the super-global variable `$_SERVER` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
151			{
152			if ($this->_token === null) {
153			$this->_token = $_SERVER['GITHUB_TOKEN'];
154			}
155
156			return $this->_token;
157			}
158			}
159

hiqdev / hidev

Push — master ( 61af37...142460 )

GithubController::setDescription() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like