gravityview / GravityView

vendor/squizlabs/php_codesniffer/src/Standards/Generic/Tests/Arrays/DisallowLongArraySyntaxUnitTest.2.inc

Doc Comments -6 removed lines

@@ -9,9 +9,3 @@
         $arr = array(
             'a' => 'a',
 <<<<<<< HEAD
-            'b' => 'b'
-=======
-            'c' => 'c'
->>>>>>> master
-        );
-    }

Indentation +3 added lines, -9 removed lines

@@ -5,13 +5,7 @@
 // This is not a merge conflict - it is a valid test case to test handling of arrays without associated closer.
 // Please do not remove.
 function test()
-    {
-        $arr = array(
-            'a' => 'a',
+	{
+		$arr = array(
+			'a' => 'a',
 <<<<<<< HEAD
-            'b' => 'b'
-=======
-            'c' => 'c'
->>>>>>> master
-        );
-    }

Switch Indentation -6 removed lines

@@ -9,9 +9,3 @@
         $arr = array(
             'a' => 'a',
 <<<<<<< HEAD
-            'b' => 'b'
-=======
-            'c' => 'c'
->>>>>>> master
-        );
-    }

Spacing +1 added lines, -7 removed lines

@@ -8,10 +8,4 @@
     {
         $arr = array(
             'a' => 'a',
-<<<<<<< HEAD
-            'b' => 'b'
-=======
-            'c' => 'c'
->>>>>>> master
-        );
-    }
+<< << <<< HEAD

Braces +1 added lines, -8 removed lines

@@ -4,14 +4,7 @@
 // The following function has a simulated git conflict for testing.
 // This is not a merge conflict - it is a valid test case to test handling of arrays without associated closer.
 // Please do not remove.
-function test()
-    {
+function test() {
         $arr = array(
             'a' => 'a',
 <<<<<<< HEAD
-            'b' => 'b'
-=======
-            'c' => 'c'
->>>>>>> master
-        );
-    }

Upper-Lower-Casing -6 removed lines

@@ -9,9 +9,3 @@
         $arr = array(
             'a' => 'a',
 <<<<<<< HEAD
-            'b' => 'b'
-=======
-            'c' => 'c'
->>>>>>> master
-        );
-    }

vendor/squizlabs/php_codesniffer/src/Standards/Generic/Tests/Debug/ClosureLinterUnitTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     /**
      * Should this test be skipped for some reason.
      *
-     * @return void
+     * @return boolean
      */
     protected function shouldSkipTest()
     {

Indentation +40 added lines, -40 removed lines

@@ -16,54 +16,54 @@
 {
 
 
-    /**
-     * Should this test be skipped for some reason.
-     *
-     * @return void
-     */
-    protected function shouldSkipTest()
-    {
-        $lintPath = Config::getExecutablePath('gjslint');
-        if ($lintPath === null) {
-            return true;
-        }
+	/**
+	 * Should this test be skipped for some reason.
+	 *
+	 * @return void
+	 */
+	protected function shouldSkipTest()
+	{
+		$lintPath = Config::getExecutablePath('gjslint');
+		if ($lintPath === null) {
+			return true;
+		}
 
-        return false;
+		return false;
 
-    }//end shouldSkipTest()
+	}//end shouldSkipTest()
 
 
-    /**
-     * 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 [];
+	/**
+	 * 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 [];
 
-    }//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 [
-            3 => 1,
-            5 => 1,
-        ];
+	/**
+	 * 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 [
+			3 => 1,
+			5 => 1,
+		];
 
-    }//end getWarningList()
+	}//end getWarningList()
 
 
 }//end class

Spacing +3 added lines, -3 removed lines

@@ -23,8 +23,8 @@ 
      */
     protected function shouldSkipTest()
     {
-        $lintPath = Config::getExecutablePath('gjslint');
-        if ($lintPath === null) {
+        $lintPath = Config::getExecutablePath( 'gjslint' );
+        if ( $lintPath === null ) {
             return true;
         }
 
@@ -43,7 +43,7 @@ 
      */
     public function getErrorList()
     {
-        return [];
+        return [ ];
 
     }//end getErrorList()
 

Braces +4 added lines, -8 removed lines

@@ -12,8 +12,7 @@ 
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 use PHP_CodeSniffer\Config;
 
-class ClosureLinterUnitTest extends AbstractSniffUnitTest
-{
+class ClosureLinterUnitTest extends AbstractSniffUnitTest {
 
 
     /**
@@ -21,8 +20,7 @@ 
      *
      * @return void
      */
-    protected function shouldSkipTest()
-    {
+    protected function shouldSkipTest() {
         $lintPath = Config::getExecutablePath('gjslint');
         if ($lintPath === null) {
             return true;
@@ -41,8 +39,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getErrorList()
-    {
+    public function getErrorList() {
         return [];
 
     }//end getErrorList()
@@ -56,8 +53,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getWarningList()
-    {
+    public function getWarningList() {
         return [
             3 => 1,
             5 => 1,

vendor/squizlabs/php_codesniffer/src/Standards/Generic/Tests/VersionControl/GitMergeConflictUnitTest.1.inc

Doc Comments +1 added lines, -2 removed lines

@@ -57,5 +57,4 @@
 
 // Test that stray boundaries, i.e. an opener without closer and such, are detected.
 <<<<<<< HEAD
-$a = 1;
-=======
+$a
\ No newline at end of file

Indentation +4 added lines, -5 removed lines

@@ -20,9 +20,9 @@ 
  */
 
 function test()
-    {
-        $arr = array(
-            'a' => 'a'
+	{
+		$arr = array(
+			'a' => 'a'
 <<<<<<< HEAD
             'b' => 'b'
 =======
@@ -57,5 +57,4 @@ 
 
 // Test that stray boundaries, i.e. an opener without closer and such, are detected.
 <<<<<<< HEAD
-$a = 1;
-=======
+$a
\ No newline at end of file

Switch Indentation +1 added lines, -2 removed lines

@@ -57,5 +57,4 @@
 
 // Test that stray boundaries, i.e. an opener without closer and such, are detected.
 <<<<<<< HEAD
-$a = 1;
-=======
+$a
\ No newline at end of file

Spacing +4 added lines, -5 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
-var_dump(1 << -1);
-var_dump(1 >> -1);
+var_dump( 1 << -1 );
+var_dump( 1 >> -1 );
 var_dump(
 1
 <<
@@ -23,7 +23,7 @@ 
     {
         $arr = array(
             'a' => 'a'
-<<<<<<< HEAD
+<< << <<< HEAD
             'b' => 'b'
 =======
             'c' => 'c'
@@ -57,5 +57,4 @@ 
 
 // Test that stray boundaries, i.e. an opener without closer and such, are detected.
 <<<<<<< HEAD
-$a = 1;
-=======
+$a
\ No newline at end of file

Braces +2 added lines, -4 removed lines

@@ -19,8 +19,7 @@ 
  * <<<<<<< HEAD
  */
 
-function test()
-    {
+function test() {
         $arr = array(
             'a' => 'a'
 <<<<<<< HEAD
@@ -57,5 +56,4 @@ 
 
 // Test that stray boundaries, i.e. an opener without closer and such, are detected.
 <<<<<<< HEAD
-$a = 1;
-=======
+$a
\ No newline at end of file

Upper-Lower-Casing +2 added lines, -3 removed lines

@@ -11,7 +11,7 @@ 
 $string =
 <<<EOD
 This is a heredoc.
-EOD;
+eod;
 
 /**
  * This is not a merge conflict, but a comment showing what
@@ -57,5 +57,4 @@ 
 
 // Test that stray boundaries, i.e. an opener without closer and such, are detected.
 <<<<<<< HEAD
-$a = 1;
-=======
+$a
\ No newline at end of file

vendor/squizlabs/php_codesniffer/src/Standards/Generic/Tests/VersionControl/GitMergeConflictUnitTest.2.inc

Doc Comments -5 removed lines

@@ -24,8 +24,3 @@
 
 // Comment
 <<<<<<< HEAD
-// Second comment line. NOTE: The above opener breaks the tokenizer.
-=======
-// New second comment line
->>>>>>> master
-// Third comment line

Indentation -5 removed lines

@@ -24,8 +24,3 @@
 
 // Comment
 <<<<<<< HEAD
-// Second comment line. NOTE: The above opener breaks the tokenizer.
-=======
-// New second comment line
->>>>>>> master
-// Third comment line

Switch Indentation -5 removed lines

@@ -24,8 +24,3 @@
 
 // Comment
 <<<<<<< HEAD
-// Second comment line. NOTE: The above opener breaks the tokenizer.
-=======
-// New second comment line
->>>>>>> master
-// Third comment line

Spacing +2 added lines, -7 removed lines

@@ -14,7 +14,7 @@ 
 <<<<<<< HEAD
  * @var string $bar
  */
-public function foo($bar){ }
+public function foo( $bar ) { }
 
 /*
 =======
@@ -23,9 +23,4 @@ 
 */
 
 // Comment
-<<<<<<< HEAD
-// Second comment line. NOTE: The above opener breaks the tokenizer.
-=======
-// New second comment line
->>>>>>> master
-// Third comment line
+<< << <<< HEAD

Braces -5 removed lines

@@ -24,8 +24,3 @@
 
 // Comment
 <<<<<<< HEAD
-// Second comment line. NOTE: The above opener breaks the tokenizer.
-=======
-// New second comment line
->>>>>>> master
-// Third comment line

Upper-Lower-Casing -5 removed lines

@@ -24,8 +24,3 @@
 
 // Comment
 <<<<<<< HEAD
-// Second comment line. NOTE: The above opener breaks the tokenizer.
-=======
-// New second comment line
->>>>>>> master
-// Third comment line

vendor/squizlabs/php_codesniffer/src/Standards/Generic/Tests/VersionControl/GitMergeConflictUnitTest.3.inc

Doc Comments +1 added lines, -19 removed lines

@@ -22,22 +22,4 @@
 <<<<<<< HEAD
 $a = 1;
 =======
-$a = 2;
->>>>>>> master
-
-/*
- * The above tests are based on "normal" tokens.
- * The below test checks that once the tokenizer breaks down because of
- * unexpected merge conflict boundaries - i.e. after the first merge conflict
- * opener in non-comment, non-heredoc/nowdoc, non-inline HTML code -, subsequent
- * merge conflict boundaries will still be detected correctly.
- */
-?>
-
-<div class="abc">
-<<<<<<< HEAD
-	<p id="test-this">Testing a merge conflict.</p>
-=======
-	<p id="test-that">Another text string.</p>
->>>>>>> ref/heads/feature-branch
-</div>
+$a
\ No newline at end of file

Indentation +1 added lines, -19 removed lines

@@ -22,22 +22,4 @@
 <<<<<<< HEAD
 $a = 1;
 =======
-$a = 2;
->>>>>>> master
-
-/*
- * The above tests are based on "normal" tokens.
- * The below test checks that once the tokenizer breaks down because of
- * unexpected merge conflict boundaries - i.e. after the first merge conflict
- * opener in non-comment, non-heredoc/nowdoc, non-inline HTML code -, subsequent
- * merge conflict boundaries will still be detected correctly.
- */
-?>
-
-<div class="abc">
-<<<<<<< HEAD
-	<p id="test-this">Testing a merge conflict.</p>
-=======
-	<p id="test-that">Another text string.</p>
->>>>>>> ref/heads/feature-branch
-</div>
+$a
\ No newline at end of file

Switch Indentation +1 added lines, -19 removed lines

@@ -22,22 +22,4 @@
 <<<<<<< HEAD
 $a = 1;
 =======
-$a = 2;
->>>>>>> master
-
-/*
- * The above tests are based on "normal" tokens.
- * The below test checks that once the tokenizer breaks down because of
- * unexpected merge conflict boundaries - i.e. after the first merge conflict
- * opener in non-comment, non-heredoc/nowdoc, non-inline HTML code -, subsequent
- * merge conflict boundaries will still be detected correctly.
- */
-?>
-
-<div class="abc">
-<<<<<<< HEAD
-	<p id="test-this">Testing a merge conflict.</p>
-=======
-	<p id="test-that">Another text string.</p>
->>>>>>> ref/heads/feature-branch
-</div>
+$a
\ No newline at end of file

Spacing +2 added lines, -20 removed lines

@@ -19,25 +19,7 @@
 <?php
 
 // Break the tokenizer.
-<<<<<<< HEAD
+<< << <<< HEAD
 $a = 1;
 =======
-$a = 2;
->>>>>>> master
-
-/*
- * The above tests are based on "normal" tokens.
- * The below test checks that once the tokenizer breaks down because of
- * unexpected merge conflict boundaries - i.e. after the first merge conflict
- * opener in non-comment, non-heredoc/nowdoc, non-inline HTML code -, subsequent
- * merge conflict boundaries will still be detected correctly.
- */
-?>
-
-<div class="abc">
-<<<<<<< HEAD
-	<p id="test-this">Testing a merge conflict.</p>
-=======
-	<p id="test-that">Another text string.</p>
->>>>>>> ref/heads/feature-branch
-</div>
+$a
\ No newline at end of file

Braces +1 added lines, -19 removed lines

@@ -22,22 +22,4 @@
 <<<<<<< HEAD
 $a = 1;
 =======
-$a = 2;
->>>>>>> master
-
-/*
- * The above tests are based on "normal" tokens.
- * The below test checks that once the tokenizer breaks down because of
- * unexpected merge conflict boundaries - i.e. after the first merge conflict
- * opener in non-comment, non-heredoc/nowdoc, non-inline HTML code -, subsequent
- * merge conflict boundaries will still be detected correctly.
- */
-?>
-
-<div class="abc">
-<<<<<<< HEAD
-	<p id="test-this">Testing a merge conflict.</p>
-=======
-	<p id="test-that">Another text string.</p>
->>>>>>> ref/heads/feature-branch
-</div>
+$a
\ No newline at end of file

Upper-Lower-Casing +1 added lines, -19 removed lines

@@ -22,22 +22,4 @@
 <<<<<<< HEAD
 $a = 1;
 =======
-$a = 2;
->>>>>>> master
-
-/*
- * The above tests are based on "normal" tokens.
- * The below test checks that once the tokenizer breaks down because of
- * unexpected merge conflict boundaries - i.e. after the first merge conflict
- * opener in non-comment, non-heredoc/nowdoc, non-inline HTML code -, subsequent
- * merge conflict boundaries will still be detected correctly.
- */
-?>
-
-<div class="abc">
-<<<<<<< HEAD
-	<p id="test-this">Testing a merge conflict.</p>
-=======
-	<p id="test-that">Another text string.</p>
->>>>>>> ref/heads/feature-branch
-</div>
+$a
\ No newline at end of file

vendor/squizlabs/php_codesniffer/src/Standards/Generic/Tests/VersionControl/GitMergeConflictUnitTest.4.inc

Doc Comments +1 added lines, -5 removed lines

@@ -64,8 +64,4 @@
 <<<<<<< HEAD
 can be problematic.
 EOD;
-$a = 1;
-=======
-should also be detected.
-EOT;
->>>>>>> ref/heads/other-branchname
+$a
\ No newline at end of file

Indentation +1 added lines, -5 removed lines

@@ -64,8 +64,4 @@
 <<<<<<< HEAD
 can be problematic.
 EOD;
-$a = 1;
-=======
-should also be detected.
-EOT;
->>>>>>> ref/heads/other-branchname
+$a
\ No newline at end of file

Switch Indentation +1 added lines, -5 removed lines

@@ -64,8 +64,4 @@
 <<<<<<< HEAD
 can be problematic.
 EOD;
-$a = 1;
-=======
-should also be detected.
-EOT;
->>>>>>> ref/heads/other-branchname
+$a
\ No newline at end of file

Spacing +3 added lines, -7 removed lines

@@ -22,11 +22,11 @@ 
 =======
 should also be detected.
 EOT;
->>>>>>> ref/heads/other-branchname
+>> >> >> > ref / heads / other - branchname
 
 // Merge conflict starter outside with a closer inside of the heredoc.
 // This breaks the tokenizer.
-<<<<<<< HEAD
+<< << <<< HEAD
 $string =
 <<<EOT
 Merge conflicts in heredocs
@@ -64,8 +64,4 @@ 
 <<<<<<< HEAD
 can be problematic.
 EOD;
-$a = 1;
-=======
-should also be detected.
-EOT;
->>>>>>> ref/heads/other-branchname
+$a
\ No newline at end of file

Braces +1 added lines, -5 removed lines

@@ -64,8 +64,4 @@
 <<<<<<< HEAD
 can be problematic.
 EOD;
-$a = 1;
-=======
-should also be detected.
-EOT;
->>>>>>> ref/heads/other-branchname
+$a
\ No newline at end of file

Upper-Lower-Casing +3 added lines, -7 removed lines

@@ -9,7 +9,7 @@ 
 should also be detected.
 >>>>>>> ref/heads/other-branchname
 And now they are.
-EOD;
+eod;
 
 // Heredoc with a merge conflict starter, the closer is outside the heredoc.
 $string =
@@ -21,7 +21,7 @@ 
 $a = 1;
 =======
 should also be detected.
-EOT;
+eot;
 >>>>>>> ref/heads/other-branchname
 
 // Merge conflict starter outside with a closer inside of the heredoc.
@@ -64,8 +64,4 @@ 
 <<<<<<< HEAD
 can be problematic.
 EOD;
-$a = 1;
-=======
-should also be detected.
-EOT;
->>>>>>> ref/heads/other-branchname
+$a
\ No newline at end of file

vendor/squizlabs/php_codesniffer/src/Standards/Generic/Tests/VersionControl/GitMergeConflictUnitTest.5.inc

Doc Comments +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-<<<'EOD'
-can be problematic.
-<<<<<<< HEAD
-were previously not detected.
-=======
-should also be detected.
->>>>>>> ref/heads/other-branchname
-And now they are.
-EOD;
+$string
\ No newline at end of file

Indentation +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-<<<'EOD'
-can be problematic.
-<<<<<<< HEAD
-were previously not detected.
-=======
-should also be detected.
->>>>>>> ref/heads/other-branchname
-And now they are.
-EOD;
+$string
\ No newline at end of file

Switch Indentation +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-<<<'EOD'
-can be problematic.
-<<<<<<< HEAD
-were previously not detected.
-=======
-should also be detected.
->>>>>>> ref/heads/other-branchname
-And now they are.
-EOD;
+$string
\ No newline at end of file

Spacing +2 added lines, -11 removed lines

@@ -12,7 +12,7 @@ 
 EOD;
 
 // Break the tokenizer.
-<<<<<<< HEAD
+<< << <<< HEAD
 
 /*
  * The above tests are based on "normal" tokens.
@@ -22,13 +22,4 @@ 
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-<<<'EOD'
-can be problematic.
-<<<<<<< HEAD
-were previously not detected.
-=======
-should also be detected.
->>>>>>> ref/heads/other-branchname
-And now they are.
-EOD;
+$string
\ No newline at end of file

Braces +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-<<<'EOD'
-can be problematic.
-<<<<<<< HEAD
-were previously not detected.
-=======
-should also be detected.
->>>>>>> ref/heads/other-branchname
-And now they are.
-EOD;
+$string
\ No newline at end of file

Upper-Lower-Casing +2 added lines, -11 removed lines

@@ -9,7 +9,7 @@ 
 should also be detected.
 >>>>>>> ref/heads/other-branchname
 And now they are.
-EOD;
+eod;
 
 // Break the tokenizer.
 <<<<<<< HEAD
@@ -22,13 +22,4 @@ 
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-<<<'EOD'
-can be problematic.
-<<<<<<< HEAD
-were previously not detected.
-=======
-should also be detected.
->>>>>>> ref/heads/other-branchname
-And now they are.
-EOD;
+$string
\ No newline at end of file

vendor/squizlabs/php_codesniffer/src/Standards/Generic/Tests/VersionControl/GitMergeConflictUnitTest.6.inc

Doc Comments +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-		<<<"EOD"
-		Merge conflicts in PHP 7.3 indented heredocs
-<<<<<<< HEAD
-		can be problematic.
-=======
-		should also be detected.
->>>>>>> ref/heads/other-branchname
-		And now they are.
-		EOD;
+$string
\ No newline at end of file

Indentation +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-		<<<"EOD"
-		Merge conflicts in PHP 7.3 indented heredocs
-<<<<<<< HEAD
-		can be problematic.
-=======
-		should also be detected.
->>>>>>> ref/heads/other-branchname
-		And now they are.
-		EOD;
+$string
\ No newline at end of file

Switch Indentation +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-		<<<"EOD"
-		Merge conflicts in PHP 7.3 indented heredocs
-<<<<<<< HEAD
-		can be problematic.
-=======
-		should also be detected.
->>>>>>> ref/heads/other-branchname
-		And now they are.
-		EOD;
+$string
\ No newline at end of file

Spacing +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-		<<<"EOD"
-		Merge conflicts in PHP 7.3 indented heredocs
-<<<<<<< HEAD
-		can be problematic.
-=======
-		should also be detected.
->>>>>>> ref/heads/other-branchname
-		And now they are.
-		EOD;
+$string
\ No newline at end of file

Braces +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-		<<<"EOD"
-		Merge conflicts in PHP 7.3 indented heredocs
-<<<<<<< HEAD
-		can be problematic.
-=======
-		should also be detected.
->>>>>>> ref/heads/other-branchname
-		And now they are.
-		EOD;
+$string
\ No newline at end of file

Upper-Lower-Casing +1 added lines, -10 removed lines

@@ -22,13 +22,4 @@
  * merge conflict boundaries will still be detected correctly.
  */
 
-$string =
-		<<<"EOD"
-		Merge conflicts in PHP 7.3 indented heredocs
-<<<<<<< HEAD
-		can be problematic.
-=======
-		should also be detected.
->>>>>>> ref/heads/other-branchname
-		And now they are.
-		EOD;
+$string
\ No newline at end of file

vendor/squizlabs/php_codesniffer/src/Standards/MySource/Sniffs/Channels/IncludeOwnSystemSniff.php

Doc Comments +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {
@@ -82,7 +82,7 @@ 
      * @param int                         $stackPtr  The position in the tokens array of the
      *                                               potentially included class.
      *
-     * @return string
+     * @return boolean
      */
     protected function getIncludedClassFromToken(
         $phpcsFile,

Indentation +77 added lines, -77 removed lines

@@ -16,83 +16,83 @@
 {
 
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return [T_DOUBLE_COLON];
-
-    }//end register()
-
-
-    /**
-     * Processes this sniff, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
-     * @param int                         $stackPtr  The position of the current token in
-     *                                               the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $fileName = $phpcsFile->getFilename();
-        $matches  = [];
-        if (preg_match('|/systems/(.*)/([^/]+)?actions.inc$|i', $fileName, $matches) === 0) {
-            // Not an actions file.
-            return;
-        }
-
-        $ownClass = $matches[2];
-        $tokens   = $phpcsFile->getTokens();
-
-        $typeName = $phpcsFile->findNext(T_CONSTANT_ENCAPSED_STRING, ($stackPtr + 2), null, false, true);
-        $typeName = trim($tokens[$typeName]['content'], " '");
-        switch (strtolower($tokens[($stackPtr + 1)]['content'])) {
-        case 'includesystem' :
-            $included = strtolower($typeName);
-            break;
-        case 'includeasset' :
-            $included = strtolower($typeName).'assettype';
-            break;
-        case 'includewidget' :
-            $included = strtolower($typeName).'widgettype';
-            break;
-        default:
-            return;
-        }
-
-        if ($included === strtolower($ownClass)) {
-            $error = "You do not need to include \"%s\" from within the system's own actions file";
-            $data  = [$ownClass];
-            $phpcsFile->addError($error, $stackPtr, 'NotRequired', $data);
-        }
-
-    }//end process()
-
-
-    /**
-     * Determines the included class name from given token.
-     *
-     * @param \PHP_CodeSniffer\Files\File $phpcsFile The file where this token was found.
-     * @param array                       $tokens    The array of file tokens.
-     * @param int                         $stackPtr  The position in the tokens array of the
-     *                                               potentially included class.
-     *
-     * @return string
-     */
-    protected function getIncludedClassFromToken(
-        $phpcsFile,
-        array $tokens,
-        $stackPtr
-    ) {
-
-        return false;
-
-    }//end getIncludedClassFromToken()
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return [T_DOUBLE_COLON];
+
+	}//end register()
+
+
+	/**
+	 * Processes this sniff, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
+	 * @param int                         $stackPtr  The position of the current token in
+	 *                                               the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$fileName = $phpcsFile->getFilename();
+		$matches  = [];
+		if (preg_match('|/systems/(.*)/([^/]+)?actions.inc$|i', $fileName, $matches) === 0) {
+			// Not an actions file.
+			return;
+		}
+
+		$ownClass = $matches[2];
+		$tokens   = $phpcsFile->getTokens();
+
+		$typeName = $phpcsFile->findNext(T_CONSTANT_ENCAPSED_STRING, ($stackPtr + 2), null, false, true);
+		$typeName = trim($tokens[$typeName]['content'], " '");
+		switch (strtolower($tokens[($stackPtr + 1)]['content'])) {
+		case 'includesystem' :
+			$included = strtolower($typeName);
+			break;
+		case 'includeasset' :
+			$included = strtolower($typeName).'assettype';
+			break;
+		case 'includewidget' :
+			$included = strtolower($typeName).'widgettype';
+			break;
+		default:
+			return;
+		}
+
+		if ($included === strtolower($ownClass)) {
+			$error = "You do not need to include \"%s\" from within the system's own actions file";
+			$data  = [$ownClass];
+			$phpcsFile->addError($error, $stackPtr, 'NotRequired', $data);
+		}
+
+	}//end process()
+
+
+	/**
+	 * Determines the included class name from given token.
+	 *
+	 * @param \PHP_CodeSniffer\Files\File $phpcsFile The file where this token was found.
+	 * @param array                       $tokens    The array of file tokens.
+	 * @param int                         $stackPtr  The position in the tokens array of the
+	 *                                               potentially included class.
+	 *
+	 * @return string
+	 */
+	protected function getIncludedClassFromToken(
+		$phpcsFile,
+		array $tokens,
+		$stackPtr
+	) {
+
+		return false;
+
+	}//end getIncludedClassFromToken()
 
 
 }//end class

Switch Indentation +11 added lines, -11 removed lines

@@ -52,17 +52,17 @@
         $typeName = $phpcsFile->findNext(T_CONSTANT_ENCAPSED_STRING, ($stackPtr + 2), null, false, true);
         $typeName = trim($tokens[$typeName]['content'], " '");
         switch (strtolower($tokens[($stackPtr + 1)]['content'])) {
-        case 'includesystem' :
-            $included = strtolower($typeName);
-            break;
-        case 'includeasset' :
-            $included = strtolower($typeName).'assettype';
-            break;
-        case 'includewidget' :
-            $included = strtolower($typeName).'widgettype';
-            break;
-        default:
-            return;
+        	case 'includesystem' :
+            	$included = strtolower($typeName);
+            	break;
+        	case 'includeasset' :
+            	$included = strtolower($typeName).'assettype';
+            	break;
+        	case 'includewidget' :
+            	$included = strtolower($typeName).'widgettype';
+            	break;
+        	default:
+            	return;
         }
 
         if ($included === strtolower($ownClass)) {

Spacing +14 added lines, -14 removed lines

@@ -23,7 +23,7 @@ 
      */
     public function register()
     {
-        return [T_DOUBLE_COLON];
+        return [ T_DOUBLE_COLON ];
 
     }//end register()
 
@@ -37,38 +37,38 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
         $fileName = $phpcsFile->getFilename();
-        $matches  = [];
-        if (preg_match('|/systems/(.*)/([^/]+)?actions.inc$|i', $fileName, $matches) === 0) {
+        $matches  = [ ];
+        if ( preg_match( '|/systems/(.*)/([^/]+)?actions.inc$|i', $fileName, $matches ) === 0 ) {
             // Not an actions file.
             return;
         }
 
-        $ownClass = $matches[2];
+        $ownClass = $matches[ 2 ];
         $tokens   = $phpcsFile->getTokens();
 
-        $typeName = $phpcsFile->findNext(T_CONSTANT_ENCAPSED_STRING, ($stackPtr + 2), null, false, true);
-        $typeName = trim($tokens[$typeName]['content'], " '");
-        switch (strtolower($tokens[($stackPtr + 1)]['content'])) {
+        $typeName = $phpcsFile->findNext( T_CONSTANT_ENCAPSED_STRING, ( $stackPtr + 2 ), null, false, true );
+        $typeName = trim( $tokens[ $typeName ][ 'content' ], " '" );
+        switch ( strtolower( $tokens[ ( $stackPtr + 1 ) ][ 'content' ] ) ) {
         case 'includesystem' :
-            $included = strtolower($typeName);
+            $included = strtolower( $typeName );
             break;
         case 'includeasset' :
-            $included = strtolower($typeName).'assettype';
+            $included = strtolower( $typeName ) . 'assettype';
             break;
         case 'includewidget' :
-            $included = strtolower($typeName).'widgettype';
+            $included = strtolower( $typeName ) . 'widgettype';
             break;
         default:
             return;
         }
 
-        if ($included === strtolower($ownClass)) {
+        if ( $included === strtolower( $ownClass ) ) {
             $error = "You do not need to include \"%s\" from within the system's own actions file";
-            $data  = [$ownClass];
-            $phpcsFile->addError($error, $stackPtr, 'NotRequired', $data);
+            $data  = [ $ownClass ];
+            $phpcsFile->addError( $error, $stackPtr, 'NotRequired', $data );
         }
 
     }//end process()

Braces +3 added lines, -6 removed lines

@@ -12,8 +12,7 @@ 
 use PHP_CodeSniffer\Sniffs\Sniff;
 use PHP_CodeSniffer\Files\File;
 
-class IncludeOwnSystemSniff implements Sniff
-{
+class IncludeOwnSystemSniff implements Sniff {
 
 
     /**
@@ -21,8 +20,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return [T_DOUBLE_COLON];
 
     }//end register()
@@ -37,8 +35,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         $fileName = $phpcsFile->getFilename();
         $matches  = [];
         if (preg_match('|/systems/(.*)/([^/]+)?actions.inc$|i', $fileName, $matches) === 0) {