Applications\Mail\NewApplication

Complexity

Total Complexity

23

Size/Duplication

Total Lines	157
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Importance

Changes	3
Bugs	0	Features	2

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	init()	0	42	6
A	setJob()	0	8	2
A	setUser()	0	5	1
A	setAdmin()	0	5	1
B	setReciptient()	0	16	7
A	getDepartmentManagers()	0	9	3
A	getWorkflowSettings()	0	8	2

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

/** NewApplication.php */
namespace Applications\Mail;

use Auth\Entity\UserInterface;
use Jobs\Entity\JobInterface;
use Core\Mail\StringTemplateMessage;
use Organizations\Entity\EmployeeInterface;

/**
 * Sends Information about a new Application to the recruiter
 *
 * Class NewApplication
 * @package Applications\Mail
 */
class NewApplication extends StringTemplateMessage
{
    /**
     * Job posting
     *
     * @var \Jobs\Entity\Job $job
     */
    protected $job;

    /**
     * Owner of the job posting
     *
     * @var \Auth\Entity\User $user
     */
    protected $user;

    /**
     * Organization Admin
     *
     * @var bool|\Auth\Entity\User $admin
     */
    protected $admin;

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

    /**
     * @param array $options
     */
    public function __construct(array $options = array())
    {
        parent::__construct($options);
        $this->callInitOnSetJob = true;
    }
    
    public function init()
    {
        if (!$this->job) {
            return false;
        }

        /* @var \Auth\Entity\Info $userInfo */
        $userInfo = $this->user->getInfo();
        $name = $userInfo->getDisplayName();
        if ('' == trim($name)) {
            $name = $userInfo->getEmail();
        }
        
        $variables = array(
            'name' => $name,
            'title' => $this->job->getTitle()
        );

        $this->setReciptient();

        $this->setVariables($variables);
        $subject = /*@translate*/ 'New application for your vacancy "%s"';

        if ($this->isTranslatorEnabled()) {
            $subject = $this->getTranslator()->translate($subject);
        }
        $this->setSubject(sprintf($subject, $this->job->getTitle()));
        
        /* @var \Applications\Entity\Settings $settings */
        $settings = $this->user->getSettings('Applications');

        $body = $settings->getMailAccessText();

The method getMailAccessText does not exist on object<Applications\Entity\Settings>? Since you implemented __call, maybe consider adding a @method annotation.

        if ('' == $body) {
            $body = /*@translate*/ "Hello ##name##,\n\nThere is a new application for your vacancy:\n\"##title##\"\n\n";
            if ($this->isTranslatorEnabled()) {
                $body = $this->getTranslator()->translate($body);
            }
        }
        
        $this->setBody($body);
        return $this;
    }

    /**
     * @param JobInterface $job
     * @param bool $init
     * @return $this
     */
    public function setJob(JobInterface $job, $init = true)

The parameter $init is not used and could be removed.

    {
        $this->job = $job;

$job is of type object<Jobs\Entity\JobInterface>, but the property $job was declared to be of type object<Jobs\Entity\Job>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        if ($this->callInitOnSetJob) {
            $this->init();
        }
        return $this;
    }

    /**
     * @param \Auth\Entity\User $user
     * @return $this
     */
    public function setUser($user)
    {
        $this->user=$user;
        return $this;
    }

    /**
     * @param \Auth\Entity\User $admin
     * @return $this
     */
    public function setAdmin($admin)
    {
        $this->admin = $admin;
        return $this;
    }

    protected function setReciptient() {
        $workflowSettings = $this->getWorkflowSettings();
        if ($workflowSettings->getAcceptApplicationByDepartmentManager()){
            $departmentManagers = $this->getDepartmentManagers();
            foreach ($departmentManagers as $employee) { /* @var \Organizations\Entity\Employee $employee */

The expression $departmentManagers of type boolean|object<Doctrine\Common\Collections\Collection> is not guaranteed to be traversable. How about adding an additional type check?

                $this->setTo($employee->getUser()->getInfo()->getEmail(), $employee->getUser()->getInfo()->getDisplayName());
            }
        } else {
            $this->setTo($this->user->getInfo()->getEmail(), $this->user->getInfo()->getDisplayName());
        }
        if (true === $this->admin && $this->user->getSettings('Applications')->getMailBCC()) {
            $this->addBcc($this->user->getInfo()->getEmail(), $this->user->getInfo()->getDisplayName());
        } elseif($this->admin instanceof UserInterface && $this->admin->getSettings('Applications')->getMailBCC()) {
            $this->addBcc($this->admin->getInfo()->getEmail(), $this->admin->getInfo()->getDisplayName());
        }
    }

    /**
     * @return bool|\Doctrine\Common\Collections\Collection
     */
    protected function getDepartmentManagers(){
        if (true === $this->admin) {
            return $this->user->getOrganization()->getOrganization()->getEmployeesByRole(EmployeeInterface::ROLE_DEPARTMENT_MANAGER);
        } elseif($this->admin) {
            return $this->admin->getOrganization()->getOrganization()->getEmployeesByRole(EmployeeInterface::ROLE_DEPARTMENT_MANAGER);
        } else {
            return false;
        }
    }

    /**
     * @return bool|\Organizations\Entity\WorkflowSettings
     */
    protected function getWorkflowSettings()
    {
        if ($this->user->getOrganization()->hasAssociation()) {
            return $this->user->getOrganization()->getOrganization()->getWorkflowSettings();
        } else {
            return false;
        }
    }

}