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-27 16:19 UTC

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

@@ -15,34 +15,34 @@
 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 [1 => 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 [1 => 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.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -25,7 +25,7 @@  discard block
 block discarded – undo
      */
     public function getErrorList()
     {
-        return [1 => 1];
+        return [ 1 => 1 ];
 
     }//end getErrorList()
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
      */
     public function getWarningList()
     {
-        return [];
+        return [ ];
 
     }//end getWarningList()
 

Please login to merge, or discard this patch.

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

@@ -11,8 +11,7 @@  discard block
 block discarded – undo
 
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 
-class ByteOrderMarkUnitTest extends AbstractSniffUnitTest
-{
+class ByteOrderMarkUnitTest extends AbstractSniffUnitTest {
 
 
     /**
@@ -23,8 +22,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList()
-    {
+    public function getErrorList() {
         return [1 => 1];
 
     }//end getErrorList()
@@ -38,8 +36,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList()
-    {
+    public function getWarningList() {
         return [];
 
     }//end getWarningList()

Please login to merge, or discard this patch.

php_codesniffer/src/Standards/Generic/Tests/Files/InlineHTMLUnitTest.php 4 patches

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

@@ -15,51 +15,51 @@
 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='')
-    {
-        switch ($testFile) {
-        case 'InlineHTMLUnitTest.3.inc':
-            return [4 => 1];
-            break;
-        case 'InlineHTMLUnitTest.4.inc':
-            return [1 => 1];
-            break;
-        case 'InlineHTMLUnitTest.7.inc':
-            return [1 => 1];
-            break;
-        default:
-            return [];
-            break;
-        }//end switch
+	/**
+	 * 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='')
+	{
+		switch ($testFile) {
+		case 'InlineHTMLUnitTest.3.inc':
+			return [4 => 1];
+			break;
+		case 'InlineHTMLUnitTest.4.inc':
+			return [1 => 1];
+			break;
+		case 'InlineHTMLUnitTest.7.inc':
+			return [1 => 1];
+			break;
+		default:
+			return [];
+			break;
+		}//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.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList($testFile='')
-    {
-        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.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList($testFile='')
+	{
+		return [];
 
-    }//end getWarningList()
+	}//end getWarningList()
 
 
 }//end class

Please login to merge, or discard this patch.

Switch Indentation +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -28,18 +28,18 @@
 block discarded – undo
     public function getErrorList($testFile='')
     {
         switch ($testFile) {
-        case 'InlineHTMLUnitTest.3.inc':
-            return [4 => 1];
-            break;
-        case 'InlineHTMLUnitTest.4.inc':
-            return [1 => 1];
-            break;
-        case 'InlineHTMLUnitTest.7.inc':
-            return [1 => 1];
-            break;
-        default:
-            return [];
-            break;
+        	case 'InlineHTMLUnitTest.3.inc':
+            	return [4 => 1];
+            	break;
+        	case 'InlineHTMLUnitTest.4.inc':
+            	return [1 => 1];
+            	break;
+        	case 'InlineHTMLUnitTest.7.inc':
+            	return [1 => 1];
+            	break;
+        	default:
+            	return [];
+            	break;
         }//end switch
 
     }//end getErrorList()

Please login to merge, or discard this patch.

Spacing +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -25,20 +25,20 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='')
+    public function getErrorList( $testFile = '' )
     {
-        switch ($testFile) {
+        switch ( $testFile ) {
         case 'InlineHTMLUnitTest.3.inc':
-            return [4 => 1];
+            return [ 4 => 1 ];
             break;
         case 'InlineHTMLUnitTest.4.inc':
-            return [1 => 1];
+            return [ 1 => 1 ];
             break;
         case 'InlineHTMLUnitTest.7.inc':
-            return [1 => 1];
+            return [ 1 => 1 ];
             break;
         default:
-            return [];
+            return [ ];
             break;
         }//end switch
 
@@ -55,9 +55,9 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='')
+    public function getWarningList( $testFile = '' )
     {
-        return [];
+        return [ ];
 
     }//end getWarningList()
 

Please login to merge, or discard this patch.

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

@@ -11,8 +11,7 @@  discard block
 block discarded – undo
 
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 
-class InlineHTMLUnitTest extends AbstractSniffUnitTest
-{
+class InlineHTMLUnitTest extends AbstractSniffUnitTest {
 
 
     /**
@@ -25,8 +24,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='')
-    {
+    public function getErrorList($testFile='') {
         switch ($testFile) {
         case 'InlineHTMLUnitTest.3.inc':
             return [4 => 1];
@@ -55,8 +53,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='')
-    {
+    public function getWarningList($testFile='') {
         return [];
 
     }//end getWarningList()

Please login to merge, or discard this patch.

src/Standards/Generic/Tests/Files/LowercasedFilenameUnitTest.php 3 patches

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

@@ -11,8 +11,7 @@  discard block
 block discarded – undo
 
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 
-class LowercasedFilenameUnitTest extends AbstractSniffUnitTest
-{
+class LowercasedFilenameUnitTest extends AbstractSniffUnitTest {
 
 
     /**
@@ -23,8 +22,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList()
-    {
+    public function getErrorList() {
         return [1 => 1];
 
     }//end getErrorList()
@@ -38,8 +36,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList()
-    {
+    public function getWarningList() {
         return [];
 
     }//end getWarningList()

Please login to merge, or discard this patch.

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

@@ -15,34 +15,34 @@
 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 [1 => 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 [1 => 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.

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -25,7 +25,7 @@  discard block
 block discarded – undo
      */
     public function getErrorList()
     {
-        return [1 => 1];
+        return [ 1 => 1 ];
 
     }//end getErrorList()
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
      */
     public function getWarningList()
     {
-        return [];
+        return [ ];
 
     }//end getWarningList()
 

Please login to merge, or discard this patch.

src/Standards/Generic/Tests/Files/OneClassPerFileUnitTest.php 3 patches

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

@@ -11,8 +11,7 @@  discard block
 block discarded – undo
 
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 
-class OneClassPerFileUnitTest extends AbstractSniffUnitTest
-{
+class OneClassPerFileUnitTest extends AbstractSniffUnitTest {
 
 
     /**
@@ -23,8 +22,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList()
-    {
+    public function getErrorList() {
         return [
             6  => 1,
             10 => 1,
@@ -41,8 +39,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList()
-    {
+    public function getWarningList() {
         return [];
 
     }//end getWarningList()

Please login to merge, or discard this patch.

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

@@ -15,37 +15,37 @@
 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,
-            10 => 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,
+			10 => 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.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -64,7 +64,7 @@
 block discarded – undo
      */
     public function getWarningList()
     {
-        return [];
+        return [ ];
 
     }//end getWarningList()
 

Please login to merge, or discard this patch.

src/Standards/Generic/Tests/Files/EndFileNoNewlineUnitTest.6.inc 5 patches

Indentation -1 removed lines patch added patch discarded remove patch

		@@ -1,2 +1,1 @@
		block discarded – undo
1	1	<?php
2		-/** Why me?

Please login to merge, or discard this patch.

Switch Indentation -1 removed lines patch added patch discarded remove patch

		@@ -1,2 +1,1 @@
		block discarded – undo
1	1	<?php
2		-/** Why me?

Please login to merge, or discard this patch.

Spacing -1 removed lines patch added patch discarded remove patch

		@@ -1,2 +1,1 @@
		block discarded – undo
1	1	<?php
2		-/** Why me?

Please login to merge, or discard this patch.

Braces -1 removed lines patch added patch discarded remove patch

		@@ -1,2 +1,1 @@
		block discarded – undo
1	1	<?php
2		-/** Why me?

Please login to merge, or discard this patch.

Upper-Lower-Casing -1 removed lines patch added patch discarded remove patch

		@@ -1,2 +1,1 @@
		block discarded – undo
1	1	<?php
2		-/** Why me?

Please login to merge, or discard this patch.

src/Standards/Generic/Tests/Files/EndFileNoNewlineUnitTest.7.inc 1 patch

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

@@ -3,4 +3,4 @@
 block discarded – undo
 // Testing that the fixer does not remove superfluous whitespace (not the concern of this sniff).
 echo 'foo';
 
-         
\ No newline at end of file
+		 
\ No newline at end of file

Please login to merge, or discard this patch.

php_codesniffer/src/Standards/Generic/Tests/Files/LineLengthUnitTest.php 4 patches

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

@@ -15,93 +15,93 @@
 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.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList($testFile='')
-    {
-        switch ($testFile) {
-        case 'LineLengthUnitTest.1.inc':
-            return [
-                31 => 1,
-                34 => 1,
-                45 => 1,
-                82 => 1,
-            ];
-            break;
-        case 'LineLengthUnitTest.2.inc':
-        case 'LineLengthUnitTest.3.inc':
-            return [7 => 1];
-            break;
-        default:
-            return [];
-            break;
-        }//end switch
+	/**
+	 * 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='')
+	{
+		switch ($testFile) {
+		case 'LineLengthUnitTest.1.inc':
+			return [
+				31 => 1,
+				34 => 1,
+				45 => 1,
+				82 => 1,
+			];
+			break;
+		case 'LineLengthUnitTest.2.inc':
+		case 'LineLengthUnitTest.3.inc':
+			return [7 => 1];
+			break;
+		default:
+			return [];
+			break;
+		}//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.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList($testFile='')
-    {
-        switch ($testFile) {
-        case 'LineLengthUnitTest.1.inc':
-            return [
-                9  => 1,
-                15 => 1,
-                21 => 1,
-                24 => 1,
-                29 => 1,
-                37 => 1,
-                63 => 1,
-                73 => 1,
-                75 => 1,
-            ];
-            break;
-        case 'LineLengthUnitTest.2.inc':
-        case 'LineLengthUnitTest.3.inc':
-            return [6 => 1];
-            break;
-        case 'LineLengthUnitTest.4.inc':
-            return [10 => 1];
-            break;
-        default:
-            return [];
-            break;
-        }//end switch
+	/**
+	 * 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='')
+	{
+		switch ($testFile) {
+		case 'LineLengthUnitTest.1.inc':
+			return [
+				9  => 1,
+				15 => 1,
+				21 => 1,
+				24 => 1,
+				29 => 1,
+				37 => 1,
+				63 => 1,
+				73 => 1,
+				75 => 1,
+			];
+			break;
+		case 'LineLengthUnitTest.2.inc':
+		case 'LineLengthUnitTest.3.inc':
+			return [6 => 1];
+			break;
+		case 'LineLengthUnitTest.4.inc':
+			return [10 => 1];
+			break;
+		default:
+			return [];
+			break;
+		}//end switch
 
-    }//end getWarningList()
+	}//end getWarningList()
 
 
 }//end class

Please login to merge, or discard this patch.

Switch Indentation +38 added lines, -38 removed lines patch added patch discarded remove patch

@@ -43,21 +43,21 @@  discard block
 block discarded – undo
     public function getErrorList($testFile='')
     {
         switch ($testFile) {
-        case 'LineLengthUnitTest.1.inc':
-            return [
-                31 => 1,
-                34 => 1,
-                45 => 1,
-                82 => 1,
-            ];
-            break;
-        case 'LineLengthUnitTest.2.inc':
-        case 'LineLengthUnitTest.3.inc':
-            return [7 => 1];
-            break;
-        default:
-            return [];
-            break;
+        	case 'LineLengthUnitTest.1.inc':
+            	return [
+                	31 => 1,
+                	34 => 1,
+                	45 => 1,
+                	82 => 1,
+            	];
+            	break;
+        	case 'LineLengthUnitTest.2.inc':
+        	case 'LineLengthUnitTest.3.inc':
+            	return [7 => 1];
+            	break;
+        	default:
+            	return [];
+            	break;
         }//end switch
 
     }//end getErrorList()
@@ -76,29 +76,29 @@  discard block
 block discarded – undo
     public function getWarningList($testFile='')
     {
         switch ($testFile) {
-        case 'LineLengthUnitTest.1.inc':
-            return [
-                9  => 1,
-                15 => 1,
-                21 => 1,
-                24 => 1,
-                29 => 1,
-                37 => 1,
-                63 => 1,
-                73 => 1,
-                75 => 1,
-            ];
-            break;
-        case 'LineLengthUnitTest.2.inc':
-        case 'LineLengthUnitTest.3.inc':
-            return [6 => 1];
-            break;
-        case 'LineLengthUnitTest.4.inc':
-            return [10 => 1];
-            break;
-        default:
-            return [];
-            break;
+        	case 'LineLengthUnitTest.1.inc':
+            	return [
+                	9  => 1,
+                	15 => 1,
+                	21 => 1,
+                	24 => 1,
+                	29 => 1,
+                	37 => 1,
+                	63 => 1,
+                	73 => 1,
+                	75 => 1,
+            	];
+            	break;
+        	case 'LineLengthUnitTest.2.inc':
+        	case 'LineLengthUnitTest.3.inc':
+            	return [6 => 1];
+            	break;
+        	case 'LineLengthUnitTest.4.inc':
+            	return [10 => 1];
+            	break;
+        	default:
+            	return [];
+            	break;
         }//end switch
 
     }//end getWarningList()

Please login to merge, or discard this patch.

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -23,7 +23,7 @@  discard block
 block discarded – undo
      *
      * @return void
      */
-    public function setCliValues($testFile, $config)
+    public function setCliValues( $testFile, $config )
     {
         $config->tabWidth = 4;
 
@@ -40,9 +40,9 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='')
+    public function getErrorList( $testFile = '' )
     {
-        switch ($testFile) {
+        switch ( $testFile ) {
         case 'LineLengthUnitTest.1.inc':
             return [
                 31 => 1,
@@ -53,10 +53,10 @@  discard block
 block discarded – undo
             break;
         case 'LineLengthUnitTest.2.inc':
         case 'LineLengthUnitTest.3.inc':
-            return [7 => 1];
+            return [ 7 => 1 ];
             break;
         default:
-            return [];
+            return [ ];
             break;
         }//end switch
 
@@ -73,9 +73,9 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='')
+    public function getWarningList( $testFile = '' )
     {
-        switch ($testFile) {
+        switch ( $testFile ) {
         case 'LineLengthUnitTest.1.inc':
             return [
                 9  => 1,
@@ -91,13 +91,13 @@  discard block
 block discarded – undo
             break;
         case 'LineLengthUnitTest.2.inc':
         case 'LineLengthUnitTest.3.inc':
-            return [6 => 1];
+            return [ 6 => 1 ];
             break;
         case 'LineLengthUnitTest.4.inc':
-            return [10 => 1];
+            return [ 10 => 1 ];
             break;
         default:
-            return [];
+            return [ ];
             break;
         }//end switch
 

Please login to merge, or discard this patch.

Braces +4 added lines, -8 removed lines patch added patch discarded remove patch

@@ -11,8 +11,7 @@  discard block
 block discarded – undo
 
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 
-class LineLengthUnitTest extends AbstractSniffUnitTest
-{
+class LineLengthUnitTest extends AbstractSniffUnitTest {
 
 
     /**
@@ -23,8 +22,7 @@  discard block
 block discarded – undo
      *
      * @return void
      */
-    public function setCliValues($testFile, $config)
-    {
+    public function setCliValues($testFile, $config) {
         $config->tabWidth = 4;
 
     }//end setCliValues()
@@ -40,8 +38,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='')
-    {
+    public function getErrorList($testFile='') {
         switch ($testFile) {
         case 'LineLengthUnitTest.1.inc':
             return [
@@ -73,8 +70,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='')
-    {
+    public function getWarningList($testFile='') {
         switch ($testFile) {
         case 'LineLengthUnitTest.1.inc':
             return [

Please login to merge, or discard this patch.

src/Standards/Generic/Tests/Files/OneObjectStructurePerFileUnitTest.php 3 patches

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

@@ -15,39 +15,39 @@
 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,
-            10 => 1,
-            14 => 1,
-            18 => 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,
+			10 => 1,
+			14 => 1,
+			18 => 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.

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

@@ -11,8 +11,7 @@  discard block
 block discarded – undo
 
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 
-class OneObjectStructurePerFileUnitTest extends AbstractSniffUnitTest
-{
+class OneObjectStructurePerFileUnitTest extends AbstractSniffUnitTest {
 
 
     /**
@@ -23,8 +22,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList()
-    {
+    public function getErrorList() {
         return [
             6  => 1,
             10 => 1,
@@ -43,8 +41,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList()
-    {
+    public function getWarningList() {
         return [];
 
     }//end getWarningList()

Please login to merge, or discard this patch.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -64,7 +64,7 @@
 block discarded – undo
      */
     public function getWarningList()
     {
-        return [];
+        return [ ];
 
     }//end getWarningList()
 

Please login to merge, or discard this patch.

php_codesniffer/src/Standards/Generic/Tests/Files/LineEndingsUnitTest.php 3 patches

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

@@ -15,49 +15,49 @@
 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;
-
-    }//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 [1 => 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()
+	/**
+	 * 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()
+
+
+	/**
+	 * 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 [1 => 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.

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -23,7 +23,7 @@  discard block
 block discarded – undo
      *
      * @return void
      */
-    public function setCliValues($testFile, $config)
+    public function setCliValues( $testFile, $config )
     {
         $config->tabWidth = 4;
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
      */
     public function getErrorList()
     {
-        return [1 => 1];
+        return [ 1 => 1 ];
 
     }//end getErrorList()
 
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
      */
     public function getWarningList()
     {
-        return [];
+        return [ ];
 
     }//end getWarningList()
 

Please login to merge, or discard this patch.

Braces +4 added lines, -8 removed lines patch added patch discarded remove patch

@@ -11,8 +11,7 @@  discard block
 block discarded – undo
 
 use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
 
-class LineEndingsUnitTest extends AbstractSniffUnitTest
-{
+class LineEndingsUnitTest extends AbstractSniffUnitTest {
 
 
     /**
@@ -23,8 +22,7 @@  discard block
 block discarded – undo
      *
      * @return void
      */
-    public function setCliValues($testFile, $config)
-    {
+    public function setCliValues($testFile, $config) {
         $config->tabWidth = 4;
 
     }//end setCliValues()
@@ -38,8 +36,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getErrorList()
-    {
+    public function getErrorList() {
         return [1 => 1];
 
     }//end getErrorList()
@@ -53,8 +50,7 @@  discard block
 block discarded – undo
      *
      * @return array<int, int>
      */
-    public function getWarningList()
-    {
+    public function getWarningList() {
         return [];
 
     }//end getWarningList()

Please login to merge, or discard this patch.

		@@ -15,34 +15,34 @@
		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 [1 => 1];
29		-
30		- }//end getErrorList()
31		-
32		-
33		- /**
34		- * Returns the lines where warnings should occur.
35		- *
36		- * The key of the array should represent the line number and the value
37		- * should represent the number of warnings that should occur on that line.
38		- *
39		- * @return array<int, int>
40		- */
41		- public function getWarningList()
42		- {
43		- return [];
44		-
45		- }//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 [1 => 1];
	29	+
	30	+ }//end getErrorList()
	31	+
	32	+
	33	+ /**
	34	+ * Returns the lines where warnings should occur.
	35	+ *
	36	+ * The key of the array should represent the line number and the value
	37	+ * should represent the number of warnings that should occur on that line.
	38	+ *
	39	+ * @return array<int, int>
	40	+ */
	41	+ public function getWarningList()
	42	+ {
	43	+ return [];
	44	+
	45	+ }//end getWarningList()
46	46
47	47
48	48	}//end class

		@@ -25,7 +25,7 @@ discard block
		block discarded – undo
25	25	*/
26	26	public function getErrorList()
27	27	{
28		- return [1 => 1];
	28	+ return [ 1 => 1 ];
29	29
30	30	}//end getErrorList()
31	31
		@@ -40,7 +40,7 @@ discard block
		block discarded – undo
40	40	*/
41	41	public function getWarningList()
42	42	{
43		- return [];
	43	+ return [ ];
44	44
45	45	}//end getWarningList()
46	46

		@@ -11,8 +11,7 @@ discard block
		block discarded – undo
11	11
12	12	use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
13	13
14		-class ByteOrderMarkUnitTest extends AbstractSniffUnitTest
15		-{
	14	+class ByteOrderMarkUnitTest extends AbstractSniffUnitTest {
16	15
17	16
18	17	/**
		@@ -23,8 +22,7 @@ discard block
		block discarded – undo
23	22	*
24	23	* @return array<int, int>
25	24	*/
26		- public function getErrorList()
27		- {
	25	+ public function getErrorList() {
28	26	return [1 => 1];
29	27
30	28	}//end getErrorList()
		@@ -38,8 +36,7 @@ discard block
		block discarded – undo
38	36	*
39	37	* @return array<int, int>
40	38	*/
41		- public function getWarningList()
42		- {
	39	+ public function getWarningList() {
43	40	return [];
44	41
45	42	}//end getWarningList()

		@@ -15,51 +15,51 @@
		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='')
29		- {
30		- switch ($testFile) {
31		- case 'InlineHTMLUnitTest.3.inc':
32		- return [4 => 1];
33		- break;
34		- case 'InlineHTMLUnitTest.4.inc':
35		- return [1 => 1];
36		- break;
37		- case 'InlineHTMLUnitTest.7.inc':
38		- return [1 => 1];
39		- break;
40		- default:
41		- return [];
42		- break;
43		- }//end switch
	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='')
	29	+ {
	30	+ switch ($testFile) {
	31	+ case 'InlineHTMLUnitTest.3.inc':
	32	+ return [4 => 1];
	33	+ break;
	34	+ case 'InlineHTMLUnitTest.4.inc':
	35	+ return [1 => 1];
	36	+ break;
	37	+ case 'InlineHTMLUnitTest.7.inc':
	38	+ return [1 => 1];
	39	+ break;
	40	+ default:
	41	+ return [];
	42	+ break;
	43	+ }//end switch
44	44
45		- }//end getErrorList()
	45	+ }//end getErrorList()
46	46
47	47
48		- /**
49		- * Returns the lines where warnings should occur.
50		- *
51		- * The key of the array should represent the line number and the value
52		- * should represent the number of warnings that should occur on that line.
53		- *
54		- * @param string $testFile The name of the file being tested.
55		- *
56		- * @return array<int, int>
57		- */
58		- public function getWarningList($testFile='')
59		- {
60		- return [];
	48	+ /**
	49	+ * Returns the lines where warnings should occur.
	50	+ *
	51	+ * The key of the array should represent the line number and the value
	52	+ * should represent the number of warnings that should occur on that line.
	53	+ *
	54	+ * @param string $testFile The name of the file being tested.
	55	+ *
	56	+ * @return array<int, int>
	57	+ */
	58	+ public function getWarningList($testFile='')
	59	+ {
	60	+ return [];
61	61
62		- }//end getWarningList()
	62	+ }//end getWarningList()
63	63
64	64
65	65	}//end class

		@@ -28,18 +28,18 @@
		block discarded – undo
28	28	public function getErrorList($testFile='')
29	29	{
30	30	switch ($testFile) {
31		- case 'InlineHTMLUnitTest.3.inc':
32		- return [4 => 1];
33		- break;
34		- case 'InlineHTMLUnitTest.4.inc':
35		- return [1 => 1];
36		- break;
37		- case 'InlineHTMLUnitTest.7.inc':
38		- return [1 => 1];
39		- break;
40		- default:
41		- return [];
42		- break;
	31	+ case 'InlineHTMLUnitTest.3.inc':
	32	+ return [4 => 1];
	33	+ break;
	34	+ case 'InlineHTMLUnitTest.4.inc':
	35	+ return [1 => 1];
	36	+ break;
	37	+ case 'InlineHTMLUnitTest.7.inc':
	38	+ return [1 => 1];
	39	+ break;
	40	+ default:
	41	+ return [];
	42	+ break;
43	43	}//end switch
44	44
45	45	}//end getErrorList()

		@@ -25,20 +25,20 @@ discard block
		block discarded – undo
25	25	*
26	26	* @return array<int, int>
27	27	*/
28		- public function getErrorList($testFile='')
	28	+ public function getErrorList( $testFile = '' )
29	29	{
30		- switch ($testFile) {
	30	+ switch ( $testFile ) {
31	31	case 'InlineHTMLUnitTest.3.inc':
32		- return [4 => 1];
	32	+ return [ 4 => 1 ];
33	33	break;
34	34	case 'InlineHTMLUnitTest.4.inc':
35		- return [1 => 1];
	35	+ return [ 1 => 1 ];
36	36	break;
37	37	case 'InlineHTMLUnitTest.7.inc':
38		- return [1 => 1];
	38	+ return [ 1 => 1 ];
39	39	break;
40	40	default:
41		- return [];
	41	+ return [ ];
42	42	break;
43	43	}//end switch
44	44
		@@ -55,9 +55,9 @@ discard block
		block discarded – undo
55	55	*
56	56	* @return array<int, int>
57	57	*/
58		- public function getWarningList($testFile='')
	58	+ public function getWarningList( $testFile = '' )
59	59	{
60		- return [];
	60	+ return [ ];
61	61
62	62	}//end getWarningList()
63	63

		@@ -15,37 +15,37 @@
		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		- 10 => 1,
31		- ];
32		-
33		- }//end getErrorList()
34		-
35		-
36		- /**
37		- * Returns the lines where warnings should occur.
38		- *
39		- * The key of the array should represent the line number and the value
40		- * should represent the number of warnings that should occur on that line.
41		- *
42		- * @return array<int, int>
43		- */
44		- public function getWarningList()
45		- {
46		- return [];
47		-
48		- }//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	+ 10 => 1,
	31	+ ];
	32	+
	33	+ }//end getErrorList()
	34	+
	35	+
	36	+ /**
	37	+ * Returns the lines where warnings should occur.
	38	+ *
	39	+ * The key of the array should represent the line number and the value
	40	+ * should represent the number of warnings that should occur on that line.
	41	+ *
	42	+ * @return array<int, int>
	43	+ */
	44	+ public function getWarningList()
	45	+ {
	46	+ return [];
	47	+
	48	+ }//end getWarningList()
49	49
50	50
51	51	}//end class

		@@ -64,7 +64,7 @@
		block discarded – undo
64	64	*/
65	65	public function getWarningList()
66	66	{
67		- return [];
	67	+ return [ ];
68	68
69	69	}//end getWarningList()
70	70

		@@ -3,4 +3,4 @@
		block discarded – undo
3	3	// Testing that the fixer does not remove superfluous whitespace (not the concern of this sniff).
4	4	echo 'foo';
5	5
6		-
7	6	\ No newline at end of file
	7	+
8	8	\ No newline at end of file

		@@ -15,93 +15,93 @@
		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		- * @param string $testFile The name of the file being tested.
40		- *
41		- * @return array<int, int>
42		- */
43		- public function getErrorList($testFile='')
44		- {
45		- switch ($testFile) {
46		- case 'LineLengthUnitTest.1.inc':
47		- return [
48		- 31 => 1,
49		- 34 => 1,
50		- 45 => 1,
51		- 82 => 1,
52		- ];
53		- break;
54		- case 'LineLengthUnitTest.2.inc':
55		- case 'LineLengthUnitTest.3.inc':
56		- return [7 => 1];
57		- break;
58		- default:
59		- return [];
60		- break;
61		- }//end switch
	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	+ * @param string $testFile The name of the file being tested.
	40	+ *
	41	+ * @return array<int, int>
	42	+ */
	43	+ public function getErrorList($testFile='')
	44	+ {
	45	+ switch ($testFile) {
	46	+ case 'LineLengthUnitTest.1.inc':
	47	+ return [
	48	+ 31 => 1,
	49	+ 34 => 1,
	50	+ 45 => 1,
	51	+ 82 => 1,
	52	+ ];
	53	+ break;
	54	+ case 'LineLengthUnitTest.2.inc':
	55	+ case 'LineLengthUnitTest.3.inc':
	56	+ return [7 => 1];
	57	+ break;
	58	+ default:
	59	+ return [];
	60	+ break;
	61	+ }//end switch
62	62
63		- }//end getErrorList()
	63	+ }//end getErrorList()
64	64
65	65
66		- /**
67		- * Returns the lines where warnings should occur.
68		- *
69		- * The key of the array should represent the line number and the value
70		- * should represent the number of warnings that should occur on that line.
71		- *
72		- * @param string $testFile The name of the file being tested.
73		- *
74		- * @return array<int, int>
75		- */
76		- public function getWarningList($testFile='')
77		- {
78		- switch ($testFile) {
79		- case 'LineLengthUnitTest.1.inc':
80		- return [
81		- 9 => 1,
82		- 15 => 1,
83		- 21 => 1,
84		- 24 => 1,
85		- 29 => 1,
86		- 37 => 1,
87		- 63 => 1,
88		- 73 => 1,
89		- 75 => 1,
90		- ];
91		- break;
92		- case 'LineLengthUnitTest.2.inc':
93		- case 'LineLengthUnitTest.3.inc':
94		- return [6 => 1];
95		- break;
96		- case 'LineLengthUnitTest.4.inc':
97		- return [10 => 1];
98		- break;
99		- default:
100		- return [];
101		- break;
102		- }//end switch
	66	+ /**
	67	+ * Returns the lines where warnings should occur.
	68	+ *
	69	+ * The key of the array should represent the line number and the value
	70	+ * should represent the number of warnings that should occur on that line.
	71	+ *
	72	+ * @param string $testFile The name of the file being tested.
	73	+ *
	74	+ * @return array<int, int>
	75	+ */
	76	+ public function getWarningList($testFile='')
	77	+ {
	78	+ switch ($testFile) {
	79	+ case 'LineLengthUnitTest.1.inc':
	80	+ return [
	81	+ 9 => 1,
	82	+ 15 => 1,
	83	+ 21 => 1,
	84	+ 24 => 1,
	85	+ 29 => 1,
	86	+ 37 => 1,
	87	+ 63 => 1,
	88	+ 73 => 1,
	89	+ 75 => 1,
	90	+ ];
	91	+ break;
	92	+ case 'LineLengthUnitTest.2.inc':
	93	+ case 'LineLengthUnitTest.3.inc':
	94	+ return [6 => 1];
	95	+ break;
	96	+ case 'LineLengthUnitTest.4.inc':
	97	+ return [10 => 1];
	98	+ break;
	99	+ default:
	100	+ return [];
	101	+ break;
	102	+ }//end switch
103	103
104		- }//end getWarningList()
	104	+ }//end getWarningList()
105	105
106	106
107	107	}//end class

		@@ -43,21 +43,21 @@ discard block
		block discarded – undo
43	43	public function getErrorList($testFile='')
44	44	{
45	45	switch ($testFile) {
46		- case 'LineLengthUnitTest.1.inc':
47		- return [
48		- 31 => 1,
49		- 34 => 1,
50		- 45 => 1,
51		- 82 => 1,
52		- ];
53		- break;
54		- case 'LineLengthUnitTest.2.inc':
55		- case 'LineLengthUnitTest.3.inc':
56		- return [7 => 1];
57		- break;
58		- default:
59		- return [];
60		- break;
	46	+ case 'LineLengthUnitTest.1.inc':
	47	+ return [
	48	+ 31 => 1,
	49	+ 34 => 1,
	50	+ 45 => 1,
	51	+ 82 => 1,
	52	+ ];
	53	+ break;
	54	+ case 'LineLengthUnitTest.2.inc':
	55	+ case 'LineLengthUnitTest.3.inc':
	56	+ return [7 => 1];
	57	+ break;
	58	+ default:
	59	+ return [];
	60	+ break;
61	61	}//end switch
62	62
63	63	}//end getErrorList()
		@@ -76,29 +76,29 @@ discard block
		block discarded – undo
76	76	public function getWarningList($testFile='')
77	77	{
78	78	switch ($testFile) {
79		- case 'LineLengthUnitTest.1.inc':
80		- return [
81		- 9 => 1,
82		- 15 => 1,
83		- 21 => 1,
84		- 24 => 1,
85		- 29 => 1,
86		- 37 => 1,
87		- 63 => 1,
88		- 73 => 1,
89		- 75 => 1,
90		- ];
91		- break;
92		- case 'LineLengthUnitTest.2.inc':
93		- case 'LineLengthUnitTest.3.inc':
94		- return [6 => 1];
95		- break;
96		- case 'LineLengthUnitTest.4.inc':
97		- return [10 => 1];
98		- break;
99		- default:
100		- return [];
101		- break;
	79	+ case 'LineLengthUnitTest.1.inc':
	80	+ return [
	81	+ 9 => 1,
	82	+ 15 => 1,
	83	+ 21 => 1,
	84	+ 24 => 1,
	85	+ 29 => 1,
	86	+ 37 => 1,
	87	+ 63 => 1,
	88	+ 73 => 1,
	89	+ 75 => 1,
	90	+ ];
	91	+ break;
	92	+ case 'LineLengthUnitTest.2.inc':
	93	+ case 'LineLengthUnitTest.3.inc':
	94	+ return [6 => 1];
	95	+ break;
	96	+ case 'LineLengthUnitTest.4.inc':
	97	+ return [10 => 1];
	98	+ break;
	99	+ default:
	100	+ return [];
	101	+ break;
102	102	}//end switch
103	103
104	104	}//end getWarningList()

		@@ -23,7 +23,7 @@ discard block
		block discarded – undo
23	23	*
24	24	* @return void
25	25	*/
26		- public function setCliValues($testFile, $config)
	26	+ public function setCliValues( $testFile, $config )
27	27	{
28	28	$config->tabWidth = 4;
29	29
		@@ -40,9 +40,9 @@ discard block
		block discarded – undo
40	40	*
41	41	* @return array<int, int>
42	42	*/
43		- public function getErrorList($testFile='')
	43	+ public function getErrorList( $testFile = '' )
44	44	{
45		- switch ($testFile) {
	45	+ switch ( $testFile ) {
46	46	case 'LineLengthUnitTest.1.inc':
47	47	return [
48	48	31 => 1,
		@@ -53,10 +53,10 @@ discard block
		block discarded – undo
53	53	break;
54	54	case 'LineLengthUnitTest.2.inc':
55	55	case 'LineLengthUnitTest.3.inc':
56		- return [7 => 1];
	56	+ return [ 7 => 1 ];
57	57	break;
58	58	default:
59		- return [];
	59	+ return [ ];
60	60	break;
61	61	}//end switch
62	62
		@@ -73,9 +73,9 @@ discard block
		block discarded – undo
73	73	*
74	74	* @return array<int, int>
75	75	*/
76		- public function getWarningList($testFile='')
	76	+ public function getWarningList( $testFile = '' )
77	77	{
78		- switch ($testFile) {
	78	+ switch ( $testFile ) {
79	79	case 'LineLengthUnitTest.1.inc':
80	80	return [
81	81	9 => 1,
		@@ -91,13 +91,13 @@ discard block
		block discarded – undo
91	91	break;
92	92	case 'LineLengthUnitTest.2.inc':
93	93	case 'LineLengthUnitTest.3.inc':
94		- return [6 => 1];
	94	+ return [ 6 => 1 ];
95	95	break;
96	96	case 'LineLengthUnitTest.4.inc':
97		- return [10 => 1];
	97	+ return [ 10 => 1 ];
98	98	break;
99	99	default:
100		- return [];
	100	+ return [ ];
101	101	break;
102	102	}//end switch
103	103

		@@ -15,39 +15,39 @@
		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		- 10 => 1,
31		- 14 => 1,
32		- 18 => 1,
33		- ];
34		-
35		- }//end getErrorList()
36		-
37		-
38		- /**
39		- * Returns the lines where warnings should occur.
40		- *
41		- * The key of the array should represent the line number and the value
42		- * should represent the number of warnings that should occur on that line.
43		- *
44		- * @return array<int, int>
45		- */
46		- public function getWarningList()
47		- {
48		- return [];
49		-
50		- }//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	+ 10 => 1,
	31	+ 14 => 1,
	32	+ 18 => 1,
	33	+ ];
	34	+
	35	+ }//end getErrorList()
	36	+
	37	+
	38	+ /**
	39	+ * Returns the lines where warnings should occur.
	40	+ *
	41	+ * The key of the array should represent the line number and the value
	42	+ * should represent the number of warnings that should occur on that line.
	43	+ *
	44	+ * @return array<int, int>
	45	+ */
	46	+ public function getWarningList()
	47	+ {
	48	+ return [];
	49	+
	50	+ }//end getWarningList()
51	51
52	52
53	53	}//end class

		@@ -15,49 +15,49 @@
		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;
29		-
30		- }//end setCliValues()
31		-
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 [1 => 1];
44		-
45		- }//end getErrorList()
46		-
47		-
48		- /**
49		- * Returns the lines where warnings should occur.
50		- *
51		- * The key of the array should represent the line number and the value
52		- * should represent the number of warnings that should occur on that line.
53		- *
54		- * @return array<int, int>
55		- */
56		- public function getWarningList()
57		- {
58		- return [];
59		-
60		- }//end getWarningList()
	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	+
	30	+ }//end setCliValues()
	31	+
	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 [1 => 1];
	44	+
	45	+ }//end getErrorList()
	46	+
	47	+
	48	+ /**
	49	+ * Returns the lines where warnings should occur.
	50	+ *
	51	+ * The key of the array should represent the line number and the value
	52	+ * should represent the number of warnings that should occur on that line.
	53	+ *
	54	+ * @return array<int, int>
	55	+ */
	56	+ public function getWarningList()
	57	+ {
	58	+ return [];
	59	+
	60	+ }//end getWarningList()
61	61
62	62
63	63	}//end class

gravityview / GravityView

Pull Request — develop (#1492)

Category

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

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

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

Switch Indentation +12 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +8 added lines, -8 removed lines patch added patch discarded remove patch

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

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

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

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

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation -1 removed lines patch added patch discarded remove patch

Switch Indentation -1 removed lines patch added patch discarded remove patch

Spacing -1 removed lines patch added patch discarded remove patch

Braces -1 removed lines patch added patch discarded remove patch

Upper-Lower-Casing -1 removed lines patch added patch discarded remove patch

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

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

Switch Indentation +38 added lines, -38 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

Braces +4 added lines, -8 removed lines patch added patch discarded remove patch

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

Braces +3 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

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

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Braces +4 added lines, -8 removed lines patch added patch discarded remove patch