bristol-su / control

src/Repositories/DataRole.php

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Repositories;


use Illuminate\Database\Eloquent\ModelNotFoundException;
use Illuminate\Support\Collection;

class DataRole implements \BristolSU\ControlDB\Contracts\Repositories\DataRole

Missing doc comment for class DataRole

{

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

Missing parameter comment

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

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

     */
    public function getById(int $id): \BristolSU\ControlDB\Contracts\Models\DataRole
    {
        return \BristolSU\ControlDB\Models\DataRole::findOrFail($id);

The expression return BristolSU\ControlDB\Models\DataRole::findOrFail($id) could return the type null which is incompatible with the type-hinted return BristolSU\ControlDB\Contracts\Models\DataRole. Consider adding an additional type-check to rule them out.

    }

    /**
     * Get a data role with the given attributes, including additional attributes.
     *
     * @param array $attributes

Missing parameter comment

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

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

     */
    public function getWhere($attributes = []): \BristolSU\ControlDB\Contracts\Models\DataRole
    {
        $roles = $this->getAllWhere($attributes);

        if($roles->count() > 0) {

Expected "if (...) {\n"; found "if(...) {\n"

            return $roles->first();
        }
        throw (new ModelNotFoundException())->setModel(DataRole::class);
    }

    /**
     * Create a new data role
     *
     * @param string|null $roleName

Missing parameter comment

     * @param string|null $email

Missing parameter comment

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

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

     */
    public function create(?string $roleName = null, ?string $email = null): \BristolSU\ControlDB\Contracts\Models\DataRole
    {
        return \BristolSU\ControlDB\Models\DataRole::create([

The expression return BristolSU\ControlDB\Models\DataRole::create(array('role_name' => $roleName, 'email' => $email)) could return the type null which is incompatible with the type-hinted return BristolSU\ControlDB\Contracts\Models\DataRole. Consider adding an additional type-check to rule them out.

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

            'role_name' => $roleName,
            'email' => $email,
        ]);

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

    }

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

Missing parameter comment

     * @return Collection

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

     */
    public function getAllWhere($attributes = []): Collection
    {
        $baseAttributes = $attributes;
        $additionalAttributes = [];
        foreach (\BristolSU\ControlDB\Models\DataRole::getAdditionalAttributes() as $property) {
            if (array_key_exists($property, $baseAttributes)) {
                $additionalAttributes[$property] = $baseAttributes[$property];
                unset($baseAttributes[$property]);
            }
        }
        return \BristolSU\ControlDB\Models\DataRole::where(function($query) use ($baseAttributes) {

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

            foreach($baseAttributes as $key => $value) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

                $query = $query->where($key, 'LIKE', '%' . $value . '%');
            }
            return $query;
        })->get()->filter(function (\BristolSU\ControlDB\Models\DataRole $dataRole) use ($additionalAttributes) {

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

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

            foreach ($additionalAttributes as $additionalAttribute => $value) {
                if ($dataRole->getAdditionalAttribute($additionalAttribute) !== $value) {
                    return false;
                }
            }
            return true;
        })->values();

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

    }

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

Missing parameter comment

Expected 9 spaces after parameter type; 1 found

     * @param string|null $roleName Custom name for the role
     * @param string|null $email Email of the role

Expected 4 spaces after parameter name; 1 found

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

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

     */
    public function update(int $id, ?string $roleName = null, ?string $email = null): \BristolSU\ControlDB\Contracts\Models\DataRole
    {
        $dataRole = $this->getById($id)->fill([

The method fill() does not exist on BristolSU\ControlDB\Contracts\Models\DataRole. Since it exists in all sub-types, consider adding an abstract or default implementation to BristolSU\ControlDB\Contracts\Models\DataRole.

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

            'role_name' => $roleName, 'email' => $email
        ]);

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

        $dataRole->save();
        return $dataRole;
    }
}