OptionGroup

Complexity

Total Complexity

18

Size/Duplication

Total Lines	103
Duplicated Lines	19.42 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	8	1
B	requireDefaultRecords()	20	24	5
A	getCMSValidator()	0	3	1
A	canView()	0	3	1
A	canCreate()	0	3	1
A	onBeforeDelete()	0	14	3
A	canEdit()	0	11	2
A	canDelete()	0	3	1
A	validate()	0	14	3

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

class OptionGroup extends DataObject{

	private static $db = array(
		'Title' => 'Varchar(100)'
	);

    private static $singular_name = 'Product Option Group';
	private static $plural_name = 'Product Option Groups';
	private static $description = 'Groups of product options, e.g. size, color, etc';

	function getCMSFields(){

It is generally recommended to explicitly declare the visibility for methods.

		$fields = parent::getCMSFields();

		$this->extend('getCMSFields', $fields);

		return $fields;
	}

	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();
        }
	}

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

	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();
			}
		}
	}

	public function canView($member = false) {
		return true;
	}

	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');
				break;

break is not strictly necessary here and could be removed.

		}

	}

	public function canDelete($member = null) {
		return $this->canEdit();
	}

	public function canCreate($member = null) {
		return Permission::check('Product_CANCRUD');
	}

}