feat(): add template tag for history table display

Author: gbataille

simple_history/templates/simple_history/_object_history_list.html

@@ -0,0 +1,36 @@
+{% load i18n %}
+{% load url from simple_history_compat %}
+{% load admin_urls %}
+
+{{ foo }}
+<table id="change-history" class="table table-bordered table-striped">
+  <thead>
+    <tr>
+      <th scope="col">{% trans 'Object' %}</th>
+      <th scope="col">{% trans 'Date/time' %}</th>
+      <th scope="col">{% trans 'Comment' %}</th>
+      <th scope="col">{% trans 'Changed by' %}</th>
+    </tr>
+  </thead>
+  <tbody>
+    {% for action in action_list %}
+      <tr>
+        <td><a href="{% url opts|admin_urlname:'simple_history' object.pk action.pk %}">{{ action.history_object }}</a></td>
+        <td>{{ action.history_date }}</td>
+        <td>{{ action.get_history_type_display }}</td>
+        <td>
+          {% if action.history_user %}
+            {% url admin_user_view action.history_user_id as admin_user_url %}
+            {% if admin_user_url %}
+              <a href="{{ admin_user_url }}">{{ action.history_user }}</a>
+            {% else %}
+              {{ action.history_user }}
+            {% endif %}
+          {% else %}
+            {% trans "None" %}
+          {% endif %}
+        </td>
+      </tr>
+    {% endfor %}
+  </tbody>
+</table>

simple_history/templates/simple_history/object_history.html

@@ -2,6 +2,7 @@
 {% load i18n %}
 {% load url from simple_history_compat %}
 {% load admin_urls %}
+{% load display_list from simple_history_admin_list %}
 
 
 {% block content %}
@@ -11,37 +12,7 @@
 
     <div class="module">
       {% if action_list %}
-        <table id="change-history" class="table table-bordered table-striped">
-          <thead>
-            <tr>
-              <th scope="col">{% trans 'Object' %}</th>
-              <th scope="col">{% trans 'Date/time' %}</th>
-              <th scope="col">{% trans 'Comment' %}</th>
-              <th scope="col">{% trans 'Changed by' %}</th>
-            </tr>
-          </thead>
-          <tbody>
-            {% for action in action_list %}
-              <tr>
-                <td><a href="{% url opts|admin_urlname:'simple_history' object.pk action.pk %}">{{ action.history_object }}</a></td>
-                <td>{{ action.history_date }}</td>
-                <td>{{ action.get_history_type_display }}</td>
-                <td>
-                  {% if action.history_user %}
-                    {% url admin_user_view action.history_user_id as admin_user_url %}
-                    {% if admin_user_url %}
-                      <a href="{{ admin_user_url }}">{{ action.history_user }}</a>
-                    {% else %}
-                      {{ action.history_user }}
-                    {% endif %}
-                  {% else %}
-                  {% trans "None" %}
-                  {% endif %}
-                </td>
-              </tr>
-            {% endfor %}
-          </tbody>
-        </table>
+        {% display_list module_name %}
       {% else %}
         <p>{% trans "This object doesn't have a change history." %}</p>
       {% endif %}

simple_history/templatetags/simple_history_admin_list.py

@@ -0,0 +1,11 @@
+from django import template
+
+register = template.Library()
+
+
+@register.inclusion_tag("simple_history/_object_history_list.html",
+                        takes_context=True)
+def display_list(context, cl):
+    print str(cl)
+    context.update({'foo': str(cl)})
+    return context