ApiConsoleControl::createComponentConsoleForm() - Code Metrics - Inspection of "Added asterisk to required nette control in consol..." - tomaj/nette-api - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5fd6e4...a21112 )

by Tomas

created 2016-01-05 22:10 UTC

ApiConsoleControl::createComponentConsoleForm() B

↳ Parent: ApiConsoleControl

Complexity

Conditions	9
Paths	66

Size

Total Lines	55
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	90

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	55
ccs	0
cts	45
cp	0
rs	7.2447
cc	9
eloc	37
nc	66
nop	0
crap	90

How to fix Long Method

<?php

namespace Tomaj\NetteApi\Component;

use Nette\Application\UI\Control;
use Nette\Application\UI\Form;
use Nette\Http\Request;
use Tomaj\Form\Renderer\BootstrapRenderer;
use Tomaj\NetteApi\Authorization\ApiAuthorizationInterface;
use Tomaj\NetteApi\Authorization\BearerTokenAuthorization;
use Tomaj\NetteApi\Authorization\NoAuthorization;
use Tomaj\NetteApi\EndpointIdentifier;
use Tomaj\NetteApi\Handlers\ApiHandlerInterface;
use Tomaj\NetteApi\Misc\ConsoleRequest;
use Tomaj\NetteApi\Params\InputParam;

class ApiConsoleControl extends Control
{
    private $endpoint;

    private $handler;

    private $authorization;

    private $request;

    public function __construct(Request $request, EndpointIdentifier $endpoint, ApiHandlerInterface $handler, ApiAuthorizationInterface $authorization)
    {
        parent::__construct(null, 'apiconsolecontrol');
        $this->endpoint = $endpoint;
        $this->handler = $handler;
        $this->authorization = $authorization;
        $this->request = $request;
    }

    public function render()
    {
        $this->template->setFile(__DIR__ . '/console.latte');
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $this->template->render();
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }

    protected function createComponentConsoleForm()
// 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);
    }
}
    {
        $form = new Form();

        $defaults = [];

        $form->setRenderer(new BootstrapRenderer());

        $uri = $this->request->getUrl();
        $scheme = $uri->scheme;
        if (isset($_SERVER['HTTP_X_FORWARDED_PROTO'])) {
            $scheme = $_SERVER['HTTP_X_FORWARDED_PROTO'];
        }
        $url = $scheme . '://' . $uri->host . '/api/' . $this->endpoint->getUrl();

        $form->addText('api_url', 'Api Url');
        $defaults['api_url'] = $url;

        $form->addText('method', 'Method');
        $defaults['method'] = $this->endpoint->getMethod();

        if ($this->authorization instanceof BearerTokenAuthorization) {
            $form->addText('token', 'Token:')
                ->setAttribute('placeholder', 'napište token');
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        } elseif ($this->authorization instanceof NoAuthorization) {
            $form->addText('authorization', 'Authorization')
                ->setDisabled(true);
            $defaults['authorization'] = 'No authorization - global access';
        }

        $params = $this->handler->params();
        foreach ($params as $param) {
            $count = $param->isMulti() ? 5 : 1;
            for ($i = 0; $i < $count; $i++) {
                $key = $param->getKey();
                if ($param->isMulti()) {
                    $key = $key . '___' . $i;
                }
                $c = $form->addText($key, $this->getParamLabel($param));
                if ($param->getAvailableValues()) {
                    $c->setOption('description', 'available values: ' . implode(' | ', $param->getAvailableValues()));
                }
            }
        }

        $form->addSubmit('send', 'Otestuj')
            ->getControlPrototype()
            ->setName('button')
            ->setHtml('<i class="fa fa-cloud-upload"></i> Try api');

        $form->setDefaults($defaults);

        $form->onSuccess[] = array($this, 'formSucceeded');
        return $form;
    }

    private function getParamLabel(InputParam $param)
    {
        $title = $param->getKey();
        if ($param->isRequired()) {
            $title .= ' *';
        }
        return $title;
    }

    public function formSucceeded($form, $values)
    {
        $url = $values['api_url'];

        $token = false;
        if (isset($values['token'])) {
            $token = $values['token'];
            unset($values['token']);
        }

        $method = $values['method'];
        unset($values['method']);

        $consoleRequest = new ConsoleRequest($this->handler);
        $result = $consoleRequest->makeRequest($url, $method, $values, $token);

        $this->template->add('response', $result);
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }
}


