Inspection of "Version 2.10" - gravityview/GravityView - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#1492)

by Zack

created 2021-02-22 21:00 UTC

Indentation +128 added lines, -128 removed lines patch added patch discarded remove patch

@@ -4,157 +4,157 @@
 block discarded – undo
 
 function complexityFive()
 {
-    if ($condition) {
-    }
+	if ($condition) {
+	}
 
-    switch ($condition) {
-        case '1':
-        break;
-        case '2':
-        break;
-        case '3':
-        break;
-    }
+	switch ($condition) {
+		case '1':
+		break;
+		case '2':
+		break;
+		case '3':
+		break;
+	}
 }
 
 function complexityTen()
 {
-    while ($condition === true) {
-        if ($condition) {
-        }
-    }
+	while ($condition === true) {
+		if ($condition) {
+		}
+	}
 
-    switch ($condition) {
-        case '1':
-            if ($condition) {
-            } else if ($cond) {
-            }
-        break;
-        case '2':
-            while ($cond) {
-                echo 'hi';
-            }
-        break;
-        case '3':
-        break;
-        default:
-        break;
-    }
+	switch ($condition) {
+		case '1':
+			if ($condition) {
+			} else if ($cond) {
+			}
+		break;
+		case '2':
+			while ($cond) {
+				echo 'hi';
+			}
+		break;
+		case '3':
+		break;
+		default:
+		break;
+	}
 }
 
 function complexityEleven()
 {
-    while ($condition === true) {
-        if ($condition) {
-        } else if ($cond) {
-        }
-    }
+	while ($condition === true) {
+		if ($condition) {
+		} else if ($cond) {
+		}
+	}
 
-    switch ($condition) {
-        case '1':
-            if ($condition) {
-            } else if ($cond) {
-            }
-        break;
-        case '2':
-            while ($cond) {
-                echo 'hi';
-            }
-        break;
-        case '3':
-        break;
-        default:
-        break;
-    }
+	switch ($condition) {
+		case '1':
+			if ($condition) {
+			} else if ($cond) {
+			}
+		break;
+		case '2':
+			while ($cond) {
+				echo 'hi';
+			}
+		break;
+		case '3':
+		break;
+		default:
+		break;
+	}
 }
 
 
 function complexityTwenty()
 {
-    while ($condition === true) {
-        if ($condition) {
-        } else if ($cond) {
-        }
-    }
+	while ($condition === true) {
+		if ($condition) {
+		} else if ($cond) {
+		}
+	}
 
-    switch ($condition) {
-        case '1':
-            if ($condition) {
-            } else if ($cond) {
-            }
-        break;
-        case '2':
-            while ($cond) {
-                echo 'hi';
-            }
-        break;
-        case '3':
-            switch ($cond) {
-                case '1':
-                break;
-                case '2':
-                break;
-            }
-        break;
-        case '4':
-            do {
-                if ($condition) {
-                    if ($cond) {
-                    } else if ($con) {
-                    }
-                }
-            } while ($cond);
-        break;
-        default:
-            if ($condition) {
-            }
-        break;
-    }
+	switch ($condition) {
+		case '1':
+			if ($condition) {
+			} else if ($cond) {
+			}
+		break;
+		case '2':
+			while ($cond) {
+				echo 'hi';
+			}
+		break;
+		case '3':
+			switch ($cond) {
+				case '1':
+				break;
+				case '2':
+				break;
+			}
+		break;
+		case '4':
+			do {
+				if ($condition) {
+					if ($cond) {
+					} else if ($con) {
+					}
+				}
+			} while ($cond);
+		break;
+		default:
+			if ($condition) {
+			}
+		break;
+	}
 }
 
 
 function complexityTwentyOne()
 {
-    while ($condition === true) {
-        if ($condition) {
-        } else if ($cond) {
-        }
-    }
+	while ($condition === true) {
+		if ($condition) {
+		} else if ($cond) {
+		}
+	}
 
-    switch ($condition) {
-        case '1':
-            if ($condition) {
-            } else if ($cond) {
-            }
-        break;
-        case '2':
-            while ($cond) {
-                echo 'hi';
-            }
-        break;
-        case '3':
-            switch ($cond) {
-                case '1':
-                break;
-                case '2':
-                break;
-            }
-        break;
-        case '4':
-            do {
-                if ($condition) {
-                    if ($cond) {
-                    } else if ($con) {
-                    }
-                }
-            } while ($cond);
-        break;
-        default:
-            if ($condition) {
-            } else if ($cond) {
-            }
-        break;
-    }
+	switch ($condition) {
+		case '1':
+			if ($condition) {
+			} else if ($cond) {
+			}
+		break;
+		case '2':
+			while ($cond) {
+				echo 'hi';
+			}
+		break;
+		case '3':
+			switch ($cond) {
+				case '1':
+				break;
+				case '2':
+				break;
+			}
+		break;
+		case '4':
+			do {
+				if ($condition) {
+					if ($cond) {
+					} else if ($con) {
+					}
+				}
+			} while ($cond);
+		break;
+		default:
+			if ($condition) {
+			} else if ($cond) {
+			}
+		break;
+	}
 }
 
 ?>

Please login to merge, or discard this patch.

php_codesniffer/src/Standards/Generic/Tests/Commenting/FixmeUnitTest.php 1 patch

Indentation +41 added lines, -41 removed lines patch added patch discarded remove patch

@@ -16,47 +16,47 @@
 block discarded – undo
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList($testFile='FixmeUnitTest.inc')
-    {
-        return [
-            3  => 1,
-            4  => 1,
-            7  => 1,
-            10 => 1,
-            13 => 1,
-            16 => 1,
-            18 => 1,
-            21 => 1,
-        ];
-
-    }//end getErrorList()
-
-
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList($testFile='FixmeUnitTest.inc')
-    {
-        return [];
-
-    }//end getWarningList()
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList($testFile='FixmeUnitTest.inc')
+	{
+		return [
+			3  => 1,
+			4  => 1,
+			7  => 1,
+			10 => 1,
+			13 => 1,
+			16 => 1,
+			18 => 1,
+			21 => 1,
+		];
+
+	}//end getErrorList()
+
+
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList($testFile='FixmeUnitTest.inc')
+	{
+		return [];
+
+	}//end getWarningList()
 
 
 }//end class

Please login to merge, or discard this patch.

src/Standards/Generic/Tests/Commenting/DocCommentUnitTest.inc 1 patch

Indentation +8 added lines, -9 removed lines patch added patch discarded remove patch

@@ -213,15 +213,15 @@  discard block
 block discarded – undo
 
 /**doc comment */
 
-        /**
-         * Document behaviour with missing blank lines with indented docblocks.
-         * @param
-         * @param
-         * @tag one
-         *
-         */
+		/**
+		 * Document behaviour with missing blank lines with indented docblocks.
+		 * @param
+		 * @param
+		 * @tag one
+		 *
+		 */
 
-        /** Indented doc comment */
+		/** Indented doc comment */
 
 /**
  * Verify and document sniff behaviour when the "tag value" is indented with a mix of tabs and spaces.
@@ -249,4 +249,3 @@  discard block
 block discarded – undo
  * @link		http://pear.php.net/package/PHP_CodeSniffer
  */
 
