Merge pull request #126 from VincentLanglet/fixComment

🐛 Fix delimiter spacing for comment with new line

Author: VincentLanglet

TwigCS/src/Sniff/AbstractSpacingSniff.php

@@ -65,7 +65,7 @@ private function checkSpaceAfter(int $tokenPosition, array $tokens, int $expecte
 
         // Ignore new line
         $next = $this->findNext(Token::WHITESPACE_TOKENS, $tokens, $tokenPosition + 1, true);
-        if (false !== $next && $this->isTokenMatching($tokens[$next], Token::EOL_TYPE)) {
+        if (false !== $next && $this->isTokenMatching($tokens[$next], [Token::EOL_TYPE, Token::COMMENT_EOL_TYPE])) {
             return;
         }
 
@@ -106,7 +106,7 @@ private function checkSpaceBefore(int $tokenPosition, array $tokens, int $expect
 
         // Ignore new line
         $previous = $this->findPrevious(Token::WHITESPACE_TOKENS, $tokens, $tokenPosition - 1, true);
-        if ($this->isTokenMatching($tokens[$previous], Token::EOL_TYPE)) {
+        if ($this->isTokenMatching($tokens[$previous], [Token::EOL_TYPE, Token::COMMENT_EOL_TYPE])) {
             return;
         }
 

TwigCS/tests/Ruleset/Generic/DelimiterSpacing/DelimiterSpacingTest.fixed.twig

@@ -9,6 +9,9 @@
 {{
     shouldNotCareAboutNewLine
 }}
+{#
+    shouldNotCareAboutNewLine
+#}
 {%- if foo -%}{%- endif -%}
 
 {{ foo({'bar': {'baz': 'shouldNotCareAboutDoubleHashes'}}) }}

TwigCS/tests/Ruleset/Generic/DelimiterSpacing/DelimiterSpacingTest.php

@@ -18,10 +18,10 @@ class DelimiterSpacingTest extends AbstractSniffTest
     public function testSniff(): void
     {
         $this->checkSniff(new DelimiterSpacingSniff(), [
-            [12 => 1],
-            [12 => 12],
-            [12 => 15],
-            [12 => 25],
+            [15 => 1],
+            [15 => 12],
+            [15 => 15],
+            [15 => 25],
         ]);
     }
 }

TwigCS/tests/Ruleset/Generic/DelimiterSpacing/DelimiterSpacingTest.twig

@@ -9,6 +9,9 @@
 {{
     shouldNotCareAboutNewLine
 }}
+{#
+    shouldNotCareAboutNewLine
+#}
 {%-if foo  -%}{%-  endif-%}
 
 {{ foo({'bar': {'baz': 'shouldNotCareAboutDoubleHashes'}}) }}