BristolSU\ControlDB\Observers\NotifyObservers\DataGroupNotifier

Complexity

Total Complexity

6

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Test Coverage

Coverage

78.95%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	5	1
A	getAllWhere()	0	3	1
A	__construct()	0	4	1
A	update()	0	6	1
A	getWhere()	0	3	1
A	getById()	0	3	1

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Observers\NotifyObservers;

use BristolSU\ControlDB\Contracts\Repositories\DataGroup as DataGroupRepository;
use BristolSU\ControlDB\Observers\NotifyObservers\Framework\Notifier;
use BristolSU\ControlDB\Observers\NotifyObservers\Framework\ObserverStore;
use Illuminate\Support\Collection;

class DataGroupNotifier extends Notifier implements DataGroupRepository

Missing doc comment for class DataGroupNotifier

{

    /**

Missing short description in doc comment

     * @var DataGroupRepository
     */
    private $dataGroupRepository;

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

    public function __construct(DataGroupRepository $dataGroupRepository, ObserverStore $observerStore)

Missing doc comment for function __construct()

    {
        parent::__construct($observerStore, DataGroupRepository::class);
        $this->dataGroupRepository = $dataGroupRepository;
    }

    /**
     * Get a data group by ID
     *
     * @param int $id

Missing parameter comment

     * @return \BristolSU\ControlDB\Contracts\Models\DataGroup

Tag @return cannot be grouped with parameter tags in a doc comment

     */
    public function getById(int $id): \BristolSU\ControlDB\Contracts\Models\DataGroup
    {
        return $this->dataGroupRepository->getById($id);
    }

    /**
     * Get a data group where the given attributes match, including additional attributes.
     *
     * @param array $attributes

Missing parameter comment

     * @return \BristolSU\ControlDB\Contracts\Models\DataGroup

Tag @return cannot be grouped with parameter tags in a doc comment

     */
    public function getWhere($attributes = []): \BristolSU\ControlDB\Contracts\Models\DataGroup
    {
        return $this->dataGroupRepository->getWhere($attributes);
    }

    /**
     * Get all data groups where the given attributes match, including additional attributes.
     *
     * @param array $attributes

Missing parameter comment

     * @return Collection|\BristolSU\ControlDB\Contracts\Models\DataGroup[]

Tag @return cannot be grouped with parameter tags in a doc comment

     */
    public function getAllWhere($attributes = []): Collection
    {
        return $this->dataGroupRepository->getAllWhere($attributes);
    }

    /**
     * Create a group with the given attributes
     *
     * @param string|null $name Name of the group

Expected 2 spaces after parameter name; 1 found

     * @param string|null $email Email of the group
     * @return \BristolSU\ControlDB\Contracts\Models\DataGroup

Tag @return cannot be grouped with parameter tags in a doc comment

     */
    public function create(?string $name = null, ?string $email = null): \BristolSU\ControlDB\Contracts\Models\DataGroup
    {
        $dataGroup = $this->dataGroupRepository->create($name, $email);
        $this->notify('create', $dataGroup);
        return $dataGroup;
    }

    /**
     * Update a group with the given attributes
     *
     * @param int $id

Missing parameter comment

Expected 9 spaces after parameter type; 1 found

     * @param string|null $name Name of the group

Expected 2 spaces after parameter name; 1 found

     * @param string|null $email Email of the group
     * @return \BristolSU\ControlDB\Contracts\Models\DataGroup

Tag @return cannot be grouped with parameter tags in a doc comment

     */
    public function update(int $id, ?string $name = null, ?string $email = null): \BristolSU\ControlDB\Contracts\Models\DataGroup
    {
        $oldDataGroup = $this->getById($id);
        $newDataGroup = $this->dataGroupRepository->update($id, $name, $email);
        $this->notify('update', $oldDataGroup, $newDataGroup);
        return $newDataGroup;
    }
}