OperationManual

Complexity

Total Complexity

6

Size/Duplication

Total Lines	76
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	8	8	1
A	providePermissions()	8	8	1
A	canCreate()	4	4	1
A	canEdit()	4	4	1
A	canDelete()	4	4	1

<?php

namespace Dynamic\ProductCatalog\Docs;

use SilverStripe\Security\Permission;
use SilverStripe\Security\PermissionProvider;

class OperationManual extends ProductDoc implements PermissionProvider

This class seems to be duplicated in your project.

{
    /**
     * @var string
     */
    private static $singular_name = 'Operation Manual';

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

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

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

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

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

        return $fields;
    }

    /**
     * @return array
     */
    public function providePermissions()
    {
        return array(
            'Operation_EDIT' => 'Edit Operation Manuals',
            'Operation_DELETE' => 'Delete Operation Manuals',
            'Operation_CREATE' => 'Create Operation Manuals',
        );
    }

    /**
     * @param null $member
     *
     * @return bool|int
     */
    public function canCreate($member = null, $context = [])
    {
        return Permission::check('Operation_CREATE', 'any', $member);
    }

    /**
     * @param null $member
     *
     * @return bool|int
     */
    public function canEdit($member = null, $context = [])

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

    {
        return Permission::check('Operation_EDIT', 'any', $member);
    }

    /**
     * @param null $member
     *
     * @return bool|int
     */
    public function canDelete($member = null, $context = [])

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

    {
        return Permission::check('Operation_DELETE', 'any', $member);
    }

    /**
     * @param null $member
     *
     * @return bool
     */
    public function canView($member = null, $context = [])

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

    {
        return true;
    }
}