dynamic / silverstripe-manageable-dataobject

tests/Model/SampleManageableDataObject.php

<?php

namespace Dynamic\ManageableDataObject\Test\Model;

use Dynamic\ManageableDataObject\Extensions\ManageableObjectExtension;
use Dynamic\ManageableDataObject\Interfaces\ManageableDataObjectInterface;
use Dynamic\ViewableDataObject\Extensions\ViewableDataObject;
use SilverStripe\Dev\TestOnly;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\RequiredFields;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Permission;
use SilverStripe\Security\PermissionProvider;

/**
 * Class SampleManageableDataObject
 * @package Dynamic\ManageableDataObject\Test\Model
 *
 * @mixin ViewableDataObject
 * @mixin ManageableObjectExtension
 */
class SampleManageableDataObject extends DataObject implements PermissionProvider, ManageableDataObjectInterface, TestOnly
{

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

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

    /**
     * @var string
     */
    private static $listing_page_class = SampleManageableObjectPage::class;

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

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

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

        ViewableDataObject::class,
        ManageableObjectExtension::class,
    ];

    /**
     * @return array
     */
    public function providePermissions()
    {
        return [
            'MDO_Create',
            'MDO_Edit',
            'MDO_Delete',
            'MDO_View',
        ];
    }

	/**
	 * @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|int
	 */
    public function canCreate($member = null, $context = array())
    {
        return Permission::check('MDO_Create', 'any', $member);
    }

    /**
     * @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|int
     */
    public function canEdit($member = null)
    {
        return Permission::check('MDO_Edit', 'any', $member);
    }

    /**
     * @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|int
     */
    public function canDelete($member = null)
    {
        return Permission::check('MDO_Delete', 'any', $member);
    }

    /**
     * @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|int
     */
    public function canView($member = null)
    {
        return Permission::check('MDO_View', 'any', $member);
    }

    /**
     * @param null $params

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

     *
     * @return FieldList
     */
    public function getFrontEndFields($params = null)
    {
        return parent::getFrontEndFields();
    }

    /**
     * @return FieldList
     */
    public function getFrontEndActions()
    {
        return FieldList::create();
    }

    /**
     * @return RequiredFields
     */
    public function getFrontEndRequiredFields()
    {
        return RequiredFields::create();
    }
}