Method renderOutput has 244 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

File PhpSecInfo.php has 502 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php declare(strict_types=1);

/**
 * Main class file
 * @author Ed Finkler <coj@funkatron.com>

Function _outputRenderTable has a Cognitive Complexity of 31 (exceeds 5 allowed). Consider refactoring.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up

Method _outputRenderTable has 96 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

The class PhpSecInfo has an overall complexity of 56 which is very high. The configured complexity threshold is 50.

Open

class PhpSecInfo
{
    /**
     * An array of tests to run
     *

• Exclude checks
  • Disable engine
  • Disable check

Function renderOutput has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Read up
  Read up

Method runTests has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function runTests()
    {
        // initialize a bunch of arrays
        $this->test_results = [];

Function loadTests has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up

Function runTests has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function runTests()
    {
        // initialize a bunch of arrays
        $this->test_results = [];

• Read up
  Read up

Function _outputRenderStatsTable has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function _outputRenderStatsTable()
    {
        // Add by
        // https://github.com/bigdeej/PhpSecInfo/tree/master/PhpSecInfo/Test/Core
        $score = 100;

• Read up
  Read up

Avoid too many return statements within this method.

Open

                return 'Invalid Result Code';

Avoid too many return statements within this method.

Open

                return 'value-notrun';

Avoid too many return statements within this method.

Open

                return 'value-error';

Avoid too many return statements within this method.

Open

                return 'Error';

The method _outputRenderTable() has 124 lines of code. Current threshold is set to 100. Avoid really long methods.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Exclude checks
  • Disable engine
  • Disable check

The method _outputRenderTable() has an NPath complexity of 1068. The configured NPath complexity threshold is 200.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method renderOutput() has 315 lines of code. Current threshold is set to 100. Avoid really long methods.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Exclude checks
  • Disable engine
  • Disable check

The method _outputRenderTable() has a Cyclomatic Complexity of 16. The configured cyclomatic complexity threshold is 10.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

Avoid using undefined variables such as '$classNames' which will lead to PHP notices.

Open

                    $classNames[] = 'PhpSecInfo_Test_' . $test_dir . '_' . basename($entry, '.php');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$stats' which will lead to PHP notices.

Open

                $stats[$this->_outputGetResultTypeFromCode($code)] = [

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$stats' which will lead to PHP notices.

Open

        $this->_outputRenderTable('Test Results Summary', $stats);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$classNames' which will lead to PHP notices.

Open

        $this->tests_to_run = $classNames;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Missing class import via use statement (line '869', column '16').

Open

    $psi = new PhpSecInfo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

The method runTests uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $rs = [
                    'result'            => $test->getResult(),
                    'message'           => $test->getMessage(),
                    'value_current'     => null,

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid unused local variables such as '$score'.

Open

        $score = 100;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class PhpSecInfo

• Exclude checks
  • Disable engine
  • Disable check

The property $test_results is not named in camelCase.

Open

class PhpSecInfo
{
    /**
     * An array of tests to run
     *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

The property $tests_to_run is not named in camelCase.

Open

class PhpSecInfo
{
    /**
     * An array of tests to run
     *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

The parameter $group_name is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething($user_name) {
    }
}

Avoid variables with short names like $rs. Configured minimum length is 3.

Open

                $rs = [

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

The parameter $page_title is not named in camelCase.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething($user_name) {
    }
}

The property $tests_not_run is not named in camelCase.

Open

class PhpSecInfo
{
    /**
     * An array of tests to run
     *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

The property $result_counts is not named in camelCase.

Open

class PhpSecInfo
{
    /**
     * An array of tests to run
     *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

The parameter $group_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething($user_name) {
    }
}

The property $num_tests_run is not named in camelCase.

Open

class PhpSecInfo
{
    /**
     * An array of tests to run
     *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

Method name "_outputRenderTable" should not be prefixed with an underscore to indicate visibility

Open

    public function _outputRenderTable($group_name, $group_results)

• Exclude checks
  • Disable engine
  • Disable check

Method name "_outputRenderStatsTable" should not be prefixed with an underscore to indicate visibility

Open

    public function _outputRenderStatsTable()

• Exclude checks
  • Disable engine
  • Disable check

Method name "_outputGetResultTypeFromCode" should not be prefixed with an underscore to indicate visibility

Open

    public function _outputGetResultTypeFromCode($code)

• Exclude checks
  • Disable engine
  • Disable check

Method name "_outputRenderNotRunTable" should not be prefixed with an underscore to indicate visibility

Open

    public function _outputRenderNotRunTable()

• Exclude checks
  • Disable engine
  • Disable check

Method name "_outputGetCssClassFromResult" should not be prefixed with an underscore to indicate visibility

Open

    public function _outputGetCssClassFromResult($code)

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 123 characters

Open

                                        <td><?php echo wordwrap($test_results['value_current'], 55, '<br />', true) ?></td>

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 128 characters

Open

                            <div class="result"><?php echo $this->_outputGetResultTypeFromCode($test_results['result']) ?></div>

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 24

Open

                        case 'fr':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 20

Open

                    } ?></a>

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 16

Open

                switch (PHPSECINFO_LANG_DEFAULT) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 68 spaces, found 28

Open

                            echo 'Participate from Github';

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 24

Open

                        case 'fr':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 44

Open

                                            } ?></td>

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 20

Open

                    switch (PHPSECINFO_LANG_DEFAULT) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 60 spaces, found 20

Open

                    switch (PHPSECINFO_LANG_DEFAULT) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 44 spaces, found 48

Open

                                                case 'fr':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 44

Open

                                            switch (PHPSECINFO_LANG_DEFAULT) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 20

Open

                    default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 20

Open

                    switch (PHPSECINFO_LANG_DEFAULT) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 20

Open

                    case 'fr':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 68 spaces, found 28

Open

                            echo 'Participer depuis Github';

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 44 spaces, found 48

Open

                                                case 'fr':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 64 spaces, found 24

Open

                        default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 44

Open

                                            switch (PHPSECINFO_LANG_DEFAULT) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 24

Open

                        default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 16

Open

                } ?> <?php echo PHPSECINFO_BUILD ?> - <a href="https://github.com/ZerooCool/phpsecinfo/"

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 20

Open

                    } ?></th>

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 44 spaces, found 48

Open

                                                default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 12 spaces, found 16

Open

                switch (PHPSECINFO_LANG_DEFAULT) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 20

Open

                    default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 12 spaces, found 16

Open

                } ?> - <?php

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 24

Open

                        case 'fr':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 44

Open

                                            } ?></td>

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 20

Open

                    case 'fr':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 24

Open

                        default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 20

Open

                    } ?></th>

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 24

Open

                        default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 44 spaces, found 48

Open

                                                default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 64 spaces, found 24

Open

                        case 'fr':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 60 spaces, found 20

Open

                    } ?></a> - <a href="PhpSecInfo/phpinfo.php"><?php

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 20

Open

                    switch (PHPSECINFO_LANG_DEFAULT) {

• Exclude checks
  • Disable engine
  • Disable check

The variable $test_root is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $this_dir is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_root is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_dirs is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_dir is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_dirs is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_dir is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_root is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $this_dir is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_name is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_name is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $page_title is not named in camelCase.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_name is not named in camelCase.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $page_title is not named in camelCase.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_name is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_name is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $this_dir is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_name is not named in camelCase.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_root is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_dir is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $this_dir is not named in camelCase.

Open

    public function loadTests()
    {
        $test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');

        // echo "<pre>"; echo print_r($test_root, true); echo "</pre>";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_results is not named in camelCase.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_results is not named in camelCase.

Open

    public function renderOutput($page_title = 'PHP Security Information')
    {
        /**
         * We need to use PhpSecInfo_Test::getBooleanIniValue() below
         *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $group_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $test_results is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The method _outputRenderNotRunTable is not named in camelCase.

Open

    public function _outputRenderNotRunTable()
    {
        $this->_outputRenderTable('Tests Not Run', $this->tests_not_run);
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method _outputGetCssClassFromResult is not named in camelCase.

Open

    public function _outputGetCssClassFromResult($code)
    {
        switch ($code) {
            case PHPSECINFO_TEST_RESULT_OK:
                return 'value-ok';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method _outputRenderStatsTable is not named in camelCase.

Open

    public function _outputRenderStatsTable()
    {
        // Add by
        // https://github.com/bigdeej/PhpSecInfo/tree/master/PhpSecInfo/Test/Core
        $score = 100;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method _outputRenderTable is not named in camelCase.

Open

    public function _outputRenderTable($group_name, $group_results)
    {
        // exit out if $group_results was empty or not an array. This sorta seems a little hacky...
        if (!is_array($group_results) || count($group_results) < 1) {
            return false;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method _outputGetResultTypeFromCode is not named in camelCase.

Open

    public function _outputGetResultTypeFromCode($code)
    {
        switch ($code) {
            case PHPSECINFO_TEST_RESULT_OK:
                return 'Pass';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}