Silverback\ApiComponentBundle\Entity\RestrictedResourceTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	addSecurityRole()	0	4	1
A	setSecurityRoles()	0	11	3
A	getSecurityRoles()	0	3	1

<?php

declare(strict_types=1);

namespace Silverback\ApiComponentBundle\Entity;

use Doctrine\ORM\Mapping as ORM;

trait RestrictedResourceTrait
{
    /**
     * @ORM\Column(type="json_array", nullable=true)
     * @var array|null
     */
    protected $securityRoles;

    public function getSecurityRoles(): ?array
    {
        return $this->securityRoles;
    }

    /**
     * @param string $securityRole
     * @return static
     */
    public function addSecurityRole(string $securityRole)
    {
        $this->securityRoles[] = $securityRole;
        return $this;
    }

    /**
     * @param array|null $securityRoles
     * @return static
     */
    public function setSecurityRoles(?array $securityRoles)
    {
        if (!$securityRoles) {

The expression $securityRoles of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $this->securityRoles = $securityRoles;
            return $this;
        }
        $this->securityRoles = [];
        foreach ($securityRoles as $securityRole) {
            $this->addSecurityRole($securityRole);
        }
        return $this;
    }
}