feat(): includes a custom list of fields

Author: gbataille

simple_history/admin.py

@@ -57,6 +57,7 @@ def history_view(self, request, object_id, extra_context=None):
         history = getattr(model, model._meta.simple_history_manager_attribute)
         object_id = unquote(object_id)
         action_list = history.filter(**{pk_name: object_id})
+        history_list_display = getattr(self, "history_list_display", [])
         # If no history was found, see whether this object even exists.
         try:
             obj = model.objects.get(**{pk_name: object_id})
@@ -77,7 +78,8 @@ def history_view(self, request, object_id, extra_context=None):
             'root_path': getattr(self.admin_site, 'root_path', None),
             'app_label': app_label,
             'opts': opts,
-            'admin_user_view': admin_user_view
+            'admin_user_view': admin_user_view,
+            'history_list_display': history_list_display,
         }
         context.update(extra_context or {})
         extra_kwargs = {}

simple_history/templates/simple_history/_object_history_list.html

@@ -1,12 +1,15 @@
 {% load i18n %}
 {% load url from simple_history_compat %}
 {% load admin_urls %}
+{% load getattribute from getattributes %}
 
-{{ foo }}
 <table id="change-history" class="table table-bordered table-striped">
   <thead>
     <tr>
       <th scope="col">{% trans 'Object' %}</th>
+      {% for column in history_list_display %}
+        <th scope="col">{% trans column %}</th>
+      {% endfor %}
       <th scope="col">{% trans 'Date/time' %}</th>
       <th scope="col">{% trans 'Comment' %}</th>
       <th scope="col">{% trans 'Changed by' %}</th>
@@ -16,6 +19,9 @@
     {% for action in action_list %}
       <tr>
         <td><a href="{% url opts|admin_urlname:'simple_history' object.pk action.pk %}">{{ action.history_object }}</a></td>
+        {% for column in history_list_display %}
+        <td scope="col">{{ action|getattribute:column }}</th>
+        {% endfor %}
         <td>{{ action.history_date }}</td>
         <td>{{ action.get_history_type_display }}</td>
         <td>

simple_history/templates/simple_history/object_history.html

@@ -12,7 +12,7 @@
 
     <div class="module">
       {% if action_list %}
-        {% display_list module_name %}
+        {% display_list %}
       {% else %}
         <p>{% trans "This object doesn't have a change history." %}</p>
       {% endif %}

simple_history/templatetags/getattributes.py

@@ -0,0 +1,12 @@
+from django import template
+
+register = template.Library()
+
+
+def getattribute(value, arg):
+    """Gets an attribute of an object dynamically from a string name"""
+
+    if hasattr(value, str(arg)):
+        return getattr(value, arg)
+
+register.filter('getattribute', getattribute)

simple_history/templatetags/simple_history_admin_list.py

@@ -5,7 +5,5 @@
 
 @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)})
+def display_list(context):
     return context