Modules\Account\Policies\AccountPolicy

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

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

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

namespace Modules\Account\Policies;

use Foundation\Exceptions\Exception;
use Foundation\Policies\OwnershipPolicy;
use Modules\Account\Contracts\AccountServiceContract;
use Modules\User\Entities\User;

class AccountPolicy extends OwnershipPolicy
{
    protected $service;

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

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

        $maxAccountCount = 20;

        if ($AccountCount > $maxAccountCount) {
            throw new Exception('You Cannot create more than 20 Accounts. 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.

    }
}