OptionGroup

Complexity

Total Complexity

17

Size/Duplication

Total Lines	149
Duplicated Lines	13.42 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

58.33%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
B	requireDefaultRecords()	20	25	5
A	getCMSValidator()	0	4	1
A	validate()	0	15	3
A	onBeforeDelete()	0	15	3
A	canView()	0	4	1
A	canEdit()	0	11	2
A	canDelete()	0	4	1
A	canCreate()	0	4	1

<?php
/**
 *
 * @package FoxyStripe
 *
 */

class OptionGroup extends DataObject
{
    /**
     * @var array
     */
    private static $db = array(
        'Title' => 'Varchar(100)'
    );

    /**
     * @var array
     */
    private static $has_many = array(
        'Options' => 'OptionItem',
    );

    /**
     * @var string
     */
    private static $singular_name = 'Product Option Group';

    /**
     * @var string
     */
    private static $plural_name = 'Product Option Groups';

    /**
     * @var string
     */
    private static $description = 'Groups of product options, e.g. size, color, etc';

    /**
     *
     */
    public function requireDefaultRecords()
    {
        parent::requireDefaultRecords();
        // create a catch-all group
        if (!OptionGroup::get()->filter(array('Title' => 'Options'))->first()) {

This code seems to be duplicated across your project.

            $do = new OptionGroup();
            $do->Title = "Options";

The property Title does not exist on object<OptionGroup>. Since you implemented __set, maybe consider adding a @property annotation.

            $do->write();
        }
        if (!OptionGroup::get()->filter(array('Title' => 'Size'))->first()) {

This code seems to be duplicated across your project.

            $do = new OptionGroup();
            $do->Title = "Size";

The property Title does not exist on object<OptionGroup>. Since you implemented __set, maybe consider adding a @property annotation.

            $do->write();
        }
        if (!OptionGroup::get()->filter(array('Title' => 'Color'))->first()) {

This code seems to be duplicated across your project.

            $do = new OptionGroup();
            $do->Title = "Color";

The property Title does not exist on object<OptionGroup>. Since you implemented __set, maybe consider adding a @property annotation.

            $do->write();
        }
        if (!OptionGroup::get()->filter(array('Title' => 'Type'))->first()) {

This code seems to be duplicated across your project.

            $do = new OptionGroup();
            $do->Title = "Type";

The property Title does not exist on object<OptionGroup>. Since you implemented __set, maybe consider adding a @property annotation.

            $do->write();
        }
    }

    /**
     * @return RequiredFields
     */
    public function getCMSValidator()
    {
        return new RequiredFields(array('Title'));
    }

    /**
     * @return ValidationResult
     */
    public function validate()
    {
        $result = parent::validate();

        $title = $this->Title;

The property Title does not exist on object<OptionGroup>. Since you implemented __set, maybe consider adding a @property annotation.

        $firstChar = substr($title, 0, 1);
        if (preg_match('/[^a-zA-Z]/', $firstChar)) {
            $result->error('The first character of the Title can only be a letter', 'bad');
        }
        if (preg_match('/[^a-zA-Z]\s/', $title)) {
            $result->error('Please only use letters, numbers and spaces in the title', 'bad');
        }

        return $result;
    }

    /**
     *
     */
    public function onBeforeDelete()
    {
        parent::onBeforeDelete();

        //make sure that if we delete this option group, we reassign the group's option items to the 'None' group.
        $items = OptionItem::get()->filter(array('ProductOptionGroupID' => $this->ID));

        if (isset($items)) {
            $noneGroup = OptionGroup::get()->filter(array('Title' => 'Options'))->first();
            foreach ($items as $item) {
                $item->ProductOptionGroupID = $noneGroup->ID;
                $item->write();
            }
        }
    }

    /**
     * @param bool $member
     * @return bool
     */
    public function canView($member = false)
    {
        return true;
    }

    /**
     * @param null $member
     * @return bool|int
     */
    public function canEdit($member = null)
    {
        switch ($this->Title) {

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

            case 'Options':
                return false;
                break;

break is not strictly necessary here and could be removed.

            default:
                return Permission::check('Product_CANCRUD', 'any', $member);
                break;

break is not strictly necessary here and could be removed.

        }
    }

    /**
     * @param null $member
     * @return bool|int
     */
    public function canDelete($member = null)
    {
        return $this->canEdit($member);
    }

    /**
     * @param null $member
     * @return bool|int
     */
    public function canCreate($member = null)
    {
        return Permission::check('Product_CANCRUD', 'any', $member);
    }
}