Ecodev\Felix\Acl\Assertion\IsOwner

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

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

<?php

declare(strict_types=1);

namespace Ecodev\Felix\Acl\Assertion;

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

class IsOwner implements AssertionInterface
{
    /**
     * Assert that the object belongs to the current user
     *
     * @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)
    {
        /** @var HasOwner $object */
        $object = $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() === $object->getOwner()) {
            return true;
        }

        return $acl->reject('the object does not belong to the user');

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?

    }
}