Update BlankLineBefore Return/If sniffs to be satisfied if preceded by doc block comment

Author: robocoder

Sniffs/Formatting/BlankLineBeforeReturnSniff.php

@@ -67,6 +67,7 @@ public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
         while ($current >= 0 && $tokens[$current]['line'] >= $previousLine) {
             if ($tokens[$current]['line'] == $previousLine
                 && $tokens[$current]['type'] !== 'T_WHITESPACE'
+                && $tokens[$current]['type'] !== 'T_DOC_COMMENT'
                 && $tokens[$current]['type'] !== 'T_COMMENT'
             ) {
                 $prevLineTokens[] = $tokens[$current]['type'];

Sniffs/Instaclick/BlankLineBeforeIfSniff.php

@@ -67,6 +67,7 @@ public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
         while ($current >= 0 && $tokens[$current]['line'] >= $previousLine) {
             if ($tokens[$current]['line'] == $previousLine
                 && $tokens[$current]['type'] !== 'T_WHITESPACE'
+                && $tokens[$current]['type'] !== 'T_DOC_COMMENT'
                 && $tokens[$current]['type'] !== 'T_COMMENT'
             ) {
                 $prevLineTokens[] = $tokens[$current]['type'];

Tests/Formatting/BlankLineBeforeReturnUnitTest.inc

@@ -17,6 +17,20 @@ function validFunctionReturnThree()
     return;
 }
 
+function validFunctionReturnFour()
+{
+    // comment
+    return;
+}
+
+function validFunctionReturnFive()
+{
+    /**
+     * multi-line
+     */
+    return;
+}
+
 function invalidFunctionReturnOne()
 {
     echo "";

Tests/Formatting/BlankLineBeforeReturnUnitTest.php

@@ -38,7 +38,7 @@ class Symfony2_Tests_Formatting_BlankLineBeforeReturnUnitTest extends AbstractSn
     public function getErrorList()
     {
         return array(
-            23 => 1,
+            37 => 1,
         );
     }
 

Tests/Instaclick/BlankLineBeforeIfUnitTest.inc

@@ -20,6 +20,29 @@ function validFunctionIfThree()
     }
 }
 
+function validFunctionIfFour()
+{
+    /* comment */
+    if (true) {
+    }
+}
+
+function validFunctionIfFive()
+{
+    // single line comment
+    if (true) {
+    }
+}
+
+function validFunctionIfSix()
+{
+    /**
+     * multi-line doc block
+     */
+    if (true) {
+    }
+}
+
 function invalidFunctionIfOne()
 {
     echo "";

Tests/Instaclick/BlankLineBeforeIfUnitTest.php

@@ -38,7 +38,7 @@ class Symfony2_Tests_Instaclick_BlankLineBeforeIfUnitTest extends AbstractSniffU
     public function getErrorList()
     {
         return array(
-            26 => 1,
+            49 => 1,
         );
     }