SilverCommerce\OrdersAdmin\Model\Notification

Complexity

Total Complexity

14

Size/Duplication

Total Lines	128
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
C	sendNotification()	0	56	11
B	getCMSFields()	0	31	3

<?php

namespace SilverCommerce\OrdersAdmin\Model;

use SilverStripe\ORM\DataObject;
use SilverStripe\Forms\DropdownField;
use SilverStripe\Control\Email\Email;
use SilverStripe\SiteConfig\SiteConfig;

class Notification extends DataObject
{
    private static $table_name = 'OrderNotification';

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

    /**
     * @config
     */
    private static $db = [

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

        "Status" => "Varchar",
        "SendNotificationTo" => "Enum('Customer,Vendor,Both','Customer')",
        "CustomSubject" => "Varchar(255)",
        "FromEmail" => "Varchar",
        "VendorEmail" => "Varchar"
    ];
    
    /**
     * @config
     */
    private static $has_one = [

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

        "Parent" => SiteConfig::class
    ];
    
    /**
     * @config
     */
    private static $summary_fields = [

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

        "Status",
        "SendNotificationTo",
        "FromEmail",
        "VendorEmail",
        "CustomSubject"
    ];
    
    public function getCMSFields()
    {
        $this->beforeUpdateCMSFields(function ($fields) {
            $status_field = DropdownField::create(
                "Status",

'Status' of type string is incompatible with the type array expected by parameter $args of SilverStripe\View\ViewableData::create().

                $this->fieldLabel("Status"),
                Invoice::config()->get("statuses")
            );
            
            $fields->replaceField("Status", $status_field);
            
            $vendor = $fields->dataFieldByName("VendorEmail");
            
            if ($vendor) {
                $vendor->setDescription(_t(
                    "Orders.VendorEmailDescription",
                    "Only needed when notification sent to vendor (or both)"
                ));
            }
            
            $subject = $fields->dataFieldByName("CustomSubject");
            
            if ($subject) {
                $subject->setDescription(_t(
                    "Orders.CustomSubjectDescription",
                    "Overwrite the default subject created in the notification email"
                ));
            }
        });

        return parent::getCMSFields();
    }
    
    /**
     * Deal with sending a notification. This is assumed to be an email
     * by default, but can be extended through "augmentSend" to allow
     * adding of additional notification types (such as SMS, XML, etc)
     * 
     */
    public function sendNotification($order)
    {
        // Deal with customer email
        if ($order->Email && ($this->SendNotificationTo == 'Customer' || $this->SendNotificationTo == "Both")) {

The property SendNotificationTo does not exist on SilverCommerce\OrdersAdmin\Model\Notification. Since you implemented __get, consider adding a @property annotation.

            if ($this->CustomSubject) {

The property CustomSubject does not exist on SilverCommerce\OrdersAdmin\Model\Notification. Since you implemented __get, consider adding a @property annotation.

                $subject = $this->CustomSubject;
            } else {
                $subject = _t('Orders.Order', 'Order') . " {$order->OrderNumber} {$order->Status}";
            }

            $email = Email::create()
                ->setSubject($subject)
                ->setTo($order->Email)
                ->setHTMLTemplate("\\SilverCommerce\\OrdersAdmin\\Email\\OrderNotificationEmail_Customer")
                ->setData([
                    "Order" => $order,
                    "SiteConfig" => $this->Parent(),

The method Parent() does not exist on SilverCommerce\OrdersAdmin\Model\Notification. Since you implemented __call, consider adding a @method annotation.

                    "Notification" => $this
                ]);

            if ($this->FromEmail) {

The property FromEmail does not exist on SilverCommerce\OrdersAdmin\Model\Notification. Since you implemented __get, consider adding a @property annotation.

                $email->setFrom($this->FromEmail);
            }
            
            $this->extend("augmentEmailCustomer", $email, $order);
            
            $email->send();
        }

        // Deal with vendor email
        if ($this->VendorEmail && ($this->SendNotificationTo == 'Vendor' || $this->SendNotificationTo == "Both")) {

The property VendorEmail does not exist on SilverCommerce\OrdersAdmin\Model\Notification. Since you implemented __get, consider adding a @property annotation.

            if ($this->CustomSubject) {
                $subject = $this->CustomSubject;
            } else {
                $subject = _t('Orders.Order', 'Order') . " {$order->OrderNumber} {$order->Status}";
            }
            
            $email = Email::create()
                ->setSubject($subject)
                ->setTo($this->VendorEmail)
                ->setHTMLTemplate("\\SilverCommerce\\OrdersAdmin\\Email\\OrderNotificationEmail_Vendor")
                ->setData([
                    "Order" => $order,
                    "Notification" => $this
                ]);

            if ($this->FromEmail) {
                $email->setFrom($this->FromEmail);
            }
            
            $this->extend("augmentEmailVendor", $email, $order);
            
            $email->send();
        }
        
        $this->extend("augmentSend", $order);
    }
}