-/** No docblock close tag. Must be last test without new line.
\ No newline at end of file

Please login to merge, or discard this patch.

src/Standards/Generic/Tests/Commenting/DocCommentUnitTest.php 1 patch

Indentation +84 added lines, -84 removed lines patch added patch discarded remove patch

@@ -15,97 +15,97 @@
 block discarded – undo
 {
 
 
-    /**
-     * Get a list of CLI values to set before the file is tested.
-     *
-     * @param string                  $testFile The name of the file being tested.
-     * @param \PHP_CodeSniffer\Config $config   The config data for the test run.
-     *
-     * @return void
-     */
-    public function setCliValues($testFile, $config)
-    {
-        $config->tabWidth = 4;
+	/**
+	 * Get a list of CLI values to set before the file is tested.
+	 *
+	 * @param string                  $testFile The name of the file being tested.
+	 * @param \PHP_CodeSniffer\Config $config   The config data for the test run.
+	 *
+	 * @return void
+	 */
+	public function setCliValues($testFile, $config)
+	{
+		$config->tabWidth = 4;
 
-    }//end setCliValues()
+	}//end setCliValues()
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @return array(int => int)
-     */
-    public function getErrorList()
-    {
-        return [
-            14  => 1,
-            16  => 1,
-            18  => 1,
-            23  => 1,
-            26  => 1,
-            30  => 1,
-            32  => 1,
-            38  => 2,
-            40  => 1,
-            41  => 1,
-            51  => 1,
-            54  => 1,
-            58  => 1,
-            60  => 2,
-            67  => 1,
-            69  => 2,
-            80  => 1,
-            81  => 2,
-            88  => 1,
-            91  => 1,
-            95  => 1,
-            156 => 1,
-            158 => 1,
-            170 => 3,
-            171 => 3,
-            179 => 1,
-            183 => 1,
-            184 => 2,
-            185 => 1,
-            186 => 1,
-            187 => 2,
-            193 => 1,
-            196 => 1,
-            199 => 1,
-            203 => 1,
-            206 => 1,
-            211 => 1,
-            214 => 4,
-            218 => 1,
-            220 => 2,
-            222 => 1,
-            224 => 3,
-            243 => 1,
-            244 => 1,
-            246 => 1,
-            248 => 1,
-            249 => 1,
-        ];
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @return array(int => int)
+	 */
+	public function getErrorList()
+	{
+		return [
+			14  => 1,
+			16  => 1,
+			18  => 1,
+			23  => 1,
+			26  => 1,
+			30  => 1,
+			32  => 1,
+			38  => 2,
+			40  => 1,
+			41  => 1,
+			51  => 1,
+			54  => 1,
+			58  => 1,
+			60  => 2,
+			67  => 1,
+			69  => 2,
+			80  => 1,
+			81  => 2,
+			88  => 1,
+			91  => 1,
+			95  => 1,
+			156 => 1,
+			158 => 1,
+			170 => 3,
+			171 => 3,
+			179 => 1,
+			183 => 1,
+			184 => 2,
+			185 => 1,
+			186 => 1,
+			187 => 2,
+			193 => 1,
+			196 => 1,
+			199 => 1,
+			203 => 1,
+			206 => 1,
+			211 => 1,
+			214 => 4,
+			218 => 1,
+			220 => 2,
+			222 => 1,
+			224 => 3,
+			243 => 1,
+			244 => 1,
+			246 => 1,
+			248 => 1,
+			249 => 1,
+		];
 
-    }//end getErrorList()
+	}//end getErrorList()
 
 
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @return array(int => int)
-     */
-    public function getWarningList()
-    {
-        return [];
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array(int => int)
+	 */
+	public function getWarningList()
+	{
+		return [];
 
-    }//end getWarningList()
+	}//end getWarningList()
 
 
 }//end class

Please login to merge, or discard this patch.

php_codesniffer/src/Standards/Generic/Tests/Commenting/TodoUnitTest.php 1 patch

Indentation +41 added lines, -41 removed lines patch added patch discarded remove patch

@@ -15,47 +15,47 @@
 block discarded – undo
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList($testFile='TodoUnitTest.inc')
-    {
-        return [];
-
-    }//end getErrorList()
-
-
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList($testFile='TodoUnitTest.inc')
-    {
-        return [
-            3  => 1,
-            4  => 1,
-            7  => 1,
-            10 => 1,
-            13 => 1,
-            16 => 1,
-            18 => 1,
-            21 => 1,
-        ];
-
-    }//end getWarningList()
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList($testFile='TodoUnitTest.inc')
+	{
+		return [];
+
+	}//end getErrorList()
+
+
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList($testFile='TodoUnitTest.inc')
+	{
+		return [
+			3  => 1,
+			4  => 1,
+			7  => 1,
+			10 => 1,
+			13 => 1,
+			16 => 1,
+			18 => 1,
+			21 => 1,
+		];
+
+	}//end getWarningList()
 
 
 }//end class

Please login to merge, or discard this patch.

Generic/Tests/ControlStructures/InlineControlStructureUnitTest.1.inc 1 patch

Indentation +95 added lines, -95 removed lines patch added patch discarded remove patch

@@ -3,11 +3,11 @@  discard block
 block discarded – undo
 if ($something) echo 'hello';
 
 if ($something) {
-    echo 'hello';
+	echo 'hello';
 } else echo 'hi';
 
 if ($something) {
-    echo 'hello';
+	echo 'hello';
 } else if ($else) echo 'hi';
 
 foreach ($something as $thing) echo 'hello';
@@ -17,61 +17,61 @@  discard block
 block discarded – undo
 while ($something) echo 'hello';
 
 do {
-    $i--;
+	$i--;
 } while ($something);
 
 if(true)
   $someObject->{$name};
 
 if (true) :
-    $foo = true;
+	$foo = true;
 endif;
 
 while (true) :
-    $foo = true;
+	$foo = true;
 endwhile;
 
 for ($i; $i > 0; $i--) :
-    echo 'hello';
+	echo 'hello';
 endfor;
 
 foreach ($array as $element) :
-    echo 'hello';
+	echo 'hello';
 endforeach;
 
 while (!$this->readLine($tokens, $tag));
 while (!$this->readLine($tokens, $tag)); //skip to end of file
 
 foreach ($cookies as $cookie)
-    if ($cookie->match($uri, $matchSessionCookies, $now))
-        $ret[] = $cookie;
+	if ($cookie->match($uri, $matchSessionCookies, $now))
+		$ret[] = $cookie;
 
 foreach ($stringParade as $hit)
-    $hitParade[] = $hit + 0; //cast to integer
+	$hitParade[] = $hit + 0; //cast to integer
 
 if ($foo) :
-    echo 'true';
+	echo 'true';
 elseif ($something) :
-    echo 'foo';
+	echo 'foo';
 else:
-    echo 'false';
+	echo 'false';
 endif;
 
 function test()
 {
-    if ($a)
-        $a.=' '.($b ? 'b' : ($c ? ($d ? 'd' : 'c') : ''));
+	if ($a)
+		$a.=' '.($b ? 'b' : ($c ? ($d ? 'd' : 'c') : ''));
 }
 
 if ($a)
-    foreach ($b as $c) {
-        if ($d) {
-            $e=$f;
-            $g=$h;
-        } elseif ($i==0) {
-            $j=$k;
-        }
-    }
+	foreach ($b as $c) {
+		if ($d) {
+			$e=$f;
+			$g=$h;
+		} elseif ($i==0) {
+			$j=$k;
+		}
+	}
 
 ?>
 <div style="text-align: right;">
@@ -80,136 +80,136 @@  discard block
 block discarded – undo
 
 <?php
 switch ($this->error):
-    case Shop_Customer :: ERROR_INVALID_GENDER: ?>
+	case Shop_Customer :: ERROR_INVALID_GENDER: ?>
         Ung&uuml;ltiges Geschlecht!
     <?php break;
-    case Shop_Customer :: ERROR_EMAIL_IN_USE: ?>
+	case Shop_Customer :: ERROR_EMAIL_IN_USE: ?>
         Die eingetragene E-Mail-Adresse ist bereits registriert.
     <?php break;
 endswitch;
 
 if ($this->allowShopping !== true):
-    if ($this->status != Shop_Cart :: OK):
-        switch ($this->status):
-            case Shop_Cart :: NOT_FOUND:
-            echo 'foo';
-        endswitch;
-    endif;
+	if ($this->status != Shop_Cart :: OK):
+		switch ($this->status):
+			case Shop_Cart :: NOT_FOUND:
+			echo 'foo';
+		endswitch;
+	endif;
 else:
-    echo 'foo';
+	echo 'foo';
 endif;
 
 // ELSE IF split over multiple lines (not inline)
 if ($test) {
 } else
