Merge pull request #6 from SmileyChris/fix-component-vars

Fix component variables not getting parsed correctly.

changes/5.bugfix.md

@@ -0,0 +1 @@
+Fix a bug where the component context wasn't being set correctly, especially noticeable inside of a loop.

includecontents/templatetags/includecontents.py

@@ -233,6 +233,7 @@ def set_component_attrs(self, context: Context, new_context: Context):
         # Don't use the extra context for the include tag if it's a component
         # since we've handled adding it to the new context ourselves.
         extra_context = self.include_node.extra_context
+        self.include_node.extra_context = {}
         yield
         self.include_node.extra_context = extra_context
 

tests/templates/test_component/multi.html

@@ -0,0 +1,6 @@
+<main>
+  <include:card title="hello">Title</include:card>
+  {% for i in "123" %}
+    <include:card title={i}>loop {{ i }}</include:card>
+  {% endfor %}
+</main>

tests/test_component.py

@@ -179,3 +179,32 @@ def test_shorthand_attrs():
         Context({"food": "pizza"})
     )
     assert output == "pizza"
+
+
+def test_multi():
+    output = render_to_string("test_component/multi.html")
+    assert output == (
+        """<main>
+  <section class="card">
+  <h3 >hello</h3>
+  <div>Title</div>
+</section>
+  
+    <section class="card">
+  <h3 >1</h3>
+  <div>loop 1</div>
+</section>
+  
+    <section class="card">
+  <h3 >2</h3>
+  <div>loop 2</div>
+</section>
+  
+    <section class="card">
+  <h3 >3</h3>
+  <div>loop 3</div>
+</section>
+  
+</main>
+"""
+    )