SilverCommerce\Discounts\Model\PercentageDiscount

Complexity

Total Complexity

8

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	4
Bugs	1	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	appliedAmount()	0	3	1
B	calculateAmount()	0	36	7

<?php

namespace SilverCommerce\Discounts\Model;

use SilverStripe\ORM\ArrayList;
use SilverCommerce\Discounts\Model\Discount;
use SilverCommerce\OrdersAdmin\Model\Estimate;
use SilverCommerce\TaxAdmin\Helpers\MathsHelper;

class PercentageDiscount extends Discount
{
    private static $table_name = 'Discount_Percentage';

The private property $table_name is not used, and could be removed.

    private static $description = "A simple cost-based discount";

The private property $description is not used, and could be removed.

    private static $db = [

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

        "Amount"    => "Decimal"
    ];

    public function appliedAmount(AppliedDiscount $item)
    {
        return $this->calculateAmount($item->Estimate());

The method Estimate() does not exist on SilverCommerce\Discounts\Model\AppliedDiscount. Since you implemented __call, consider adding a @method annotation.

    }

    public function calculateAmount(Estimate $estimate)
    {
        $cats = $this->Categories();

The method Categories() does not exist on SilverCommerce\Discounts\Model\PercentageDiscount. Since you implemented __call, consider adding a @method annotation.

        $all_products = ArrayList::create();
        $value = $estimate->getSubTotal();
        $min = (float) $this->MinOrder;

The property MinOrder does not exist on SilverCommerce\Discounts\Model\PercentageDiscount. Since you implemented __get, consider adding a @property annotation.

        if ($cats->count() > 0) {
            $value = 0;
            foreach ($cats as $cat) {
                $all_products->merge($cat->Products());
            }

            foreach ($estimate->Items() as $line_item) {
                $match = $line_item->FindStockItem();
                if ($all_products->find('ID', $match->ID)) {
                    $value += ($line_item->Quantity * $line_item->UnitPrice);
                }
            }
        }

        $converted_value = (int) ($value * 100);

        $converted_amount = $converted_value * ($this->Amount / 100);

The property Amount does not exist on SilverCommerce\Discounts\Model\PercentageDiscount. Since you implemented __get, consider adding a @property annotation.

        $amount = MathsHelper::round($converted_amount, 0)/100;

        if ($amount > $value) {
            $amount = $value;
        }

        if ($value < $min) {
            $amount = 0;
        }

        return $amount;
    }
}