LightSaml\Context\Profile\AbstractProfileContext

Complexity

Total Complexity

4

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getProfileContext()	0	12	4

<?php

/*
 * This file is part of the LightSAML-Core package.
 *
 * (c) Milos Tomic <tmilos@lightsaml.com>
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace LightSaml\Context\Profile;

use LightSaml\Context\AbstractContext;
use LightSaml\Error\LightSamlContextException;

abstract class AbstractProfileContext extends AbstractContext
{
    /**
     * @return ProfileContext
     */
    public function getProfileContext()
    {
        $result = $this;
        while ($result && false == $result instanceof ProfileContext) {

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

            $result = $result->getParent();
        }

        if ($result) {

$result is of type null, thus it always evaluated to false.

            return $result;
        }

        throw new LightSamlContextException($this, 'Missing ProfileContext');
    }
}