BristolSU\ControlDB\Cache\User

Complexity

Total Complexity

9

Size/Duplication

Total Lines	109
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	getByDataProviderId()	0	4	1
A	getById()	0	4	1
A	create()	0	3	1
A	__construct()	0	4	1
A	paginate()	0	3	1
A	update()	0	3	1
A	all()	0	3	1
A	count()	0	4	1
A	delete()	0	3	1

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Cache;

use BristolSU\ControlDB\Contracts\Models\User as UserModel;
use BristolSU\ControlDB\Contracts\Repositories\User as UserRepositoryContract;
use BristolSU\ControlDB\Contracts\Repositories\User as UserRepository;
use Illuminate\Contracts\Cache\Repository;
use Illuminate\Support\Collection;

class User implements UserRepository

Missing doc comment for class User

{
    /**

Missing short description in doc comment

     * @var UserRepositoryContract
     */
    private $userRepository;

Private member variable "userRepository" 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(UserRepositoryContract $userRepository, Repository $cache)

Missing doc comment for function __construct()

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

    /**
     * Get a user by ID
     *
     * @param int $id ID of the user
     * @return UserModel

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

     */
    public function getById(int $id): UserModel
    {
        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->userRepository->getById($id);
        });

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

    }

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

Missing parameter comment

     * @return UserModel

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

     */
    public function getByDataProviderId(int $dataProviderId): UserModel
    {
        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->userRepository->getByDataProviderId($dataProviderId);
        });

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

    }

    /**
     * Get all users
     *
     * @return Collection|UserModel[]
     */
    public function all(): Collection
    {
        return $this->userRepository->all();
    }

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

Missing parameter comment

     * @return UserModel

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

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

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

Missing parameter comment

     */

Missing @return tag in function comment

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

    /**
     * Paginate through all the users
     *
     * @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|UserModel[]
     */
    public function paginate(int $page, int $perPage): Collection
    {
        return $this->userRepository->paginate($page, $perPage);
    }

    /**
     * Get the number of users
     *
     * @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->userRepository->count();
        });

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

    }

    /**
     * Update the user model
     *
     * @param int $id

Missing parameter comment

     * @param int $dataProviderId New data provider ID
     * @return UserModel

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

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