Application\Acl\Assertion\StatusIsNew

Complexity

Total Complexity

3

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Test Coverage

Coverage

71.43%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	assert()	0	10	2
A	getName()	0	3	1

<?php

declare(strict_types=1);

namespace Application\Acl\Assertion;

use Application\DBAL\Types\ExpenseClaimStatusType;
use Application\Model\ExpenseClaim;
use Ecodev\Felix\Acl\Assertion\NamedAssertion;
use Laminas\Permissions\Acl\Acl;
use Laminas\Permissions\Acl\Resource\ResourceInterface;
use Laminas\Permissions\Acl\Role\RoleInterface;

class StatusIsNew implements NamedAssertion
{
    public function getName(): string
    {
        return "la demande n'est pas encore traitée";
    }

    /**
     * Assert that the expense claim is new (not processed yet).
     *
     * @param \Application\Acl\Acl $acl
     * @param RoleInterface $role
     * @param ResourceInterface $resource
     * @param string $privilege
     *
     * @return bool
     */
    public function assert(Acl $acl, ?RoleInterface $role = null, ?ResourceInterface $resource = null, $privilege = null)
    {
        /** @var ExpenseClaim $expenseClaim */
        $expenseClaim = $resource->getInstance();

The method getInstance() does not exist on null.

The method getInstance() does not exist on Laminas\Permissions\Acl\Resource\ResourceInterface.

        if ($expenseClaim->getStatus() === ExpenseClaimStatusType::NEW) {
            return true;
        }

        return $acl->reject('the expense claim status is not new but instead: ' . $expenseClaim->getStatus());

The method reject() does not exist on Laminas\Permissions\Acl\Acl. It seems like you code against a sub-type of Laminas\Permissions\Acl\Acl such as Ecodev\Felix\Acl\Acl.

    }
}