BristolSU\ControlDB\Observers\PositionObserverCascadeDelete

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	removeTags()	0	4	2
A	__construct()	0	4	1
A	deleteRoles()	0	4	2
A	delete()	0	4	1

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Observers;

use BristolSU\ControlDB\Contracts\Models\Position;
use BristolSU\ControlDB\Contracts\Repositories\Pivots\Tags\PositionPositionTag;
use BristolSU\ControlDB\Contracts\Repositories\Role;

class PositionObserverCascadeDelete

Missing doc comment for class PositionObserverCascadeDelete

{

    /**

Missing short description in doc comment

     * @var Role
     */
    private $roleRepository;

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

    /**

Missing short description in doc comment

     * @var PositionPositionTag
     */
    private $positionPositionTag;

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

    public function __construct(Role $roleRepository, PositionPositionTag $positionPositionTag)

Missing doc comment for function __construct()

    {
        $this->roleRepository = $roleRepository;
        $this->positionPositionTag = $positionPositionTag;
    }

    public function delete(Position $position)

Missing doc comment for function delete()

    {
        $this->deleteRoles($position);
        $this->removeTags($position);
    }

    private function deleteRoles(Position $position)

Private method name "PositionObserverCascadeDelete::deleteRoles" must be prefixed with an underscore

Missing doc comment for function deleteRoles()

    {
        foreach($this->roleRepository->allThroughPosition($position) as $role) {

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

            $this->roleRepository->delete($role->id());
        }
    }

    private function removeTags(Position $position)

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

Missing doc comment for function removeTags()

    {
        foreach($this->positionPositionTag->getTagsThroughPosition($position) as $tag) {

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

            $this->positionPositionTag->removeTagFromPosition($tag, $position);
        }
    }

}