PickUpOrDeliveryModifierOptions_WeightBracket

Complexity

Total Complexity

9

Size/Duplication

Total Lines	102
Duplicated Lines	20.59 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	i18n_singular_name()	0	4	1
A	i18n_plural_name()	0	4	1
A	canCreate()	7	7	2
A	canView()	0	4	1
A	canEdit()	7	7	2
A	canDelete()	7	7	2

<?php


/**
 * below we record options for weight brackets with fixed cost
 * e.g. if Order.Weight > 10 and Order.Weight < 20 => Charge is $111.
 *
 *
 *
 */
class PickUpOrDeliveryModifierOptions_WeightBracket extends DataObject

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    private static $db = array(

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

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

        "Name" => "Varchar",
        "MinimumWeight" => "Int",
        "MaximumWeight" => "Int",
        "FixedCost" => "Currency"
    );

    private static $belongs_many_many = array(

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

The property $belongs_many_many is not used and could be removed.

        "PickUpOrDeliveryModifierOptions" => "PickUpOrDeliveryModifierOptions"
    );

    private static $indexes = array(

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

The property $indexes is not used and could be removed.

        "MinimumWeight" => true,
        "MaximumWeight" => true
    );

    private static $searchable_fields = array(

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

The property $searchable_fields is not used and could be removed.

        "Name" => "PartialMatchFilter"
    );

    private static $field_labels = array(

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

The property $field_labels is not used and could be removed.

        "Name" => "Description (e.g. small parcel)",
        "MinimumWeight" => "The minimum weight in grams",
        "MaximumWeight" => "The maximum weight in grams",
        "FixedCost" => "Total price (fixed cost)"
    );

    private static $summary_fields = array(

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

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

        "Name",
        "MinimumWeight",
        "MaximumWeight",
        "FixedCost"
    );

    private static $singular_name = "Weight Bracket";

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

The property $singular_name is not used and could be removed.

    public function i18n_singular_name()
    {
        return _t("PickUpOrDeliveryModifierOptions.WEIGHTBRACKET", "Weight Bracket");
    }

    private static $plural_name = "Weight Brackets";

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

The property $plural_name is not used and could be removed.

    public function i18n_plural_name()
    {
        return _t("PickUpOrDeliveryModifierOptions.WEIGHTBRACKETS", "Weight Brackets");
    }

    private static $default_sort = "MinimumWeight ASC, MaximumWeight ASC";

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

The property $default_sort is not used and could be removed.

    /**
     * standard SS method
     * @param Member | NULL
     * @return Boolean

Should the return type not be boolean|string|null?

     */
    public function canCreate($member = null)

This method seems to be duplicated in your project.

    {
        if (Permission::checkMember($member, Config::inst()->get("EcommerceRole", "admin_permission_code"))) {
            return true;
        }
        return parent::canCreate($member);
    }

    /**
     * standard SS method
     * @param Member | NULL
     * @return Boolean
     */
    public function canView($member = null)
    {
        return true;
    }

    /**
     * standard SS method
     * @param Member | NULL
     * @return Boolean

Should the return type not be boolean|string|null?

     */
    public function canEdit($member = null)

This method seems to be duplicated in your project.

    {
        if (Permission::checkMember($member, Config::inst()->get("EcommerceRole", "admin_permission_code"))) {
            return true;
        }
        return parent::canEdit($member);
    }

    /**
     * standard SS method
     * @param Member | NULL
     * @return Boolean

Should the return type not be boolean|string|null?

     */
    public function canDelete($member = null)

This method seems to be duplicated in your project.

    {
        if (Permission::checkMember($member, Config::inst()->get("EcommerceRole", "admin_permission_code"))) {
            return true;
        }
        return parent::canDelete($member);
    }
}