dynamic / silverstripe-recipe-book

src/Model/RecipeDirection.php

<?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) {
            $recipe = $fields->dataFieldByName('RecipeID');
            $fields->replaceField('RecipeID', $recipe->performReadonlyTransformation());

            $fields->addFieldToTab(
                'Root.Main',
                HTMLEditorField::create('Title')
                    ->setTitle('Direction Step')
                    ->setRows(5)
            );

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

        return parent::getCMSFields();
    }

    /**
     *
     */
    protected function onBeforeWrite()
    {
        parent::onBeforeWrite();

        if (!$this->Sort) {
            $this->Sort = static::get()->filter('RecipeID', $this->RecipeID)->max('Sort') + 1;
        }
    }

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