Slince\CakePermission\Model\Table\PermissionsTableTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	buildPermissionRelationship()	0	11	2
A	validationPermission()	0	9	1
A	refreshCache()	0	4	1

<?php
/**
 * CakePHP permission handling library
 * @author Tao <taosikai@yeah.net>
 */
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', [

It seems like belongsToMany() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            'className' => TableFactory::getModelClass('Roles'),
            'foreignKey' => 'permission_id',
            'targetForeignKey' => 'role_id',
            'joinTable' => Configure::read('Permission.tableNameMap.roles_permissions') ?: 'roles_permissions',
            'saveStrategy' => 'append'
        ]);
        $this->addBehavior('Timestamp');

It seems like addBehavior() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @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));
    }
}