BristolSU\ControlDB\Observers\RoleObserverCascadeDelete

Complexity

Total Complexity

6

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Test Coverage

Coverage

86.67%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	delete()	0	4	1
A	removeUsers()	0	4	2
A	__construct()	0	4	1
A	removeTags()	0	4	2

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Observers;

use BristolSU\ControlDB\Contracts\Models\Role;
use BristolSU\ControlDB\Contracts\Repositories\Pivots\Tags\RoleRoleTag;
use BristolSU\ControlDB\Contracts\Repositories\Pivots\UserRole;

class RoleObserverCascadeDelete

Missing doc comment for class RoleObserverCascadeDelete

{

    /**

Missing short description in doc comment

     * @var UserRole
     */
    private $userRole;

Private member variable "userRole" must be prefixed with an underscore

    /**

Missing short description in doc comment

     * @var RoleRoleTag
     */
    private $roleRoleTag;

Private member variable "roleRoleTag" must be prefixed with an underscore

    public function __construct(UserRole $userRole, RoleRoleTag $roleRoleTag)

Missing doc comment for function __construct()

    {
        $this->userRole = $userRole;
        $this->roleRoleTag = $roleRoleTag;
    }

    public function delete(Role $role)

Missing doc comment for function delete()

    {
        $this->removeUsers($role);
        $this->removeTags($role);
    }

    private function removeUsers(Role $role)

Private method name "RoleObserverCascadeDelete::removeUsers" must be prefixed with an underscore

Missing doc comment for function removeUsers()

    {
        foreach($this->userRole->getUsersThroughRole($role) as $user) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

            $this->userRole->removeUserFromRole($user, $role);
        }
    }

    private function removeTags(Role $role)

Private method name "RoleObserverCascadeDelete::removeTags" must be prefixed with an underscore

Missing doc comment for function removeTags()

    {
        foreach($this->roleRoleTag->getTagsThroughRole($role) as $tag) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

            $this->roleRoleTag->removeTagFromRole($tag, $role);
        }
    }

}