Sign upLogin

AOEpeople/StackFormation

View on GitHub
Star
src/AwsInspector/Model/SecurityGroup/SecurityGroup.php

Summary

Maintainability
B
5 hrs
Test Coverage

Showing 2 of 2 total issues

Function hasAccess has a Cognitive Complexity of 33 (exceeds 5 allowed). Consider refactoring.
Open

    public function hasAccess($origin, $port, $protocol='tcp') {
        foreach ($this->getIpPermissions() as $permission) {
            if ($permission['IpProtocol'] != $protocol || $permission['FromPort'] != $port) {
                continue;
            }
Severity: Minor
Found in src/AwsInspector/Model/SecurityGroup/SecurityGroup.php - About 4 hrs to fix

Method hasAccess has 26 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    public function hasAccess($origin, $port, $protocol='tcp') {
        foreach ($this->getIpPermissions() as $permission) {
            if ($permission['IpProtocol'] != $protocol || $permission['FromPort'] != $port) {
                continue;
            }
Severity: Minor
Found in src/AwsInspector/Model/SecurityGroup/SecurityGroup.php - About 1 hr to fix
    Category
    Status