bristol-su / control

src/Repositories/Group.php

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Repositories;

use BristolSU\ControlDB\Contracts\Models\Group as GroupModel;
use BristolSU\ControlDB\Contracts\Repositories\Group as GroupContract;
use Illuminate\Support\Collection;

/**
 * Handles groups
 */

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 Group implements GroupContract
{

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

    /**
     * Get all groups
     *
     * @return Collection|GroupModel[]
     */
    public function all(): Collection
    {
        return \BristolSU\ControlDB\Models\Group::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 \BristolSU\ControlDB\Models\Group::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 group by ID
     *
     * @param int $id

Missing parameter comment

     */

Missing @return tag in function comment

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

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

Opening brace should be on a new line

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

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

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

The expression return BristolSU\ControlDB\Models\Group::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 group model
     *
     * @param int $id

Missing parameter comment

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