Client::send() - Code Metrics - Inspection of "Margin for PDF output can now be defined as an obj..." - jonnnnyw/php-phantomjs - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c11e1b...0baec8 )

by Jonny

created 2016-11-15 15:13 UTC

Client::send() A

↳ Parent: Client

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1.064

Importance

Changes

Metric	Value
dl	0
loc	10
ccs	3
cts	5
cp	0.6
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	5
nc	1
nop	2
crap	1.064

<?php

/*
 * This file is part of the php-phantomjs.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace JonnyW\PhantomJs;

use JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface;
use JonnyW\PhantomJs\Procedure\ProcedureCompilerInterface;
use JonnyW\PhantomJs\Http\MessageFactoryInterface;
use JonnyW\PhantomJs\Http\RequestInterface;
use JonnyW\PhantomJs\Http\ResponseInterface;
use JonnyW\PhantomJs\DependencyInjection\ServiceContainer;

/**
 * PHP PhantomJs
 *
 * @author Jon Wenmoth <[email protected]>
 */
class Client implements ClientInterface
{
    /**
     * Client.
     *
     * @var \JonnyW\PhantomJs\ClientInterface
     * @access private
     */
    private static $instance;

    /**
     * PhantomJs engine.
     *
     * @var \JonnyW\PhantomJs\Engine
     * @access protected
     */
    protected $engine;

    /**
     * Procedure loader.
     *
     * @var \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface
     * @access protected
     */
    protected $procedureLoader;

    /**
     * Procedure validator.
     *
     * @var \JonnyW\PhantomJs\Procedure\ProcedureCompilerInterface
     * @access protected
     */
    protected $procedureCompiler;

    /**
     * Message factory.
     *
     * @var \JonnyW\PhantomJs\Http\MessageFactoryInterface
     * @access protected
     */
    protected $messageFactory;

    /**
     * Procedure template
     *
     * @var string
     * @access protected
     */
    protected $procedure;

    /**
     * Internal constructor
     *
     * @access public
     * @param  \JonnyW\PhantomJs\Engine                               $engine
     * @param  \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface   $procedureLoader
     * @param  \JonnyW\PhantomJs\Procedure\ProcedureCompilerInterface $procedureCompiler
     * @param  \JonnyW\PhantomJs\Http\MessageFactoryInterface         $messageFactory
     * @return void

     */
    public function __construct(Engine $engine, ProcedureLoaderInterface $procedureLoader, ProcedureCompilerInterface $procedureCompiler, MessageFactoryInterface $messageFactory)
    {
        $this->engine            = $engine;
        $this->procedureLoader   = $procedureLoader;
        $this->procedureCompiler = $procedureCompiler;
        $this->messageFactory    = $messageFactory;
        $this->procedure         = 'http_default';
    }

