Waca\Background\CreationTaskBase::execute()

Complexity

Conditions	12
Paths	63

Size

Total Lines	68
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	156

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function execute()
    {
        $this->request = $this->getRequest();
        $user = $this->getTriggerUser();
        $parameters = $this->getParameters();

        if ($this->request->getStatus() !== RequestStatus::JOBQUEUE) {
            $this->markCancelled('Request is not deferred to the job queue');

            return;
        }

        if ($this->request->getEmailSent() != 0 && !isset($parameters->emailText)) {
            $this->markFailed('Request has already been sent a templated email');

            return;
        }

        if ($this->request->getEmail() === $this->getSiteConfiguration()->getDataClearEmail()) {
            $this->markFailed('Private data of request has been purged.');

            return;
        }

        $emailText = null;
        $ccMailingList = null;
        $logTarget = null;

        if (isset($parameters->emailText) && isset($parameters->ccMailingList)) {
            $emailText = $parameters->emailText;
            $ccMailingList = $parameters->ccMailingList;
            $logTarget = "custom-y";
        }

        if ($this->getEmailTemplate() !== null) {
            $emailText = $this->getEmailTemplate()->getText();
            $ccMailingList = false;
            $logTarget = $this->getEmailTemplate()->getId();
        }

        if ($emailText === null || $ccMailingList === null) {
            $this->markFailed('Unable to get closure email text');

            return;
        }

        try {
            $this->performCreation($user);

            $this->request->setStatus(RequestStatus::CLOSED);
            $this->request->setReserved(null);
            $this->request->setEmailSent(true);
            $this->request->save();

            // Log the closure as the user
            $logComment = $this->getEmailTemplate() === null ? $emailText : null;
            Logger::closeRequest($this->getDatabase(), $this->request, $logTarget, $logComment, $this->getTriggerUser());

It seems like $logTarget can also be of type string; however, parameter $target of Waca\Helpers\Logger::closeRequest() does only seem to accept integer, maybe add an additional type check?

            $requestEmailHelper = new RequestEmailHelper($this->getEmailHelper());
            $requestEmailHelper->sendMail($this->request, $emailText, $this->getTriggerUser(), $ccMailingList);
        }
        catch (Exception $ex) {
            $this->markFailed($ex->getMessage());

            return;
        }

        $this->markComplete();
    }
}