Squiz/SuperfluousWhiteSpace: bug fix - ignoreBlankLines should only i…

…gnore blank lines

Setting `ignoreBlankLines` to true, would now also cause the sniff to ignore single-line comments with superfluous whitespace at the end.

Includes unit test

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

@@ -192,6 +192,7 @@ public function process(File $phpcsFile, $stackPtr)
 
             // Ignore blank lines if required.
             if ($this->ignoreBlankLines === true
+                && trim($tokens[$stackPtr]['content']) === ''
                 && $tokens[($stackPtr - 1)]['line'] !== $tokens[$stackPtr]['line']
             ) {
                 return;

src/Standards/Squiz/Tests/WhiteSpace/SuperfluousWhitespaceUnitTest.inc

@@ -46,6 +46,8 @@ function myFunction2()
 
 }
 
+// Ordinary comment with extra whitespace at the end    
+
 // phpcs:set Squiz.WhiteSpace.SuperfluousWhitespace ignoreBlankLines false
 
 // Уберём из системных свойств все кроме информации об услугах

src/Standards/Squiz/Tests/WhiteSpace/SuperfluousWhitespaceUnitTest.inc.fixed

@@ -42,6 +42,8 @@ function myFunction2()
 
 }
 
+// Ordinary comment with extra whitespace at the end
+
 // phpcs:set Squiz.WhiteSpace.SuperfluousWhitespace ignoreBlankLines false
 
 // Уберём из системных свойств все кроме информации об услугах

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

@@ -39,7 +39,8 @@ public function getErrorList($testFile='SuperfluousWhitespaceUnitTest.inc')
                 23 => 1,
                 28 => 1,
                 33 => 1,
-                53 => 1,
+                49 => 1,
+                55 => 1,
             ];
             break;
         case 'SuperfluousWhitespaceUnitTest.1.js':