EmailTemplatesAdmin::SendEmailing() - Code Metrics - lekoala/silverstripe-email-templates - Measure and Improve Code Quality continuously with Scrutinizer

EmailTemplatesAdmin::SendEmailing() B
last analyzed 2025-04-22 23:35 UTC

↳ Parent: EmailTemplatesAdmin

Complexity

Conditions	8
Paths	24

Size

Total Lines	54
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	8
eloc	34
c	1
b	0
f	0
nc	24
nop	0
dl	0
loc	54
rs	8.1315

How to fix Long Method

<?php

namespace LeKoala\EmailTemplates\Admin;

use Exception;
use SilverStripe\Admin\ModelAdmin;
use SilverStripe\Control\Director;
use SilverStripe\View\Requirements;
use SilverStripe\Control\HTTPResponse;
use LeKoala\EmailTemplates\Models\Emailing;
use LeKoala\EmailTemplates\Models\SentEmail;
use LeKoala\EmailTemplates\Helpers\FluentHelper;
use LeKoala\EmailTemplates\Models\EmailTemplate;
filter:
    dependency_paths: ["lib/*"]
use SilverStripe\Core\Environment;
use SilverStripe\Security\Member;

/**
 * Manage your email templates
 *
 * @author lekoala
 */
class EmailTemplatesAdmin extends ModelAdmin
{

    private static $managed_models = [

        EmailTemplate::class,
        SentEmail::class,
        Emailing::class,
    ];
    private static $url_segment = 'email-templates';

    private static $menu_title = 'Emails';

    private static $menu_icon = 'lekoala/silverstripe-email-templates:images/mail.png';

    private static $allowed_actions = [

        'ImportForm',
        'SearchForm',
        'PreviewEmail',
        'PreviewEmailing',
        'SendEmailing',
        'ViewSentEmail',
        'SendTestEmailTemplate',
    ];

    public function subsiteCMSShowInMenu()
    {
        return true;
    }

    public function getList()
    {
        $list = parent::getList();
        return $list;
    }

    /**
     * Called by EmailTemplate
     *
     * @return string
     */
    public function SendEmailing()
    {
        Environment::increaseTimeLimitTo();

        $id = (int) $this->getRequest()->getVar('id');

        /* @var $Emailing Emailing */
        $Emailing = self::getEmailingById($id);
        $emails = $Emailing->getEmailsByLocales();

        $errors = 0;
        $messages = [];
        foreach ($emails as $locale => $emails) {
            foreach ($emails as $email) {
                $res = null;
                $msg = null;
                // Wrap with withLocale to make sure any environment variable (urls, etc) are properly set when sending
                FluentHelper::withLocale($locale, function () use ($email, &$res, &$msg) {
                    try {
                        $res = $email->send();
                    } catch (Exception $ex) {
                        $res = false;
                        $msg = $ex->getMessage();
                    }
                });
                if (!$res) {
                    $errors++;
                    $messages[] = $msg;
                }
            }
        }

        if ($errors == 0) {
            $Emailing->LastSent = date('Y-m-d H:i:s');
            $Emailing->LastSentCount = count($emails);
            $Emailing->write();
            $message = _t('EmailTemplatesAdmin.EMAILING_SENT', 'Emailing sent');
        } else {
            $Emailing->LastError = implode(", ", $messages);
            $Emailing->write();

            $message =  _t('EmailTemplatesAdmin.EMAILING_ERROR', 'There was an error sending email');
            $message .= ": " . implode(", ", $messages);
        }

        if (Director::is_ajax()) {
            $this->getResponse()->addHeader('X-Status', rawurlencode($message));
            if ($errors > 0) {
                // $this->getResponse()->setStatusCode(400);
            }
            return $this->getResponse();
        }

        return $message;
    }

    /**
     * Called by EmailTemplate::previewTab
     *
     * @return string
     */
    public function PreviewEmail()
    {
        // Prevent CMS styles to interfere with preview
        Requirements::clear();

        $id = (int) $this->getRequest()->getVar('id');

        $EmailTemplate = self::getEmailTemplateById($id);
        $html = $EmailTemplate->renderTemplate(true);

        Requirements::restore();

        return $html;
    }

    /**
     * Called by Emailing::previewTab
     *
     * @return string
     */
    public function PreviewEmailing()
    {
        // Prevent CMS styles to interfere with preview
        Requirements::clear();

        $id = (int) $this->getRequest()->getVar('id');

        $Emailing = self::getEmailingById($id);
        $html = $Emailing->renderTemplate();

        Requirements::restore();

        return $html;
    }

