don't reset Liquid tag's whitespace control from comment tag

Shopify · Jan 5, 2024 · dab4000 · dab4000
1 parent 96a0363
commit dab4000
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/lib/liquid/tags/comment.rb b/lib/liquid/tags/comment.rb
@@ -63,7 +63,7 @@ def parse_body(body, tokenizer)
             comment_tag_depth -= 1
 
             if comment_tag_depth.zero?
-              parse_context.trim_whitespace = (token[-3] == WhitespaceControl)
+              parse_context.trim_whitespace = (token[-3] == WhitespaceControl) unless tokenizer.for_liquid_tag
               return false
             end
           end

diff --git a/test/unit/tags/comment_tag_unit_test.rb b/test/unit/tags/comment_tag_unit_test.rb
@@ -187,4 +187,16 @@ def test_with_whitespace_control
       Hello!
     LIQUID
   end
+
+  def test_dont_override_liquid_tag_whitespace_control
+    assert_template_result("Hello!World!", <<~LIQUID.chomp)
+      Hello!
+      {%- liquid
+        comment
+         this is inside a liquid tag
+        endcomment
+      -%}
+      World!
+    LIQUID
+  end
 end