dynamic / silverstripe-recipe-book

src/Model/RecipeIngredient.php

<?php

namespace Dynamic\RecipeBook\Model;

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

/**
 * Class RecipeIngredient
 * @package Dynamic\Nucu\Model
 *
 * @property string $Title
 * @property int $Sort
 * @property int $RecipeID
 * @method RecipePage Recipe()
 */
class RecipeIngredient extends DataObject
{
    /**
     * @var string
     */
    private static $singular_name = 'Recipe Ingredient';

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

    /**
     * @var string
     */
    private static $plural_name = 'Recipe Ingredients';

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

    /**
     * @var string
     */
    private static $description = 'An ingredient used in a Recipe';

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

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

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

        'Title' => 'Varchar(255)',
        '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 = 'RecipeIngredient';

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', TextField::create('Title')->setTitle('Ingredient and measurement'));
            $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;
    }
}