Dynamic\ProductCatalog\Docs\OperationManual

Complexity

Total Complexity

6

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

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

<?php

namespace Dynamic\ProductCatalog\Docs;

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

class OperationManual extends ProductDoc implements PermissionProvider
{
    /**
     * @var string
     */
    private static $singular_name = 'Operation Manual';

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

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

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

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

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

    /**
     * @return FieldList

The type Dynamic\ProductCatalog\Docs\FieldList was not found. Maybe you did not declare it correctly or list all dependencies?

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

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

        return $fields;

The expression return $fields returns the type SilverStripe\Forms\FieldList which is incompatible with the documented return type Dynamic\ProductCatalog\Docs\FieldList.

    }

    /**
     * @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

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('Operation_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('Operation_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('Operation_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;
    }
}