Create::query() - Code Metrics - Inspection of "fix test XmlHandler getAccept" - agoalofalife/bpm-online - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — 2.0.0 (c5bb78)

by Chubarov

created 2017-04-25 12:12 UTC

Create::query() A

↳ Parent: Create

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
eloc	9
nc	1
nop	0
dl	0
loc	14
ccs	0
cts	11
cp	0
crap	2
rs	9.4285
c	0
b	0
f	0

<?php
namespace agoalofalife\bpm\Actions;

use agoalofalife\bpm\Assistants\QueryBuilder;
use agoalofalife\bpm\Contracts\Action;
use agoalofalife\bpm\Contracts\ActionSet;
use agoalofalife\bpm\Contracts\Authentication;
use agoalofalife\bpm\KernelBpm;


/**
 * Class Create
 * Class to create a new object in the BPM
 *
 * @property KernelBpm kernel
 * @property string HTTP_TYPE
 * @property array data
 * @package agoalofalife\bpm\Actions
 */
class Create implements Action, ActionSet
{
    use QueryBuilder;

    protected $kernel;
    protected $url = '/';
    /**
     * Request type to created
     * @var string
     */
    protected $HTTP_TYPE = 'POST';
    protected $data = [];

    public function injectionKernel(KernelBpm $bpm)
    {
        $this->kernel = $bpm;
    }

    public function getUrl()
    {
        return $this->url;
    }

    public function processData()
    {
        $this->query();
        return $this->kernel->getHandler();
    }

    public function setData(array $data)
    {
        $this->data = $data;
    }

    private function query()
    {
        $parameters = str_replace(' ', '%20', $this->url);

        $url        = $this->kernel->getCollection() . $parameters;
        $urlHome    = config($this->kernel->getPrefixConfig() . '.UrlHome');

        $this->headers()->getCookie()->body()->httpErrorsFalse()->get();
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
        $response   =  $this->kernel->getCurl()->request($this->HTTP_TYPE, $urlHome . $url,
                       $this->headers()->getCookie()->body()->httpErrorsFalse()->get()
        );
        $body       = $response->getBody();
        $this->kernel->getHandler()->parse($body->getContents());

    }
}

1		<?php
2		namespace agoalofalife\bpm\Actions;
3
4		use agoalofalife\bpm\Assistants\QueryBuilder;
5		use agoalofalife\bpm\Contracts\Action;
6		use agoalofalife\bpm\Contracts\ActionSet;
7		use agoalofalife\bpm\Contracts\Authentication;
8		use agoalofalife\bpm\KernelBpm;
9
10
11		/**
12		* Class Create
13		* Class to create a new object in the BPM
14		*
15		* @property KernelBpm kernel
16		* @property string HTTP_TYPE
17		* @property array data
18		* @package agoalofalife\bpm\Actions
19		*/
20		class Create implements Action, ActionSet
21		{
22		use QueryBuilder;
23
24		protected $kernel;
25		protected $url = '/';
26		/**
27		* Request type to created
28		* @var string
29		*/
30		protected $HTTP_TYPE = 'POST';
31		protected $data = [];
32
33		public function injectionKernel(KernelBpm $bpm)
34		{
35		$this->kernel = $bpm;
36		}
37
38	1	public function getUrl()
39		{
40	1	return $this->url;
41		}
42
43		public function processData()
44		{
45		$this->query();
46		return $this->kernel->getHandler();
47		}
48
49	1	public function setData(array $data)
50		{
51	1	$this->data = $data;
52	1	}
53
54		private function query()
55		{
56		$parameters = str_replace(' ', '%20', $this->url);
57
58		$url = $this->kernel->getCollection() . $parameters;
59		$urlHome = config($this->kernel->getPrefixConfig() . '.UrlHome');
60
61		$this->headers()->getCookie()->body()->httpErrorsFalse()->get();
		0 ignored issues – show Unused Code introduced 2017-04-25 12:21 UTC by Report Bug Copy Issue Report The call to the method `agoalofalife\bpm\Actions\Create::get()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
62		$response = $this->kernel->getCurl()->request($this->HTTP_TYPE, $urlHome . $url,
63		$this->headers()->getCookie()->body()->httpErrorsFalse()->get()
64		);
65		$body = $response->getBody();
66		$this->kernel->getHandler()->parse($body->getContents());
		0 ignored issues – show Bug introduced 2017-04-25 12:21 UTC by Report Bug Copy Issue Report The method `parse()` does not seem to exist on `object<Illuminate\Container\Container>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
67		}
68		}

agoalofalife / bpm-online

Branch — 2.0.0 (c5bb78)

Create::query() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like