Dynamic\RecipeBook\Model\RecipeDirection

Complexity

Total Complexity

5

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

5 Methods

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

<?php

namespace Dynamic\RecipeBook\Model;

use Dynamic\RecipeBook\Page\RecipePage;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\HTMLEditor\HTMLEditorField;
use SilverStripe\ORM\DataObject;

/**
 * Class RecipeDirection
 * @package Dynamic\RecipeBook\Page
 *
 * @property string $Title
 * @property int $Sort
 * @property int $RecipeID
 * @method RecipePage Recipe()
 */
class RecipeDirection extends DataObject
{
    /**
     * @var array
     */
    private static $db = [

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

        'Title' => 'HTMLText',
        'Sort' => 'Int',
    ];

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

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

        'Recipe' => RecipePage::class,
    ];

    /**
     * @var string
     */
    private static $default_sort = 'Sort';

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

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

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

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $this->beforeUpdateCMSFields(function (FieldList $fields) {
            $fields->addFieldToTab(
                'Root.Main',
                HTMLEditorField::create('Title')
                    ->setTitle('Direction Step')
                    ->setRows(5)
            );

            $fields->removeByName('Sort');
        });

        return parent::getCMSFields();
    }

    /**
     * @param null $member

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

     * @param array $context
     * @return bool
     */
    public function canCreate($member = null, $context = [])
    {
        return true;
    }

    /**
     * @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 true;
    }

    /**
     * @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 canDelete($member = null)
    {
        return true;
    }

    /**
     * @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 canView($member = null)
    {
        return true;
    }
}