SilverCommerce\Discounts\Model\FreePostageDiscount

Complexity

Total Complexity

8

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

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

<?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 FreePostageDiscount extends Discount
{
    private static $table_name = 'Discount_FreePostage';

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

    private static $description = "removes the postage cost from an order";

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

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

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

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

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

        if ($cats->count() > 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 = 0;
                }
            }
        }

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

        $converted_amount = $converted_value;

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

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

        return $amount;
    }

    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.

    }
}