LPTracker\models\LeadFile

Complexity

Total Complexity

8

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	3
A	toArray()	0	11	3
A	getId()	0	4	1
A	getName()	0	4	1

<?php

namespace LPTracker\models;

use LPTracker\exceptions\LPTrackerSDKException;

class LeadFile extends Model

There is one abstract method validate in this class; you could implement it, or declare this class as abstract.

{
    /**
     * @var integer
     */
    protected $id;

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

    public function __construct(array $fileData = [])
    {
        if (!empty($fileData['id'])) {
            $this->id = (int) $fileData['id'];
        }
        if (!empty($fileData['name'])) {
            $this->name = (int) $fileData['name'];

The property $name was declared of type string, but (int) $fileData['name'] is of type integer. Maybe add a type cast?

        }
    }

    /**
     * @return array
     */
    public function toArray()
    {
        $result = [];
        if (!empty($this->id)) {
            $result['id'] = $this->getId();
        }
        if (!empty($this->name)) {
            $result['name'] = $this->getName();
        }
        return $result;
    }

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

    /**
     * @return string
     */
    public function getName()
    {
        return $this->name;
    }
}