JsonResponse::__construct() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/master'" - psfs/core - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 1b7368...050678 )

by Fran

created 2023-06-03 17:53 UTC

JsonResponse::__construct() A

↳ Parent: JsonResponse

Complexity

Conditions	4
Paths	4

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	7
c	1
b	0
f	0
nc	4
nop	5
dl	0
loc	9
ccs	0
cts	8
cp	0
crap	20
rs	10

<?php
namespace PSFS\base\dto;

class JsonResponse extends Dto
{
    /**
     * Response message
     * @var string $message
     */
    public $message = 'No message';
    /**
     * Response result
     * @var bool $success
     */
    public $success = false;
    /**
     * Data of response
     * @var array $data
     */
    public $data = null;
    /**
     * Number of total results
     * @var int total
     */
    public $total = 0;
    /**
     * Number of pages availables
     * @var int pages
     */
    public $pages = 1;

    /**
     * JsonResponse constructor.
     * @param array $data
     * @param bool $result
     * @param integer $total
     * @param int $pages
     * @param string $message
     * @throws \PSFS\base\exception\GeneratorException
     */
    public function __construct($data = array(), $result = false, $total = null, $pages = 1, $message = null)
    {
        parent::__construct(false);
        $this->data = $data;
        $this->success = $result;
        $this->total = $total ?: (is_array($data) ? count($data) : 0);

        $this->pages = $pages;
        if(null !== $message) {
            $this->message = $message;
        }
    }
}

1			<?php
2			namespace PSFS\base\dto;
3
4			class JsonResponse extends Dto
5			{
6			/**
7			* Response message
8			* @var string $message
9			*/
10			public $message = 'No message';
11			/**
12			* Response result
13			* @var bool $success
14			*/
15			public $success = false;
16			/**
17			* Data of response
18			* @var array $data
19			*/
20			public $data = null;
21			/**
22			* Number of total results
23			* @var int total
24			*/
25			public $total = 0;
26			/**
27			* Number of pages availables
28			* @var int pages
29			*/
30			public $pages = 1;
31
32			/**
33			* JsonResponse constructor.
34			* @param array $data
35			* @param bool $result
36			* @param integer $total
37			* @param int $pages
38			* @param string $message
39			* @throws \PSFS\base\exception\GeneratorException
40			*/
41			public function __construct($data = array(), $result = false, $total = null, $pages = 1, $message = null)
42			{
43			parent::__construct(false);
44			$this->data = $data;
45			$this->success = $result;
46			$this->total = $total ?: (is_array($data) ? count($data) : 0);
			0 ignored issues – show introduced 2018-11-03 11:20 UTC by Report Bug Copy Issue Report The condition `is_array($data)` is always `true`. Loading history...
47			$this->pages = $pages;
48			if(null !== $message) {
49			$this->message = $message;
50			}
51			}
52			}

psfs / core

Push — master ( 1b7368...050678 )

JsonResponse::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like