Use a macro for material_list

Author: encukou

naucse/templates/_macros.html

@@ -25,3 +25,13 @@
         </a>
     </li>
 {%- endmacro -%}
+
+{%- macro material_list(materials, filter_type=None) %}
+    <ul>
+        {% for mat in materials %}
+            {% if filter_type == None or mat.url_type == filter_type %}
+                {{ material_li(mat) }}
+            {% endif %}
+        {% endfor %}
+    </ul>
+{%- endmacro -%}

naucse/templates/backpage.html

@@ -1,5 +1,5 @@
 {% extends "_base.html" %}
-{% from "_macros.html" import material_li %}
+{% from "_macros.html" import material_list with context %}
 
 {% block content %}
 
@@ -22,25 +22,12 @@ <h1>
 
         {% if homework_section %}
             <h2>Domácí projekty</h2>
-            <ul>
-            {% for mat in session.materials %}
-                {% if mat.url_type == "homework" %}
-                    {{ material_li(mat) }}
-                {% endif %}
-            {% endfor %}
-            </ul>
+            {{ material_list(session.materials, filter_type='homework') }}
         {% endif %}
 
         {% if link_section %}
             <h2>Zajímavé odkazy</h2>
-
-            <ul>
-            {% for mat in session.materials %}
-                {% if mat.url_type == "link" %}
-                    {{ material_li(mat) }}
-                {% endif %}
-            {% endfor %}
-            </ul>
+            {{ material_list(session.materials, filter_type='link') }}
         {% endif %}
 
         {% block coverpage_content %}

naucse/templates/course.html

@@ -1,5 +1,5 @@
 {% extends "_base.html" %}
-{% from "_macros.html" import material_li %}
+{% from "_macros.html" import material_list with context %}
 
 {% macro session_heading(session, index, plan) %}
     {% if plan|length > 1 %}
@@ -46,11 +46,7 @@ <h4>
                         {{ session_heading(session, loop.index, plan) }}
                         </a>
                     </h4>
-                    <ul>
-                        {% for mat in session.materials %}
-                            {{ material_li(mat) }}
-                        {% endfor %}
-                    </ul>
+                    {{ material_list(session.materials) }}
                 </div>
                 {% endfor %}
 

naucse/templates/coverpage.html

@@ -1,5 +1,5 @@
 {% extends "_base.html" %}
-{% from "_macros.html" import material_li %}
+{% from "_macros.html" import material_list with context %}
 
 {% block content %}
 
@@ -29,24 +29,11 @@ <h1>
         {% endif %}
 
         <h2>Materiály</h2>
-        <ul>
-        {% for mat in session.materials %}
-            {% if mat.url_type == "lesson" %}
-                {{ material_li(mat) }}
-            {% endif %}
-        {% endfor %}
-        </ul>
+        {{ material_list(session.materials, filter_type='lesson') }}
 
         {% if cheatsheet_section %}
             <h2>Taháky</h2>
-
-            <ul>
-            {% for mat in session.materials %}
-                {% if mat.url_type == "cheatsheet" %}
-                    {{ material_li(mat) }}
-                {% endif %}
-            {% endfor %}
-            </ul>
+            {{ material_list(session.materials, filter_type='cheatsheet') }}
         {% endif %}
 
         {% block coverpage_content %}