Organizations\Entity\Organization

Complexity

Total Complexity

73

Size/Duplication

Total Lines	688
Duplicated Lines	2.62 %

Coupling/Cohesion

Components	5
Dependencies	13

Importance

Changes	7
Bugs	0	Features	4

39 Methods

Rating	Name	Duplication	Size	Complexity
A	setParent()	0	6	1
A	getParent()	0	4	3
A	getHiringOrganizations()	0	4	1
A	isHiringOrganization()	0	4	1
A	setExternalId()	0	5	1
A	getExternalId()	0	4	1
A	setHydrator()	0	4	1
A	getHydrator()	0	4	1
A	setOrganizationName()	0	10	2
A	getOrganizationName()	0	4	1
A	getName()	0	7	2
A	getSearchableProperties()	0	4	1
A	getPermissions()	0	11	3
A	setPermissions()	9	9	2
A	getPermissionsResourceId()	0	4	1
C	getPermissionsUserIds()	0	40	11
A	setImage()	0	5	1
A	getImage()	0	4	1
A	setContact()	0	8	2
A	getContact()	0	7	2
A	isDraft()	0	4	1
A	setIsDraft()	0	5	1
A	getDescription()	0	4	1
A	setDescription()	0	6	1
A	setEmployees()	0	9	2
A	getEmployees()	0	13	3
A	getEmployee()	0	13	4
A	getEmployeesByRole()	0	11	3
A	isOwner()	0	4	1
A	isEmployee()	0	4	2
A	isAssociated()	0	4	2
A	updatePermissions()	0	22	3
A	setUser()	9	9	2
A	getUser()	0	4	1
A	getJobs()	0	4	1
A	getTemplate()	0	7	2
A	setTemplate()	0	5	1
A	getWorkflowSettings()	0	6	2
A	setWorkflowSettings()	0	4	1

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

namespace Organizations\Entity;

use Auth\Entity\UserInterface;
use Core\Entity\AbstractIdentifiableModificationDateAwareEntity as BaseEntity;
use Core\Entity\Collection\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Doctrine\ODM\MongoDB\Mapping\Annotations as ODM;
use Core\Entity\Permissions;
use Core\Entity\PermissionsInterface;
use Core\Entity\EntityInterface;
use Zend\Hydrator\HydratorInterface;
use Core\Entity\Hydrator\EntityHydrator;
use Core\Entity\DraftableEntityInterface;

/**
 * The organization.
 *
 * @ODM\Document(collection="organizations", repositoryClass="Organizations\Repository\Organization")
 * @ODM\HasLifecycleCallbacks
 * @ODM\Indexes({
 *      @ODM\Index(keys={
 *          "_organizationName"="text"
 *      }, name="fulltext")
 * })
 *
 * @todo write test
 * @author Mathias Weitz <weitz@cross-solution.de>
 * @author Mathias Gelhausen <gelhausen@cross-solution.de>
 */
class Organization extends BaseEntity implements OrganizationInterface, DraftableEntityInterface

The property $_organizationName is not named in camelCase.

