✨ Add fixer for BlankLineBeforeReturnSniff

Author: Vincent Langlet

Symfony3Custom/Sniffs/Formatting/BlankLineBeforeReturnSniff.php

@@ -62,10 +62,21 @@ public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
         ) {
             return;
         } else if (count($prevLineTokens) > 0) {
-            $phpcsFile->addError(
+            $fix = $phpcsFile->addFixableError(
                 'Missing blank line before return statement',
-                $stackPtr
+                $stackPtr,
+                'MissedBlankLineBeforeRetrun'
             );
+
+            if ($fix === true) {
+                $phpcsFile->fixer->beginChangeset();
+                $i = 1;
+                while($tokens[$stackPtr-$i]['type'] == "T_WHITESPACE") {
+                    $i++;
+                }
+                $phpcsFile->fixer->addNewLine($stackPtr-$i);
+                $phpcsFile->fixer->endChangeset();
+            }
         }
 
         return;

Symfony3Custom/Tests/Formatting/BlankLineBeforeReturnUnitTest.inc.fixed

@@ -0,0 +1,47 @@
+<?php
+function validFunctionReturnOne()
+{
+    return;
+}
+
+function validFunctionReturnTwo()
+{
+
+    return;
+}
+
+function validFunctionReturnThree()
+{
+    echo "";
+
+    return;
+}
+
+function validFunctionReturnFour()
+{
+    // comment
+    return;
+}
+
+function validFunctionReturnFive()
+{
+    /**
+     * multi-line
+     */
+    return;
+}
+
+function invalidFunctionReturnOne()
+{
+    echo "";
+
+    return;
+}
+
+switch ($condition) {
+    case 'foo':
+        return true;
+
+    default:
+        return false;
+}