PhpAbac\Example\User

Complexity

Total Complexity

21

Size/Duplication

Total Lines	184
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

86.54%

Importance

Changes	1
Bugs	0	Features	0

18 Methods

Rating	Name	Duplication	Size	Complexity
A	setId()	0	5	1
A	getId()	0	3	1
A	setName()	0	5	1
A	getName()	0	3	1
A	setAge()	0	5	1
A	getAge()	0	3	1
A	setParentNationality()	0	5	1
A	getParentNationality()	0	3	1
A	addVisa()	0	5	1
A	removeVisa()	0	6	2
A	getVisas()	0	3	1
A	getVisa()	0	9	3
A	setHasDoneJapd()	0	5	1
A	getHasDoneJapd()	0	3	1
A	setHasDrivingLicense()	0	5	1
A	getHasDrivingLicense()	0	3	1
A	setCountry()	0	5	1
A	getCountry()	0	3	1

<?php

namespace PhpAbac\Example;

class User {
    /** @var int **/
    private $id;
    /** @var string **/
    private $name;
    /** @var int **/
    private $age;
    /** @var string **/
    private $parentNationality;
    /** @var array **/
    private $visas;
    /** @var bool **/
    private $hasDoneJapd;
    /** @var bool **/
    private $hasDrivingLicense;
    /** @var string Iso code of user country */
    private $country;

    /**
     * @param int $id
     * @return \PhpAbac\Example\User
     */
    public function setId($id) {
        $this->id = $id;
        
        return $this;
    }
    
    /**
     * @return int
     */
    public function getId() {
        return $this->id;
    }
    
    /**
     * @param string $name
     * @return \PhpAbac\Example\User
     */
    public function setName($name) {
        $this->name = $name;
        
        return $this;
    }
    
    /**
     * @return string
     */
    public function getName() {
        return $this->name;
    }
    
    /**
     * @param int $age
     * @return \PhpAbac\Example\User
     */
    public function setAge($age) {
        $this->age = $age;
        
        return $this;
    }
    
    /**
     * @return int
     */
    public function getAge() {
        return $this->age;
    }
    
    /**
     * @param string $parentNationality
     * @return \PhpAbac\Example\User
     */
    public function setParentNationality($parentNationality) {
        $this->parentNationality = $parentNationality;
        
        return $this;
    }
    
    /**
     * @return bool
     */
    public function getParentNationality() {
        return $this->parentNationality;
    }
    
    /**
     * @param \PhpAbac\Example\Visa $visa
     * @return \PhpAbac\Example\User
     */
    public function addVisa(Visa $visa) {
        $this->visas[$visa->getId()] = $visa;
        
        return $this;
    }
    
    /**
     * @param \PhpAbac\Example\Visa $visa
     * @return \PhpAbac\Example\User
     */
    public function removeVisa(Visa $visa) {
        if(isset($this->visas[$visa->getId()])) {

Expected 1 space after IF keyword; 0 found

            unset($this->visas[$visa->getId()]);
        }
        return $this;
    }
    
    /**
     * @return array
     */
    public function getVisas() {
        return $this->visas;
    }
	
	/**
	 * Return a specific visa
	 *
	 * @param Visa $visa

There is no parameter named $visa. Was it maybe removed?

	 *
	 * @return mixed|null
	 */
    public function getVisa($country_code) {
    	/** @var Visa $visa */
    	$visas = [];
		foreach($this->visas as $visa) {

Expected 1 space after FOREACH keyword; 0 found

    		if ($visa->getCountry()->getCode() == $country_code)

It is generally a best practice to always use braces with control structures.

				$visas[] = $visa;
		}
		return $visas;
	}
    
    /**
     * @param bool $hasDoneJapd
     * @return \PhpAbac\Example\User
     */
    public function setHasDoneJapd($hasDoneJapd) {
        $this->hasDoneJapd = $hasDoneJapd;
        
        return $this;
    }
    
    /**
     * @return bool
     */
    public function getHasDoneJapd() {
        return $this->hasDoneJapd;
    }
    
    /**
     * @param bool $hasDrivingLicense
     * @return \PhpAbac\Example\User
     */
    public function setHasDrivingLicense($hasDrivingLicense) {
        $this->hasDrivingLicense = $hasDrivingLicense;
        
        return $this;
    }
    
    /**
     * @return bool
     */
    public function getHasDrivingLicense() {
        return $this->hasDrivingLicense;
    }


    /**
     * Function to set the iso code of the user country
     *
     * @param $country
     */
    public function setCountry($country) {
        $this->country = $country;

        return $this;
    }

    /**
     * @return string Iso code of the user country
     */
    public function getCountry() {
        return $this->country;
    }
}