{
    /**
     * Event name of post construct event.
     *
     * @var string
     */
    const postConstruct = 'postRepositoryConstruct';

This class constant is not uppercase (expected POSTCONSTRUCT).

    /**
     * externalId. Allows external applications to reference their primary key.
     *
     * @var string
     * @ODM\Field(type="string")
     * @ODM\Index
     */
    protected $externalId;
    
    /**
     * The actual name of the organization.
     *
     * @var \Organizations\Entity\OrganizationName
     * @ODM\ReferenceOne(targetDocument="\Organizations\Entity\OrganizationName", simple=true, cascade="persist")
     */
    protected $organizationName;
    
    /**
     * Only for sorting/searching purposes
     *
     * @var string
     * @ODM\Field(type="string")
     */
    protected $_organizationName;

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

    /**
     * Assigned permissions.
     *
     * @var PermissionsInterface
     * @ODM\EmbedOne(targetDocument="\Core\Entity\Permissions")
     */
    protected $permissions;
    
    /**
     * primary logo of an organization
     *
     * @var \Organizations\Entity\OrganizationImage
     * @ODM\ReferenceOne(targetDocument="\Organizations\Entity\OrganizationImage", inversedBy="organization", simple=true, nullable="true", cascade={"all"})
     */
    protected $image;

    /**
     * Flag indicating draft state of this job.
     *
     * @var bool
     * @ODM\Boolean
     */
    protected $isDraft = false;
    
    /**
     * Organization contact data.
     *
     * @ODM\EmbedOne(targetDocument="\Organizations\Entity\OrganizationContact") */
    protected $contact;

    /**
     * The organizations' description.
     *
     * @var string
     * @ODM\Field(type="string")
     */
    protected $description;

    /**
     * The parent of this organization.
     *
     * @see setParent()
     * @var OrganizationInterface | null
     * @ODM\ReferenceOne(targetDocument="\Organizations\Entity\Organization", simple=true, nullable=true)
     * @since 0.18
     */
    protected $parent;

    /**
     * The hiring organizations of this organization.
     *
     * @var Collection
     * @ODM\ReferenceMany(
     *      targetDocument="Organizations\Entity\Organization",
     *      repositoryMethod="getHiringOrganizationsCursor"
     * )
     * @since 0.18
     */
    protected $hiringOrganizations;

    /**
     * The associated employees (users)
     *
     * @ODM\EmbedMany(targetDocument="\Organizations\Entity\Employee")
     * @var Collection
     */
    protected $employees;

    /**
     * Jobs of this organization.
     *
     * @var Collection
     * @ODM\ReferenceMany(targetDocument="\Jobs\Entity\Job", simple=true, mappedBy="organization")
     * @since 0.18
     */
    protected $jobs;

    /**
     * the owner of a Organization
     *
     * @var UserInterface $user
     * @ODM\ReferenceOne(targetDocument="\Auth\Entity\User", simple=true)
     * @ODM\Index
     */
    protected $user;

    /**
     * Default values of an organizations job template
     *
     * @var TemplateInterface;
     * @ODM\EmbedOne(targetDocument="\Organizations\Entity\Template")
     */
    protected $template;

    /**
     * Default values Workflow
     *
     * @var WorkflowSettingsInterface $workflowSettings;
     * @ODM\EmbedOne(targetDocument="\Organizations\Entity\WorkflowSettings")
     */
    protected $workflowSettings;

    /**
     * Sets the parent of an organization
     *
     * @param OrganizationInterface $parent
     *
     * @return $this
     */
    public function setParent(OrganizationInterface $parent)
    {
        $this->parent = $parent;

        return $this;
    }

    /**
     * Gets the parent of an organization
     *
     * @param bool $returnSelf returns itself, if this organization does not have a parent?
     *
     * @return null|OrganizationInterface
     */
    public function getParent($returnSelf = false)
    {
        return $this->parent ?: ($returnSelf ? $this : null);
    }

    /**
     * Gets linked organizations
     *
     * @return Collection
     */
    public function getHiringOrganizations()
    {
        return $this->hiringOrganizations;
    }

    /**
     * @return bool
     */
    public function isHiringOrganization()
    {
        return null !== $this->parent;
    }

    /**
     * Sets the external id.
     *
     * @param $externalId
     *
     * @return self
     */
    public function setExternalId($externalId)
    {
        $this->externalId = $externalId;
        return  $this;
    }

    /**
     * Gets the internal id.
     *
     * @return string
     */
    public function getExternalId()
    {
        return $this->externalId;
    }

    /**
     * @todo review this
     * @param HydratorInterface $hydrator
     *
     * @return $this
     */
    public function setHydrator(HydratorInterface $hydrator)
    {
        return $this;
    }

    /**
     * * @todo review this
     * @return EntityHydrator
     */
    public function getHydrator()
    {
        return new EntityHydrator();
    }

    /**
     * Sets the name of an organization
     *
     * @param OrganizationName $organizationName
     *
     * @return $this
     */
    public function setOrganizationName(OrganizationName $organizationName)
    {
        if (isset($this->organizationName)) {
            $this->organizationName->refCounterDec()->refCompanyCounterDec();
        }
        $this->organizationName = $organizationName;
        $this->organizationName->refCounterInc()->refCompanyCounterInc();
        $this->_organizationName = $organizationName->getName();
        return $this;
    }

    /**
     * Gets the organization name entity of an organisation
     *
     * @return OrganizationName
     */
    public function getOrganizationName()
    {
        return $this->organizationName;
    }


    /**
     * Gets the organization name
     *
     * @return string
     */
    public function getName()
    {
        if (empty($this->organizationName)) {
            return '';
        }
        return $this->organizationName->name;

The property $name is declared protected in Organizations\Entity\OrganizationName. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

    }

    /**
     * @deprecated
     * @return array
     */
    public function getSearchableProperties()
    {
        return array();
    }

    /**
     * Gets the Permissions of an organization
     *
     * @return PermissionsInterface
     */
    public function getPermissions()
    {
        if (!$this->permissions) {
            $permissions = new Permissions();
            if ($this->user) {
                $permissions->grant($this->user, Permissions::PERMISSION_ALL);

$this->user is of type object<Auth\Entity\UserInterface>, but the function expects a string|object<Core\Entity\PermissionsResourceInterface>.

            }
            $this->setPermissions($permissions);
        }
        return $this->permissions;
    }

    /**
     * Sets the Permissions of an Organization
     *
     * @param PermissionsInterface $permissions
     *
     * @return $this
     */
    public function setPermissions(PermissionsInterface $permissions)

This method seems to be duplicated in your project.

    {
        // Assure the user has always all rights.
        if ($this->user) {
            $permissions->grant($this->user, Permissions::PERMISSION_ALL);
        }
        $this->permissions = $permissions;
        return $this;
    }

    /**
     * Gets the Permissions Resource ID
     *
     * @return string
     */
    public function getPermissionsResourceId()
    {
        return 'organization:' . $this->getId();
    }

    /**
     * @param null $type
     *
     * @return array
     */
    public function getPermissionsUserIds($type = null)
    {
        // if we have a user, grant him full access to all associated permissions.
        $user = $this->getUser();
        $spec = $user
              ? $spec = array(PermissionsInterface::PERMISSION_ALL => array($this->getUser()->getId()))
              : array();

        if (null === $type || ('Job/Permissions' != $type && 'Application' != $type)) {
            return $spec;
        }

        if ('Job/Permissions' == $type) {
            $change = EmployeePermissionsInterface::JOBS_CHANGE;
            $view = EmployeePermissionsInterface::JOBS_VIEW;
        } else {
            $change = EmployeePermissionsInterface::APPLICATIONS_CHANGE;
            $view = EmployeePermissionsInterface::APPLICATIONS_VIEW;
        }

        $employees = $this->isHiringOrganization()
            ? $this->getParent()->getEmployees()
            : $this->getEmployees();

        foreach ($employees as $emp) {
            /* @var $emp EmployeeInterface */
            if ($emp->isUnassigned()) {
                continue;
            }

            $perm = $emp->getPermissions();
            if ($perm->isAllowed($change)) {
                $spec[PermissionsInterface::PERMISSION_CHANGE][] = $emp->getUser()->getId();
            } elseif ($perm->isAllowed($view)) {
                $spec[PermissionsInterface::PERMISSION_VIEW][] = $emp->getUser()->getId();
            }
        }

        return $spec;
    }

    /**
     * Sets the logo of an organization
     *
     * @param OrganizationImage $image

Should the type for parameter $image not be null|OrganizationImage?

     *
     * @return self
     */
    public function setImage(OrganizationImage $image = null)
    {
        $this->image = $image;
        return $this;
    }

    /**
     * Gets the Logo of an organization
     *
     * @return OrganizationImage
     */
    public function getImage()
    {
        return $this->image;
    }

    /**
     * Sets the Contact Data of an organization
     *
     * @param EntityInterface $contact

Should the type for parameter $contact not be null|EntityInterface?

     *
     * @return $this
     */
    public function setContact(EntityInterface $contact = null)
    {
        if (!$contact instanceof OrganizationContact) {
            $contact = new OrganizationContact($contact);

The call to OrganizationContact::__construct() has too many arguments starting with $contact.

        }
        $this->contact = $contact;
        return $this;
    }

    /**
     * Gets the contact Data of an organization
     *
     * @return OrganizationContact
     */
    public function getContact()
    {
        if (!$this->contact instanceof OrganizationContact) {
            $this->contact = new OrganizationContact();
        }
        return $this->contact;
    }

    /**
     * Gets the Draft flag
     *
     * @return bool
     */
    public function isDraft()
    {
        return $this->isDraft;
    }

    /**
     * Sets the draft flag
     *
     * @param bool $flag
     *
     * @return $this
     */
    public function setIsDraft($flag)
    {
        $this->isDraft = (bool) $flag;
        return $this;
    }

    /**
     * Gets the default description of an organization.
     *
     * This description is used as the default of the company_description
     * used in a job template
     *
     * @return string
     */
    public function getDescription()
    {
        return $this->description;
    }

    /**
     * Set the default description af an organization
     *
     * @param string $description
     *
     * @return $this
     */
    public function setDescription($description)
    {
        $this->description = $description;

        return $this;
    }

    /**
     * Sets the the list of employees
     *
     * @param Collection $employees
     *
     * @return $this
     */
    public function setEmployees(Collection $employees)
    {
        /* todo: Throw exception or at least log incidents, where employees are added to "hiring orgs" */
        if (!$this->isHiringOrganization()) {
            $this->employees = $employees;
        }

        return $this;
    }

    /**
     * Gets the list of employees
     *
     * @return ArrayCollection|Collection
     */
    public function getEmployees()
    {
        if ($this->isHiringOrganization()) {
            // Always return empty list, as we never have employees in this case.
            return new ArrayCollection();
        }

        if (!$this->employees) {
            $this->setEmployees(new ArrayCollection());
        }

        return $this->employees;
    }

    /**
     * Gets an employee by User or ID.
     *
     * @param UserInterface|string $userOrId
     *
     * @return mixed|null
     */
    public function getEmployee($userOrId)
    {
        $employees = $this->getEmployees();
        $userId    = $userOrId instanceof \Auth\Entity\UserInterface ? $userOrId->getId() : $userOrId;

        foreach ($employees as $employee) {
            if ($employee->getUser()->getId() == $userId) {
                return $employee;
            }
        }

        return null;
    }

    /**
     * Gets a list of Employees by a user role
     *
     * @param string $role
     * @return ArrayCollection
     */
    public function getEmployeesByRole($role){
        $employees = new ArrayCollection();

        /* @var \Organizations\Entity\Employee $employee */
        foreach ($this->getEmployees() as $employee) {
            if ($role === $employee->getRole()) {
                $employees->add($employee);
            }
        }
        return $employees;
    }

    /**
     * Checks, if a User is the owner of an organization
     *
     * @param UserInterface $user
     *
     * @return bool
     */
    public function isOwner(UserInterface $user)
    {
        return $this->getUser()->getId() == $user->getId();
    }

    /**
     * Returns true, if a User is an employee of the organization
     *
     * @param UserInterface $user
     *
     * @return bool
     */
    public function isEmployee(UserInterface $user)
    {
        return $this->refs && in_array($user->getId(), $this->refs->getEmployeeIds());

The property refs does not exist on object<Organizations\Entity\Organization>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * Returns true, if the user belongs to the organization.
     *
     * @param UserInterface $user
     *
     * @return bool
     */
    public function isAssociated(UserInterface $user)
    {
        return $this->isOwner($user) || $this->isEmployee($user);
    }

    /**
     * Updates the organizationsPermissions to allow all employees to view this organization.
     *
     * In case of a HiringOrganization Permissions are granted to all employees of the parent
     * organization.
     *
     * @ODM\PreUpdate
     * @ODM\PrePersist
     * @since 0.18
     */
    public function updatePermissions()
    {
        if ($this->isHiringOrganization()) {
            $organization = $this->getParent();
            $owner        = $organization->getUser();

            $this->setUser($owner);
        } else {
            $organization = $this;
        }

        /* @var $employees null | ArrayCollection | \Doctrine\ODM\MongoDB\PersistentCollection */
        $employees = $organization->getEmployees();

        $perms = $this->getPermissions();

        foreach ($employees as $emp) {

The expression $employees of type null is not traversable.

            /* @var $emp \Organizations\Entity\Employee */
            $perms->grant($emp->getUser(), PermissionsInterface::PERMISSION_CHANGE, false);

The call to PermissionsInterface::grant() has too many arguments starting with false.

        }
        $perms->build();
    }

    public function setUser(UserInterface $user)

This method seems to be duplicated in your project.

    {
        if ($this->user) {
            $this->getPermissions()->revoke($this->user, Permissions::PERMISSION_ALL, false);

The call to PermissionsInterface::revoke() has too many arguments starting with false.

        }
        $this->user = $user;
        $this->getPermissions()->grant($user, Permissions::PERMISSION_ALL);
        return $this;
    }

    /**
     * Gets the owner of the organization
     *
     * @return UserInterface
     */
    public function getUser()
    {
        return $this->user;
    }

    /**
     * Gets the Jobs of an organization
     *
     * @return Collection
     */
    public function getJobs()
    {
        return $this->jobs;
    }

    /**
     * Gets default values of an organizations job template
     *
     * @return TemplateInterface
     */
    public function getTemplate()
    {
        if (null === $this->template){
            $this->template = new Template();
        }
        return $this->template;
    }

    /**
     * Sets default values of an organizations job template
     *
     * @return self
     */
    public function setTemplate(TemplateInterface $template)
    {
        $this->template=$template;
        return $this;
    }

    /**
     * Gets Workflow Settings
     *
     * @return WorkflowSettings|WorkflowSettingsInterface
     */
    public function getWorkflowSettings(){
        if (null == $this->workflowSettings) {
            $this->workflowSettings = new WorkflowSettings();
        }
        return $this->workflowSettings;
    }

    /**
     * Sets Workflow Settings
     *
     * @param $workflowSettings
     *
     * @return self
     */
    public function setWorkflowSettings($workflowSettings){
        $this->workflowSettings=$workflowSettings;
        return $this;
    }
}