bristol-su / control

src/Cache/Group.php

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Cache;

use BristolSU\ControlDB\Contracts\Models\Group as GroupModel;
use BristolSU\ControlDB\Contracts\Repositories\Group as GroupRepositoryContract;
use BristolSU\ControlDB\Contracts\Repositories\Group as GroupRepository;
use Illuminate\Contracts\Cache\Repository;
use Illuminate\Support\Collection;

class Group implements GroupRepository

Missing doc comment for class Group

{
    /**

Missing short description in doc comment

     * @var GroupRepositoryContract
     */
    private $groupRepository;

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

    /**

Missing short description in doc comment

     * @var Repository
     */
    private $cache;

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

    public function __construct(GroupRepositoryContract $groupRepository, Repository $cache)

Missing doc comment for function __construct()

    {
        $this->groupRepository = $groupRepository;
        $this->cache = $cache;
    }

    /**
     * Get a group by ID
     *
     * @param int $id ID of the group
     * @return GroupModel

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

     */
    public function getById(int $id): GroupModel
    {
        return $this->cache->rememberForever(static::class . '@getById:' . $id, function() use ($id) {

The opening parenthesis of a multi-line function call should be the last content on the line.

Expected 1 space after FUNCTION keyword; 0 found

            return $this->groupRepository->getById($id);
        });

For multi-line function calls, the closing parenthesis should be on a new line.

    }

    /**
     * Get a group by its data provider ID
     *
     * @param int $dataProviderId

Missing parameter comment

     * @return GroupModel

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

     */
    public function getByDataProviderId(int $dataProviderId): GroupModel
    {
        return $this->cache->rememberForever(static::class . '@getByDataProviderId:' . $dataProviderId, function() use ($dataProviderId) {

The opening parenthesis of a multi-line function call should be the last content on the line.

Expected 1 space after FUNCTION keyword; 0 found

            return $this->groupRepository->getByDataProviderId($dataProviderId);
        });

For multi-line function calls, the closing parenthesis should be on a new line.

    }

    /**
     * Get all groups
     *
     * @return Collection|GroupModel[]
     */
    public function all(): Collection
    {
        return $this->groupRepository->all();
    }

    /**
     * Create a new group
     *
     * @param int $dataProviderId

Missing parameter comment

     * @return GroupModel

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

     */
    public function create(int $dataProviderId): GroupModel
    {
        return $this->groupRepository->create($dataProviderId);
    }

    /**
     * Delete a group by ID
     *
     * @param int $id

Missing parameter comment

     */

Missing @return tag in function comment

    public function delete(int $id): void
    {
        $this->groupRepository->delete($id);
    }

    /**
     * Paginate through all the groups
     *
     * @param int $page The page number to return

Expected 4 spaces after parameter name; 1 found

     * @param int $perPage The number of results to return per page
     *
     * @return Collection|GroupModel[]
     */
    public function paginate(int $page, int $perPage): Collection
    {
        return $this->groupRepository->paginate($page, $perPage);
    }

    /**
     * Get the number of groups
     *
     * @return int
     */
    public function count(): int
    {
        return $this->cache->rememberForever(static::class . '@count', function() {

The opening parenthesis of a multi-line function call should be the last content on the line.

Expected 1 space after FUNCTION keyword; 0 found

            return $this->groupRepository->count();
        });

For multi-line function calls, the closing parenthesis should be on a new line.

    }

    /**
     * Update the group model
     *
     * @param int $id 
     * @param int $dataProviderId New data provider ID
     * @return GroupModel

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

     */
    public function update(int $id, int $dataProviderId): GroupModel
    {
        return $this->groupRepository->update($id, $dataProviderId);
    }
}