Modules\Schedule\Policies\SchedulePolicy

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	create()	0	8	2

<?php
/**
 * Created by PhpStorm.
 * User: arthur
 * Date: 12.10.18
 * Time: 01:25.
 */

namespace Modules\Schedule\Policies;

use Foundation\Exceptions\Exception;
use Foundation\Policies\OwnershipPolicy;
use Modules\Schedule\Contracts\ScheduleServiceContract;
use Modules\User\Entities\User;

class SchedulePolicy extends OwnershipPolicy
{
    protected $service;

    /**
     * SchedulePolicy constructor.
     *
     * @param $service
     */
    public function __construct(ScheduleServiceContract $service)
    {
        $this->service = $service;
    }

    public function create(User $user): bool
    {
        $ScheduleCount = $this->service->getByUserId($user->id)->count();

        $maxScheduleCount = 20;

        if ($ScheduleCount > $maxScheduleCount) {
            throw new Exception('You Cannot create more than 20 Schedules. Delete one first', 401);
        }

The function implicitly returns null when the if condition on line 36 is false. This is incompatible with the type-hinted return boolean. Consider adding a return statement or allowing null as return value.

    }
}