From 88c0ce4e9f334d53ddf8a754dcb9f8707ca83741 Mon Sep 17 00:00:00 2001
From: Peter Inglesby <peter.inglesby@gmail.com>
Date: Wed, 22 May 2013 08:49:39 +0100
Subject: [PATCH 1/4] Remove stray closing tag.

---
 djadmin2/templates/admin2/bootstrap/base.html | 1 -
 1 file changed, 1 deletion(-)

diff --git a/djadmin2/templates/admin2/bootstrap/base.html b/djadmin2/templates/admin2/bootstrap/base.html
index 445b587..c042746 100644
--- a/djadmin2/templates/admin2/bootstrap/base.html
+++ b/djadmin2/templates/admin2/bootstrap/base.html
@@ -33,7 +33,6 @@
                                 <b class="caret"></b>
                             </a>
                             <ul class="dropdown-menu">
-</strong>
                                 {% if user.has_usable_password %}
                                     <li><a tabindex="-1" href="TODO">{% trans "Change password" %}</a></li>
                                 {% endif %}

From 8b6b56a1a80881043108106173413691adf4209d Mon Sep 17 00:00:00 2001
From: Peter Inglesby <peter.inglesby@gmail.com>
Date: Wed, 22 May 2013 08:50:04 +0100
Subject: [PATCH 2/4] Add messages to base.html

---
 djadmin2/templates/admin2/bootstrap/base.html | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/djadmin2/templates/admin2/bootstrap/base.html b/djadmin2/templates/admin2/bootstrap/base.html
index c042746..2c6b179 100644
--- a/djadmin2/templates/admin2/bootstrap/base.html
+++ b/djadmin2/templates/admin2/bootstrap/base.html
@@ -44,6 +44,14 @@
             </div>
         </div>
 
+        {% if messages %}
+        <ul class="messages">
+            {% for message in messages %}
+            <li{% if message.tags %} class="{{ message.tags }}"{% endif %}>{{ message }}</li>
+            {% endfor %}
+        </ul>
+        {% endif %}
+
         <div class="container-fluid">
             <div class="row">
                 <div class="span10">

From 79e6dfad839fa2df97e763929abcbc726d4766c0 Mon Sep 17 00:00:00 2001
From: Peter Inglesby <peter.inglesby@gmail.com>
Date: Wed, 22 May 2013 11:07:54 +0100
Subject: [PATCH 3/4] Add delete_selected action

---
 djadmin2/actions.py                           | 60 +++++++++++++++++++
 .../delete_selected_confirmation.html         | 23 +++++++
 2 files changed, 83 insertions(+)
 create mode 100644 djadmin2/actions.py
 create mode 100644 djadmin2/templates/admin2/bootstrap/delete_selected_confirmation.html