    public function SendTestEmailTemplate()
    {
        $id = (int) $this->getRequest()->getVar('id');
        $to = (string) $this->getRequest()->getVar('to');

        if (!$to) {
            die("Please set a [email protected]");

        }

        $member = Member::get()->filter('Email', $to)->first();

        $emailTemplate = self::getEmailTemplateById($id);

        $email = $emailTemplate->getEmail();

        $emailTemplate->setPreviewData($email);
        if ($member) {
            $email->setToMember($member);
            $email->addData("Member", $member);
        } else {
            $email->setTo($to);
        }
        $result = $email->doSend();

        print_r($result);
        die();

    }

    /**
     * Called by SentEmail
     *
     * @return string
     */
    public function ViewSentEmail()
    {
        // Prevent CMS styles to interfere with preview
        Requirements::clear();

        $id = (int) $this->getRequest()->getVar('id');

        $SentEmail = self::getSentEmailById($id);
        $html = $SentEmail->Body;

        Requirements::restore();

        return $html;
    }

    /**
     * @param int $id
     * @return Emailing
     */
    protected static function getEmailingById($id)
    {
        return Emailing::get()->byID($id);
    }

    /**
     * @param int $id
     * @return EmailTemplate
     */
    protected static function getEmailTemplateById($id)
    {
        return EmailTemplate::get()->byID($id);
    }

    /**
     * @param int $id
     * @return SentEmail
     */
    protected static function getSentEmailById($id)
    {
        return SentEmail::get()->byID($id);
    }
}


1			<?php
2
3			namespace LeKoala\EmailTemplates\Admin;
4
5			use Exception;
6			use SilverStripe\Admin\ModelAdmin;
7			use SilverStripe\Control\Director;
8			use SilverStripe\View\Requirements;
9			use SilverStripe\Control\HTTPResponse;
10			use LeKoala\EmailTemplates\Models\Emailing;
11			use LeKoala\EmailTemplates\Models\SentEmail;
12			use LeKoala\EmailTemplates\Helpers\FluentHelper;
13			use LeKoala\EmailTemplates\Models\EmailTemplate;
			0 ignored issues – show Bug introduced 2024-11-04 01:37 UTC by Report Bug Copy Issue Report The type `LeKoala\EmailTemplates\Models\EmailTemplate` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
