NStack\Tests\objects\CollectionShow2

Complexity

Total Complexity

6

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getId()	0	3	1
A	getPrice()	0	3	1
A	getDescription()	0	3	1
A	toArray()	0	7	1
A	parse()	0	6	1
A	getTitle()	0	3	1

<?php

namespace NStack\Tests\objects;

use NStack\Models\Model;

/**
 * Class CollectionShow2
 *
 * @package NStack\Tests\objects
 * @author  Tiago Araujo <tiar@nodesagency.com>
 */
class CollectionShow2 extends Model
{
    /** @var int */
    protected $id;

    /** @var string */
    protected $title;

    /** @var string */
    protected $description;

    /** @var float */
    protected $price;

    /**
     * parse
     *
     * @param array $data
     */
    public function parse(array $data)
    {
        $this->id           = (int)$data['id'];
        $this->title        = (string)$data['Title'];
        $this->description  = (string)$data['Description'];
        $this->price        = (string)$data['Price'];

The property $price was declared of type double, but (string)$data['Price'] is of type string. Maybe add a type cast?

    }

    /**
     * toArray
     *
     * @return array
     */
    public function toArray(): array
    {
        return [
            'id'            => $this->id,
            'Title'         => $this->title,
            'Description'   => $this->description,
            'Price'         => $this->price,
        ];
    }

    /**
     * @return int
     */
    public function getId(): int
    {
        return $this->id;
    }

    /**
     * @return string
     */
    public function getTitle(): string
    {
        return $this->title;
    }

    /**
     * @return string
     */
    public function getDescription(): string
    {
        return $this->description;
    }

    /**
     * @return float
     */
    public function getPrice(): float
    {
        return $this->price;
    }

}