Emails\CertificationEmail

Complexity

Total Complexity

12

Size/Duplication

Total Lines	87
Duplicated Lines	31.03 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	8	27	5
A	getSubject()	0	4	1
A	getBody()	19	36	4
A	getHTMLBody()	0	5	1
A	getTextBody()	0	5	1

<?php
namespace Emails;

class CertificationEmail extends VolunteerEmail
{
    protected $text;
    protected $additionalProps;

    public function __construct($profile, $emailTypeSource, $certType, $other = array())
    {
        parent::__construct($profile);
        $dataTable = \DataSetFactory::getDataTableByNames('fvs', 'longText');
        $entries = $dataTable->read(new \Data\Filter("id eq $emailTypeSource"));
        if(empty($entries))
        {
            throw new \Exception("Could not locate email with source type $emailTypeSource");
        }
        if(isset($entries['value']))

This code seems to be duplicated across your project.

        {
            $this->text = $entries['value'];
        }
        else if(isset($entries[0]['value']))
        {
            $this->text = $entries[0]['value'];
        }
        $this->addToAddress($this->profile->email);
        $this->additionalProps = $other;
        $dataTable = \DataSetFactory::getDataTableByNames('fvs', 'certifications');
        $entries = $dataTable->read(new \Data\Filter("certID eq $certType"));
        if(empty($entries))
        {
            throw new \Exception("Could not locate certification with type $certType");
        }
        $this->additionalProps['certType'] = $entries[0]['name'];
    }

    public function getSubject()
    {
        return 'Certification Notification - Burning Flipside Volunteer System';
    }

    protected function getBody($html = true)
    {
        $firstName = $this->profile->firstName;
        $lastName = $this->profile->lastName;
        $paperName = $this->profile->getDisplayName('paperName');
        $webName = $this->profile->getDisplayName('webName');
        $certType = $this->additionalProps['certType'];
        $rejectReason = $this->additionalProps['reason'];
        $vars = array(
          '{$firstName}' => $firstName,
          '{$lastName}' => $lastName,
          '{$paperName}' => $paperName,
          '{$webName}' => $webName,
          '{$certType}' => $certType,
          '{$rejectReason}' => $rejectReason
        );
        if($html === true)

This code seems to be duplicated across your project.

        {
            $text = strtr($this->text, $vars);
            return $text;
        }
        else
        {
            $rawText = $this->text;
            $index = strpos($rawText, "<script");
            if($index !== false)
            {
                $end = strpos($rawText, "</script>");
                if($index === 0)
                {
                    $rawText = substr($rawText, $end+9);
                }
            }
            return strtr(strip_tags($rawText), $vars);
        }
    }

    public function getHTMLBody()
    {
        $body = $this->getBody();
        return $body;
    }

    public function getTextBody()
    {
        $body = $this->getBody(false);
        return $body;
    }
}