dynamic / silverstripe-elemental-testimonials

src/Model/Testimonial.php

<?php

namespace Dynamic\Elements\Model;

use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Permission;
use SilverStripe\Security\PermissionProvider;

/**
 * Class Testimonial
 * @package Dynamic\Elements\Model
 *
 * @property string $Title
 * @property string $Content
 * @property string $Name
 * @property string $Position
 * @property string $Affiliation
 */
class Testimonial extends DataObject implements PermissionProvider
{
    /**
     * @var string
     */
    private static $singular_name = 'Testimonial';

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

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

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

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

The private property $table_name 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)',
        'Content' => 'Text',
        'Name' => 'Varchar(255)',
        'Position' => 'Varchar(255)',
        'Affiliation' => 'Varchar(255)',
    ];

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

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

        'TestimonialCategories' => TestimonialCategory::class,
    ];

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

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

        'Title' => 'Title',
        'Content.Summary' => 'Testimonial',
        'Name' => 'Name',
    ];

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

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

        'Title',
        'Content.Summary',
        'Name',
    ];

    /**
     * @return array
     */
    public function providePermissions()
    {
        return ['Testimonial_MANAGE' => 'Manage Testimonials'];
    }

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