Ticket #8472: recent_action_on_appindex.diff

django/contrib/admin/templatetags/log.py

@@ -1 +1 @@
 from django import template
 from django.contrib.admin.models import LogEntry
+from django.contrib.contenttypes.models import ContentType
 
 register = template.Library()
 
 class AdminLogNode(template.Node):
-    def __init__(self, limit, varname, user):
-        self.limit, self.varname, self.user = limit, varname, user
+    def __init__(self, limit, varname, user, current_app):
+        self.limit, self.varname, self.user, self.current_app = limit, varname, user, current_app
 
     def __repr__(self):
         return "<GetAdminLog Node>"
@@ -17 +18 @@
             user_id = self.user
             if not user_id.isdigit():
                 user_id = context[self.user].id
-            context[self.varname] = LogEntry.objects.filter(user__id__exact=user_id).select_related('content_type', 'user')[:self.limit]
+            filters = {'user__id__exact': user_id}
+
+            if self.current_app and len(context['app_list']) == 1: # If there are more than one app don't use the filter
+                current_app = context['app_list'][0]['name'].lower()
+                contenttypes = ContentType.objects.filter(app_label=current_app)
+                ids = [contenttype.id for contenttype in contenttypes]
+                filters['content_type__in'] = ids
+
+            context[self.varname] = LogEntry.objects.filter(**filters).select_related('content_type', 'user')[:self.limit]
         return ''
 
 class DoGetAdminLog:
@@ -26 +35 @@
 
     Usage::
 
-        {% get_admin_log [limit] as [varname] for_user [context_var_containing_user_obj] %}
+        {% get_admin_log [limit] as [varname] for_user [context_var_containing_user_obj] for_current_app %}
 
     Examples::
 
+        {% get_admin_log 10 as admin_log for_user 23 for_current_app %}
         {% get_admin_log 10 as admin_log for_user 23 %}
         {% get_admin_log 10 as admin_log for_user user %}
         {% get_admin_log 10 as admin_log %}
 
+    The option ``for_current_app`` should be used only inside of an
+    application index. It will look up for the current app inside of app_list.
+    If this option is used on any other place, it just won't make any filter.
     Note that ``context_var_containing_user_obj`` can be a hard-coded integer
     (user ID) or the name of a template context variable containing the user
     object whose ID you want.
@@ -52 +65 @@
         if len(tokens) > 4:
             if tokens[4] != 'for_user':
                 raise template.TemplateSyntaxError("Fourth argument in '%s' must be 'for_user'" % self.tag_name)
-        return AdminLogNode(limit=tokens[1], varname=tokens[3], user=(len(tokens) > 5 and tokens[5] or None))
+        if len(tokens) > 6:
+            if tokens[6] != 'for_current_app':
+                raise template.TemplateSyntaxError("Fifth argument in '%s' must be 'for_current_app'" % self.tag_name)
+        return AdminLogNode(limit=tokens[1], varname=tokens[3], user=(len(tokens) > 5 and tokens[5] or None), current_app=(len(tokens) > 6))
 
 register.tag('get_admin_log', DoGetAdminLog('get_admin_log'))

django/contrib/admin/templates/admin/app_index.html

@@ -12 +12 @@
 
 {% endif %} 
 
-{% block sidebar %}{% endblock %}
- No newline at end of file
+{% block sidebar %}
+<div id="content-related">
+    <div class="module" id="recent-actions-module">
+        <h2>{% trans 'Recent Actions' %}</h2>
+        <h3>{% trans 'My Actions' %}</h3>
+            {% load log %}
+            {% get_admin_log 10 as admin_log for_user user for_current_app %}
+            {% if not admin_log %}
+            <p>{% trans 'None available' %}</p>
+            {% else %}
+            <ul class="actionlist">
+            {% for entry in admin_log %}
+            <li class="{% if entry.is_addition %}addlink{% endif %}{% if entry.is_change %}changelink{% endif %}{% if entry.is_deletion %}deletelink{% endif %}">
+                {% if entry.is_deletion or not entry.get_admin_url %}
+                    {{ entry.object_repr }}
+                {% else %}
+                    <a href="{{ entry.get_admin_url }}">{{ entry.object_repr }}</a>
+                {% endif %}
+                <br/>
+                {% if entry.content_type %}
+                    <span class="mini quiet">{% filter capfirst %}{% trans entry.content_type.name %}{% endfilter %}</span>
+                {% else %}
+                    <span class="mini quiet">{% trans 'Unknown content' %}</span>
+                {% endif %}
+            </li>
+            {% endfor %}
+            </ul>
+            {% endif %}
+    </div>
+</div>
+{% endblock %}