diff --git a/djadmin2/actions.py b/djadmin2/actions.py
new file mode 100644
index 0000000..5937e3a
--- /dev/null
+++ b/djadmin2/actions.py
@@ -0,0 +1,60 @@
+from django.core.exceptions import PermissionDenied
+from django.template.response import TemplateResponse
+from django.utils.text import capfirst
+
+from django.contrib import messages
+
+def get_description(action):
+    if hasattr(action, 'description'):
+        return action.description
+    else:
+        return capfirst(action.__name__.replace('_', ' '))
+
+def delete_selected(request, queryset):
+    # We check whether the user has permission to delete the objects in the
+    # queryset.  
+    # 
+    # TODO: This duplicates some of the permission-checking functionality in
+    # BaseAdmin2.  Investigate how to DRY this out.
+    #
+    # TODO: Check that user has permission to delete all related obejcts.  See
+    # `get_deleted_objects` in contrib.admin.util for how this is currently
+    # done.  (Hint: I think we can do better.)
+
+    model = queryset.model
+    opts = model._meta
+    permission_name = '%s.delete.%s' \
+            % (opts.app_label, opts.object_name.lower())
+    has_permission = request.user.has_perm(permission_name)
+
+    if len(queryset) == 1:
+        objects_name = opts.verbose_name
+    else:
+        objects_name = opts.verbose_name_plural
+
+    if request.POST.get('confirmed'):
+        # The user has confirmed that they want to delete the objects.
+        if has_permission:
+            queryset.delete()
+            message = "Successfully deleted %d %s" % \
+                    (len(queryset), objects_name)
+            messages.add_message(request, messages.INFO, message)
+            return None
+        else:
+            raise PermissionDenied
+    else:
+        # The user has not confirmed that they want to delete the objects, so
+        # render a template asking for their confirmation.
+        if has_permission:
+            template = 'admin2/bootstrap/delete_selected_confirmation.html'
+            context = {
+                'queryset': queryset,
+                'objects_name': objects_name
+            }
+            return TemplateResponse(request, template, context)
+        else:
+            message = "Permission to delete %s denied" % objects_name
+            messages.add_message(request, messages.INFO, message)
+            return None
+
+delete_selected.description = "Delete selected items"
diff --git a/djadmin2/templates/admin2/bootstrap/delete_selected_confirmation.html b/djadmin2/templates/admin2/bootstrap/delete_selected_confirmation.html
new file mode 100644
index 0000000..72f1452
--- /dev/null
+++ b/djadmin2/templates/admin2/bootstrap/delete_selected_confirmation.html
@@ -0,0 +1,23 @@
+{% extends "admin2/bootstrap/base.html" %}
+
+{% block content %}
+
+<p>Are you sure you want to delete the selected {{ objects_name }}?</p>
+
+<ul>
+    {% for item in queryset %}
+    <li>{{ item }}</li>
+    {% endfor %}
+</ul>
+
+<form method="post">
+    {% csrf_token %}
+    <input type="hidden" name="confirmed" value="yes" />
+    <input type="hidden" name="action" value="delete_selected" />
+    {% for item in queryset %}
+    <input type="hidden" name="selected_model_id" value="{{ item.id }}" />
+    {% endfor %}
+    <input type="submit"/>
+</form>
+
+{% endblock content %}

From ade92956d02996543e39b0f821ade7be125cf139 Mon Sep 17 00:00:00 2001
From: Peter Inglesby <peter.inglesby@gmail.com>
Date: Wed, 22 May 2013 11:09:01 +0100
Subject: [PATCH 4/4] Handle actions by POSTing to ModelListView

---
 djadmin2/models.py                            | 19 +++++
 .../static/themes/bootstrap/js/actions.js     | 26 +++++++
 .../admin2/bootstrap/model_list.html          | 75 +++++++++++--------
 djadmin2/views.py                             | 21 ++++++
 example/blog/tests/test_views.py              | 16 ++++
 5 files changed, 125 insertions(+), 32 deletions(-)
 create mode 100644 djadmin2/static/themes/bootstrap/js/actions.js

diff --git a/djadmin2/models.py b/djadmin2/models.py
index 8e74b98..8459a77 100644
--- a/djadmin2/models.py
+++ b/djadmin2/models.py
@@ -13,11 +13,13 @@ from django.db.models import get_models, signals
 
 from djadmin2 import apiviews
 from djadmin2 import views
+from djadmin2 import actions
 
 MODEL_ADMIN_ATTRS = (
                     'list_display', 'list_display_links', 'list_filter',
                     'admin', 'has_permission', 'has_add_permission',
                     'has_edit_permission', 'has_delete_permission',
+                    'get_actions',
                          )
 
 
@@ -126,6 +128,9 @@ class ModelAdmin2(BaseAdmin2):
     api_list_view = apiviews.ListCreateAPIView
     api_detail_view = apiviews.RetrieveUpdateDestroyAPIView
 
+    # Actions
+    actions = [actions.delete_selected]
+
     def __init__(self, model, admin, **kwargs):
         self.model = model
         self.admin = admin
@@ -243,6 +248,20 @@ class ModelAdmin2(BaseAdmin2):
     def api_urls(self):
         return self.get_api_urls(), None, None
 
