WBW\Bundle\BootstrapBundle\Helper\UserHelper

Complexity

Total Complexity

7

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	hasRoles()	0	26	7

<?php

/**
 * This file is part of the bootstrap-bundle package.
 *
 * (c) 2018 WEBEWEB
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace WBW\Bundle\BootstrapBundle\Helper;

use Symfony\Component\Security\Core\User\UserInterface;

/**
 * User helper.
 *
 * @author webeweb <https://github.com/webeweb/>
 * @package WBW\Bundle\BootstrapBundle\Helper
 */
class UserHelper {

    /**
     * Determines if the connected user entity has role $roles.
     *
     * @param object $user The user.
     * @param string|array $roles The role or roles.
     * @param boolean $or OR ? If true, matches a role cause a break and the method returns true.
     * @return boolean Returns true in case of success, false otherwise.
     */
    public function hasRoles($user, $roles, $or = true) {

        // Check the user.
        if (null === $user || false === ($user instanceof UserInterface)) {
            return false;
        }

        // Check the roles.
        if (false === is_array($roles)) {
            $roles = [$roles];
        }

        // Initialize the result.
        $result = 1 <= count($roles);

        // Handle each role.
        foreach ($roles as $role) {
            $result &= in_array($role, $user->getRoles());
            if (true == $result && true === $or) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

                break;
            }
        }

        // Return the result.
        return boolval($result);
    }

}