ilateral\SilverStripe\Notifier\Types\EmailNotification

Complexity

Total Complexity

5

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getRenderedSubject()	0	3	1
A	send()	0	43	4

<?php

namespace ilateral\SilverStripe\Notifier\Types;

use SilverStripe\Control\Email\Email;

/**
 * Simple wrapper for SilverStripe email notoifications
 */
class EmailNotification extends NotificationType
{
    private static $table_name = "Notifications_EmailNotification";

The private property $table_name is not used, and could be removed.

    private static $singular_name = 'Email Notification';

The private property $singular_name is not used, and could be removed.

    private static $plural_name = 'Email Notifications';

The private property $plural_name is not used, and could be removed.

    private static $template = self::class;

    private static $db = [

The private property $db is not used, and could be removed.

        'Subject' => 'Varchar'
    ];

    private static $casting = [

The private property $casting is not used, and could be removed.

        'RenderedSubject' => 'Varchar'
    ];

    /**
     * Return a rendered version of this notification's subject using the
     * current object as a base
     *
     * @return string
     */
    public function getRenderedSubject(): string
    {
        return $this->renderString((string) $this->Subject);

The property Subject does not exist on ilateral\SilverStripe\Notifier\Types\EmailNotification. Since you implemented __get, consider adding a @property annotation.

    }

    public function send(
        array $custom_recipients = [],
        array $custom_data = []
    ) {
        $recipients = array_merge(
            $this->getRecipients(),
            $custom_recipients
        );

        $from = $this->getSender();
        $template = $this->config()->template;
        $object = $this->getObject();
        $subject = $this->getRenderedSubject();
        $content = $this->getRenderedContent();
        $data = [
            'Object' => $object,
            'Content' => $content
        ];
        $data = array_merge($data, $custom_data);

        if (empty($from)) {
            $from =  Email::config()->admin_email;
        }

        foreach ($recipients as $recipient) {
            // If recipient is blank for some reason
            // then skip sending
            $recipient = trim($recipient);

            if (empty($recipient)) {
                continue;
            }

            Email::create()
                ->setTo($recipient)
                ->setFrom($from)
                ->setSubject($subject)
                ->setData($data)
                ->setHTMLTemplate($template)
                ->send();
        }

        return;
    }
}