PermissionsTableTrait::validationPermission() - Code Metrics - slince/cakephp-permission - Measure and Improve Code Quality continuously with Scrutinizer

PermissionsTableTrait::validationPermission() A
last analyzed 2017-05-17 08:33 UTC

↳ Parent: PermissionsTableTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
c	0
b	0
f	0
rs	9.6666
cc	1
eloc	6
nc	1
nop	1

<?php
/**
 * CakePHP permission handling library
 * @author Tao <[email protected]>
 */
namespace Slince\CakePermission\Model\Table;

use Cake\Cache\Cache;
use Cake\Core\Configure;
use Cake\Validation\Validator;
use Slince\CakePermission\Constants;
use Slince\CakePermission\TableFactory;

trait PermissionsTableTrait
{
    public function buildPermissionRelationship()
    {
        $this->belongsToMany('Roles', [
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            'className' => TableFactory::getModelClass('Roles'),
            'foreignKey' => 'permission_id',
            'targetForeignKey' => 'role_id',
            'joinTable' => Configure::read('Permission.tableNameMap.roles_permissions') ?: 'roles_permissions',
            'saveStrategy' => 'append'
        ]);
        $this->addBehavior('Timestamp');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * @param Validator $validator
     * @return Validator
     */
    public function validationPermission(Validator $validator)
    {
        $validator->add('name', 'unique', [
            'rule' => 'validateUnique',
            'message' => 'The permission already exists',
            'provider' => 'table'
        ]);
        return $validator;
    }

    /**
     * Refreshes the cache
     * @param int $roleId
     */
    public static function refreshCache($roleId)
    {
        Cache::delete(sprintf(Constants::CACHE_PERMISSIONS, $roleId));
    }
}

1			<?php
2			/**
3			* CakePHP permission handling library
4			* @author Tao <[email protected]>
5			*/
6			namespace Slince\CakePermission\Model\Table;
7
8			use Cake\Cache\Cache;
9			use Cake\Core\Configure;
10			use Cake\Validation\Validator;
11			use Slince\CakePermission\Constants;
12			use Slince\CakePermission\TableFactory;
13
14			trait PermissionsTableTrait
15			{
16			public function buildPermissionRelationship()
17			{
18			$this->belongsToMany('Roles', [
			0 ignored issues – show Bug introduced 2017-05-14 09:05 UTC by Report Bug Copy Issue Report It seems like `belongsToMany()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
19			'className' => TableFactory::getModelClass('Roles'),
20			'foreignKey' => 'permission_id',
21			'targetForeignKey' => 'role_id',
22			'joinTable' => Configure::read('Permission.tableNameMap.roles_permissions') ?: 'roles_permissions',
23			'saveStrategy' => 'append'
24			]);
25			$this->addBehavior('Timestamp');
			0 ignored issues – show Bug introduced 2017-05-14 09:05 UTC by Report Bug Copy Issue Report It seems like `addBehavior()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
26			}
27
28			/**
29			* @param Validator $validator
30			* @return Validator
31			*/
32			public function validationPermission(Validator $validator)
33			{
34			$validator->add('name', 'unique', [
35			'rule' => 'validateUnique',
36			'message' => 'The permission already exists',
37			'provider' => 'table'
38			]);
39			return $validator;
40			}
41
42			/**
43			* Refreshes the cache
44			* @param int $roleId
45			*/
46			public static function refreshCache($roleId)
47			{
48			Cache::delete(sprintf(Constants::CACHE_PERMISSIONS, $roleId));
49			}
50			}

slince / cakephp-permission

PermissionsTableTrait::validationPermission() A last analyzed 2017-05-17 08:33 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

PermissionsTableTrait::validationPermission() A
last analyzed 2017-05-17 08:33 UTC