EcommerceOrderStepCountryData

Complexity

Total Complexity

5

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	i18n_singular_name()	0	4	1
A	i18n_plural_name()	0	4	1
A	getCMSFields()	0	22	1
A	validate()	0	13	2

<?php

/**
 * Data for each in orderstep
 * for each country
 *
 */

class EcommerceOrderStepCountryData extends DataObject
{
    private static $singular_name = "Country specific Order Step Information";

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

    public function i18n_singular_name()
    {
        return "Country specific Order Step Information";
    }

    private static $plural_name = "Country specific Order Step Information Items";

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

    public function i18n_plural_name()
    {
        return "Country specific Order Step Information Items";
    }

    private static $db = array(

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

        'CountrySpecificEmailSubject' => 'Varchar(255)',
        'CountrySpecificEmailMessage' => 'HTMLText'
    );

    private static $has_one = array(

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

        "OrderStep" => "OrderStep",
        "EcommerceCountry" => "EcommerceCountry"
    );

    private static $required_fields = array(
        "EcommerceCountryID"
    );

    private static $summary_fields = array(

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

        "EcommerceCountry.Title" => "Country",
        "OrderStep.Title" => "Step",
        "CountrySpecificEmailSubject" => "Subject"

    );

    private static $field_labels = array(

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

        "CountrySpecificEmailSubject" => "Email Subject",
        "CountrySpecificEmailMessage" => "Email Message"
    );


    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $messageField = new HtmlEditorField('CountrySpecificEmailMessage', "Email Content");
        $messageField->setRows(7);
        $fields->addFieldsToTab(
            'Root.Main',
            array(
            new TextField('CountrySpecificEmailSubject', "Subject"),
            $messageField
        )
        );
        $fields->addFieldToTab(
            "Root.Main",
            new DropdownField(
                "EcommerceCountryID",
                "Country",
                EcommerceCountry::get()->map()
            )
        );
        return $fields;
    }

    /**
     * make sure this entry does not exist yet...
     */
    public function validate()
    {
        $valid = parent::validate();
        $filter = array(
            "OrderStepID" => $this->OrderStepID,

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

            "EcommerceCountryID" => $this->EcommerceCountryID

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

        );
        $exclude = array("ID" => $this->ID);
        if (EcommerceOrderStepCountryData::get()->filter($filter)->exclude($exclude)->count()) {
            $valid->error('An entry for this country and order step already exists. Please change the country or review existing records.');
        }
        return $valid;
    }
}