ErrorResponse::__construct() - Code Metrics - nilportugues/jsend-transformer - Measure and Improve Code Quality continuously with Scrutinizer

ErrorResponse::__construct() A
last analyzed 2015-12-15 18:07 UTC

↳ Parent: ErrorResponse

Complexity

Conditions	2
Paths	2

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	19
rs	9.4286
cc	2
eloc	10
nc	2
nop	3

<?php

/**
 * Author: Nil Portugués Calderó <[email protected]>
 * Date: 7/27/15
 * Time: 11:52 PM.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace NilPortugues\Api\JSend\Http\Message;

use NilPortugues\Api\Http\Message\AbstractResponse;

/**
 * Class ErrorResponse.
 */
class ErrorResponse extends AbstractResponse
{
    /**
     * @param string $message
     * @param int    $code
     * @param null   $data
     */
    public function __construct($message, $code = 500, $data = null)
    {
        $body = json_encode(
            array_filter(
                [
                    'status' => 'error',
                    'code' => $code,
                    'message' => (string) $message,
                    'data' => '',
                ]
            )
        );

        if ($data) {
            $body = str_replace('"data": ""', substr(substr($data, 1), 0, -1), $body);
        }

        $this->response = parent::instance($body, 500, $this->headers);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
    }
}


1			<?php
2
3			/**
4			* Author: Nil Portugués Calderó <[email protected]>
5			* Date: 7/27/15
6			* Time: 11:52 PM.
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11			namespace NilPortugues\Api\JSend\Http\Message;
12
13			use NilPortugues\Api\Http\Message\AbstractResponse;
14
15			/**
16			* Class ErrorResponse.
17			*/
18			class ErrorResponse extends AbstractResponse
19			{
20			/**
21			* @param string $message
22			* @param int $code
23			* @param null $data
24			*/
25			public function __construct($message, $code = 500, $data = null)
26			{
27			$body = json_encode(
28			array_filter(
29			[
30			'status' => 'error',
31			'code' => $code,
32			'message' => (string) $message,
33			'data' => '',
34			]
35			)
36			);
37
38			if ($data) {
39			$body = str_replace('"data": ""', substr(substr($data, 1), 0, -1), $body);
40			}
41
42			$this->response = parent::instance($body, 500, $this->headers);
			0 ignored issues – show Comprehensibility Bug introduced 2015-12-15 18:07 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`instance()` instead of `__construct()`). Are you sure this is correct? If so, you might want to change this to `$this->instance()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
43			}
44			}
45

nilportugues / jsend-transformer

ErrorResponse::__construct() A last analyzed 2015-12-15 18:07 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ErrorResponse::__construct() A
last analyzed 2015-12-15 18:07 UTC