LightSaml\Resolver\Credential\AlgorithmFilterResolver

Complexity

Total Complexity

8

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Test Coverage

Coverage

30%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	resolve()	0	20	8

<?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\Resolver\Credential;

use LightSaml\Credential\CredentialInterface;
use LightSaml\Credential\Criteria\AlgorithmCriteria;
use LightSaml\Criteria\CriteriaSet;

class AlgorithmFilterResolver extends AbstractQueryableResolver
{
    /**
     * @param CriteriaSet           $criteriaSet
     * @param CredentialInterface[] $arrCredentials
     *
     * @return CredentialInterface[]
     */
    public function resolve(CriteriaSet $criteriaSet, array $arrCredentials = array())
    {
        if (false == $criteriaSet->has(AlgorithmCriteria::class)) {

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

            return $arrCredentials;
        }

        $result = array();
        foreach ($criteriaSet->get(AlgorithmCriteria::class) as $criteria) {
            /* @var AlgorithmCriteria $criteria */
            foreach ($arrCredentials as $credential) {
                if (($credential->getPrivateKey() && $credential->getPrivateKey()->getAlgorith() == $criteria->getAlgorithm()) ||

The method RobRichards\XMLSecLibs\XMLSecurityKey::getAlgorith() has been deprecated.

                    ($credential->getPublicKey() && $credential->getPublicKey()->getAlgorith() == $criteria->getAlgorithm())

The method RobRichards\XMLSecLibs\XMLSecurityKey::getAlgorith() has been deprecated.

                ) {
                    $result[] = $credential;
                }
            }
        }

        return $result;
    }
}