-    if ($test) {
-    } else {
-    }
+	if ($test) {
+	} else {
+	}
 
 switch($response = \Bar::baz('bat', function ($foo) {
-    return 'bar';
+	return 'bar';
 })) {
-    case 1:
-        return 'test';
+	case 1:
+		return 'test';
 
-    case 2:
-        return 'other';
+	case 2:
+		return 'other';
 }
 
 $stuff = [1,2,3];
 foreach($stuff as $num)
-    if ($num %2 ) {
-        echo "even";
-    } else {
-        echo "odd";
-    }
+	if ($num %2 ) {
+		echo "even";
+	} else {
+		echo "odd";
+	}
 
 $i = 0;
 foreach($stuff as $num)
-    do {
-        echo $i;
-        $i++;
-    } while ($i < 5);
+	do {
+		echo $i;
+		$i++;
+	} while ($i < 5);
 
 foreach($stuff as $num)
-    if (true) {
-        echo "true1\n";
-    }
-    if (true) {
-        echo "true2\n";
-    }
+	if (true) {
+		echo "true1\n";
+	}
+	if (true) {
+		echo "true2\n";
+	}
 
 if ($foo) echo 'foo';
 elseif ($bar) echo 'bar';
 else echo 'baz';
 
 switch ($type) {
-    case 1:
-        if ($foo) {
-            return true;
-        } elseif ($baz)
-            return true;
-        else {
-            echo 'else';
-        }
-    break;
+	case 1:
+		if ($foo) {
+			return true;
+		} elseif ($baz)
+			return true;
+		else {
+			echo 'else';
+		}
+	break;
 }
 
 foreach ($sql as $s)
-        if (!$this->execute) echo "<pre>",$s.";\n</pre>";
-        else {
-            $ok = $this->connDest->Execute($s);
-            if (!$ok)
-                if ($this->neverAbort) $ret = false;
-                else return false;
-        }
+		if (!$this->execute) echo "<pre>",$s.";\n</pre>";
+		else {
+			$ok = $this->connDest->Execute($s);
+			if (!$ok)
+				if ($this->neverAbort) $ret = false;
+				else return false;
+		}
 
 if ($bar)
-    if ($foo) echo 'hi'; // lol
+	if ($foo) echo 'hi'; // lol
 
 if ($level == 'district')
-    \DB::update(<<<EOD
+	\DB::update(<<<EOD
 some
 text
 here
 EOD
-    );
+	);
 
 if ($level == 'district')
-    $var = <<<EOD
+	$var = <<<EOD
 some
 text
 here
 EOD;
 
 if ($a && $a === Foo::VARIABLE && ($a === Foo::METHOD || $a === Foo::FUNCTION))
-    echo 'hi';
+	echo 'hi';
 
 $out = array_map(function ($test) { if ($test) return 1; else return 2; }, $input); // comment
 
 for ($x=0;$x<5;$x++):
-    if ($x) continue;
+	if ($x) continue;
 endfor;
 
 for ($x=0;$x<5;$x++):
-    if ($x) continue ?> <?php
+	if ($x) continue ?> <?php
 endfor;
 
 if (true)
-    try {
-    }
-    catch(Exception $e) {
-    }
+	try {
+	}
+	catch(Exception $e) {
+	}
 
 switch ($num) {
-    case 0:
-        if (1 > $num)
-            return bar(
-                baz(
-                    "foobarbaz"
-                )
-            );
-        break;
+	case 0:
+		if (1 > $num)
+			return bar(
+				baz(
+					"foobarbaz"
+				)
+			);
+		break;
 }
 
 do {
@@ -219,8 +219,8 @@  discard block
 block discarded – undo
 while ($i < 10);
 
 if ($this) {
-    if ($that)
-        foo(${$a[$b]});
+	if ($that)
+		foo(${$a[$b]});
 }
 
 while (!$this->readLine($tokens, $tag)); //phpcs:ignore Standard.Category.Sniff
@@ -230,11 +230,11 @@  discard block
 block discarded – undo
 while (!$this->readLine($tokens, $tag)); /* comment */
 
 foreach ($stringParade as $hit)
-    $hitParade[] = $hit + 0; // phpcs:ignore Standard.Category.Sniff
+	$hitParade[] = $hit + 0; // phpcs:ignore Standard.Category.Sniff
 
 if ($bar)
-    if ($foo) echo 'hi'; /* @phpcs:ignore Standard.Category.Sniff */
+	if ($foo) echo 'hi'; /* @phpcs:ignore Standard.Category.Sniff */
 
 if (true) $callable = function () {
-    return true;
+	return true;
 };

Please login to merge, or discard this patch.

Generic/Tests/ControlStructures/InlineControlStructureUnitTest.php 1 patch

Indentation +88 added lines, -88 removed lines patch added patch discarded remove patch

@@ -15,100 +15,100 @@
 block discarded – undo
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList($testFile='InlineControlStructureUnitTest.1.inc')
-    {
-        switch ($testFile) {
-        case 'InlineControlStructureUnitTest.1.inc':
-            return [
-                3   => 1,
-                7   => 1,
-                11  => 1,
-                13  => 1,
-                15  => 1,
-                17  => 1,
-                23  => 1,
-                42  => 1,
-                43  => 1,
-                45  => 1,
-                46  => 1,
-                49  => 1,
-                62  => 1,
-                66  => 1,
-                78  => 1,
-                120 => 1,
-                128 => 1,
-                134 => 1,
-                142 => 1,
-                143 => 1,
-                144 => 1,
-                150 => 1,
-                158 => 1,
-                159 => 1,
-                162 => 1,
-                163 => 1,
-                164 => 1,
-                167 => 1,
-                168 => 1,
-                170 => 1,
-                178 => 1,
-                185 => 1,
-                188 => 2,
-                191 => 1,
-                195 => 1,
-                198 => 1,
-                206 => 1,
-                222 => 1,
-                226 => 1,
-                228 => 1,
-                230 => 1,
-                232 => 1,
-                235 => 1,
-                236 => 1,
-                238 => 1,
-            ];
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList($testFile='InlineControlStructureUnitTest.1.inc')
+	{
+		switch ($testFile) {
+		case 'InlineControlStructureUnitTest.1.inc':
+			return [
+				3   => 1,
+				7   => 1,
+				11  => 1,
+				13  => 1,
+				15  => 1,
+				17  => 1,
+				23  => 1,
+				42  => 1,
+				43  => 1,
+				45  => 1,
+				46  => 1,
+				49  => 1,
+				62  => 1,
+				66  => 1,
+				78  => 1,
+				120 => 1,
+				128 => 1,
+				134 => 1,
+				142 => 1,
+				143 => 1,
+				144 => 1,
+				150 => 1,
+				158 => 1,
+				159 => 1,
+				162 => 1,
+				163 => 1,
+				164 => 1,
+				167 => 1,
+				168 => 1,
+				170 => 1,
+				178 => 1,
+				185 => 1,
+				188 => 2,
+				191 => 1,
+				195 => 1,
+				198 => 1,
+				206 => 1,
+				222 => 1,
+				226 => 1,
+				228 => 1,
+				230 => 1,
+				232 => 1,
+				235 => 1,
+				236 => 1,
+				238 => 1,
+			];
 
-        case 'InlineControlStructureUnitTest.js':
-            return [
-                3  => 1,
-                7  => 1,
-                11 => 1,
-                13 => 1,
-                15 => 1,
-                21 => 1,
-                27 => 1,
-                30 => 1,
-            ];
+		case 'InlineControlStructureUnitTest.js':
+			return [
+				3  => 1,
+				7  => 1,
+				11 => 1,
+				13 => 1,
+				15 => 1,
+				21 => 1,
+				27 => 1,
+				30 => 1,
+			];
 
-        default:
-            return [];
-        }//end switch
+		default:
+			return [];
+		}//end switch
 
-    }//end getErrorList()
+	}//end getErrorList()
 
 
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList()
-    {
-        return [];
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList()
+	{
+		return [];
 
-    }//end getWarningList()
+	}//end getWarningList()
 
 
 }//end class

