Epesi\Core\System\User\Access\AccessSettings::showEditPermissions()

Complexity

Conditions	2
Paths	2

Size

Total Lines	62
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
	protected function showEditPermissions()
	{
		$permissionsData = Permission::all(['id', 'name'])->map(function($permission){
			$permission['name'] = trans(ucwords($permission['name']));
			
			return $permission;
		})->toArray();

		$table = $this->columns()->addColumn()->add([Table::class, 'selectable'])->addStyle('cursor', 'pointer');
		$table->setModel($this->getModel($permissionsData), false);
		
		$table->addColumn('name', Table\Column\Text::class, ['caption' => __('Permissions')]);
		
		$table->addColumn(null, new Table\Column\Template([['i', 'class' => 'indicator arrow circle right icon']]), ['caption' => 'Test'])->setAttr('class', ['right aligned']);

array(array('i', 'class' => 'indicator arrow circle right icon')) of type array<integer,string[]> is incompatible with the type string expected by parameter $template of atk4\ui\Table\Column\Template::__construct().

		
		$this->getApp()->addStyle('
			table.selectable tr:not(.active) .indicator {
				display: none;
			}'
		);
		
		$table->on('click', 'tr', $this->reload(new \atk4\ui\JsExpression('$(this).data("id")')));
		
		if ($permissionId = $this->permissionId()) {
			$permission = Permission::findById($permissionId);

It seems like $permissionId can also be of type string; however, parameter $id of Spatie\Permission\Models\Permission::findById() does only seem to accept integer, maybe add an additional type check?

			
			$this->putModuleVariable('permission', $permissionId);
			
			$table->js(true)->find("tr[data-id=$permission->id]")->addClass('active');

Accessing id on the interface Spatie\Permission\Contracts\Permission suggest that you code against a concrete implementation. How about adding an instanceof check?

			
			$rolesData = $permission->roles()->get()->map(function($role) {
				$role['name'] = trans($role['name']);
				
				return $role;
			})->toArray();

			$column = $this->columns()->addColumn();
			$rolesTable = Table::addTo($column);
			$rolesTable->setModel($this->getModel($rolesData), false);
			
			$rolesTable->addColumn('name', [Table\Column\Text::class], ['caption' => __('Roles allowed to :permission', ['permission' => $permission->name])]);

Accessing name on the interface Spatie\Permission\Contracts\Permission suggest that you code against a concrete implementation. How about adding an instanceof check?

			
			$roleActions = $rolesTable->addColumn(null, [Table\Column\ActionButtons::class]);
			$roleActions->addButton($this->deleteButton(), function($jQuery, $roleId) use ($permission) {

The method addButton() does not exist on atk4\ui\Table\Column. It seems like you code against a sub-type of atk4\ui\Table\Column such as atk4\ui\Table\Column\ActionButtons.

				Role::findById($roleId)->revokePermissionTo($permission);
				
				return $this->reload();
			}, __('Revoke permission to role?'));
				
			$usersData = $permission->users()->get()->toArray();

			$usersTable = Table::addTo($column);
			$usersTable->setModel($this->getModel($usersData), false);
				
			$usersTable->addColumn('name', [Table\Column\Text::class], ['caption' => __('Users allowed to :permission', ['permission' => $permission->name])]);
				
			$userActions = $usersTable->addColumn(null, [Table\Column\ActionButtons::class]);
			$userActions->addButton($this->deleteButton(), function($jQuery, $userId) use ($permission) {
				User::find($userId)->revokePermissionTo($permission);
					
				return $this->reload();
			}, __('Revoke permission to user?'));
		}