fucongcong/framework

Method html has 36 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function html($text, $tags = null) {
        $text = trim($text);
        //过滤换行符
        $text = preg_replace('/\r?\n/', '', $text);
        //完全过滤注释

The method text has a boolean flag argument $parse_br, which is a certain sign of a Single Responsibility Principle violation.

Open

    public static function text($text, $parse_br = false, $quote_style = ENT_NOQUOTES) {

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

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

Open

        } else {
            $text = nl2br($text);
        }

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

The parameter $quote_style is not named in camelCase.

Open

    public static function text($text, $parse_br = false, $quote_style = ENT_NOQUOTES) {
        if (is_numeric($text))
            $text = (string)$text;
 
        if (!is_string($text))

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

The parameter $parse_br is not named in camelCase.

Open

    public static function text($text, $parse_br = false, $quote_style = ENT_NOQUOTES) {
        if (is_numeric($text))
            $text = (string)$text;
 
        if (!is_string($text))

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

Inline control structures are not allowed

Open

        if (!is_string($text))

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 127 characters

Open

        while (preg_match('/(<[a-z]+)(window\.|javascript:|js:|about:|file:|document\.|vbs:|cookie)([^><]*)/i', $text, $mat)) {

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

        if (is_numeric($text))

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 174 characters

Open

     * @param int     $quote_style ENT_NOQUOTES:(默认)不过滤单引号和双引号 ENT_QUOTES:过滤单引号和双引号 ENT_COMPAT:过滤双引号,而不过滤单引号

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 158 characters

Open

        while (preg_match('/(<[a-z]+)(lang|on|action|background|codebase|dynsrc|lowsrc|style|class|width|height|align|hspace|valign)[^><]+/i', $text, $mat)) {

• Exclude checks
  • Disable engine
  • Disable check

Opening brace should be on a new line

Open

    public static function html($text, $tags = null) {

• Exclude checks
  • Disable engine
  • Disable check

Opening brace should be on a new line

Open

    public static function text($text, $parse_br = false, $quote_style = ENT_NOQUOTES) {

• Exclude checks
  • Disable engine
  • Disable check

No space found after comma in function call

Open

        $text = str_replace('\\','\\\\',$text);

• Exclude checks
  • Disable engine
  • Disable check

No space found after comma in function call

Open

        $text = str_replace('\\','\\\\',$text);

• Exclude checks
  • Disable engine
  • Disable check

The variable $parse_br is not named in camelCase.

Open

    public static function text($text, $parse_br = false, $quote_style = ENT_NOQUOTES) {
        if (is_numeric($text))
            $text = (string)$text;
 
        if (!is_string($text))

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

The variable $quote_style is not named in camelCase.

Open

    public static function text($text, $parse_br = false, $quote_style = ENT_NOQUOTES) {
        if (is_numeric($text))
            $text = (string)$text;
 
        if (!is_string($text))

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