Please login to merge, or discard this patch.

Standards/Generic/Tests/NamingConventions/UpperCaseConstantNameUnitTest.php 1 patch

Indentation +36 added lines, -36 removed lines patch added patch discarded remove patch

@@ -15,42 +15,42 @@
 block discarded – undo
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList()
-    {
-        return [
-            8  => 1,
-            10 => 1,
-            12 => 1,
-            14 => 1,
-            19 => 1,
-            28 => 1,
-            30 => 1,
-        ];
-
-    }//end getErrorList()
-
-
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList()
-    {
-        return [];
-
-    }//end getWarningList()
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList()
+	{
+		return [
+			8  => 1,
+			10 => 1,
+			12 => 1,
+			14 => 1,
+			19 => 1,
+			28 => 1,
+			30 => 1,
+		];
+
+	}//end getErrorList()
+
+
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList()
+	{
+		return [];
+
+	}//end getWarningList()
 
 
 }//end class

Please login to merge, or discard this patch.

src/Standards/Generic/Tests/NamingConventions/ConstructorNameUnitTest.php 1 patch

Indentation +34 added lines, -34 removed lines patch added patch discarded remove patch

@@ -15,40 +15,40 @@
 block discarded – undo
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList()
-    {
-        return [
-            6  => 1,
-            11 => 1,
-            47 => 1,
-            62 => 1,
-            91 => 1,
-        ];
-
-    }//end getErrorList()
-
-
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList()
-    {
-        return [];
-
-    }//end getWarningList()
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList()
+	{
+		return [
+			6  => 1,
+			11 => 1,
+			47 => 1,
+			62 => 1,
+			91 => 1,
+		];
+
+	}//end getErrorList()
+
+
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList()
+	{
+		return [];
+
+	}//end getWarningList()
 
 
 }//end class

Please login to merge, or discard this patch.

		@@ -4,157 +4,157 @@
		block discarded – undo
4	4
5	5	function complexityFive()
6	6	{
7		- if ($condition) {
8		- }
	7	+ if ($condition) {
	8	+ }
9	9
10		- switch ($condition) {
11		- case '1':
12		- break;
13		- case '2':
14		- break;
15		- case '3':
16		- break;
17		- }
	10	+ switch ($condition) {
	11	+ case '1':
	12	+ break;
	13	+ case '2':
	14	+ break;
	15	+ case '3':
	16	+ break;
	17	+ }
18	18	}
19	19
20	20	function complexityTen()
21	21	{
22		- while ($condition === true) {
23		- if ($condition) {
24		- }
25		- }
	22	+ while ($condition === true) {
	23	+ if ($condition) {
	24	+ }
	25	+ }
26	26
27		- switch ($condition) {
28		- case '1':
29		- if ($condition) {
30		- } else if ($cond) {
31		- }
32		- break;
33		- case '2':
34		- while ($cond) {
35		- echo 'hi';
36		- }
37		- break;
38		- case '3':
39		- break;
40		- default:
41		- break;
42		- }
	27	+ switch ($condition) {
	28	+ case '1':
	29	+ if ($condition) {
	30	+ } else if ($cond) {
	31	+ }
	32	+ break;
	33	+ case '2':
	34	+ while ($cond) {
	35	+ echo 'hi';
	36	+ }
	37	+ break;
	38	+ case '3':
	39	+ break;
	40	+ default:
	41	+ break;
	42	+ }
43	43	}
44	44
45	45	function complexityEleven()
46	46	{
47		- while ($condition === true) {
48		- if ($condition) {
49		- } else if ($cond) {
50		- }
51		- }
	47	+ while ($condition === true) {
	48	+ if ($condition) {
	49	+ } else if ($cond) {
	50	+ }
	51	+ }
52	52
53		- switch ($condition) {
54		- case '1':
55		- if ($condition) {
56		- } else if ($cond) {
57		- }
58		- break;
59		- case '2':
60		- while ($cond) {
61		- echo 'hi';
62		- }
63		- break;
64		- case '3':
65		- break;
66		- default:
67		- break;
68		- }
	53	+ switch ($condition) {
	54	+ case '1':
	55	+ if ($condition) {
	56	+ } else if ($cond) {
	57	+ }
	58	+ break;
	59	+ case '2':
	60	+ while ($cond) {
	61	+ echo 'hi';
	62	+ }
	63	+ break;
	64	+ case '3':
	65	+ break;
	66	+ default:
	67	+ break;
	68	+ }
69	69	}
70	70
71	71
72	72	function complexityTwenty()
73	73	{
74		- while ($condition === true) {
75		- if ($condition) {
76		- } else if ($cond) {
77		- }
78		- }
	74	+ while ($condition === true) {
	75	+ if ($condition) {
	76	+ } else if ($cond) {
	77	+ }
	78	+ }
79	79
80		- switch ($condition) {
81		- case '1':
82		- if ($condition) {
83		- } else if ($cond) {
84		- }
85		- break;
86		- case '2':
87		- while ($cond) {
88		- echo 'hi';
89		- }
90		- break;
91		- case '3':
92		- switch ($cond) {
93		- case '1':
94		- break;
95		- case '2':
96		- break;
97		- }
98		- break;
99		- case '4':
100		- do {
101		- if ($condition) {
102		- if ($cond) {
103		- } else if ($con) {
104		- }
105		- }
106		- } while ($cond);
107		- break;
108		- default:
109		- if ($condition) {
110		- }
111		- break;
112		- }
	80	+ switch ($condition) {
	81	+ case '1':
	82	+ if ($condition) {
	83	+ } else if ($cond) {
	84	+ }
	85	+ break;
	86	+ case '2':
	87	+ while ($cond) {
	88	+ echo 'hi';
	89	+ }
	90	+ break;
	91	+ case '3':
	92	+ switch ($cond) {
	93	+ case '1':
	94	+ break;
	95	+ case '2':
	96	+ break;
	97	+ }
	98	+ break;
	99	+ case '4':
	100	+ do {
	101	+ if ($condition) {
	102	+ if ($cond) {
	103	+ } else if ($con) {
	104	+ }
	105	+ }
	106	+ } while ($cond);
	107	+ break;
	108	+ default:
	109	+ if ($condition) {
	110	+ }
	111	+ break;
	112	+ }
113	113	}
114	114
115	115
116	116	function complexityTwentyOne()
117	117	{
118		- while ($condition === true) {
119		- if ($condition) {
120		- } else if ($cond) {
121		- }
122		- }
	118	+ while ($condition === true) {
	119	+ if ($condition) {
	120	+ } else if ($cond) {
	121	+ }
	122	+ }
123	123
124		- switch ($condition) {
125		- case '1':
126		- if ($condition) {
127		- } else if ($cond) {
128		- }
129		- break;
130		- case '2':
131		- while ($cond) {
132		- echo 'hi';
133		- }
134		- break;
135		- case '3':
136		- switch ($cond) {
137		- case '1':
138		- break;
139		- case '2':
140		- break;
141		- }
142		- break;
143		- case '4':
144		- do {
145		- if ($condition) {
146		- if ($cond) {
147		- } else if ($con) {
148		- }
149		- }
150		- } while ($cond);
151		- break;
152		- default:
153		- if ($condition) {
154		- } else if ($cond) {
155		- }
156		- break;
157		- }
	124	+ switch ($condition) {
	125	+ case '1':
	126	+ if ($condition) {
	127	+ } else if ($cond) {
	128	+ }
	129	+ break;
	130	+ case '2':
	131	+ while ($cond) {
	132	+ echo 'hi';
	133	+ }
	134	+ break;
	135	+ case '3':
	136	+ switch ($cond) {
	137	+ case '1':
	138	+ break;
	139	+ case '2':
	140	+ break;
	141	+ }
	142	+ break;
	143	+ case '4':
	144	+ do {
	145	+ if ($condition) {
	146	+ if ($cond) {
	147	+ } else if ($con) {
	148	+ }
	149	+ }
	150	+ } while ($cond);
	151	+ break;
	152	+ default:
	153	+ if ($condition) {
	154	+ } else if ($cond) {
	155	+ }
	156	+ break;
	157	+ }
158	158	}
159	159
160	160	?>

		@@ -16,47 +16,47 @@
		block discarded – undo
