Nip\MailModule\Models\EmailsTable\EmailTrait

Complexity

Total Complexity

27

Size/Duplication

Total Lines	188
Duplicated Lines	0 %

Test Coverage

Coverage

5.33%

Importance

Changes	3
Bugs	0	Features	1

16 Methods

Rating	Name	Duplication	Size	Complexity
A	getBody()	0	3	1
A	populateFromConfig()	0	5	1
A	IsHTML()	0	7	3
A	insert()	0	6	1
A	delete()	0	4	1
A	getFrom()	0	3	1
A	getTos()	0	14	4
A	getPreviewBody()	0	3	1
A	buildMailMessageAttachments()	0	6	2
A	populateFromItem()	0	4	1
A	clearAttachments()	0	3	1
A	afterSend()	0	14	2
A	getActivitiesByEmail()	0	12	3
A	getLinksByEmail()	0	12	3
A	getCustomArgs()	0	7	1
A	getSubject()	0	3	1

<?php

namespace Nip\MailModule\Models\EmailsTable;

use ByTIC\MediaLibrary\HasMedia\HasMediaTrait;
use Nip\Mail\Models\Mailable\RecordTrait as MailableRecordTrait;
use Nip\MailModule\Models\EmailsTable\Traits\MergeTags\MergeTagsRecordTrait;
use Nip\Records\AbstractModels\Record;
use Nip_File_System;

The type Nip_File_System was not found. Maybe you did not declare it correctly or list all dependencies?

use Swift_Attachment;
use Nip\Mail\Mailer;
use Nip\Mail\Message;

/**
 * Trait EmailTrait
 * @package Nip\Mail\Models\EmailsTable
 *
 * @property int $id_item
 * @property string $type
 * @property string $from
 * @property string $from_name
 * @property string $smtp_host
 * @property string $smtp_user
 * @property string $smtp_password
 * @property string $to
 * @property string $subject
 * @property string $compiled_subject
 * @property string $body
 * @property string $compiled_body
 * @property string $vars
 * @property string $is_html
 * @property string $sent
 * @property string $date_sent
 * @property string $created
 */
trait EmailTrait
{
    use MailableRecordTrait;
    use HasMediaTrait;
    use MergeTagsRecordTrait;

    public function populateFromConfig()
    {
        $config = app('config');
        $this->from = $config->get('mail.from.address');
        $this->from_name = $config->get('mail.from.address');
    }

    /**
     * @param Record $item
     */
    public function populateFromItem($item)
    {
        $this->id_item = $item->id;
        $this->type = inflector()->singularize($item->getManager()->getTable());
    }

    /**
     * @inheritdoc
     * Used to decode html entities to proper chars
     */
    public function getFrom()
    {
        return [$this->from => html_entity_decode($this->from_name, ENT_QUOTES)];
    }

    /**
     * @return string
     */
    public function getPreviewBody()
    {
        return $this->getBody();
    }

    /**
     * @return string
     */
    public function getBody()
    {
        return $this->body;
    }

    /**
     * @return string
     */
    public function getSubject()
    {
        return $this->subject;
    }

    /**
     * @return array
     */
    public function getTos()
    {
        $emailsTos = [];
        if (preg_match_all('/\s*"?([^><,"]+)"?\s*((?:<[^><,]+>)?)\s*/', $this->to, $matches, PREG_SET_ORDER) > 0) {
            foreach ($matches as $m) {
                if (!empty($m[2])) {
                    $emailsTos[trim($m[2], '<>')] = html_entity_decode($m[1]);
                } else {
                    $emailsTos[$m[1]] = '';
                }
            }
        }

        return $emailsTos;
    }

    /**
     * @return mixed
     */
    public function insert()
    {
        $this->saveMergeTagsToDbField();
        $this->created = date('Y-m-d H:i:s');

        return parent::insert();
    }

    public function delete()
    {
        $this->clearAttachments();
        return parent::delete();
    }

    public function clearAttachments()
    {
        $file = $this->getFiles()->delete();

The assignment to $file is dead and can be removed.

Are you sure the assignment to $file is correct as $this->getFiles()->delete() targeting ByTIC\MediaLibrary\Collections\Collection::delete() seems to always return null.

    }


    /**
     * @param Message $message
     */
    public function buildMailMessageAttachments(&$message)
    {
        $emailFiles = $this->getFiles();
        foreach ($emailFiles as $emailFile) {
            $message->attach(
                Swift_Attachment::newInstance($emailFile->read(), $emailFile->getName())
            );
        }
    }

    /**
     * @return array|null
     */
    protected function getCustomArgs()
    {
        $args = [];
        $args['category'] = $this->type;
        $args['id_email'] = $this->id;

        return $args;
    }

    /**
     * @param null $value

Are you sure the doc-type for parameter $value is correct as it would always require null to be passed?

     * @return bool
     */
    public function IsHTML($value = null)
    {
        if (is_bool($value)) {

The condition is_bool($value) is always false.

            $this->is_html = $value ? 'yes' : 'no';
        }

        return $this->is_html == 'yes';
    }

    /**
     * @return mixed
     */
    public function getActivitiesByEmail()
    {
        if (!$this->getRegistry()->exists('activities-email')) {

It seems like getRegistry() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $actEmail = [];
            $activities = $this->getActivities();

The method getActivities() does not exist on Nip\MailModule\Models\EmailsTable\EmailTrait. Did you maybe mean getActivitiesByEmail()?

            foreach ($activities as $activity) {
                $actEmail[$activity->email][] = $activity;
            }
            $this->getRegistry()->set('activities-email', $actEmail);
        }

        return $this->getRegistry()->get('activities-email');
    }

    /**
     * @return mixed
     */
    public function getLinksByEmail()
    {
        if (!$this->getRegistry()->exists('links-email')) {
            $linksEmail = [];

The assignment to $linksEmail is dead and can be removed.

            $links = $this->getLinks();

The method getLinks() does not exist on Nip\MailModule\Models\EmailsTable\EmailTrait. Did you maybe mean getLinksByEmail()?

            foreach ($links as $link) {
                $actEmail[$link->url][$link->email] = $link;
            }
            $this->getRegistry()->set('links-email', $actEmail);

The variable $actEmail seems to be defined by a foreach iteration on line 196. Are you sure the iterator is never empty, otherwise this variable is not defined?

        }

        return $this->getRegistry()->get('links-email');
    }

    /**
     * @param Mailer $mailer
     * @param Message $message
     * @param $response
     */
    protected function afterSend($mailer, $message, $response)

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

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

    {
        if ($response > 0) {
            $this->sent = 'yes';
            $this->smtp_user = '';
            $this->smtp_host = '';
            $this->smtp_password = '';
//            $this->subject = '';
//            $this->body = '';
            //        $this->vars = '';
            $this->date_sent = date(DATE_DB);

The constant Nip\MailModule\Models\EmailsTable\DATE_DB was not found. Maybe you did not declare it correctly or list all dependencies?

            $this->update();

It seems like update() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->clearAttachments();
        }
    }
}