1			<?php
2
3			namespace Tomaj\NetteApi\Component;
4
5			use Nette\Application\UI\Control;
6			use Nette\Application\UI\Form;
7			use Nette\Http\Request;
8			use Tomaj\Form\Renderer\BootstrapRenderer;
9			use Tomaj\NetteApi\Authorization\ApiAuthorizationInterface;
10			use Tomaj\NetteApi\Authorization\BearerTokenAuthorization;
11			use Tomaj\NetteApi\Authorization\NoAuthorization;
12			use Tomaj\NetteApi\EndpointIdentifier;
13			use Tomaj\NetteApi\Handlers\ApiHandlerInterface;
14			use Tomaj\NetteApi\Misc\ConsoleRequest;
15			use Tomaj\NetteApi\Params\InputParam;
16
17			class ApiConsoleControl extends Control
18			{
19			private $endpoint;
20
21			private $handler;
22
23			private $authorization;
24
25			private $request;
26
27			public function __construct(Request $request, EndpointIdentifier $endpoint, ApiHandlerInterface $handler, ApiAuthorizationInterface $authorization)
28			{
29			parent::__construct(null, 'apiconsolecontrol');
30			$this->endpoint = $endpoint;
31			$this->handler = $handler;
32			$this->authorization = $authorization;
33			$this->request = $request;
34			}
35
36			public function render()
37			{
38			$this->template->setFile(__DIR__ . '/console.latte');
			0 ignored issues – show Documentation introduced 2015-12-25 23:18 UTC by Report Bug Copy Issue Report The property `$template` is declared private in `Nette\Application\UI\Control`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
39			$this->template->render();
			0 ignored issues – show Documentation introduced 2015-12-25 23:18 UTC by Report Bug Copy Issue Report The property `$template` is declared private in `Nette\Application\UI\Control`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
40			}
41
42			protected function createComponentConsoleForm()
			0 ignored issues – show Coding Style introduced 2015-12-25 23:18 UTC by Report Bug Copy Issue Report `createComponentConsoleForm` 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...
43			{
44			$form = new Form();
45
46			$defaults = [];
47
48			$form->setRenderer(new BootstrapRenderer());
49
50			$uri = $this->request->getUrl();
51			$scheme = $uri->scheme;
52			if (isset($_SERVER['HTTP_X_FORWARDED_PROTO'])) {
53			$scheme = $_SERVER['HTTP_X_FORWARDED_PROTO'];
54			}
55			$url = $scheme . '://' . $uri->host . '/api/' . $this->endpoint->getUrl();
56
57			$form->addText('api_url', 'Api Url');
58			$defaults['api_url'] = $url;
59
60			$form->addText('method', 'Method');
61			$defaults['method'] = $this->endpoint->getMethod();
62
63			if ($this->authorization instanceof BearerTokenAuthorization) {
64			$form->addText('token', 'Token:')
65			->setAttribute('placeholder', 'napište token');
			0 ignored issues – show Documentation introduced 2015-12-29 22:14 UTC by Report Bug Copy Issue Report `'napište token'` is of type `string`, but the function expects a `boolean`. 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...
66			} elseif ($this->authorization instanceof NoAuthorization) {
67			$form->addText('authorization', 'Authorization')
68			->setDisabled(true);
69			$defaults['authorization'] = 'No authorization - global access';
70			}
71
72			$params = $this->handler->params();
73			foreach ($params as $param) {
74			$count = $param->isMulti() ? 5 : 1;
75			for ($i = 0; $i < $count; $i++) {
76			$key = $param->getKey();
77			if ($param->isMulti()) {
78			$key = $key . '___' . $i;
79			}
80			$c = $form->addText($key, $this->getParamLabel($param));
81			if ($param->getAvailableValues()) {
82			$c->setOption('description', 'available values: ' . implode(' \| ', $param->getAvailableValues()));
83			}
84			}
85			}
86
87			$form->addSubmit('send', 'Otestuj')
88			->getControlPrototype()
89			->setName('button')
90			->setHtml('<i class="fa fa-cloud-upload"></i> Try api');
91
92			$form->setDefaults($defaults);
93
94			$form->onSuccess[] = array($this, 'formSucceeded');
95			return $form;
96			}
97
98			private function getParamLabel(InputParam $param)
99			{
100			$title = $param->getKey();
101			if ($param->isRequired()) {
102			$title .= ' *';
103			}
104			return $title;
105			}
106
107			public function formSucceeded($form, $values)
108			{
109			$url = $values['api_url'];
110
111			$token = false;
112			if (isset($values['token'])) {
113			$token = $values['token'];
114			unset($values['token']);
115			}
116
117			$method = $values['method'];
118			unset($values['method']);
119
120			$consoleRequest = new ConsoleRequest($this->handler);
121			$result = $consoleRequest->makeRequest($url, $method, $values, $token);
122
123			$this->template->add('response', $result);
			0 ignored issues – show Documentation introduced 2015-12-25 23:18 UTC by Report Bug Copy Issue Report The property `$template` is declared private in `Nette\Application\UI\Control`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
124			}
125			}
126

tomaj / nette-api

Push — master ( 5fd6e4...a21112 )

ApiConsoleControl::createComponentConsoleForm() B

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like