16	16	{
17	17
18	18
19		- /**
20		- * Returns the lines where errors should occur.
21		- *
22		- * The key of the array should represent the line number and the value
23		- * should represent the number of errors that should occur on that line.
24		- *
25		- * @param string $testFile The name of the file being tested.
26		- *
27		- * @return array<int, int>
28		- */
29		- public function getErrorList($testFile='FixmeUnitTest.inc')
30		- {
31		- return [
32		- 3 => 1,
33		- 4 => 1,
34		- 7 => 1,
35		- 10 => 1,
36		- 13 => 1,
37		- 16 => 1,
38		- 18 => 1,
39		- 21 => 1,
40		- ];
41		-
42		- }//end getErrorList()
43		-
44		-
45		- /**
46		- * Returns the lines where warnings should occur.
47		- *
48		- * The key of the array should represent the line number and the value
49		- * should represent the number of warnings that should occur on that line.
50		- *
51		- * @param string $testFile The name of the file being tested.
52		- *
53		- * @return array<int, int>
54		- */
55		- public function getWarningList($testFile='FixmeUnitTest.inc')
56		- {
57		- return [];
58		-
59		- }//end getWarningList()
	19	+ /**
	20	+ * Returns the lines where errors should occur.
	21	+ *
	22	+ * The key of the array should represent the line number and the value
	23	+ * should represent the number of errors that should occur on that line.
	24	+ *
	25	+ * @param string $testFile The name of the file being tested.
	26	+ *
	27	+ * @return array<int, int>
	28	+ */
	29	+ public function getErrorList($testFile='FixmeUnitTest.inc')
	30	+ {
	31	+ return [
	32	+ 3 => 1,
	33	+ 4 => 1,
	34	+ 7 => 1,
	35	+ 10 => 1,
	36	+ 13 => 1,
	37	+ 16 => 1,
	38	+ 18 => 1,
	39	+ 21 => 1,
	40	+ ];
	41	+
	42	+ }//end getErrorList()
	43	+
	44	+
	45	+ /**
	46	+ * Returns the lines where warnings should occur.
	47	+ *
	48	+ * The key of the array should represent the line number and the value
	49	+ * should represent the number of warnings that should occur on that line.
	50	+ *
	51	+ * @param string $testFile The name of the file being tested.
	52	+ *
	53	+ * @return array<int, int>
	54	+ */
	55	+ public function getWarningList($testFile='FixmeUnitTest.inc')
	56	+ {
	57	+ return [];
	58	+
	59	+ }//end getWarningList()
60	60
61	61
62	62	}//end class

		@@ -213,15 +213,15 @@ discard block
		block discarded – undo
213	213
214	214	/*doc comment /
215	215
216		- /**
217		- * Document behaviour with missing blank lines with indented docblocks.
218		- * @param
219		- * @param
220		- * @tag one
221		- *
222		- */
	216	+ /**
	217	+ * Document behaviour with missing blank lines with indented docblocks.
	218	+ * @param
	219	+ * @param
	220	+ * @tag one
	221	+ *
	222	+ */
223	223
224		- /** Indented doc comment */
	224	+ /** Indented doc comment */
225	225
226	226	/**
227	227	* Verify and document sniff behaviour when the "tag value" is indented with a mix of tabs and spaces.
		@@ -249,4 +249,3 @@ discard block
		block discarded – undo
249	249	* @link http://pear.php.net/package/PHP_CodeSniffer
250	250	*/
251	251
252		-/** No docblock close tag. Must be last test without new line.
253	252	\ No newline at end of file

		@@ -15,97 +15,97 @@
		block discarded – undo
15	15	{
16	16
17	17
18		- /**
19		- * Get a list of CLI values to set before the file is tested.
20		- *
21		- * @param string $testFile The name of the file being tested.
22		- * @param \PHP_CodeSniffer\Config $config The config data for the test run.
23		- *
24		- * @return void
25		- */
26		- public function setCliValues($testFile, $config)
27		- {
28		- $config->tabWidth = 4;
	18	+ /**
	19	+ * Get a list of CLI values to set before the file is tested.
	20	+ *
	21	+ * @param string $testFile The name of the file being tested.
	22	+ * @param \PHP_CodeSniffer\Config $config The config data for the test run.
	23	+ *
	24	+ * @return void
	25	+ */
	26	+ public function setCliValues($testFile, $config)
	27	+ {
	28	+ $config->tabWidth = 4;
29	29
30		- }//end setCliValues()
	30	+ }//end setCliValues()
31	31
32	32
33		- /**
34		- * Returns the lines where errors should occur.
35		- *
36		- * The key of the array should represent the line number and the value
37		- * should represent the number of errors that should occur on that line.
38		- *
39		- * @return array(int => int)
40		- */
41		- public function getErrorList()
42		- {
43		- return [
44		- 14 => 1,
45		- 16 => 1,
46		- 18 => 1,
47		- 23 => 1,
48		- 26 => 1,
49		- 30 => 1,
50		- 32 => 1,
51		- 38 => 2,
52		- 40 => 1,
53		- 41 => 1,
54		- 51 => 1,
55		- 54 => 1,
56		- 58 => 1,
57		- 60 => 2,
58		- 67 => 1,
59		- 69 => 2,
60		- 80 => 1,
61		- 81 => 2,
62		- 88 => 1,
63		- 91 => 1,
64		- 95 => 1,
65		- 156 => 1,
66		- 158 => 1,
67		- 170 => 3,
68		- 171 => 3,
69		- 179 => 1,
70		- 183 => 1,
71		- 184 => 2,
72		- 185 => 1,
73		- 186 => 1,
74		- 187 => 2,
75		- 193 => 1,
76		- 196 => 1,
77		- 199 => 1,
78		- 203 => 1,
79		- 206 => 1,
80		- 211 => 1,
81		- 214 => 4,
82		- 218 => 1,
83		- 220 => 2,
84		- 222 => 1,
85		- 224 => 3,
86		- 243 => 1,
87		- 244 => 1,
88		- 246 => 1,
89		- 248 => 1,
90		- 249 => 1,
91		- ];
	33	+ /**
	34	+ * Returns the lines where errors should occur.
	35	+ *
	36	+ * The key of the array should represent the line number and the value
	37	+ * should represent the number of errors that should occur on that line.
	38	+ *
	39	+ * @return array(int => int)
	40	+ */
	41	+ public function getErrorList()
	42	+ {
	43	+ return [
	44	+ 14 => 1,
	45	+ 16 => 1,
	46	+ 18 => 1,
	47	+ 23 => 1,
	48	+ 26 => 1,
	49	+ 30 => 1,
	50	+ 32 => 1,
	51	+ 38 => 2,
	52	+ 40 => 1,
	53	+ 41 => 1,
	54	+ 51 => 1,
	55	+ 54 => 1,
	56	+ 58 => 1,
	57	+ 60 => 2,
	58	+ 67 => 1,
	59	+ 69 => 2,
	60	+ 80 => 1,
	61	+ 81 => 2,
	62	+ 88 => 1,
	63	+ 91 => 1,
	64	+ 95 => 1,
	65	+ 156 => 1,
	66	+ 158 => 1,
	67	+ 170 => 3,
	68	+ 171 => 3,
	69	+ 179 => 1,
	70	+ 183 => 1,
	71	+ 184 => 2,
	72	+ 185 => 1,
	73	+ 186 => 1,
	74	+ 187 => 2,
	75	+ 193 => 1,
	76	+ 196 => 1,
	77	+ 199 => 1,
	78	+ 203 => 1,
	79	+ 206 => 1,
	80	+ 211 => 1,
	81	+ 214 => 4,
	82	+ 218 => 1,
	83	+ 220 => 2,
	84	+ 222 => 1,
	85	+ 224 => 3,
	86	+ 243 => 1,
	87	+ 244 => 1,
	88	+ 246 => 1,
	89	+ 248 => 1,
	90	+ 249 => 1,
	91	+ ];
92	92
93		- }//end getErrorList()
	93	+ }//end getErrorList()
