Fixed bug #2601 : Squiz.WhiteSpace.FunctionSpacing incorrect fix when spacing is 0

Author: gsherwood

package.xml

@@ -170,6 +170,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
   - Fixed bug #2566 : Author tag email validation doesn't support all TLDs
     -- Thanks to Juliette Reinders Folmer for the patch
   - Fixed bug #2575 : Custom error messages don't have data replaced when cache is enabled
+  - Fixed bug #2601 : Squiz.WhiteSpace.FunctionSpacing incorrect fix when spacing is 0
  </notes>
  <contents>
   <dir name="/">

src/Standards/Squiz/Sniffs/WhiteSpace/FunctionSpacingSniff.php

@@ -318,6 +318,10 @@ public function process(File $phpcsFile, $stackPtr)
                     $nextContent = $phpcsFile->findNext(T_WHITESPACE, ($nextSpace + 1), null, true);
                     $phpcsFile->fixer->beginChangeset();
                     for ($i = $nextSpace; $i < $nextContent; $i++) {
+                        if ($tokens[$i]['line'] === $tokens[$prevContent]['line']) {
+                            continue;
+                        }
+
                         if ($tokens[$i]['line'] === $tokens[$nextContent]['line']) {
                             $phpcsFile->fixer->addContentBefore($i, str_repeat($phpcsFile->eolChar, $requiredSpacing));
                             break;
@@ -327,7 +331,7 @@ public function process(File $phpcsFile, $stackPtr)
                     }
 
                     $phpcsFile->fixer->endChangeset();
-                }
+                }//end if
             }//end if
         }//end if
 

src/Standards/Squiz/Tests/WhiteSpace/FunctionSpacingUnitTest.1.inc

@@ -482,3 +482,19 @@ class Foo {
 if (function_exists('foo') === false) {
     function foo(){}
 }
+
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacing 0
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacingBeforeFirst 0
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacingAfterLast 0
+
+interface OneBlankLineBeforeFirstFunctionClassInterface
+{
+
+
+    /** @return mixed */
+    public function interfaceMethod();
+}
+
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacing 2
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacingBeforeFirst 2
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacingAfterLast 2

src/Standards/Squiz/Tests/WhiteSpace/FunctionSpacingUnitTest.1.inc.fixed

@@ -566,3 +566,17 @@ if (function_exists('foo') === false) {
 
 
 }
+
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacing 0
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacingBeforeFirst 0
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacingAfterLast 0
+
+interface OneBlankLineBeforeFirstFunctionClassInterface
+{
+    /** @return mixed */
+    public function interfaceMethod();
+}
+
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacing 2
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacingBeforeFirst 2
+// phpcs:set Squiz.WhiteSpace.FunctionSpacing spacingAfterLast 2

src/Standards/Squiz/Tests/WhiteSpace/FunctionSpacingUnitTest.php

@@ -87,6 +87,7 @@ public function getErrorList($testFile='')
                 478 => 2,
                 479 => 1,
                 483 => 2,
+                495 => 1,
             ];
 
         case 'FunctionSpacingUnitTest.2.inc':