Dynamic\FlexSlider\Model\Slide

Complexity

Total Complexity

40

Size/Duplication

Total Lines	341
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	canPublish()	0	17	5
C	canView()	0	44	14
A	getSlideType()	0	3	1
A	canEdit()	0	19	5
A	getCMSFields()	0	31	1
A	providePermissions()	0	26	1
A	canCreate()	0	20	6
A	getSiteConfig()	0	8	2
A	canDelete()	0	22	5

<?php

namespace Dynamic\FlexSlider\Model;

use DNADesign\Elemental\Forms\TextCheckboxGroupField;

The type DNADesign\Elemental\Forms\TextCheckboxGroupField was not found. Maybe you did not declare it correctly or list all dependencies?

use Dynamic\FlexSlider\Interfaces\SlideInterface;
use SilverStripe\Forms\FieldList;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\InheritedPermissions;
use SilverStripe\Security\InheritedPermissionsExtension;
use SilverStripe\Security\Permission;
use SilverStripe\Security\PermissionProvider;
use SilverStripe\Security\Security;
use SilverStripe\SiteConfig\SiteConfig;
use SilverStripe\Versioned\Versioned;

/**
 * Class Slide
 * @package Dynamic\FlexSlider\Model
 *
 * @mixin Versioned
 * @mixin InheritedPermissionsExtension
 */
class Slide extends DataObject implements PermissionProvider, SlideInterface
{
    /**
     * The Create permission
     */
    const SLIDE_CREATE = 'SLIDE_CREATE';

    /**
     * The Publish permission
     */
    const SLIDE_PUBLISH = 'SLIDE_PUBLISH';

    /**
     * The Edit permission
     */
    const SLIDE_EDIT = 'SLIDE_EDIT';

    /**
     * The Delete permission
     */
    const SLIDE_DELETE = 'SLIDE_DELETE';

