thephplab /
lightsaml
| Conditions | 6 |
| Paths | 5 |
| Total Lines | 18 |
| Lines | 0 |
| Ratio | 0 % |
| Tests | 8 |
| CRAP Score | 6.0493 |
| Changes | 0 | ||
| 1 | <?php |
||
| 26 | 10 | public function resolve(CriteriaSet $criteriaSet, array $arrCredentials = array()) |
|
| 27 | { |
||
| 28 | 10 | if (false == $criteriaSet->has(UsageCriteria::class)) { |
|
|
|
|||
| 29 | return $arrCredentials; |
||
| 30 | } |
||
| 31 | |||
| 32 | 10 | $result = array(); |
|
| 33 | 10 | foreach ($criteriaSet->get(UsageCriteria::class) as $criteria) { |
|
| 34 | /* @var UsageCriteria $criteria */ |
||
| 35 | 10 | foreach ($arrCredentials as $credential) { |
|
| 36 | 10 | if (false == $credential->getUsageType() || $criteria->getUsage() == $credential->getUsageType()) { |
|
| 37 | 10 | $result[] = $credential; |
|
| 38 | } |
||
| 39 | } |
||
| 40 | } |
||
| 41 | |||
| 42 | 10 | return $result; |
|
| 43 | } |
||
| 44 | } |
||
| 45 |
When comparing two booleans, it is generally considered safer to use the strict comparison operator.