bkdotcom/PHPDebugConsole

View on GitHub
src/Debug/Utility/HtmlParse.php

Summary

Maintainability
A
0 mins
Test Coverage
A
100%

Reduce the number of returns of this function 6, down to the maximum allowed 3.
Open

    public static function parseAttribValue($name, $val, $options)
Severity: Major
Found in src/Debug/Utility/HtmlParse.php by sonar-php

Having too many return statements in a function increases the function's essential complexity because the flow of execution is broken each time a return statement is encountered. This makes it harder to read and understand the logic of the function.

Noncompliant Code Example

With the default threshold of 3:

function myFunction(){ // Noncompliant as there are 4 return statements
  if (condition1) {
    return true;
  } else {
    if (condition2) {
      return false;
    } else {
      return true;
    }
  }
  return false;
}

Unexpected tabs found.
Open

            $decode = (bool) ($options & Html::PARSE_ATTRIB_DATA);

Unexpected tabs found.
Open

            $decode = (bool) ($options & Html::PARSE_ATTRIB_NUMERIC);

Unexpected tabs found.
Open

            $decode = (bool) ($options & Html::PARSE_ATTRIB_CLASS);

There are no issues that match your filters.

Category
Status