    /**
     * @var string
     */
    private static $singular_name = 'Slide';

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

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

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

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

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)',
        'ShowTitle' => 'Boolean',
        'Content' => 'HTMLText',
        'SortOrder' => 'Int',
    ];

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

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

        'Page' => \Page::class,
    ];

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

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

        Versioned::class,
        InheritedPermissionsExtension::class,
    ];

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

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

        "Stage", "Live",
    ];


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

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

    /**
     * Adds Publish button to SlideImage record
     *
     * @var bool
     */
    private static $versioned_gridfield_extensions = true;

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

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

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

        'Title',
        'Content',
    ];

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

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

        'SlideType',
        'Title',
    ];

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $this->beforeUpdateCMSFields(function (FieldList $fields) {
            $viewInherit = $fields->dataFieldByName('CanViewType');
            $editInherit = $fields->dataFieldByName('CanEditType');

            $fields->removeByName([
                'Title',
                'ShowTitle',
                'SortOrder',
                'CanViewType',
                'CanEditType',
                'PageID',
            ]);

            $fields->addFieldToTab(
                'Root.Main',
                TextCheckboxGroupField::create(),
                'Content'
            );

            $fields->addFieldsToTab(
                'Root.Permissions',
                [
                    $viewInherit,
                    $editInherit,
                ]
            );
        });

        return parent::getCMSFields();
    }

    /**
     * @return array
     */
    public function providePermissions()
    {
        return [
            static::SLIDE_CREATE => [
                'name' => _t(__CLASS__ . '.PERMISSION_CREATE_DESCRIPTION', 'Create Slides'),
                'help' => _t(__CLASS__ . '.PERMISSION_CREATE_HELP', 'Allow creating slides'),
                'category' => _t(__CLASS__ . '.PERMISSIONS_CATEGORY', 'FlexSlider Permissions'),
                'sort' => 100,
            ],
            static::SLIDE_PUBLISH => [
                'name' => _t(__CLASS__ . '.PERMISSION_PUBLISH_DESCRIPTION', 'Publish Slides'),
                'help' => _t(__CLASS__ . '.PERMISSION_PUBLISH_HELP', 'Allow publishing slides'),
                'category' => _t(__CLASS__ . '.PERMISSIONS_CATEGORY', 'FlexSlider Permissions'),
                'sort' => 100,
            ],
            static::SLIDE_EDIT => [
                'name' => _t(__CLASS__ . '.PERMISSION_EDIT_DESCRIPTION', 'Edit Slides'),
                'help' => _t(__CLASS__ . '.PERMISSION_EDIT_HELP', 'Allow editing slides'),
                'category' => _t(__CLASS__ . '.PERMISSIONS_CATEGORY', 'FlexSlider Permissions'),
                'sort' => 100,
            ],
            static::SLIDE_DELETE => [
                'name' => _t(__CLASS__ . '.PERMISSION_DELETE_DESCRIPTION', 'Delete Slides'),
                'help' => _t(__CLASS__ . '.PERMISSION_DELETE_HELP', 'Allow deleting slides'),
                'category' => _t(__CLASS__ . '.PERMISSIONS_CATEGORY', 'FlexSlider Permissions'),
                'sort' => 100,
            ],
        ];
    }

    /**
     * @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 = [])
    {
        if (!$member) {

$member is of type null, thus it always evaluated to false.

            $member = Security::getCurrentUser();
        }

        // Standard mechanism for accepting permission changes from extensions
        $extended = $this->extendedCan(__FUNCTION__, $member, $context);
        if ($extended !== null) {
            return $extended;
        }

        // Check permission
        if ($member && Permission::checkMember($member, "ADMIN") || Permission::checkMember($member, static::SLIDE_CREATE)) {

Consider adding parentheses for clarity. Current Interpretation: ($member && SilverStripe\Security\Permission::checkMember($member, 'ADMIN')) || SilverStripe\Security\Permission::checkMember($member, static::SLIDE_CREATE), Probably Intended Meaning: $member && (SilverStripe\Security\Permission::checkMember($member, 'ADMIN') || SilverStripe\Security\Permission::checkMember($member, static::SLIDE_CREATE))

            return true;
        }

        // This doesn't necessarily mean we are creating a root page, but that
        // we don't know if there is a parent, so default to this permission
        return SiteConfig::current_site_config()->canCreateTopLevel($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 canPublish($member = null)
    {
        if (!$member) {

$member is of type null, thus it always evaluated to false.

            $member = Security::getCurrentUser();
        }

        // Check extension
        $extended = $this->extendedCan('canPublish', $member);
        if ($extended !== null) {
            return $extended;
        }

        if (Permission::checkMember($member, "ADMIN") || Permission::checkMember($member, static::SLIDE_PUBLISH)) {
            return true;
        }

        return false;
    }

    /**
     * @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)
    {
        if (!$member) {

$member is of type null, thus it always evaluated to false.

            $member = Security::getCurrentUser();
        }

        // Standard mechanism for accepting permission changes from extensions
        $extended = $this->extendedCan('canEdit', $member);
        if ($extended !== null) {
            return $extended;
        }

        // Default permissions
        if (Permission::checkMember($member, "ADMIN") || Permission::checkMember($member, static::SLIDE_EDIT)) {
            return true;
        }


        return false;
    }

    /**
     * @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|null
     */
    public function canView($member = null)
    {
        if (!$member) {

$member is of type null, thus it always evaluated to false.

            $member = Security::getCurrentUser();
        }

        // Standard mechanism for accepting permission changes from extensions
        $extended = $this->extendedCan('canView', $member);
        if ($extended !== null) {
            return $extended;
        }

        // admin override
        if ($member && Permission::checkMember($member, ["ADMIN", "SITETREE_VIEW_ALL"])) {
            return true;
        }

        // Note: getInheritedPermissions() is disused in this instance
        // to allow parent canView extensions to influence subpage canView()

        // check for empty spec
        if (!$this->CanViewType || $this->CanViewType === InheritedPermissions::ANYONE) {

The property CanViewType does not exist on Dynamic\FlexSlider\Model\Slide. Since you implemented __get, consider adding a @property annotation.

            return true;
        }

        // check for inherit
        if ($this->CanViewType === InheritedPermissions::INHERIT) {
            return $this->getSiteConfig()->canViewPages($member);
        }

        // check for any logged-in users
        if ($this->CanViewType === InheritedPermissions::LOGGED_IN_USERS && $member && $member->ID) {
            return true;
        }

        // check for specific groups
        if ($this->CanViewType === InheritedPermissions::ONLY_THESE_USERS
            && $member
            && $member->inGroups($this->ViewerGroups())

The method ViewerGroups() does not exist on Dynamic\FlexSlider\Model\Slide. Since you implemented __call, consider adding a @method annotation.

        ) {
            return true;
        }

        return false;
    }

    /**
     * @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)
    {
        if (!$member) {

$member is of type null, thus it always evaluated to false.

            $member = Security::getCurrentUser();
        }

        // Standard mechanism for accepting permission changes from extensions
        $extended = $this->extendedCan('canDelete', $member);
        if ($extended !== null) {
            return $extended;
        }

        if (!$member) {

$member is of type SilverStripe\Security\Member, thus it always evaluated to true.

            return false;
        }

        // Default permission check
        if (Permission::checkMember($member, ["ADMIN", static::SLIDE_DELETE])) {
            return true;
        }

        return false;
    }

    /**
     * Stub method to get the site config, unless the current class can provide an alternate.
     *
     * @return SiteConfig
     */
    public function getSiteConfig()
    {
        $configs = $this->invokeWithExtensions('alternateSiteConfig');
        foreach (array_filter($configs) as $config) {
            return $config;
        }

        return SiteConfig::current_site_config();
    }

    /**
     * @return string
     */
    public function getSlideType()
    {
        return static::i18n_singular_name();

The method SilverStripe\ORM\DataObject::i18n_singular_name() is not static, but was called statically.

    }
}