94	94
95	95
96		- /**
97		- * Returns the lines where warnings should occur.
98		- *
99		- * The key of the array should represent the line number and the value
100		- * should represent the number of warnings that should occur on that line.
101		- *
102		- * @return array(int => int)
103		- */
104		- public function getWarningList()
105		- {
106		- return [];
	96	+ /**
	97	+ * Returns the lines where warnings should occur.
	98	+ *
	99	+ * The key of the array should represent the line number and the value
	100	+ * should represent the number of warnings that should occur on that line.
	101	+ *
	102	+ * @return array(int => int)
	103	+ */
	104	+ public function getWarningList()
	105	+ {
	106	+ return [];
107	107
108		- }//end getWarningList()
	108	+ }//end getWarningList()
109	109
110	110
111	111	}//end class

		@@ -15,47 +15,47 @@
		block discarded – undo
15	15	{
16	16
17	17
18		- /**
19		- * Returns the lines where errors should occur.
20		- *
21		- * The key of the array should represent the line number and the value
22		- * should represent the number of errors that should occur on that line.
23		- *
24		- * @param string $testFile The name of the file being tested.
25		- *
26		- * @return array<int, int>
27		- */
28		- public function getErrorList($testFile='TodoUnitTest.inc')
29		- {
30		- return [];
31		-
32		- }//end getErrorList()
33		-
34		-
35		- /**
36		- * Returns the lines where warnings should occur.
37		- *
38		- * The key of the array should represent the line number and the value
39		- * should represent the number of warnings that should occur on that line.
40		- *
41		- * @param string $testFile The name of the file being tested.
42		- *
43		- * @return array<int, int>
44		- */
45		- public function getWarningList($testFile='TodoUnitTest.inc')
46		- {
47		- return [
48		- 3 => 1,
49		- 4 => 1,
50		- 7 => 1,
51		- 10 => 1,
52		- 13 => 1,
53		- 16 => 1,
54		- 18 => 1,
55		- 21 => 1,
56		- ];
57		-
58		- }//end getWarningList()
	18	+ /**
	19	+ * Returns the lines where errors should occur.
	20	+ *
	21	+ * The key of the array should represent the line number and the value
	22	+ * should represent the number of errors that should occur on that line.
	23	+ *
	24	+ * @param string $testFile The name of the file being tested.
	25	+ *
	26	+ * @return array<int, int>
	27	+ */
	28	+ public function getErrorList($testFile='TodoUnitTest.inc')
	29	+ {
	30	+ return [];
	31	+
	32	+ }//end getErrorList()
	33	+
	34	+
	35	+ /**
	36	+ * Returns the lines where warnings should occur.
	37	+ *
	38	+ * The key of the array should represent the line number and the value
	39	+ * should represent the number of warnings that should occur on that line.
	40	+ *
	41	+ * @param string $testFile The name of the file being tested.
	42	+ *
	43	+ * @return array<int, int>
	44	+ */
	45	+ public function getWarningList($testFile='TodoUnitTest.inc')
	46	+ {
	47	+ return [
	48	+ 3 => 1,
	49	+ 4 => 1,
	50	+ 7 => 1,
	51	+ 10 => 1,
	52	+ 13 => 1,
	53	+ 16 => 1,
	54	+ 18 => 1,
	55	+ 21 => 1,
	56	+ ];
	57	+
	58	+ }//end getWarningList()
59	59
60	60
61	61	}//end class

		@@ -3,11 +3,11 @@ discard block
		block discarded – undo
3	3	if ($something) echo 'hello';
4	4
5	5	if ($something) {
6		- echo 'hello';
	6	+ echo 'hello';
7	7	} else echo 'hi';
8	8
9	9	if ($something) {
10		- echo 'hello';
	10	+ echo 'hello';
11	11	} else if ($else) echo 'hi';
12	12
13	13	foreach ($something as $thing) echo 'hello';
		@@ -17,61 +17,61 @@ discard block
		block discarded – undo
17	17	while ($something) echo 'hello';
18	18
19	19	do {
20		- $i--;
	20	+ $i--;
21	21	} while ($something);
22	22
23	23	if(true)
24	24	$someObject->{$name};
25	25
26	26	if (true) :
27		- $foo = true;
	27	+ $foo = true;
28	28	endif;
29	29
30	30	while (true) :
31		- $foo = true;
	31	+ $foo = true;
32	32	endwhile;
33	33
34	34	for ($i; $i > 0; $i--) :
35		- echo 'hello';
	35	+ echo 'hello';
36	36	endfor;
37	37
38	38	foreach ($array as $element) :
39		- echo 'hello';
	39	+ echo 'hello';
40	40	endforeach;
41	41
42	42	while (!$this->readLine($tokens, $tag));
43	43	while (!$this->readLine($tokens, $tag)); //skip to end of file
44	44
45	45	foreach ($cookies as $cookie)
46		- if ($cookie->match($uri, $matchSessionCookies, $now))
47		- $ret[] = $cookie;
	46	+ if ($cookie->match($uri, $matchSessionCookies, $now))
	47	+ $ret[] = $cookie;
48	48
49	49	foreach ($stringParade as $hit)
50		- $hitParade[] = $hit + 0; //cast to integer
	50	+ $hitParade[] = $hit + 0; //cast to integer
51	51
52	52	if ($foo) :
53		- echo 'true';
	53	+ echo 'true';
54	54	elseif ($something) :
55		- echo 'foo';
	55	+ echo 'foo';
56	56	else:
57		- echo 'false';
	57	+ echo 'false';
58	58	endif;
59	59
60	60	function test()
61	61	{
62		- if ($a)
63		- $a.=' '.($b ? 'b' : ($c ? ($d ? 'd' : 'c') : ''));
	62	+ if ($a)
	63	+ $a.=' '.($b ? 'b' : ($c ? ($d ? 'd' : 'c') : ''));
64	64	}
65	65
66	66	if ($a)
67		- foreach ($b as $c) {
68		- if ($d) {
69		- $e=$f;
70		- $g=$h;
71		- } elseif ($i==0) {
72		- $j=$k;
73		- }
74		- }
	67	+ foreach ($b as $c) {
	68	+ if ($d) {
	69	+ $e=$f;
	70	+ $g=$h;
	71	+ } elseif ($i==0) {
	72	+ $j=$k;
	73	+ }
	74	+ }
75	75
76	76	?>
77	77	<div style="text-align: right;">
		@@ -80,136 +80,136 @@ discard block
		block discarded – undo
80	80
81	81	<?php
82	82	switch ($this->error):
83		- case Shop_Customer :: ERROR_INVALID_GENDER: ?>
	83	+ case Shop_Customer :: ERROR_INVALID_GENDER: ?>
84	84	Ungültiges Geschlecht!
85	85	<?php break;
86		- case Shop_Customer :: ERROR_EMAIL_IN_USE: ?>
	86	+ case Shop_Customer :: ERROR_EMAIL_IN_USE: ?>
87	87	Die eingetragene E-Mail-Adresse ist bereits registriert.
88	88	<?php break;
89	89	endswitch;
90	90
91	91	if ($this->allowShopping !== true):
92		- if ($this->status != Shop_Cart :: OK):
93		- switch ($this->status):
94		- case Shop_Cart :: NOT_FOUND:
95		- echo 'foo';
96		- endswitch;
97		- endif;
	92	+ if ($this->status != Shop_Cart :: OK):
	93	+ switch ($this->status):
	94	+ case Shop_Cart :: NOT_FOUND:
	95	+ echo 'foo';
	96	+ endswitch;
	97	+ endif;
98	98	else:
99		- echo 'foo';
	99	+ echo 'foo';
100	100	endif;
101	101
102	102	// ELSE IF split over multiple lines (not inline)
103	103	if ($test) {
104	104	} else
105		- if ($test) {
106		- } else {
107		- }
	105	+ if ($test) {
	106	+ } else {
	107	+ }
