Dynamic\ProductCatalog\docs\CareCleaningDoc

Complexity

Total Complexity

6

Size/Duplication

Total Lines	79
Duplicated Lines	100 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	7	7	1
A	canView()	3	3	1
A	providePermissions()	6	6	1
A	canDelete()	3	3	1
A	canEdit()	3	3	1
A	canCreate()	3	3	1

<?php

namespace Dynamic\ProductCatalog\docs;

use SilverStripe\Forms\FieldList;
use SilverStripe\Security\Permission;
use SilverStripe\Security\PermissionProvider;

class CareCleaningDoc extends ProductDoc implements PermissionProvider

This class seems to be duplicated in your project.

{
    /**
     * @var string
     */
    private static $singular_name = 'Care & Cleaning Doc';

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

    /**
     * @var string
     */
    private static $plural_name = 'Care & Cleaning Docs';

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

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

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

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        $fields->dataFieldByName('Download')->setFolderName('Uploads/Products/CareCleaningDocs');

        return $fields;
    }

    /**
     * @return array
     */
    public function providePermissions()
    {
        return array(
            'Care_EDIT' => 'Edit Care and Cleaning Docs',
            'Care_DELETE' => 'Delete Care and Cleaning Docs',
            'Care_CREATE' => 'Create Care and Cleaning Docs',
        );
    }

    /**
     * @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 canCreate($member = null, $context = [])
    {
        return Permission::check('Care_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, $context = [])

The parameter $context is not used and could be removed.

    {
        return Permission::check('Care_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, $context = [])

The parameter $context is not used and could be removed.

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

The parameter $context is not used and could be removed.

    {
        return true;
    }
}