+    def get_actions(self):
+        actions_dict = {}
+
+        for cls in type(self).mro()[::-1]:
+            class_actions = getattr(cls, 'actions', [])
+            for action in class_actions:
+                actions_dict[action.__name__] = {
+                        'name': action.__name__,
+                        'description': actions.get_description(action),
+                        'func': action
+                }
+        return actions_dict
+
+
 
 class ImmutableAdmin(object):
     """
diff --git a/djadmin2/static/themes/bootstrap/js/actions.js b/djadmin2/static/themes/bootstrap/js/actions.js
new file mode 100644
index 0000000..daf3f0b
--- /dev/null
+++ b/djadmin2/static/themes/bootstrap/js/actions.js
@@ -0,0 +1,26 @@
+$(function() {
+	var element = $("#model-list");
+	var selectAllCheckbox = element.find('.model-select-all');
+	var selectCheckboxen = element.find('.model-select');
+	var selectedCount = element.find('#selected-count');
+
+	var updateSelectedCount = function() {
+		var count = 0;
+		for (var ix = 0; ix < selectCheckboxen.length; ix++) {
+			if ($(selectCheckboxen[ix]).prop('checked')) {
+				count++;
+			};
+		};
+		selectAllCheckbox.prop('checked', count == selectCheckboxen.length);
+		selectedCount.text(count);
+	};
+
+	selectAllCheckbox.click(function(e) {
+		selectCheckboxen.prop('checked', this.checked);
+		updateSelectedCount();
+	});
+
+	selectCheckboxen.click(function(e) {
+		updateSelectedCount();
+	});
+});
diff --git a/djadmin2/templates/admin2/bootstrap/model_list.html b/djadmin2/templates/admin2/bootstrap/model_list.html
index 3ec9c53..38acf31 100644
--- a/djadmin2/templates/admin2/bootstrap/model_list.html
+++ b/djadmin2/templates/admin2/bootstrap/model_list.html
@@ -5,46 +5,57 @@
 
 {% block page_title %}Select {{ model }} to change{% endblock %}
 
+{% block extrajs %}
+<script src="/static/themes/bootstrap/js/actions.js"></script>
+{% endblock extrajs %}
+
 {% block content %}
 
-<div class="row">
-    <div class="span12">
-        <div class="space-below">
-            <div class="btn-group">
-                <button class="btn dropdown-toggle" data-toggle="dropdown">
-                    Actions
-                    <span class="caret"></span>
-                </button>
-                <ul class="dropdown-menu">
-                    <li><a tabindex="-1" href="#">Delete selected {{ model }}{{ object_list|pluralize }}</a></li>
-                </ul>
+<div id="model-list" class="row">
+    <form id="model-list-form" class="form-inline" method="post">
+        <div class="span12">
+            <div class="navbar">
+                {% csrf_token %}
+                <select name="action">
+                    {% for action in actions %}
+                    <option value="{{ action.name }}">{{ action.description }}</option>
+                    {% endfor %}
+                </select>
+                <button type="Submit" class="btn">Go</button>
+		<span id="selected-count">0</span> of {{ object_list|length }} selected
+                <div class="pull-right">
+                    {# if has_add_permission #}
+                    <a href="{% url view|admin2_urlname:'create' %}" class="btn"><i class="icon-plus"></i> Add {{ model }}</a>
+                    {# endif #}
+                </div>
             </div>
-            <small class="muted">TODO of {{ object_list|length }} selected</small>
-            <div class="pull-right">
-                {# if has_add_permission #}
-                <a href="{% url view|admin2_urlname:'create' %}" class="btn"><i class="icon-plus"></i> Add {{ model }}</a>
-                {# endif #}
-            </div>
-        </div>
 
-        <table class="table table-bordered table-striped">
-            <thead>
-                <th class="checkbox-column"><input type="checkbox"></th>
-                <th>{{ model|title}}</th>
-            </thead>
-            <tbody>
-                {% for obj in object_list %}
+            <table class="table table-bordered table-striped">
+                <thead>
+                    <th><input type="checkbox" class="model-select-all"></th>
+                    <th>{{ model|title}}</th>
+                </thead>
+                <tbody>
+                    {% for obj in object_list %}
                     <tr>
-                        <td><input type="checkbox"></td>
+                        <td><input type="checkbox" class="model-select" name="selected_model_id" value="{{obj.id}}"></td>
                         <td>
-                            <a href="{% url view|admin2_urlname:'update' pk=obj.pk %}">{{ obj }}</a>
+                            {{ obj }} <a href="{% url view|admin2_urlname:'detail' pk=obj.pk %}">detail</a>
+                            {# if has_edit_permission #}
+                            <a href="{% url view|admin2_urlname:'update' pk=obj.pk %}">edit</a>
+                            {# endif #}
+                            {# if has_delete_permission #}
+                            <a href="{% url view|admin2_urlname:'delete' pk=obj.pk %}">delete</a>
+                            {# endif #}
                         </td>
                     </tr>
-                {% endfor %}
-            </tbody>
-        </table>
-        <p>{{ object_list|length }} {{ model }}{{ object_list|pluralize }}</p>
-    </div>
+                    {% endfor %}
+                </tbody>
+            </table>
+
+            {{ object_list|length }} {{ model }}{{ object_list|pluralize }}
+        </div>
+    </form>
 </div>
 
 {% endblock content %}
diff --git a/djadmin2/views.py b/djadmin2/views.py
index 249d31c..c1f18ca 100644
--- a/djadmin2/views.py
+++ b/djadmin2/views.py
@@ -5,6 +5,7 @@ from django.conf import settings
 from django.contrib.auth.views import redirect_to_login
 from django.core.exceptions import PermissionDenied
 from django.forms.models import modelform_factory
+from django.http import HttpResponseRedirect
 from django.views import generic
 
 from braces.views import AccessMixin
@@ -109,12 +110,32 @@ class ModelListView(Admin2Mixin, generic.ListView):
     default_template_name = "model_list.html"
     permission_type = 'view'
 
+    def post(self, request):
+        # This is where we handle actions
+        action_name = request.POST['action']
+        action_func = self.get_actions()[action_name]['func']
+        selected_model_ids = request.POST.getlist('selected_model_id')
+        queryset = self.model.objects.filter(pk__in=selected_model_ids)
+        response = action_func(request, queryset)
+        if response is None:
+            return HttpResponseRedirect(self.get_success_url())
+        else:
+            return response
+
     def get_context_data(self, **kwargs):
         context = super(ModelListView, self).get_context_data(**kwargs)
         context['model'] = self.get_model()._meta.verbose_name
         context['model_pluralized'] = self.get_model()._meta.verbose_name_plural
+        context['actions'] = self.get_actions().values()
         return context
 
+    def get_success_url(self):
+        view_name = 'admin2:{}_{}_index'.format(self.app_label, self.model_name)
+        return reverse(view_name)
+
+    def get_actions(self):
+        return self.model_admin.get_actions()
+
 
 class ModelDetailView(AdminModel2Mixin, generic.DetailView):
     default_template_name = "model_detail.html"
diff --git a/example/blog/tests/test_views.py b/example/blog/tests/test_views.py
index 0ff839d..dc4c4e1 100644
--- a/example/blog/tests/test_views.py
+++ b/example/blog/tests/test_views.py
@@ -30,6 +30,22 @@ class PostListTest(BaseIntegrationTest):
         response = self.client.get(reverse("admin2:blog_post_index"))
         self.assertContains(response, post.title)
 
+    def test_actions_displayed(self):
+        response = self.client.get(reverse("admin2:blog_post_index"))
+        self.assertInHTML('<option value="delete_selected">Delete selected items</option>', response.content)
+
+    def test_delete_selected_post(self):
+        post = Post.objects.create(title="a_post_title", body="body")
+        params = {'action': 'delete_selected', 'selected_model_id': str(post.id)}
+        response = self.client.post(reverse("admin2:blog_post_index"), params)
+        self.assertInHTML('<p>Are you sure you want to delete the selected post?</p>', response.content)
+
+    def test_delete_selected_post_confirmation(self):
+        post = Post.objects.create(title="a_post_title", body="body")
+        params = {'action': 'delete_selected', 'selected_model_id': str(post.id), 'confirmed': 'yes'}
+        response = self.client.post(reverse("admin2:blog_post_index"), params)
+        self.assertRedirects(response, reverse("admin2:blog_post_index"))
+
 
 class PostDetailViewTest(BaseIntegrationTest):
     def test_view_ok(self):