bristol-su / control

src/Repositories/User.php

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Repositories;

use BristolSU\ControlDB\Contracts\Models\User as UserModel;
use BristolSU\ControlDB\Contracts\Repositories\User as UserContract;
use Illuminate\Support\Collection;

/**
 * Handles users
 */

Missing @category tag in class comment

Missing @package tag in class comment

Missing @author tag in class comment

Missing @license tag in class comment

Missing @link tag in class comment

class User implements UserContract
{

    /**
     * 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 \BristolSU\ControlDB\Models\User::findOrFail($id);
    }

    /**
     * Get all users
     *
     * @return Collection|UserModel[]
     */
    public function all(): Collection
    {
        return \BristolSU\ControlDB\Models\User::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 \BristolSU\ControlDB\Models\User::create([

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

            'data_provider_id' => $dataProviderId
        ]);

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

    }

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

Missing parameter comment

     */

Missing @return tag in function comment

    public function delete(int $id): void
    {
        \BristolSU\ControlDB\Models\User::findOrFail($id)->delete();
    }

    /**
     * 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 {

Opening brace should be on a new line

        return \BristolSU\ControlDB\Models\User::where('data_provider_id', $dataProviderId)->firstOrFail();
    }

    /**
     * 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 \BristolSU\ControlDB\Models\User::paginate($perPage, ['*'], 'page', $page)->getCollection();
    }

    /**
     * Get the number of users
     *
     * @return int
     */
    public function count(): int
    {
        return \BristolSU\ControlDB\Models\User::count();

The expression return BristolSU\ControlDB\Models\User::count() could return the type Illuminate\Database\Eloquent\Builder which is incompatible with the type-hinted return integer. Consider adding an additional type-check to rule them out.

    }


    /**
     * 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
    {
        $user = $this->getById($id)->fill(['data_provider_id' => $dataProviderId]);
        $user->save();
        return $user;
    }
}