14			use SilverStripe\Core\Environment;
15			use SilverStripe\Security\Member;
16
17			/**
18			* Manage your email templates
19			*
20			* @author lekoala
21			*/
22			class EmailTemplatesAdmin extends ModelAdmin
23			{
24
25			private static $managed_models = [
			0 ignored issues – show introduced 2019-12-12 16:31 UTC by Report Bug Copy Issue Report The private property `$managed_models` is not used, and could be removed. Loading history...
26			EmailTemplate::class,
27			SentEmail::class,
28			Emailing::class,
29			];
30			private static $url_segment = 'email-templates';
			0 ignored issues – show introduced 2019-12-12 16:31 UTC by Report Bug Copy Issue Report The private property `$url_segment` is not used, and could be removed. Loading history...
31			private static $menu_title = 'Emails';
			0 ignored issues – show introduced 2019-12-12 16:31 UTC by Report Bug Copy Issue Report The private property `$menu_title` is not used, and could be removed. Loading history...
32			private static $menu_icon = 'lekoala/silverstripe-email-templates:images/mail.png';
			0 ignored issues – show introduced 2020-01-24 14:40 UTC by Report Bug Copy Issue Report The private property `$menu_icon` is not used, and could be removed. Loading history...
33			private static $allowed_actions = [
			0 ignored issues – show introduced 2019-12-12 16:31 UTC by Report Bug Copy Issue Report The private property `$allowed_actions` is not used, and could be removed. Loading history...
34			'ImportForm',
35			'SearchForm',
36			'PreviewEmail',
37			'PreviewEmailing',
38			'SendEmailing',
39			'ViewSentEmail',
40			'SendTestEmailTemplate',
41			];
42
43			public function subsiteCMSShowInMenu()
44			{
45			return true;
46			}
47
48			public function getList()
49			{
50			$list = parent::getList();
51			return $list;
52			}
53
54			/**
55			* Called by EmailTemplate
56			*
57			* @return string
58			*/
59			public function SendEmailing()
60			{
61			Environment::increaseTimeLimitTo();
62
63			$id = (int) $this->getRequest()->getVar('id');
64
65			/* @var $Emailing Emailing */
66			$Emailing = self::getEmailingById($id);
67			$emails = $Emailing->getEmailsByLocales();
68
69			$errors = 0;
70			$messages = [];
71			foreach ($emails as $locale => $emails) {
72			foreach ($emails as $email) {
73			$res = null;
74			$msg = null;
75			// Wrap with withLocale to make sure any environment variable (urls, etc) are properly set when sending
76			FluentHelper::withLocale($locale, function () use ($email, &$res, &$msg) {
77			try {
78			$res = $email->send();
79			} catch (Exception $ex) {
80			$res = false;
81			$msg = $ex->getMessage();
82			}
83			});
84			if (!$res) {
85			$errors++;
86			$messages[] = $msg;
87			}
88			}
89			}
90
91			if ($errors == 0) {
92			$Emailing->LastSent = date('Y-m-d H:i:s');
93			$Emailing->LastSentCount = count($emails);
94			$Emailing->write();
95			$message = _t('EmailTemplatesAdmin.EMAILING_SENT', 'Emailing sent');
96			} else {
97			$Emailing->LastError = implode(", ", $messages);
98			$Emailing->write();
99
100			$message = _t('EmailTemplatesAdmin.EMAILING_ERROR', 'There was an error sending email');
101			$message .= ": " . implode(", ", $messages);
102			}
103
104			if (Director::is_ajax()) {
105			$this->getResponse()->addHeader('X-Status', rawurlencode($message));
106			if ($errors > 0) {
107			// $this->getResponse()->setStatusCode(400);
108			}
109			return $this->getResponse();
110			}
111
112			return $message;
113			}
114
115			/**
116			* Called by EmailTemplate::previewTab
117			*
118			* @return string
119			*/
120			public function PreviewEmail()
121			{
122			// Prevent CMS styles to interfere with preview
123			Requirements::clear();
124
125			$id = (int) $this->getRequest()->getVar('id');
126
127			$EmailTemplate = self::getEmailTemplateById($id);
128			$html = $EmailTemplate->renderTemplate(true);
129
130			Requirements::restore();
131
132			return $html;
133			}
134
135			/**
136			* Called by Emailing::previewTab
137			*
138			* @return string
139			*/
140			public function PreviewEmailing()
141			{
142			// Prevent CMS styles to interfere with preview
143			Requirements::clear();
144
145			$id = (int) $this->getRequest()->getVar('id');
146
147			$Emailing = self::getEmailingById($id);
148			$html = $Emailing->renderTemplate();
149
150			Requirements::restore();
151
152			return $html;
153			}
154
155			public function SendTestEmailTemplate()
156			{
157			$id = (int) $this->getRequest()->getVar('id');
158			$to = (string) $this->getRequest()->getVar('to');
159
160			if (!$to) {
161			die("Please set a [email protected]");
			0 ignored issues – show Best Practice introduced 2020-04-16 10:14 UTC by Report Bug Copy Issue Report Using `exit` here is not recommended. In general, usage of exit should be done with care and only when running in a scripting context like a CLI script. Loading history...
162			}
163
164			$member = Member::get()->filter('Email', $to)->first();
165
166			$emailTemplate = self::getEmailTemplateById($id);
167
168			$email = $emailTemplate->getEmail();
169
170			$emailTemplate->setPreviewData($email);
171			if ($member) {
172			$email->setToMember($member);
173			$email->addData("Member", $member);
174			} else {
175			$email->setTo($to);
176			}
177			$result = $email->doSend();
178
179			print_r($result);
180			die();
			0 ignored issues – show Best Practice introduced 2019-12-12 16:31 UTC by Report Bug Copy Issue Report Using `exit` here is not recommended. In general, usage of exit should be done with care and only when running in a scripting context like a CLI script. Loading history...
181			}
182
183			/**
184			* Called by SentEmail
185			*
186			* @return string
187			*/
188			public function ViewSentEmail()
189			{
190			// Prevent CMS styles to interfere with preview
191			Requirements::clear();
192
193			$id = (int) $this->getRequest()->getVar('id');
194
195			$SentEmail = self::getSentEmailById($id);
196			$html = $SentEmail->Body;
197
198			Requirements::restore();
199
200			return $html;
201			}
202
203			/**
204			* @param int $id
205			* @return Emailing
206			*/
207			protected static function getEmailingById($id)
208			{
209			return Emailing::get()->byID($id);
210			}
211
212			/**
213			* @param int $id
214			* @return EmailTemplate
215			*/
216			protected static function getEmailTemplateById($id)
217			{
218			return EmailTemplate::get()->byID($id);
219			}
220
221			/**
222			* @param int $id
223			* @return SentEmail
224			*/
225			protected static function getSentEmailById($id)
226			{
227			return SentEmail::get()->byID($id);
228			}
229			}
230

lekoala / silverstripe-email-templates

EmailTemplatesAdmin::SendEmailing() B last analyzed 2025-04-22 23:35 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

EmailTemplatesAdmin::SendEmailing() B
last analyzed 2025-04-22 23:35 UTC