EmailReminder_EmailRecord

Complexity

Total Complexity

12

Size/Duplication

Total Lines	147
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	i18n_singular_name()	0	4	1
A	i18n_plural_name()	0	4	1
A	canCreate()	0	4	1
A	canEdit()	0	4	1
A	canDelete()	0	4	1
A	getCMSFields()	0	23	1
A	canSendAgain()	0	20	4
A	CMSEditLink()	0	3	1
A	getExportFields()	0	3	1

<?php

class EmailReminder_EmailRecord extends DataObject
{
    private static $singular_name = "Email Reminder Record";

Consider using a different property name as you override a private property of the parent class.

    public function i18n_singular_name()
    {
        return self::$singular_name;
    }

    private static $plural_name = "Email Reminder Records";

Consider using a different property name as you override a private property of the parent class.

    public function i18n_plural_name()
    {
        return self::$plural_name;
    }

    private static $db = array(

Consider using a different property name as you override a private property of the parent class.

        'EmailTo' => 'Varchar(100)',
        'ExternalRecordClassName' => 'Varchar(100)',
        'ExternalRecordID' => 'Int',
        'Result' => 'Boolean',
        'IsTestOnly' => 'Boolean',
        'EmailContent' => 'HTMLText'
    );

    private static $indexes = array(

Consider using a different property name as you override a private property of the parent class.

        'EmailTo' => true,
        'ExternalRecordClassName' => true,
        'ExternalRecordID' => true,
        'Result' => true,
        'Created' => true
    );

    private static $has_one = array(

Consider using a different property name as you override a private property of the parent class.

        'EmailReminder_NotificationSchedule' => 'EmailReminder_NotificationSchedule'
    );

    private static $summary_fields = array(

Consider using a different property name as you override a private property of the parent class.

        'Created.Nice' => 'When',
        'EmailTo' => 'Sent to',
        'Result.Nice' => 'Sent Succesfully',
        'IsTestOnly.Nice' => 'Test Only'
    );

    public static $default_sort = [
        'Created' => 'DESC',
        'ID' => 'DESC'
    ];


    public function canCreate($member = null)
    {
        return false;
    }

    public function canEdit($member = null)
    {
        return false;
    }

    public function canDelete($member = null)
    {
        return false;
    }

    /**
     * standard SS method.
     *
     * @return FieldList
     */
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $fields->addFieldsToTab(
            'Root.Details',
            array(
                $fields->dataFieldByName("EmailTo"),
                $fields->dataFieldByName("ExternalRecordClassName"),
                $fields->dataFieldByName("ExternalRecordID"),
                $fields->dataFieldByName("Result"),
                $fields->dataFieldByName("IsTestOnly"),
                $fields->dataFieldByName("EmailReminder_NotificationScheduleID"),
            )
        );
        $fields->replaceField(
            'EmailContent',
            LiteralField::create(
                'EmailContent',
                $this->EmailContent

The property EmailContent does not exist on object<EmailReminder_EmailRecord>. Since you implemented __get, maybe consider adding a @property annotation.

            )
        );
        return $fields;
    }

    /**
     *
     * tests to see if an email can be sent
     * the emails can only be sent once unless previous attempts have failed
     */
    public function canSendAgain()
    {
        $send = true;
        if ($this->Result) {

The property Result does not exist on object<EmailReminder_EmailRecord>. Since you implemented __get, maybe consider adding a @property annotation.

            if ($this->IsTestOnly) {

The property IsTestOnly does not exist on object<EmailReminder_EmailRecord>. Since you implemented __get, maybe consider adding a @property annotation.

                return true;
            } else {
                $send = false;
                $numberOfSecondsBeforeYouCanSendAgain = $this->EmailReminder_NotificationSchedule()->RepeatDays * 86400;

The method EmailReminder_NotificationSchedule does not exist on object<EmailReminder_EmailRecord>? Since you implemented __call, maybe consider adding a @method annotation.

                $todaysTS = strtotime('NOW');

                $creationTS = strtotime($this->Created);
                $difference = ($todaysTS - $creationTS);
                if ($difference > $numberOfSecondsBeforeYouCanSendAgain) {
                    $send = true;
                }
            }
        }
        return $send;
    }

    /**
     *
     * PartialMatchFilter
     */
    private static $searchable_fields = array(

Consider using a different property name as you override a private property of the parent class.

        'EmailTo' => 'PartialMatchFilter',
        'Result' => 'ExactMatchFilter',
        'IsTestOnly' => 'ExactMatchFilter',
    );

    /**
     * e.g.
     *    $controller = singleton("MyModelAdmin");
     *    return $controller->Link().$this->ClassName."/EditForm/field/".$this->ClassName."/item/".$this->ID."/edit";
      */
    public function CMSEditLink()
    {
    }


    /**
     * returns list of fields as they are exported
     * @return array

Should the return type not be array|null?

     * Field => Label
     */
    public function getExportFields()
    {
    }
}