Ecodev\Felix\Acl\Assertion\IsMyself

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Test Coverage

Coverage

80%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	assert()	0	9	3

<?php

declare(strict_types=1);

namespace Ecodev\Felix\Acl\Assertion;

use Ecodev\Felix\Model\CurrentUser;
use Laminas\Permissions\Acl\Acl;
use Laminas\Permissions\Acl\Assertion\AssertionInterface;
use Laminas\Permissions\Acl\Resource\ResourceInterface;
use Laminas\Permissions\Acl\Role\RoleInterface;

class IsMyself implements AssertionInterface
{
    /**
     * Assert that the user is the current user himself
     *
     * @param \Ecodev\Felix\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)
    {
        $user = $resource->getInstance();

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

The method getInstance() does not exist on null.

        if (CurrentUser::get() && CurrentUser::get() === $user) {
            return true;
        }

        return $acl->reject('it is not himself');

The method reject() does not exist on Laminas\Permissions\Acl\Acl. Are you sure you never get this type here, but always one of the subclasses?

    }
}