108	108
109	109	switch($response = \Bar::baz('bat', function ($foo) {
110		- return 'bar';
	110	+ return 'bar';
111	111	})) {
112		- case 1:
113		- return 'test';
	112	+ case 1:
	113	+ return 'test';
114	114
115		- case 2:
116		- return 'other';
	115	+ case 2:
	116	+ return 'other';
117	117	}
118	118
119	119	$stuff = [1,2,3];
120	120	foreach($stuff as $num)
121		- if ($num %2 ) {
122		- echo "even";
123		- } else {
124		- echo "odd";
125		- }
	121	+ if ($num %2 ) {
	122	+ echo "even";
	123	+ } else {
	124	+ echo "odd";
	125	+ }
126	126
127	127	$i = 0;
128	128	foreach($stuff as $num)
129		- do {
130		- echo $i;
131		- $i++;
132		- } while ($i < 5);
	129	+ do {
	130	+ echo $i;
	131	+ $i++;
	132	+ } while ($i < 5);
133	133
134	134	foreach($stuff as $num)
135		- if (true) {
136		- echo "true1\n";
137		- }
138		- if (true) {
139		- echo "true2\n";
140		- }
	135	+ if (true) {
	136	+ echo "true1\n";
	137	+ }
	138	+ if (true) {
	139	+ echo "true2\n";
	140	+ }
141	141
142	142	if ($foo) echo 'foo';
143	143	elseif ($bar) echo 'bar';
144	144	else echo 'baz';
145	145
146	146	switch ($type) {
147		- case 1:
148		- if ($foo) {
149		- return true;
150		- } elseif ($baz)
151		- return true;
152		- else {
153		- echo 'else';
154		- }
155		- break;
	147	+ case 1:
	148	+ if ($foo) {
	149	+ return true;
	150	+ } elseif ($baz)
	151	+ return true;
	152	+ else {
	153	+ echo 'else';
	154	+ }
	155	+ break;
156	156	}
157	157
158	158	foreach ($sql as $s)
159		- if (!$this->execute) echo "<pre>",$s.";\n</pre>";
160		- else {
161		- $ok = $this->connDest->Execute($s);
162		- if (!$ok)
163		- if ($this->neverAbort) $ret = false;
164		- else return false;
165		- }
	159	+ if (!$this->execute) echo "<pre>",$s.";\n</pre>";
	160	+ else {
	161	+ $ok = $this->connDest->Execute($s);
	162	+ if (!$ok)
	163	+ if ($this->neverAbort) $ret = false;
	164	+ else return false;
	165	+ }
166	166
167	167	if ($bar)
168		- if ($foo) echo 'hi'; // lol
	168	+ if ($foo) echo 'hi'; // lol
169	169
170	170	if ($level == 'district')
171		- \DB::update(<<<EOD
	171	+ \DB::update(<<<EOD
172	172	some
173	173	text
174	174	here
175	175	EOD
176		- );
	176	+ );
177	177
178	178	if ($level == 'district')
179		- $var = <<<EOD
	179	+ $var = <<<EOD
180	180	some
181	181	text
182	182	here
183	183	EOD;
184	184
185	185	if ($a && $a === Foo::VARIABLE && ($a === Foo::METHOD \|\| $a === Foo::FUNCTION))
186		- echo 'hi';
	186	+ echo 'hi';
187	187
188	188	$out = array_map(function ($test) { if ($test) return 1; else return 2; }, $input); // comment
189	189
190	190	for ($x=0;$x<5;$x++):
191		- if ($x) continue;
	191	+ if ($x) continue;
192	192	endfor;
193	193
194	194	for ($x=0;$x<5;$x++):
195		- if ($x) continue ?> <?php
	195	+ if ($x) continue ?> <?php
196	196	endfor;
197	197
198	198	if (true)
199		- try {
200		- }
201		- catch(Exception $e) {
202		- }
	199	+ try {
	200	+ }
	201	+ catch(Exception $e) {
	202	+ }
