phpsecinfo/PhpSecInfo/PhpSecInfo.php
Method renderOutput has 244 lines of code (exceeds 25 allowed). Consider refactoring. Open
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
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
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 upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method _outputRenderTable has 96 lines of code (exceeds 25 allowed). Consider refactoring. Open
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
Open
class PhpSecInfo
{
/**
* An array of tests to run
*- Exclude checks
Function renderOutput has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method runTests has 33 lines of code (exceeds 25 allowed). Consider refactoring. Open
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
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function runTests has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring. Open
Open
public function runTests()
{
// initialize a bunch of arrays
$this->test_results = [];
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function _outputRenderStatsTable has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring. Open
Open
public function _outputRenderStatsTable()
{
// Add by
// https://github.com/bigdeej/PhpSecInfo/tree/master/PhpSecInfo/Test/Core
$score = 100;- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Avoid too many return statements within this method. Open
Open
return 'value-notrun';Avoid too many return statements within this method. Open
Open
return 'value-error';Avoid too many return statements within this method. Open
Open
return 'Invalid Result Code';Avoid too many return statements within this method. Open
Open
return 'Error';The method _outputRenderTable() has an NPath complexity of 1068. The configured NPath complexity threshold is 200. Open
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 upRead up
- Exclude checks
NPathComplexity
Since: 0.1
The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.
Example
class Foo {
function bar() {
// lots of complicated code
}
}Source https://phpmd.org/rules/codesize.html#npathcomplexity
The method renderOutput() has 315 lines of code. Current threshold is set to 100. Avoid really long methods. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Exclude checks
The method _outputRenderTable() has 124 lines of code. Current threshold is set to 100. Avoid really long methods. Open
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
The method _outputRenderTable() has a Cyclomatic Complexity of 16. The configured cyclomatic complexity threshold is 10. Open
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 upRead up
- Exclude checks
CyclomaticComplexity
Since: 0.1
Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.
Example
// 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;
}
}
}
}Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity
Avoid using undefined variables such as '$stats' which will lead to PHP notices. Open
Open
$this->_outputRenderTable('Test Results Summary', $stats);- Read upRead up
- Exclude checks
UndefinedVariable
Since: 2.8.0
Detects when a variable is used that has not been defined before.
Example
class Foo
{
private function bar()
{
// $message is undefined
echo $message;
}
}Source https://phpmd.org/rules/cleancode.html#undefinedvariable
Avoid using undefined variables such as '$classNames' which will lead to PHP notices. Open
Open
$this->tests_to_run = $classNames;- Read upRead up
- Exclude checks
UndefinedVariable
Since: 2.8.0
Detects when a variable is used that has not been defined before.
Example
class Foo
{
private function bar()
{
// $message is undefined
echo $message;
}
}Source https://phpmd.org/rules/cleancode.html#undefinedvariable
Avoid using undefined variables such as '$classNames' which will lead to PHP notices. Open
Open
$classNames[] = 'PhpSecInfo_Test_' . $test_dir . '_' . basename($entry, '.php');- Read upRead up
- Exclude checks
UndefinedVariable
Since: 2.8.0
Detects when a variable is used that has not been defined before.
Example
class Foo
{
private function bar()
{
// $message is undefined
echo $message;
}
}Source https://phpmd.org/rules/cleancode.html#undefinedvariable
Avoid using undefined variables such as '$stats' which will lead to PHP notices. Open
Open
$stats[$this->_outputGetResultTypeFromCode($code)] = [- Read upRead up
- Exclude checks
UndefinedVariable
Since: 2.8.0
Detects when a variable is used that has not been defined before.
Example
class Foo
{
private function bar()
{
// $message is undefined
echo $message;
}
}Source https://phpmd.org/rules/cleancode.html#undefinedvariable
Missing class import via use statement (line '869', column '16'). Open
Open
$psi = new PhpSecInfo();- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}Source http://phpmd.org/rules/cleancode.html#MissingImport
The method runTests uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them. Open
Open
} else {
$rs = [
'result' => $test->getResult(),
'message' => $test->getMessage(),
'value_current' => null,- Read upRead up
- Exclude checks
ElseExpression
Since: 1.4.0
An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.
Example
class Foo
{
public function bar($flag)
{
if ($flag) {
// one branch
} else {
// another branch
}
}
}Source https://phpmd.org/rules/cleancode.html#elseexpression
Avoid unused local variables such as '$score'. Open
Open
$score = 100;- Read upRead up
- Exclude checks
UnusedLocalVariable
Since: 0.2
Detects when a local variable is declared and/or assigned, but not used.
Example
class Foo {
public function doSomething()
{
$i = 5; // Unused
}
}Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable
Each class must be in a namespace of at least one level (a top-level vendor name) Open
Open
class PhpSecInfo- Exclude checks
The property $tests_to_run is not named in camelCase. Open
Open
class PhpSecInfo
{
/**
* An array of tests to run
*- Read upRead up
- Exclude checks
CamelCasePropertyName
Since: 0.2
It is considered best practice to use the camelCase notation to name attributes.
Example
class ClassName {
protected $property_name;
}Source
The property $num_tests_run is not named in camelCase. Open
Open
class PhpSecInfo
{
/**
* An array of tests to run
*- Read upRead up
- Exclude checks
CamelCasePropertyName
Since: 0.2
It is considered best practice to use the camelCase notation to name attributes.
Example
class ClassName {
protected $property_name;
}Source
Avoid variables with short names like $rs. Configured minimum length is 3. Open
Open
$rs = [- Read upRead up
- Exclude checks
ShortVariable
Since: 0.2
Detects when a field, local, or parameter has a very short name.
Example
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;
}
}
}Source https://phpmd.org/rules/naming.html#shortvariable
The parameter $page_title is not named in camelCase. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Read upRead up
- Exclude checks
CamelCaseParameterName
Since: 0.2
It is considered best practice to use the camelCase notation to name parameters.
Example
class ClassName {
public function doSomething($user_name) {
}
}Source
The parameter $group_name is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseParameterName
Since: 0.2
It is considered best practice to use the camelCase notation to name parameters.
Example
class ClassName {
public function doSomething($user_name) {
}
}Source
The property $test_results is not named in camelCase. Open
Open
class PhpSecInfo
{
/**
* An array of tests to run
*- Read upRead up
- Exclude checks
CamelCasePropertyName
Since: 0.2
It is considered best practice to use the camelCase notation to name attributes.
Example
class ClassName {
protected $property_name;
}Source
The property $tests_not_run is not named in camelCase. Open
Open
class PhpSecInfo
{
/**
* An array of tests to run
*- Read upRead up
- Exclude checks
CamelCasePropertyName
Since: 0.2
It is considered best practice to use the camelCase notation to name attributes.
Example
class ClassName {
protected $property_name;
}Source
The property $result_counts is not named in camelCase. Open
Open
class PhpSecInfo
{
/**
* An array of tests to run
*- Read upRead up
- Exclude checks
CamelCasePropertyName
Since: 0.2
It is considered best practice to use the camelCase notation to name attributes.
Example
class ClassName {
protected $property_name;
}Source
The parameter $group_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseParameterName
Since: 0.2
It is considered best practice to use the camelCase notation to name parameters.
Example
class ClassName {
public function doSomething($user_name) {
}
}Source
Method name "_outputRenderTable" should not be prefixed with an underscore to indicate visibility Open
Open
public function _outputRenderTable($group_name, $group_results)- Exclude checks
Method name "_outputRenderNotRunTable" should not be prefixed with an underscore to indicate visibility Open
Open
public function _outputRenderNotRunTable()- Exclude checks
Method name "_outputGetCssClassFromResult" should not be prefixed with an underscore to indicate visibility Open
Open
public function _outputGetCssClassFromResult($code)- Exclude checks
Method name "_outputRenderStatsTable" should not be prefixed with an underscore to indicate visibility Open
Open
public function _outputRenderStatsTable()- Exclude checks
Method name "_outputGetResultTypeFromCode" should not be prefixed with an underscore to indicate visibility Open
Open
public function _outputGetResultTypeFromCode($code)- Exclude checks
Line exceeds 120 characters; contains 128 characters Open
Open
<div class="result"><?php echo $this->_outputGetResultTypeFromCode($test_results['result']) ?></div>- Exclude checks
Line exceeds 120 characters; contains 123 characters Open
Open
<td><?php echo wordwrap($test_results['value_current'], 55, '<br />', true) ?></td>- Exclude checks
Line indented incorrectly; expected 20 spaces, found 24 Open
Open
case 'fr':- Exclude checks
Line indented incorrectly; expected 20 spaces, found 24 Open
Open
case 'fr':- Exclude checks
Line indented incorrectly; expected 44 spaces, found 48 Open
Open
case 'fr':- Exclude checks
Line indented incorrectly; expected 64 spaces, found 24 Open
Open
default:- Exclude checks
Line indented incorrectly; expected at least 68 spaces, found 28 Open
Open
echo 'Participate from Github';- Exclude checks
Line indented incorrectly; expected 16 spaces, found 20 Open
Open
case 'fr':- Exclude checks
Line indented incorrectly; expected 60 spaces, found 20 Open
Open
switch (PHPSECINFO_LANG_DEFAULT) {- Exclude checks
Line indented incorrectly; expected 4 spaces, found 24 Open
Open
case 'fr':- Exclude checks
Line indented incorrectly; expected 4 spaces, found 24 Open
Open
default:- Exclude checks
Line indented incorrectly; expected 16 spaces, found 20 Open
Open
switch (PHPSECINFO_LANG_DEFAULT) {- Exclude checks
Line indented incorrectly; expected 40 spaces, found 44 Open
Open
switch (PHPSECINFO_LANG_DEFAULT) {- Exclude checks
Line indented incorrectly; expected 16 spaces, found 20 Open
Open
default:- Exclude checks
Line indented incorrectly; expected 20 spaces, found 24 Open
Open
default:- Exclude checks
Line indented incorrectly; expected 16 spaces, found 20 Open
Open
} ?></th>- Exclude checks
Line indented incorrectly; expected 44 spaces, found 48 Open
Open
default:- Exclude checks
Line indented incorrectly; expected 40 spaces, found 44 Open
Open
} ?></td>- Exclude checks
Line indented incorrectly; expected at least 68 spaces, found 28 Open
Open
echo 'Participer depuis Github';- Exclude checks
Line indented incorrectly; expected 12 spaces, found 16 Open
Open
} ?> - <?php- Exclude checks
Line indented incorrectly; expected 4 spaces, found 20 Open
Open
default:- Exclude checks
Line indented incorrectly; expected 16 spaces, found 20 Open
Open
} ?></th>- Exclude checks
Line indented incorrectly; expected 44 spaces, found 48 Open
Open
default:- Exclude checks
Line indented incorrectly; expected 40 spaces, found 44 Open
Open
switch (PHPSECINFO_LANG_DEFAULT) {- Exclude checks
Line indented incorrectly; expected 64 spaces, found 24 Open
Open
case 'fr':- Exclude checks
Line indented incorrectly; expected 20 spaces, found 24 Open
Open
default:- Exclude checks
Line indented incorrectly; expected 0 spaces, found 20 Open
Open
switch (PHPSECINFO_LANG_DEFAULT) {- Exclude checks
Line indented incorrectly; expected 40 spaces, found 44 Open
Open
} ?></td>- Exclude checks
Line indented incorrectly; expected 0 spaces, found 16 Open
Open
switch (PHPSECINFO_LANG_DEFAULT) {- Exclude checks
Line indented incorrectly; expected 0 spaces, found 20 Open
Open
} ?></a>- Exclude checks
Line indented incorrectly; expected 16 spaces, found 20 Open
Open
switch (PHPSECINFO_LANG_DEFAULT) {- Exclude checks
Line indented incorrectly; expected 12 spaces, found 16 Open
Open
switch (PHPSECINFO_LANG_DEFAULT) {- Exclude checks
Line indented incorrectly; expected 4 spaces, found 20 Open
Open
case 'fr':- Exclude checks
Line indented incorrectly; expected 0 spaces, found 16 Open
Open
} ?> <?php echo PHPSECINFO_BUILD ?> - <a href="https://github.com/ZerooCool/phpsecinfo/"- Exclude checks
Line indented incorrectly; expected 60 spaces, found 20 Open
Open
} ?></a> - <a href="PhpSecInfo/phpinfo.php"><?php- Exclude checks
Line indented incorrectly; expected 44 spaces, found 48 Open
Open
case 'fr':- Exclude checks
The variable $this_dir is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_results is not named in camelCase. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_root is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_dir is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_dirs is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_name is not named in camelCase. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_root is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_results is not named in camelCase. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_root is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_dirs is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_dir is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_name is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_name is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_dir is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $this_dir is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $this_dir is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $page_title is not named in camelCase. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_name is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_root is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_name is not named in camelCase. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_name is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $page_title is not named in camelCase. Open
Open
public function renderOutput($page_title = 'PHP Security Information')
{
/**
* We need to use PhpSecInfo_Test::getBooleanIniValue() below
*- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $test_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $this_dir is not named in camelCase. Open
Open
public function loadTests()
{
$test_root = dir(__DIR__ . DIRECTORY_SEPARATOR . 'Test');
// echo "<pre>"; echo print_r($test_root, true); echo "</pre>";- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $group_results is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The method _outputGetResultTypeFromCode is not named in camelCase. Open
Open
public function _outputGetResultTypeFromCode($code)
{
switch ($code) {
case PHPSECINFO_TEST_RESULT_OK:
return 'Pass';- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
The method _outputRenderStatsTable is not named in camelCase. Open
Open
public function _outputRenderStatsTable()
{
// Add by
// https://github.com/bigdeej/PhpSecInfo/tree/master/PhpSecInfo/Test/Core
$score = 100;- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
The method _outputRenderTable is not named in camelCase. Open
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 upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
The method _outputGetCssClassFromResult is not named in camelCase. Open
Open
public function _outputGetCssClassFromResult($code)
{
switch ($code) {
case PHPSECINFO_TEST_RESULT_OK:
return 'value-ok';- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
The method _outputRenderNotRunTable is not named in camelCase. Open
Open
public function _outputRenderNotRunTable()
{
$this->_outputRenderTable('Tests Not Run', $this->tests_not_run);
}- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}