Dynamic\FoxyStripe\Model\OrderDetail

Complexity

Total Complexity

6

Size/Duplication

Total Lines	120
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	16	2
A	canEdit()	0	3	1
A	canView()	0	3	1
A	canCreate()	0	3	1
A	canDelete()	0	3	1

<?php

namespace Dynamic\FoxyStripe\Model;

use Dynamic\FoxyStripe\Page\ProductPage;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\GridField\GridField;
use SilverStripe\Forms\GridField\GridFieldConfig_RecordViewer;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Permission;

class OrderDetail extends DataObject
{
    /**
     * @var string
     */
    private static $singular_name = 'Order Detail';

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

    /**
     * @var string
     */
    private static $plural_name = 'Order Details';

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

    /**
     * @var string
     */
    private static $description = '';

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

    /**
     * @var array
     */
    private static $db = array(

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

        'Quantity' => 'Int',
        'Price' => 'Currency',
        'ProductName' => 'Varchar(255)',
        'ProductCode' => 'Varchar(100)',
        'ProductImage' => 'Text',
        'ProductCategory' => 'Varchar(100)',
    );

    /**
     * @var array
     */
    private static $has_one = array(

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

        'Product' => ProductPage::class,
        'Order' => Order::class,
    );

    /**
     * @var array
     */
    private static $has_many = array(

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

        'OrderOptions' => OrderOption::class,
    );

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

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

        'OptionItems' => OptionItem::class,
    ];

    /**
     * @var array
     */
    private static $summary_fields = array(

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

        'Product.Title',
        'Quantity',
        'Price.Nice',
    );

    /**
     * @var string
     */
    private static $table_name = 'FS_OrderDetail';

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

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        if ($this->ID) {
            $fields->addFieldsToTab('Root.Options', array(
                GridField::create(
                    'Options',

'Options' of type string is incompatible with the type array expected by parameter $args of SilverStripe\View\ViewableData::create().

                    'Product Options',
                    $this->OrderOptions(),

The method OrderOptions() does not exist on Dynamic\FoxyStripe\Model\OrderDetail. Since you implemented __call, consider adding a @method annotation.

                    GridFieldConfig_RecordViewer::create()
                ),
            ));
        }

        return $fields;
    }

    /**
     * @param bool $member
     *
     * @return bool|int
     */
    public function canView($member = false)
    {
        return Permission::check('Product_ORDERS', 'any', $member);

$member of type boolean is incompatible with the type SilverStripe\Security\Member|integer expected by parameter $member of SilverStripe\Security\Permission::check().

    }

    /**
     * @param null $member

Are you sure the doc-type for parameter $member is correct as it would always require null to be passed?

     *
     * @return bool
     */
    public function canEdit($member = null)
    {
        return false;
    }

    /**
     * @param null $member

Are you sure the doc-type for parameter $member is correct as it would always require null to be passed?

     *
     * @return bool
     */
    public function canCreate($member = null, $context = [])
    {
        return false;
        //return Permission::check('Product_ORDERS');

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

    }

    public function canDelete($member = null)
    {
        return Permission::check('Product_ORDERS', 'any', $member);
    }
}