PageControllerExtension::fetchprice() - Code Metrics - Inspection of "CHANGELOG 1.0.5" - dynamic/silverstripe-foxy-discounts - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 3a5fda...57ad9e )

by Nic

created 2019-11-18 17:16 UTC

PageControllerExtension::fetchprice() C

↳ Parent: PageControllerExtension

Complexity

Conditions	17
Paths	115

Size

Total Lines	61
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	17
eloc	40
c	1
b	0
f	0
nc	115
nop	1
dl	0
loc	61
rs	5.0916

How to fix Long Method Complexity

<?php

namespace Dynamic\Foxy\Discounts\Extension;

use Dynamic\Foxy\Discounts\DiscountHelper;
use Dynamic\Foxy\Discounts\Model\Discount;
use Dynamic\Foxy\Discounts\Model\DiscountTier;
use Dynamic\Foxy\Form\AddToCartForm;
use Dynamic\Products\Page\Product;
filter:
    dependency_paths: ["lib/*"]
use SilverStripe\Control\HTTPRequest;
use SilverStripe\Core\Extension;
use SilverStripe\Dev\Debug;
use SilverStripe\Forms\HiddenField;
use SilverStripe\View\Requirements;

/**
 * Class PageControllerExtension
 * @package Dynamic\Foxy\Discounts\Extension
 */
class PageControllerExtension extends Extension
{
    /**
     * @var array
     */
    private static $allowed_actions = [

        'fetchprice',
    ];

    /**
     * @var array
     */
    private static $exempt_fields = [

        'price',
        'quantity',
        'h:product_id',
        'isAjax',
    ];

    /**
     * @param $form
     */
    public function updateAddToCartForm(&$form)
    {
        $page = $this->owner->data();
        if ($this->getIsDiscountable($page)) {
            /** @var DiscountHelper $discount */
            if ($discount = $page->getBestDiscount()) {
                Requirements::javascript('dynamic/silverstripe-foxy-discounts: client/dist/javascript/discount.js');
                $code = $page->Code;
                $fields = $form->Fields();
                $fields->push(
                    HiddenField::create(AddToCartForm::getGeneratedValue(
                        $code,
                        $discount->getDiscount()->getDiscountType(),
                        $this->getDiscountFieldValue()
                    ))->setValue($this->getDiscountFieldValue())
                        ->addExtraClass('product-discount')
                );
            }
        }
    }

    /**
     * @return string
     */
    public function getDiscountFieldValue()
    {
        /** @var Discount $discount */
        if ($discount = $this->owner->data()->getBestDiscount()->getDiscount()) {
            $tiers = $discount->DiscountTiers();
            $tiers = $discount->DiscountTiers();
            $bulkString = '';
            foreach ($tiers as $tier) {
                if ($discount->Type == 'Percent') {

                    $bulkString .= "|{$tier->Quantity}-{$tier->Percentage}";
                    $method = 'allunits';
                } elseif ($discount->Type == 'Amount') {
                    $bulkString .= "|{$tier->Quantity}-{$tier->Amount}";
                    $method = 'allunits';
                }
            }

            return "{$discount->Title}{{$method}{$bulkString}}";

        }

        return false;

    }

    public function fetchprice(HTTPRequest $request)
    {
        if (!$id = $request->getVar('h:product_id')) {
            return;
        }

        if (!$product = Product::get()->byID(explode('||', $id)[0])) {
            return;
        }

        if (!$this->getIsDiscountable($product)) {
            return;
        }

        $totalPrice = Discount::config()->get('calculate_total');
        $quantity = (int)$request->getVar('quantity');
        $cost = ($totalPrice) ? $product->Price * $quantity : $product->Price;
        $optionsQuery = $this->getOptionsQuery($request->getVars());
        $options = $product->Options()->filter($optionsQuery);

        foreach ($options as $option) {
            switch ($option->PriceModifierAction) {
                case 'Add':
                    if ($totalPrice) {
                        $cost += ($option->PriceModifier * $quantity);
                    } else {
                        $cost += $option->PriceModifier;
                    }
                    break;
                case 'Subtract':
                    if ($totalPrice) {
                        $cost -= ($option->PriceModifier * $quantity);
                    } else {
                        $cost -= $option->ProceModifier;
                    }
                    break;
                case 'Set':
                    if ($totalPrice) {
                        $cost = ($option->PriceModifier * $quantity);
                    } else {
                        $cost = $option->PriceModifier;
                    }
                    break;
            }
        }

        $discount = $this->getDiscount($quantity);

        if ($discount instanceof DiscountTier && $discount->exists()) {
            if ($discount->Discount()->Type == 'Percent') {
            if ($discount->/** @scrutinizer ignore-call */ Discount()->Type == 'Percent') {
                $discountAmount = $cost * ($discount->Percentage / 100);

            } elseif ($discount->Discount()->Type == 'Amount') {
                $discountAmount = $discount->Amount;

            }

            if (isset($discountAmount)) {
                $cost = $cost - $discountAmount;
            }
        }

        return $cost;
    }

    /**
     * @param $quantity
     * @return mixed
     */
    protected function getDiscount($quantity)
    {
        /** @var DiscountHelper $best */
        $best = $this->owner->data()->getBestDiscount();

        $best->setDiscountTier($quantity);

        $tier = $best->getDiscountTier();

        return $tier;
    }

    /**
     * @param $vars
     * @return array
     */
    protected function getOptionsQuery($vars)
    {
        $exempt = $this->owner->config()->get('exempt_fields');
        $filter['PriceModifierAction:not'] = null;


        foreach ($vars as $key => $val) {
            if (!in_array($key, $exempt)) {
                $filter['OptionModifierKey'][] = explode('||', $val)[0];
            }
        }

        return $filter;
    }

    /**
     * @param $product
     * @return bool
     */
    public function getIsDiscountable($product)
    {
        return $product->hasMethod('getHasDiscount')
            && $product->hasMethod('getBestDiscount')
            && $product->getHasDiscount();
    }
}


dynamic / silverstripe-foxy-discounts

Push — master ( 3a5fda...57ad9e )

PageControllerExtension::fetchprice() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like