ResponseEmitter::emit() - Code Metrics - Inspection of "Response emitter now respects headers_sent() value..." - venta/framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( a86aba...ac3ce9 )

by Alexey

created 2017-01-17 15:57 UTC

ResponseEmitter::emit() A

↳ Parent: ResponseEmitter

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2.0438

Importance

Changes

Metric	Value
cc	2
eloc	6
nc	2
nop	2
dl	0
loc	9
ccs	7
cts	9
cp	0.7778
crap	2.0438
rs	9.6666
c	0
b	0
f	0

<?php declare(strict_types = 1);

namespace Venta\Http;

use Psr\Http\Message\ResponseInterface;
use Venta\Contracts\Http\ResponseEmitter as ResponseEmitterContract;
use Zend\Diactoros\Response\SapiEmitter;
use Zend\Diactoros\Response\SapiEmitterTrait;

/**
 * Class Emitter
 *
 * @package Venta\Http
 */
final class ResponseEmitter extends SapiEmitter implements ResponseEmitterContract
{
    use SapiEmitterTrait;

    /**
     * Emits a response for a PHP SAPI environment.
     *
     * Emits the status line and headers via the header() function, and the
     * body content via the output buffer.
     *
     * @param ResponseInterface $response
     * @param null|int $maxBufferLevel Maximum output buffering level to unwrap.
     */
    public function emit(ResponseInterface $response, $maxBufferLevel = null)
    {
        if (!headers_sent()) {
            $this->emitStatusLine($response);
            $this->emitHeaders($response);
        }
        $this->flush($maxBufferLevel);
        $this->emitBody($response);
    }

    /**
     * Emit the message body.
     *
     * @param ResponseInterface $response
     */
    private function emitBody(ResponseInterface $response)

    {
        echo $response->getBody();
    }
}

1		<?php declare(strict_types = 1);
2
3		namespace Venta\Http;
4
5		use Psr\Http\Message\ResponseInterface;
6		use Venta\Contracts\Http\ResponseEmitter as ResponseEmitterContract;
7		use Zend\Diactoros\Response\SapiEmitter;
8		use Zend\Diactoros\Response\SapiEmitterTrait;
9
10		/**
11		* Class Emitter
12		*
13		* @package Venta\Http
14		*/
15		final class ResponseEmitter extends SapiEmitter implements ResponseEmitterContract
16		{
17		use SapiEmitterTrait;
18
19		/**
20		* Emits a response for a PHP SAPI environment.
21		*
22		* Emits the status line and headers via the header() function, and the
23		* body content via the output buffer.
24		*
25		* @param ResponseInterface $response
26		* @param null\|int $maxBufferLevel Maximum output buffering level to unwrap.
27		*/
28	3	public function emit(ResponseInterface $response, $maxBufferLevel = null)
29		{
30	3	if (!headers_sent()) {
31	1	$this->emitStatusLine($response);
32	1	$this->emitHeaders($response);
33		}
34	3	$this->flush($maxBufferLevel);
35	3	$this->emitBody($response);
36	3	}
37
38		/**
39		* Emit the message body.
40		*
41		* @param ResponseInterface $response
42		*/
43	3	private function emitBody(ResponseInterface $response)
		0 ignored issues – show Bug introduced 2017-01-17 16:07 UTC by Report Bug Copy Issue Report Consider using a different method name as you override a private method of the parent class. Overwriting private methods is generally fine as long as you also use private visibility. It might still be preferable for understandability to use a different method name. Loading history...
44		{
45	3	echo $response->getBody();
46		}
47		}

venta / framework

Push — master ( a86aba...ac3ce9 )

ResponseEmitter::emit() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like