Jobs\Entity\TemplateValues

Complexity

Total Complexity

27

Size/Duplication

Total Lines	263
Duplicated Lines	19.39 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	3
Bugs	0	Features	0

17 Methods

Rating	Name	Duplication	Size	Complexity
A	setQualifications()	0	5	1
A	getQualifications()	0	4	1
A	setRequirements()	0	5	1
A	getRequirements()	0	4	1
A	setBenefits()	0	5	1
A	getBenefits()	0	4	1
A	setTitle()	0	5	1
A	getTitle()	0	4	1
A	setDescription()	0	5	1
A	getDescription()	0	4	1
A	setLanguage()	0	5	1
A	getLanguage()	0	4	1
A	get()	10	10	3
A	set()	0	6	1
A	__get()	13	13	3
A	__set()	16	16	3
B	__isset()	12	12	5

<?php
/**
 * YAWIK
 *
 * @filesource
 * @copyright (c) 2013-2015 Cross Solution (http://cross-solution.de)
 * @license   MIT
 * @author    weitz@cross-solution.de
 */

namespace Jobs\Entity;

use Core\Entity\AbstractEntity;
use Doctrine\ODM\MongoDB\Mapping\Annotations as ODM;

/**
 * Holds various fields a o job opening template
 *
 * @ODM\EmbeddedDocument
 * @ODM\Indexes({
 *   @ODM\Index(keys={"requirements"="text",
 *                    "description"="text",
 *                    "qualifications"="text",
 *                    "benefits"="text",
 *                    "title"="text"},
 *              name="fulltext")
 * })
 */
class TemplateValues extends AbstractEntity implements TemplateValuesInterface

The property $_freeValues is not named in camelCase.

{

    /**
     * Qualification field of the job template
     *
     * @var String
     * @ODM\String
     */
    protected $qualifications='';

    /**
     * Requirements field of the job template
     *
     * @var String
     * @ODM\String
     */
    protected $requirements='';

    /**
     * Benefits field of the job template
     *
     * @var String
     * @ODM\String
     */
    protected $benefits='';

    /**
     * Job title field of the job template
     *
     * @var String
     * @ODM\String
     */
    protected $title='';

    /**
     * Company description field of the job template
     *
     * @var String
     * @ODM\String
     */
    protected $description='';

    /**
     * language of the job template values
     *
     * @var String
     * @ODM\String
     */
    protected $language='';

    /**
     * free values (currently not in use)
     *
     * @ODM\Hash
     */
    protected $_freeValues;

$_freeValues does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    /**
     * Sets the Qualification field of the job template
     *
     * @param $qualifications
     * @return $this
     */
    public function setQualifications($qualifications)
    {
        $this->qualifications= (string) $qualifications;
        return $this;
    }

    /**
     * Gets the qualification of a job template
     *
     * @return String
     */
    public function getQualifications()
    {
        return $this->qualifications;
    }

    /**
     * Sets the requirements of a job template
     *
     * @param String $requirements
     * @return $this
     */
    public function setRequirements($requirements)
    {
        $this->requirements=(string) $requirements;
        return $this;
    }

    /**
     * Gets the requirements of a job template
     *
     * @return String
     */
    public function getRequirements()
    {
        return $this->requirements;
    }

    /**
     * Sets the benefits of a job template
     *
     * @param String $benefits
     * @return $this
     */
    public function setBenefits($benefits)
    {
        $this->benefits=(string) $benefits;
        return $this;
    }

    /**
     * Gets the Benefits of a job template
     *
     * @return String
     */
    public function getBenefits()
    {
        return $this->benefits;
    }

    /**
     * Sets the job title of the job template
     *
     * @param $title
     * @return $this
     */
    public function setTitle($title)
    {
        $this->title=(string) $title;
        return $this;
    }

    /**
     * Gets the job title of the job template
     *
     * @return String
     */
    public function getTitle()
    {
        return $this->title;
    }

    /**
     * Sets the company description of the job template
     *
     * @param $description
     * @return $this
     */
    public function setDescription($description)
    {
        $this->description=(string) $description;
        return $this;
    }

    /**
     * Gets the company description of the job template
     *
     * @return String
     */
    public function getDescription()
    {
        return $this->description;
    }

    /**
     * Sets the language of the job template values
     *
     * @param $language
     * @return $this
     */
    public function setLanguage($language)
    {
        $this->language=(string) $language;
        return $this;
    }

    /**
     * Gets the language of the job template values
     *
     * @return String
     */
    public function getLanguage()
    {
        return $this->language;
    }

    /**
     * @param null $key
     * @param null $default
     * @param bool $set
     * @return null
     */
    public function get($key = null, $default = null, $set = false)

This method seems to be duplicated in your project.

    {
        if (isset($this->_freeValues[$key])) {
            return $this->_freeValues[$key];
        }
        if ($set) {
            $this->set($key, $default);
        }
        return $default;
    }

    /**
     * @param $key
     * @param $value
     * @return $this
     */
    public function set($key, $value)
    {
        //$this->checkWriteAccess();

84% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->_freeValues[$key] = $value;
        return $this;
    }

    public function __get($property)

This method seems to be duplicated in your project.

    {
        $getter = "get" . ucfirst($property);
        if (method_exists($this, $getter)) {
            return $this->$getter();
        }

        if (property_exists($this, $property)) {
            return $this->$property;
        }

        return $this->get($property);
    }

    public function __set($property, $value)

This method seems to be duplicated in your project.

    {
        //$this->checkWriteAccess();

84% of this comment could be valid code. Did you maybe forget this after debugging?

        $setter = 'set' . ucfirst($property);
        if (method_exists($this, $setter)) {
            $this->$setter($value);
            return;
        }

        if (property_exists($this, $property)) {
            $this->$property = $value;
            return;
        }

        $this->set($property, $value);
    }

    public function __isset($property)

This method seems to be duplicated in your project.

    {
        $value = $this->__get($property);

        if (is_array($value) && !count($value)) {
            return false;
        }
        if (is_bool($value) || is_object($value)) {
            return true;
        }
        return (bool) $value;
    }
}