Dynamic\Foxy\Inventory\Extension\ProductInventoryManager

Complexity

Total Complexity

24

Size/Duplication

Total Lines	144
Duplicated Lines	0 %

Importance

Changes	4
Bugs	0	Features	1

9 Methods

Rating	Name	Duplication	Size	Complexity
A	updateCMSFields()	0	21	1
A	validate()	0	6	3
A	getOrders()	0	21	6
A	getHasInventory()	0	3	2
A	updateGetIsAvailable()	0	11	5
A	getCartReservations()	0	6	1
A	getNumberPurchased()	0	9	3
A	getNumberAvailable()	0	3	1
A	getIsProductAvailable()	0	7	2

<?php

namespace Dynamic\Foxy\Inventory\Extension;

use Dynamic\Foxy\Inventory\Model\CartReservation;
use Dynamic\Foxy\Orders\Model\OrderDetail;
use SilverStripe\Forms\CheckboxField;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\NumericField;
use SilverStripe\Forms\ReadonlyField;
use SilverStripe\ORM\ArrayList;
use SilverStripe\ORM\DataExtension;
use SilverStripe\ORM\DataList;
use SilverStripe\ORM\ValidationResult;
use UncleCheese\DisplayLogic\Forms\Wrapper;

class ProductInventoryManager extends DataExtension
{
    /**
     * @var array
     */
    private static $db = [

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

        'ControlInventory' => 'Boolean',
        'PurchaseLimit' => 'Int',
    ];

    /**
     * @param FieldList $fields
     */
    public function updateCMSFields(FieldList $fields)
    {
        $fields->removeByName([
            'ControlInventory',
            'PurchaseLimit',
            'EmbargoLimit',
            'NumberPurchased',
        ]);

        $fields->addFieldsToTab('Root.Inventory', [
            Wrapper::create(
                CheckboxField::create('ControlInventory', 'Control Inventory?')
                    ->setDescription('limit the number of this product available for purchase'),
                Wrapper::create(
                    NumericField::create('PurchaseLimit')
                        ->setTitle('Number Available')
                        ->setDescription('add to cart form will be disabled once number available equals purchased'),
                    ReadonlyField::create('NumberAvailable', 'Remaining Available', $this->getNumberAvailable())
                        ->setDescription('This takes into account products added to the cart. Products removed from the cart may persist in the "Cart Reservations" until the expiration time.')//,
                )->displayIf('ControlInventory')->isChecked()->end()
            )->displayIf('Available')->isChecked()->end(),
        ]);
    }

    /**
     * @param ValidationResult $validationResult
     * @throws \SilverStripe\ORM\ValidationException
     */
    public function validate(ValidationResult $validationResult)
    {
        parent::validate($validationResult);

        if ($this->owner->ControlInventory && $this->owner->PurchaseLimit == 0) {

It seems like you are loosely comparing $this->owner->PurchaseLimit of type mixed|null to 0; this is ambiguous as not only 0 == 0 is true, but null == 0 is true, too. Consider using a strict comparison ===.

            $validationResult->addFieldError('PurchaseLimit', 'You must specify a purchase limit more than 0');
        }
    }

    /**
     * @param $available
     */
    public function updateGetIsAvailable(&$available)
    {
        if ($this->owner->Variations()->count()) {
            foreach ($this->owner->Variations() as $variation) {
                $available = false;
                if ($variation->getIsAvailable()) {
                    $available = true;
                }
            }
        } elseif ($this->getHasInventory()) {
            $available = $this->getIsProductAvailable();
        }
    }

    /**
     * @return bool
     */
    public function getHasInventory()
    {
        return $this->owner->ControlInventory && $this->owner->PurchaseLimit != 0;

It seems like you are loosely comparing $this->owner->PurchaseLimit of type mixed|null to 0; this is ambiguous as not only 0 == 0 is true, but null == 0 is true, too. Consider using a strict comparison ===.

    }

    /**
     * @return bool
     */
    public function getIsProductAvailable()
    {
        if ($this->owner->getHasInventory()) {
            return $this->getNumberAvailable() > 0;
        }

        return true;
    }

    /**
     * @return int|void
     */
    public function getNumberAvailable()
    {
        return (int)$this->owner->PurchaseLimit - (int)$this->getNumberPurchased() - (int)$this->getCartReservations()->count();
    }

    /**
     * @return int
     */
    public function getNumberPurchased()
    {
        $ct = 0;
        if ($this->getOrders()) {
            foreach ($this->getOrders() as $order) {
                $ct += $order->Quantity;
            }
        }
        return $ct;
    }

    /**
     * @return DataList|bool
     */
    public function getOrders()
    {
        if ($this->owner->ID) {
            $orderDetails = OrderDetail::get()->filter('ProductID', $this->owner->ID);
            $orders = ArrayList::create();
            foreach ($orderDetails as $orderDetail) {
                $hasVariation = false;
                foreach ($orderDetail->OrderVariations() as $variation) {
                    if ($variation->VariationID > 0) {
                        $hasVariation = true;
                    }
                }
                if (!$hasVariation) {
                    $orders->push($orderDetail);
                }
            }

            return $orders;
        }

        return false;
    }

    /**
     * @return DataList
     */
    public function getCartReservations()
    {
        return CartReservation::get()
            ->filter([
                'ProductID' => $this->owner->ID,
                'Expires:GreaterThan' => date('Y-m-d H:i:s', strtotime('now')),
            ]);
    }
}