203	203
204	204	switch ($num) {
205		- case 0:
206		- if (1 > $num)
207		- return bar(
208		- baz(
209		- "foobarbaz"
210		- )
211		- );
212		- break;
	205	+ case 0:
	206	+ if (1 > $num)
	207	+ return bar(
	208	+ baz(
	209	+ "foobarbaz"
	210	+ )
	211	+ );
	212	+ break;
213	213	}
214	214
215	215	do {
		@@ -219,8 +219,8 @@ discard block
		block discarded – undo
219	219	while ($i < 10);
220	220
221	221	if ($this) {
222		- if ($that)
223		- foo(${$a[$b]});
	222	+ if ($that)
	223	+ foo(${$a[$b]});
224	224	}
225	225
226	226	while (!$this->readLine($tokens, $tag)); //phpcs:ignore Standard.Category.Sniff
		@@ -230,11 +230,11 @@ discard block
		block discarded – undo
230	230	while (!$this->readLine($tokens, $tag)); /* comment */
231	231
232	232	foreach ($stringParade as $hit)
233		- $hitParade[] = $hit + 0; // phpcs:ignore Standard.Category.Sniff
	233	+ $hitParade[] = $hit + 0; // phpcs:ignore Standard.Category.Sniff
234	234
235	235	if ($bar)
236		- if ($foo) echo 'hi'; /* @phpcs:ignore Standard.Category.Sniff */
	236	+ if ($foo) echo 'hi'; /* @phpcs:ignore Standard.Category.Sniff */
237	237
238	238	if (true) $callable = function () {
239		- return true;
	239	+ return true;
240	240	};

		@@ -15,100 +15,100 @@
		block discarded – undo
15	15	{
16	16
17	17
18		- /**
19		- * Returns the lines where errors should occur.
20		- *
21		- * The key of the array should represent the line number and the value
22		- * should represent the number of errors that should occur on that line.
23		- *
24		- * @param string $testFile The name of the file being tested.
25		- *
26		- * @return array<int, int>
27		- */
28		- public function getErrorList($testFile='InlineControlStructureUnitTest.1.inc')
29		- {
30		- switch ($testFile) {
31		- case 'InlineControlStructureUnitTest.1.inc':
32		- return [
33		- 3 => 1,
34		- 7 => 1,
35		- 11 => 1,
36		- 13 => 1,
37		- 15 => 1,
38		- 17 => 1,
39		- 23 => 1,
40		- 42 => 1,
41		- 43 => 1,
42		- 45 => 1,
43		- 46 => 1,
44		- 49 => 1,
45		- 62 => 1,
46		- 66 => 1,
47		- 78 => 1,
48		- 120 => 1,
49		- 128 => 1,
50		- 134 => 1,
51		- 142 => 1,
52		- 143 => 1,
53		- 144 => 1,
54		- 150 => 1,
55		- 158 => 1,
56		- 159 => 1,
57		- 162 => 1,
58		- 163 => 1,
59		- 164 => 1,
60		- 167 => 1,
61		- 168 => 1,
62		- 170 => 1,
63		- 178 => 1,
64		- 185 => 1,
65		- 188 => 2,
66		- 191 => 1,
67		- 195 => 1,
68		- 198 => 1,
69		- 206 => 1,
70		- 222 => 1,
71		- 226 => 1,
72		- 228 => 1,
73		- 230 => 1,
74		- 232 => 1,
75		- 235 => 1,
76		- 236 => 1,
77		- 238 => 1,
78		- ];
	18	+ /**
	19	+ * Returns the lines where errors should occur.
	20	+ *
	21	+ * The key of the array should represent the line number and the value
	22	+ * should represent the number of errors that should occur on that line.
	23	+ *
	24	+ * @param string $testFile The name of the file being tested.
	25	+ *
	26	+ * @return array<int, int>
	27	+ */
	28	+ public function getErrorList($testFile='InlineControlStructureUnitTest.1.inc')
	29	+ {
	30	+ switch ($testFile) {
	31	+ case 'InlineControlStructureUnitTest.1.inc':
	32	+ return [
	33	+ 3 => 1,
	34	+ 7 => 1,
	35	+ 11 => 1,
	36	+ 13 => 1,
	37	+ 15 => 1,
	38	+ 17 => 1,
	39	+ 23 => 1,
	40	+ 42 => 1,
	41	+ 43 => 1,
	42	+ 45 => 1,
	43	+ 46 => 1,
	44	+ 49 => 1,
	45	+ 62 => 1,
	46	+ 66 => 1,
	47	+ 78 => 1,
	48	+ 120 => 1,
	49	+ 128 => 1,
	50	+ 134 => 1,
	51	+ 142 => 1,
	52	+ 143 => 1,
	53	+ 144 => 1,
	54	+ 150 => 1,
	55	+ 158 => 1,
	56	+ 159 => 1,
	57	+ 162 => 1,
	58	+ 163 => 1,
	59	+ 164 => 1,
	60	+ 167 => 1,
	61	+ 168 => 1,
	62	+ 170 => 1,
	63	+ 178 => 1,
	64	+ 185 => 1,
	65	+ 188 => 2,
	66	+ 191 => 1,
	67	+ 195 => 1,
	68	+ 198 => 1,
	69	+ 206 => 1,
	70	+ 222 => 1,
	71	+ 226 => 1,
	72	+ 228 => 1,
	73	+ 230 => 1,
	74	+ 232 => 1,
	75	+ 235 => 1,
	76	+ 236 => 1,
	77	+ 238 => 1,
	78	+ ];
79	79
80		- case 'InlineControlStructureUnitTest.js':
81		- return [
82		- 3 => 1,
83		- 7 => 1,
84		- 11 => 1,
85		- 13 => 1,
86		- 15 => 1,
87		- 21 => 1,
88		- 27 => 1,
89		- 30 => 1,
90		- ];
	80	+ case 'InlineControlStructureUnitTest.js':
	81	+ return [
	82	+ 3 => 1,
	83	+ 7 => 1,
	84	+ 11 => 1,
	85	+ 13 => 1,
	86	+ 15 => 1,
	87	+ 21 => 1,
	88	+ 27 => 1,
	89	+ 30 => 1,
	90	+ ];
91	91
92		- default:
93		- return [];
94		- }//end switch
	92	+ default:
	93	+ return [];
	94	+ }//end switch
95	95
96		- }//end getErrorList()
	96	+ }//end getErrorList()
97	97
98	98
99		- /**
100		- * Returns the lines where warnings should occur.
101		- *
102		- * The key of the array should represent the line number and the value
103		- * should represent the number of warnings that should occur on that line.
104		- *
105		- * @return array<int, int>
106		- */
107		- public function getWarningList()
108		- {
109		- return [];
	99	+ /**
	100	+ * Returns the lines where warnings should occur.
	101	+ *
	102	+ * The key of the array should represent the line number and the value
	103	+ * should represent the number of warnings that should occur on that line.
	104	+ *
	105	+ * @return array<int, int>
	106	+ */
	107	+ public function getWarningList()
	108	+ {
	109	+ return [];
110	110
111		- }//end getWarningList()
	111	+ }//end getWarningList()
112	112
113	113
114	114	}//end class

		@@ -15,42 +15,42 @@
		block discarded – undo
15	15	{
16	16
17	17
18		- /**
19		- * Returns the lines where errors should occur.
20		- *
21		- * The key of the array should represent the line number and the value
22		- * should represent the number of errors that should occur on that line.
23		- *
24		- * @return array<int, int>
25		- */
26		- public function getErrorList()
27		- {
28		- return [
29		- 8 => 1,
30		- 10 => 1,
31		- 12 => 1,
32		- 14 => 1,
33		- 19 => 1,
34		- 28 => 1,
35		- 30 => 1,
36		- ];
37		-
38		- }//end getErrorList()
39		-
40		-
41		- /**
42		- * Returns the lines where warnings should occur.
43		- *
44		- * The key of the array should represent the line number and the value
45		- * should represent the number of warnings that should occur on that line.
46		- *
47		- * @return array<int, int>
48		- */
49		- public function getWarningList()
50		- {
51		- return [];
52		-
53		- }//end getWarningList()
	18	+ /**
	19	+ * Returns the lines where errors should occur.
	20	+ *
	21	+ * The key of the array should represent the line number and the value
	22	+ * should represent the number of errors that should occur on that line.
	23	+ *
	24	+ * @return array<int, int>
	25	+ */
	26	+ public function getErrorList()
	27	+ {
	28	+ return [
	29	+ 8 => 1,
	30	+ 10 => 1,
	31	+ 12 => 1,
	32	+ 14 => 1,
	33	+ 19 => 1,
	34	+ 28 => 1,
	35	+ 30 => 1,
	36	+ ];
	37	+
	38	+ }//end getErrorList()
	39	+
	40	+
	41	+ /**
	42	+ * Returns the lines where warnings should occur.
	43	+ *
	44	+ * The key of the array should represent the line number and the value
	45	+ * should represent the number of warnings that should occur on that line.
	46	+ *
	47	+ * @return array<int, int>
	48	+ */
	49	+ public function getWarningList()
	50	+ {
	51	+ return [];
	52	+
	53	+ }//end getWarningList()
54	54
55	55
56	56	}//end class

		@@ -15,40 +15,40 @@
		block discarded – undo
15	15	{
16	16
17	17
18		- /**
19		- * Returns the lines where errors should occur.
20		- *
21		- * The key of the array should represent the line number and the value
22		- * should represent the number of errors that should occur on that line.
23		- *
24		- * @return array<int, int>
25		- */
26		- public function getErrorList()
27		- {
28		- return [
29		- 6 => 1,
30		- 11 => 1,
31		- 47 => 1,
32		- 62 => 1,
33		- 91 => 1,
34		- ];
35		-
36		- }//end getErrorList()
37		-
38		-
39		- /**
40		- * Returns the lines where warnings should occur.
41		- *
42		- * The key of the array should represent the line number and the value
43		- * should represent the number of warnings that should occur on that line.
44		- *
45		- * @return array<int, int>
46		- */
47		- public function getWarningList()
48		- {
49		- return [];
50		-
51		- }//end getWarningList()
	18	+ /**
	19	+ * Returns the lines where errors should occur.
	20	+ *
	21	+ * The key of the array should represent the line number and the value
	22	+ * should represent the number of errors that should occur on that line.
	23	+ *
	24	+ * @return array<int, int>
	25	+ */
	26	+ public function getErrorList()
	27	+ {
	28	+ return [
	29	+ 6 => 1,
	30	+ 11 => 1,
	31	+ 47 => 1,
	32	+ 62 => 1,
	33	+ 91 => 1,
	34	+ ];
	35	+
	36	+ }//end getErrorList()
	37	+
	38	+
	39	+ /**
	40	+ * Returns the lines where warnings should occur.
	41	+ *
	42	+ * The key of the array should represent the line number and the value
	43	+ * should represent the number of warnings that should occur on that line.
	44	+ *
	45	+ * @return array<int, int>
	46	+ */
	47	+ public function getWarningList()
	48	+ {
	49	+ return [];
	50	+
	51	+ }//end getWarningList()
52	52
53	53
54	54	}//end class

gravityview / GravityView

Pull Request — develop (#1492)

Category

Indentation +128 added lines, -128 removed lines patch added patch discarded remove patch

Indentation +41 added lines, -41 removed lines patch added patch discarded remove patch

Indentation +8 added lines, -9 removed lines patch added patch discarded remove patch

Indentation +84 added lines, -84 removed lines patch added patch discarded remove patch

Indentation +41 added lines, -41 removed lines patch added patch discarded remove patch

Indentation +95 added lines, -95 removed lines patch added patch discarded remove patch

Indentation +88 added lines, -88 removed lines patch added patch discarded remove patch

Indentation +36 added lines, -36 removed lines patch added patch discarded remove patch

Indentation +34 added lines, -34 removed lines patch added patch discarded remove patch