OrderStatusLog_WhitelistCustomer

Complexity

Total Complexity

25

Size/Duplication

Total Lines	166
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

10 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	21	1
A	member_is_security_risk()	0	4	1
A	member_is_whitelisted()	0	7	2
A	onAfterWrite()	0	12	4
C	assessCustomer()	0	58	12

<?php


/**
 * @authors: Nicolaas [at] Sunny Side Up .co.nz
 * @package: ecommerce
 * @sub-package: model
 * @inspiration: Silverstripe Ltd, Jeremy
 **/
class OrderStatusLog_WhitelistCustomer extends OrderStatusLog

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

{
    /**
     * @var int
     */
    private static $min_number_of_paid_orders_required = 1;

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

    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.

        'Whitelist' => 'Boolean'
    );

    private static $has_one = array(

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

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

        'Member' => 'Member',
        'BasedOn' => 'OrderStatusLog_WhitelistCustomer'
    );

    private static $defaults = array(

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

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

        'InternalUseOnly' => true
    );

    private static $singular_name = 'Whitelist Customer 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 = 'Whitelist Customer 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;
    }

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

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

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

    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $fields->replaceField(
            'BasedOnID',
            CMSEditLinkField::create(
                'BasedOnID',
                _t('OrderStatusLog_WhitelistCustomer.BASED_ON', 'Based on'),
                $this->BasedOn()

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

            )
        );
        $fields->replaceField(
            'MemberID',
            CMSEditLinkField::create(
                'MemberID',
                _t('OrderStatusLog_WhitelistCustomer.CUSTOMER', 'Customer'),
                $this->Member()

The method Member() does not exist on OrderStatusLog_WhitelistCustomer. Did you maybe mean member_is_security_risk()?

            )
        );
        return $fields;
    }

    /**
     *
     *
     * @param  Member  $member  the member to check
     * @return boolean          returns true of the member is a security risk
     */
    public static function member_is_security_risk(Member $member)
    {
        return $member->IsSecurityRisk;
    }

    /**
     *
     *
     * @param  Member  $member  the member to check
     * @return boolean          returns true of the member has been whitelisted before
     */
    public static function member_is_whitelisted(Member $member)
    {
        if ($member->IsSecurityRisk) {
            return false;
        }
        return $member->IsWhitelisted;
    }

    public function onAfterWrite()
    {
        parent::onAfterWrite();
        if ($this->Whitelist) {

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

            if ($member = $this->Member()) {

The method Member() does not exist on OrderStatusLog_WhitelistCustomer. Did you maybe mean member_is_security_risk()?

                if ($member->exists()) {
                    $member->IsWhitelisted = true;
                    $member->write();
                }
            }
        }
    }

    public function assessCustomer()
    {
        //already done ...
        if ($this->Whitelist) {

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

            return true;
        }
        $order = $this->Order();

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

        if ($order && $order->exists()) {
            if ($order->MemberID) {
                $this->MemberID = $order->MemberID;

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

                $this->Whitelist = false;

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

                $member = $order->Member();
                if ($member && $member->exists()) {
                    //check if member has previouly been whitelisted
                    $previousOne = OrderStatusLog_WhitelistCustomer::get()
                        ->filter(
                            array(
                                'Whitelist' => 1,
                                'MemberID' => $member->ID
                            )
                        )
                        ->exclude(
                            array('OrderID' => $order->ID)
                        )->first();
                    if ($previousOne) {
                        $this->Whitelist = true;

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

                        $this->BasedOnID = $previousOne->ID;

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

                    } else {
                        //member has placed orders before
                        $previousOrders = Order::get()
                            ->filter(
                                array(
                                    'MemberID' => $member->ID,
                                    'CancelledByID' => 0
                                )
                            )
                            ->exclude(
                                array(
                                    'ID' => $order->ID
                                )
                            );
                        $count = 0;
                        $minOrdersRequired = $this->Config()->get('min_number_of_paid_orders_required');
                        foreach ($previousOrders as $previousOrder) {
                            if ($previousOrder->IsPaid() && $previousOrder->IsArchived()) {
                                $count++;
                                if ($count >= $minOrdersRequired) {
                                    $this->Whitelist = true;

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

                                    break;
                                }
                            }
                        }
                    }
                }
                $this->write();
            }
        }
    }
}