EndorphinStudio\Detector\Data\Device

Complexity

Total Complexity

5

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	jsonSerialize()	0	5	1
A	setHasModel()	0	3	1
A	setModel()	0	3	1
A	getModel()	0	3	1
A	isHasModel()	0	3	1

<?php
/**
 * @author Serhii Nekhaienko <sergey.nekhaenko@gmail.com>
 * @license GPL
 * @copyright Serhii Nekhaienko &copy 2018
 * @version 4.0.2
 * @project endorphin-studio/browser-detector
 */

namespace EndorphinStudio\Detector\Data;

/**
 * Class Device
 * Result of device detection
 * @package EndorphinStudio\Detector\Data
 */
class Device extends AbstractDataWithVersion
{
    /**
     * @return Model|null
     */
    public function getModel(): Model
    {
        return $this->model;

The expression return $this->model could return the type null which is incompatible with the type-hinted return EndorphinStudio\Detector\Data\Model. Consider adding an additional type-check to rule them out.

    }

    /**
     * @param Model|null $model
     */
    public function setModel(Model $model)
    {
        $this->model = $model;
    }

    /**
     * @return bool
     */
    public function isHasModel(): bool
    {
        return $this->hasModel;
    }

    /**
     * @param bool $hasModel
     */
    public function setHasModel(bool $hasModel)
    {
        $this->hasModel = $hasModel;
    }

    /**
     * @var Model | null
     */
    private $model = null;

    /**
     * @var bool
     */
    private $hasModel = false;


    public function jsonSerialize()
    {
        $fields = get_object_vars($this);
        unset($fields['result']);
        return $fields;
    }
}