Dynamic\FoxyStripe\ORM\FoxyStripeInventoryManager

Complexity

Total Complexity

13

Size/Duplication

Total Lines	95
Duplicated Lines	0 %

Test Coverage

Coverage

93.75%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getHasInventory()	0	3	2
A	getIsProductAvailable()	0	6	2
A	updateCMSFields()	0	22	1
A	getOrders()	0	6	2
A	getNumberPurchased()	0	13	4
A	getNumberAvailable()	0	4	2

<?php

namespace Dynamic\FoxyStripe\ORM;

use Dynamic\FoxyStripe\Model\OrderDetail;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\CheckboxField;
use SilverStripe\Forms\NumericField;
use SilverStripe\Forms\ReadonlyField;
use SilverStripe\ORM\DataExtension;
use SilverStripe\Forms\DropdownField;
use SilverStripe\Core\Extension;
use UncleCheese\DisplayLogic\Forms\Wrapper;

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

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

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

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

        $fields->addFieldsToTab('Root.Inventory', array(
            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('NumberPurchased', 'Purchased', $this->getNumberPurchased())//,
                /*

64% of this comment could be valid code. Did you maybe forget this after debugging?

                NumericField::create('EmbargoLimit')
                ->setTitle('Embargo Time')
                ->setDescription('time in seconds to reserve an item once added to cart')
                */
            )->displayIf('ControlInventory')->isChecked()->end(),
        ));
    }

    /**
     * @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->owner->PurchaseLimit > $this->getNumberPurchased();
        }
        return true;
    }

    /**
     * @return int
     */
    public function getNumberAvailable()
    {
        if ($this->getIsProductAvailable()) {
            return (int)$this->owner->PurchaseLimit - (int)$this->getNumberPurchased();
        }
    }

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

        return $ct;
    }

    /**
     * @return DataList

The type Dynamic\FoxyStripe\ORM\DataList was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public function getOrders()
    {
        if ($this->owner->ID) {
            return OrderDetail::get()->filter('ProductID', $this->owner->ID);

The expression return Dynamic\FoxyStripe\Model\OrderDetail::get()->filter('ProductID', $this->owner->ID) returns the type SilverStripe\ORM\DataList which is incompatible with the documented return type Dynamic\FoxyStripe\ORM\DataList.

        }
        return false;

The expression return false returns the type false which is incompatible with the documented return type Dynamic\FoxyStripe\ORM\DataList.

    }
}