Moved module list item template code out into included template

robgolding · May 11, 2010 · dc4a982 · dc4a982
1 parent 1d678b2
commit dc4a982
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 6 deletions.
diff --git a/myuni/templates/course/modules/module_list.html b/myuni/templates/course/modules/module_list.html
@@ -5,9 +5,11 @@
 {% block header-block %}{% endblock header-block %}
 
 {% block content-block %}
-    {% for module in object_list %}
 	<div class="block">
-		<h3><a href="{{ module.get_absolute_url }}">{{ module.code }}: {{ module.name }}, {{ module.get_semester_display }} {{ module.get_year_display }}</a></h3>
+		<ul>
+		{% for module in object_list %}
+			{% include 'course/modules/module_list_item.html' %}
+		{% endfor %}
+		</ul>
 	</div>
-    {% endfor %}
 {% endblock content-block %}
diff --git a/myuni/templates/course/modules/module_list_all.html b/myuni/templates/course/modules/module_list_all.html
@@ -5,9 +5,11 @@
 {% block header-block %}{% endblock header-block %}
 
 {% block content-block %}
-    {% for module in object_list %}
 	<div class="block">
-		<h3><a href="{{ module.get_absolute_url }}">{{ module.code }}: {{ module.name }}, {{ module.get_semester_display }} {{ module.get_year_display }}</a></h3>
+		<ul>
+		{% for module in object_list %}
+			{% include 'course/modules/module_list_item.html' %}
+		{% endfor %}
+		</ul>
 	</div>
-    {% endfor %}
 {% endblock content-block %}
diff --git a/myuni/templates/course/modules/module_list_item.html b/myuni/templates/course/modules/module_list_item.html
@@ -0,0 +1,4 @@
+{% ifchanged module.year %}
+	<h3>{{ module.get_year_display }}</h3>
+{% endifchanged %}
+<li><a href="{{ module.get_absolute_url }}">{{ module.code }}: {{ module.name }}</a></li>