    /**
     * Get singleton instance
     *
     * @access public
     * @return \JonnyW\PhantomJs\Client
     */
    public static function getInstance()
    {
        if (!self::$instance instanceof ClientInterface) {

            $serviceContainer = ServiceContainer::getInstance();

            self::$instance = new static(
                $serviceContainer->get('engine'),
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                $serviceContainer->get('procedure_loader'),
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                $serviceContainer->get('procedure_compiler'),
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                $serviceContainer->get('message_factory')
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return self::$instance;
    }

    /**
     * Get PhantomJs engine.
     *
     * @access public
     * @return \JonnyW\PhantomJs\Engine
     */
    public function getEngine()
    {
        return $this->engine;
    }

    /**
     * Get message factory instance
     *
     * @access public
     * @return \JonnyW\PhantomJs\Http\MessageFactoryInterface
     */
    public function getMessageFactory()
    {
        return $this->messageFactory;
    }

    /**
     * Get procedure loader instance
     *
     * @access public
     * @return \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface
     */
    public function getProcedureLoader()
    {
        return $this->procedureLoader;
    }

    /**
     * Send request
     *
     * @access public
     * @param  \JonnyW\PhantomJs\Http\RequestInterface  $request
     * @param  \JonnyW\PhantomJs\Http\ResponseInterface $response
     * @return \JonnyW\PhantomJs\Http\ResponseInterface
     */
    public function send(RequestInterface $request, ResponseInterface $response)
    {
        $procedure = $this->procedureLoader->load($this->procedure);

        $this->procedureCompiler->compile($procedure, $request);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $procedure->run($request, $response);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return $response;
    }

    /**
     * Get log.
     *
     * @access public
     * @return string
     */
    public function getLog()
    {
        return $this->getEngine()->getLog();
    }

    /**
     * Set procedure template.
     *
     * @access public
     * @param  string $procedure
     * @return void
     */
    public function setProcedure($procedure)
    {
        $this->procedure = $procedure;
    }

    /**
     * Get procedure template.
     *
     * @access public
     * @return string
     */
    public function getProcedure()
    {
        return $this->procedure;
    }

    /**
     * Get procedure compiler.
     *
     * @access public
     * @return \JonnyW\PhantomJs\Procedure\ProcedureCompilerInterface
     */
    public function getProcedureCompiler()
    {
        return $this->procedureCompiler;
    }

    /**
     * Set lazy request flag.
     *
     * @access public
     * @return void
     */
    public function isLazy()
    {
        $this->procedure = 'http_lazy';
    }
}


1		<?php
2
3		/*
4		* This file is part of the php-phantomjs.
5		*
6		* For the full copyright and license information, please view the LICENSE
7		* file that was distributed with this source code.
8		*/
9		namespace JonnyW\PhantomJs;
10
11		use JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface;
12		use JonnyW\PhantomJs\Procedure\ProcedureCompilerInterface;
13		use JonnyW\PhantomJs\Http\MessageFactoryInterface;
14		use JonnyW\PhantomJs\Http\RequestInterface;
15		use JonnyW\PhantomJs\Http\ResponseInterface;
16		use JonnyW\PhantomJs\DependencyInjection\ServiceContainer;
17
18		/**
19		* PHP PhantomJs
20		*
21		* @author Jon Wenmoth <[email protected]>
22		*/
23		class Client implements ClientInterface
24		{
25		/**
26		* Client.
27		*
28		* @var \JonnyW\PhantomJs\ClientInterface
29		* @access private
30		*/
31		private static $instance;
32
33		/**
34		* PhantomJs engine.
35		*
36		* @var \JonnyW\PhantomJs\Engine
37		* @access protected
38		*/
39		protected $engine;
40
41		/**
42		* Procedure loader.
43		*
44		* @var \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface
45		* @access protected
46		*/
47		protected $procedureLoader;
48
49		/**
50		* Procedure validator.
51		*
52		* @var \JonnyW\PhantomJs\Procedure\ProcedureCompilerInterface
53		* @access protected
54		*/
55		protected $procedureCompiler;
56
57		/**
58		* Message factory.
59		*
60		* @var \JonnyW\PhantomJs\Http\MessageFactoryInterface
61		* @access protected
62		*/
63		protected $messageFactory;
64
65		/**
66		* Procedure template
67		*
68		* @var string
69		* @access protected
70		*/
71		protected $procedure;
72
73		/**
74		* Internal constructor
75		*
76		* @access public
77		* @param \JonnyW\PhantomJs\Engine $engine
78		* @param \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface $procedureLoader
79		* @param \JonnyW\PhantomJs\Procedure\ProcedureCompilerInterface $procedureCompiler
80		* @param \JonnyW\PhantomJs\Http\MessageFactoryInterface $messageFactory
81		* @return void
		0 ignored issues – show Comprehensibility Best Practice introduced 2015-12-17 10:52 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
82		*/
83	1	public function __construct(Engine $engine, ProcedureLoaderInterface $procedureLoader, ProcedureCompilerInterface $procedureCompiler, MessageFactoryInterface $messageFactory)
84		{
85	1	$this->engine = $engine;
86	1	$this->procedureLoader = $procedureLoader;
87	1	$this->procedureCompiler = $procedureCompiler;
88	1	$this->messageFactory = $messageFactory;
89	1	$this->procedure = 'http_default';
90	1	}
91
92		/**
93		* Get singleton instance
94		*
95		* @access public
96		* @return \JonnyW\PhantomJs\Client
97		*/
98		public static function getInstance()
99		{
100		if (!self::$instance instanceof ClientInterface) {
101
102		$serviceContainer = ServiceContainer::getInstance();
103
104		self::$instance = new static(
105		$serviceContainer->get('engine'),
		0 ignored issues – show Documentation introduced 2015-12-17 10:52 UTC by Report Bug Copy Issue Report `$serviceContainer->get('engine')` is of type `object\|null`, but the function expects a `object<JonnyW\PhantomJs\Engine>`. 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...
106		$serviceContainer->get('procedure_loader'),
		0 ignored issues – show Documentation introduced 2015-12-17 10:52 UTC by Report Bug Copy Issue Report `$serviceContainer->get('procedure_loader')` is of type `object\|null`, but the function expects a `object<JonnyW\PhantomJs\...ocedureLoaderInterface>`. 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...
107		$serviceContainer->get('procedure_compiler'),
		0 ignored issues – show Documentation introduced 2015-12-17 10:52 UTC by Report Bug Copy Issue Report `$serviceContainer->get('procedure_compiler')` is of type `object\|null`, but the function expects a `object<JonnyW\PhantomJs\...edureCompilerInterface>`. 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...
108		$serviceContainer->get('message_factory')
		0 ignored issues – show Documentation introduced 2015-12-17 10:52 UTC by Report Bug Copy Issue Report `$serviceContainer->get('message_factory')` is of type `object\|null`, but the function expects a `object<JonnyW\PhantomJs\...essageFactoryInterface>`. 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...
109		);
110		}
111
112		return self::$instance;
113		}
114
115		/**
116		* Get PhantomJs engine.
117		*
118		* @access public
119		* @return \JonnyW\PhantomJs\Engine
120		*/
121		public function getEngine()
122		{
123		return $this->engine;
124		}
125
126		/**
127		* Get message factory instance
128		*
129		* @access public
130		* @return \JonnyW\PhantomJs\Http\MessageFactoryInterface
131		*/
132	1	public function getMessageFactory()
133		{
134	1	return $this->messageFactory;
135		}
136
137		/**
138		* Get procedure loader instance
139		*
140		* @access public
141		* @return \JonnyW\PhantomJs\Procedure\ProcedureLoaderInterface
142		*/
143	1	public function getProcedureLoader()
144		{
145	1	return $this->procedureLoader;
146		}
147
148		/**
149		* Send request
150		*
151		* @access public
152		* @param \JonnyW\PhantomJs\Http\RequestInterface $request
153		* @param \JonnyW\PhantomJs\Http\ResponseInterface $response
154		* @return \JonnyW\PhantomJs\Http\ResponseInterface
155		*/
156	1	public function send(RequestInterface $request, ResponseInterface $response)
157		{
158	1	$procedure = $this->procedureLoader->load($this->procedure);
159
160	1	$this->procedureCompiler->compile($procedure, $request);
		0 ignored issues – show Documentation introduced 2015-12-17 10:52 UTC by Report Bug Copy Issue Report `$request` is of type `object<JonnyW\PhantomJs\Http\RequestInterface>`, but the function expects a `object<JonnyW\PhantomJs\Procedure\InputInterface>`. 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...
161
162		$procedure->run($request, $response);
		0 ignored issues – show Documentation introduced 2015-12-17 10:52 UTC by Report Bug Copy Issue Report `$request` is of type `object<JonnyW\PhantomJs\Http\RequestInterface>`, but the function expects a `object<JonnyW\PhantomJs\Procedure\InputInterface>`. 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... Documentation introduced 2015-12-17 10:52 UTC by Report Bug Copy Issue Report `$response` is of type `object<JonnyW\PhantomJs\Http\ResponseInterface>`, but the function expects a `object<JonnyW\PhantomJs\...cedure\OutputInterface>`. 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...
163
164		return $response;
165		}
166
167		/**
168		* Get log.
169		*
170		* @access public
171		* @return string
172		*/
173		public function getLog()
174		{
175		return $this->getEngine()->getLog();
176		}
177
178		/**
179		* Set procedure template.
180		*
181		* @access public
182		* @param string $procedure
183		* @return void
184		*/
185	1	public function setProcedure($procedure)
186		{
187	1	$this->procedure = $procedure;
188	1	}
189
190		/**
191		* Get procedure template.
192		*
193		* @access public
194		* @return string
195		*/
196		public function getProcedure()
197		{
198		return $this->procedure;
199		}
200
201		/**
202		* Get procedure compiler.
203		*
204		* @access public
205		* @return \JonnyW\PhantomJs\Procedure\ProcedureCompilerInterface
206		*/
207		public function getProcedureCompiler()
208		{
209		return $this->procedureCompiler;
210		}
211
212		/**
213		* Set lazy request flag.
214		*
215		* @access public
216		* @return void
217		*/
218		public function isLazy()
219		{
220		$this->procedure = 'http_lazy';
221		}
222		}
223

jonnnnyw / php-phantomjs

GitHub Access Token became invalid

Push — master ( c11e1b...0baec8 )

Client::send() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like