From 22163173feb276fa54ad8678ab20c31a6b7844b7 Mon Sep 17 00:00:00 2001
From: xiaoyu <763691951@qq.com>
Date: Fri, 23 Sep 2016 16:30:59 +0800
Subject: [PATCH] user bulk delete view

---
 apps/users/api.py                             |  13 ++
 apps/users/models.py                          |   5 +
 apps/users/templates/users/user_list.html     | 107 ++++++++++-------
 apps/users/templates/users/user_list.old.html | 113 ------------------
 4 files changed, 80 insertions(+), 158 deletions(-)
 delete mode 100644 apps/users/templates/users/user_list.old.html

diff --git a/apps/users/api.py b/apps/users/api.py
index c90bde910..6fb1fe7c5 100644
--- a/apps/users/api.py
+++ b/apps/users/api.py
@@ -98,3 +98,16 @@ class GroupDeleteApi(generics.DestroyAPIView):
 class UserBulkUpdateApi(ListBulkCreateUpdateDestroyAPIView):
     queryset = User.objects.all()
     serializer_class = UserBulkUpdateSerializer
+
+    def filter_queryset(self, queryset):
+        id_list = self.request.query_params.get('id__in')
+        if id_list:
+            import json
+            try:
+                ids = json.loads(id_list)
+            except Exception as e:
+                logger.error(str(e))
+                return queryset
+            if isinstance(ids, list):
+                queryset = queryset.filter(id__in=ids)
+        return queryset
diff --git a/apps/users/models.py b/apps/users/models.py
index 13206bdb2..200d3e038 100644
--- a/apps/users/models.py
+++ b/apps/users/models.py
@@ -215,6 +215,11 @@ class User(AbstractUser):
         user.save()
         user.groups.add(UserGroup.initial())
 
+    def delete(self):
+        if self.is_superuser:
+            return
+        return super(User, self).delete()
+
     @classmethod
     def generate_fake(cls, count=100):
         from random import seed, choice
diff --git a/apps/users/templates/users/user_list.html b/apps/users/templates/users/user_list.html
index da36ab903..0773a0871 100644
--- a/apps/users/templates/users/user_list.html
+++ b/apps/users/templates/users/user_list.html
@@ -11,12 +11,12 @@
     <thead>
         <tr>
             <th></th>
-            <th class="text-center"><a href="{% url 'users:user-list' %}?sort=name">{% trans 'Name' %}</a></th>
-            <th class="text-center"><a href="{% url 'users:user-list' %}?sort=username">{% trans 'Username' %}</a></th>
+            <th class="text-center">{% trans 'Name' %}</a></th>
+            <th class="text-center">{% trans 'Username' %}</a></th>
             <th class="text-center">{% trans 'Role' %}</th>
             <th class="text-center">{% trans 'User group' %}</th>
             <th class="text-center">{% trans 'Asset num' %}</th>
-            <th class="text-center"><a href="{% url 'users:user-list' %}?sort=date_expired">{% trans 'Active' %}</a></th>
+            <th class="text-center">{% trans 'Active' %}</a></th>
             <th class="text-center">{% trans 'Action' %}</th>
         </tr>
     </thead>
@@ -24,45 +24,12 @@
     </tbody>
 </table>
 {% endblock %}
-
-{% block table_body %}
-     {% for user in object_list %}
-         <tr class="gradeX">
-             <td class="text-center">
-                 <input type="checkbox" name="checked" value="{{ user.id }}" class="ipt_bulk_update">
-             </td>
-             <td class="text-center">
-                 <a href="{% url 'users:user-detail' pk=user.id %}">
-                     {{ user.name }}
-                 </a>
-             </td>
-             <td class="text-center">{{ user.username }}</td>
-             <td class="text-center">{{ user.get_role_display }}</td>
-             <td class="text-center" title="{% for user_group in user.group.all %} {{ user_group.name }} {% endfor %}"> {{ user.groups.all|join_queryset_attr:"name" }} </td>
-             <th class="text-center">{{ user.name }}</th>
-             <td class="text-center">
-                 {% if user.is_expired and user.is_active %}
-                     <i class="fa fa-times text-danger"></i>
-                 {% else %}
-                     <i class="fa fa-check text-navy"></i>
-                 {% endif %}
-             </td>
-             <td class="text-center">
-                 <a href="{% url 'users:user-update' pk=user.id %}" class="btn btn-xs btn-info">{% trans 'Update' %}</a>
-                 <!--<a href="{% url 'users:user-delete' pk=user.id %}" class="btn btn-xs btn-danger del {% if user.id == request.user.id or user.username == 'admin' %} disabled {% endif %}">{% trans 'Delete' %}</a>-->
-                 <a onclick="obj_del(this,'{{ user.name }}','{% url 'users:user-delete' user.id %}')" class="btn btn-xs btn-danger del {% if user.id == request.user.id or user.username == 'admin' %} disabled {% endif %}">{% trans 'Delete' %}</a>
-             </td>
-         </tr>
-     {% endfor %}
-{% endblock %}
-
 {% block content_bottom_left %}
         <div class="input-group">
             <select class="form-control m-b" style="width: auto" id="slct_bulk_update">
                 <option value="delete">{% trans 'Delete selected' %}</option>
                 <option value="update">{% trans 'Update selected' %}</option>
                 <option value="deactive">{% trans 'Deactive selected' %}</option>
-                <option value="export">{% trans 'Export selected' %}</option>
             </select>
             <div class="input-group-btn pull-left" style="padding-left: 5px;">
                 <button id='btn_bulk_update' style="height: 32px;"  class="btn btn-sm btn-primary">
@@ -108,8 +75,14 @@ $(document).ready(function(){
             {orderable: false, className: 'select-checkbox', targets: 0},
             {className: 'text-center', targets: [1, 2, 3, 4, 5, 6, 7]},
             {targets: 7,
-             createdCell: function (td, cellData) {
-                $(td).html('<a href="#" class="btn btn-xs btn-info">{% trans "Update" %}</a><a class="btn btn-xs btn-danger del">{% trans "Delete" %}</a>')
+             createdCell: function (td, cellData, rowData) {
+                var update_btn = '<a href="{% url "users:user-update" pk=99991937 %}" class="btn btn-xs btn-info">{% trans "Update" %}</a>'.replace('99991937', cellData);
+                var del_btn = '<a class="btn btn-xs btn-danger m-l-xs btn_user_delete" data-uid="99991937">{% trans "Delete" %}</a>'.replace('99991937', cellData);
+                if (rowData.role === 'Admin') {
+                    $(td).html(update_btn)
+                } else {
+                    $(td).html(update_btn + del_btn)
+                }
              }
             },
             {targets: 6,
@@ -141,9 +114,14 @@ $(document).ready(function(){
     var action = $('#slct_bulk_update').val();
     var $data_table = $('#user_list_table').DataTable()
     var id_list = [];
+    var plain_id_list = [];
     $data_table.rows({selected: true}).every(function(){
-        id_list.push({id: this.data().id})
+        id_list.push({id: this.data().id});
+        plain_id_list.push(this.data().id);
     });
+    if (id_list === []) {
+        return false;
+    };
     var the_url = "{% url 'users:user-bulk-update-api' %}";
     function doDeactive() {
         var body = $.each(id_list, function(index, user_object) {
@@ -162,11 +140,19 @@ $(document).ready(function(){
             confirmButtonText: "{% trans 'Confirm' %}",
             closeOnConfirm: false
         }, function() {
-            APIUpdateAttr({url: the_url, method: 'DELETE', body: JSON.stringify(id_list)});
-            $data_table.ajax.reload();
+            var success = function() {
+                var msg = "{% trans 'User Deleted.' %}";
+                swal("{% trans 'User Delete' %}", msg, "success");
+                $('#user_list_table').DataTable().ajax.reload();
+            };
+            var fail = function() {
+                var msg = "{% trans 'User Deleting failed.' %}";
+                swal("{% trans 'User Delete' %}", msg, "error");
+            };
+            var url_delete = the_url + '?id__in=' + JSON.stringify(plain_id_list);
+            APIUpdateAttr({url: url_delete, method: 'DELETE', success: success, error: fail});
         });
     }
-    function doExport() {}
     function doUpdate() {}
     switch(action) {
         case 'deactive':
@@ -178,12 +164,43 @@ $(document).ready(function(){
         case 'update':
             doUpdate();
             break;
-        case 'export':
-            doExport();
-            break;
         default:
             break;
     }
+}).on('click', '.btn_user_delete', function(){
+    var $this = $(this);
+    function doDelete() {
+        var uid = $this.data('uid');
+        var the_url = '{% url "users:user-patch-api" pk=99991937 %}'.replace('99991937', uid);
+        var body = {};
+        var success = function() {
+            var msg = "{% trans 'User Deleted.' %}";
+            swal("{% trans 'User Delete' %}", msg, "success");
+            $('#user_list_table').DataTable().ajax.reload();
+        };
+        var fail = function() {
+            var msg = "{% trans 'User Deleting failed.' %}";
+            swal("{% trans 'User Delete' %}", msg, "error");
+        }
+        APIUpdateAttr({
+            url: the_url,
+            body: JSON.stringify(body),
+            method: 'DELETE',
+            success: success,
+            error: fail
+        });
+    }
+    swal({
+        title: "{% trans 'Are you sure?' %}",
+        text: "{% trans 'This will delete the selected user.' %}",
+        type: "warning",
+        showCancelButton: true,
+        confirmButtonColor: "#DD6B55",
+        confirmButtonText: "{% trans 'Confirm' %}",
+        closeOnConfirm: false
+    }, function() {
+        doDelete();
+    });
 })
 </script>
 {% endblock %}
diff --git a/apps/users/templates/users/user_list.old.html b/apps/users/templates/users/user_list.old.html
deleted file mode 100644
index f9049f525..000000000
--- a/apps/users/templates/users/user_list.old.html
+++ /dev/null
@@ -1,113 +0,0 @@
-{% extends '_base_list.html' %}
-{% load i18n %}
-{% load common_tags %}
-{% block content_left_head %}
-    <a href="{% url 'users:user-create' %}" class="btn btn-sm btn-primary "> {% trans "Create user" %} </a>
-{% endblock %}
-
-{% block table_head %}
-    <th class="text-center">
-        <input type="checkbox" id="check_all" onclick="checkAll('check_all', 'checked')">
-    </th>
-    <th class="text-center"><a href="{% url 'users:user-list' %}?sort=name">{% trans 'Name' %}</a></th>
-    <th class="text-center"><a href="{% url 'users:user-list' %}?sort=username">{% trans 'Username' %}</a></th>
-    <th class="text-center">{% trans 'Role' %}</th>
-    <th class="text-center">{% trans 'User group' %}</th>
-    <th class="text-center">{% trans 'Asset num' %}</th>
-    <th class="text-center"><a href="{% url 'users:user-list' %}?sort=date_expired">{% trans 'Active' %}</a></th>
-    <th class="text-center"></th>
-{% endblock %}
-
-{% block table_body %}
-     {% for user in object_list %}
-         <tr class="gradeX">
-             <td class="text-center">
-                 <input type="checkbox" name="checked" value="{{ user.id }}" class="ipt_bulk_update">
-             </td>
-             <td class="text-center">
-                 <a href="{% url 'users:user-detail' pk=user.id %}">
-                     {{ user.name }}
-                 </a>
-             </td>
-             <td class="text-center">{{ user.username }}</td>
-             <td class="text-center">{{ user.get_role_display }}</td>
-             <td class="text-center" title="{% for user_group in user.group.all %} {{ user_group.name }} {% endfor %}"> {{ user.groups.all|join_queryset_attr:"name" }} </td>
-             <th class="text-center">{{ user.name }}</th>
-             <td class="text-center">
-                 {% if user.is_expired and user.is_active %}
-                     <i class="fa fa-times text-danger"></i>
-                 {% else %}
-                     <i class="fa fa-check text-navy"></i>
-                 {% endif %}
-             </td>
-             <td class="text-center">
-                 <a href="{% url 'users:user-update' pk=user.id %}" class="btn btn-xs btn-info">{% trans 'Update' %}</a>
-                 <!--<a href="{% url 'users:user-delete' pk=user.id %}" class="btn btn-xs btn-danger del {% if user.id == request.user.id or user.username == 'admin' %} disabled {% endif %}">{% trans 'Delete' %}</a>-->
-                 <a onclick="obj_del(this,'{{ user.name }}','{% url 'users:user-delete' user.id %}')" class="btn btn-xs btn-danger del {% if user.id == request.user.id or user.username == 'admin' %} disabled {% endif %}">{% trans 'Delete' %}</a>
-             </td>
-         </tr>
-     {% endfor %}
-{% endblock %}
-
-{% block content_bottom_left %}
-        <div class="input-group">
-            <select class="form-control m-b" style="width: auto" id="slct_bulk_update">
-                <option value="delete">{% trans 'Delete selected' %}</option>
-                <option value="update">{% trans 'Update selected' %}</option>
-                <option value="deactive">{% trans 'Deactive selected' %}</option>
-                <option value="export">{% trans 'Export selected' %}</option>
-            </select>
-            <div class="input-group-btn pull-left" style="padding-left: 5px;">
-                <button id='btn_bulk_update' style="height: 32px;"  class="btn btn-sm btn-primary">
-                 {% trans 'Submit' %}
-                </button>
-            </div>
-        </div>
-{% endblock %}
-{% block custom_foot_js %}
-<script>
-$(document).on('click', '#btn_bulk_update', function(){
-    var action = $('#slct_bulk_update').val();
-    var id_list = $('.ipt_bulk_update:checked').map(function(){return {'id': parseInt($(this).val())}}).get();
-    var the_url = "{% url 'users:user-bulk-update-api' %}";
-    function doDeactive() {
-        var body = $.each(id_list, function(index, user_object) {
-            user_object['is_active'] = false;
-        });
-        APIUpdateAttr({url: the_url, method: 'PATCH', body: JSON.stringify(body)});
-    };
-    function doDelete() {
-        swal({
-            title: "{% trans 'Are you sure?' %}",
-            text: "{% trans 'This will delete the selected users !!!' %}",
-            type: "warning",
-            showCancelButton: true,
-            confirmButtonColor: "#DD6B55",
-            confirmButtonText: "{% trans 'Confirm' %}",
-            closeOnConfirm: false
-        }, function() {
-            APIUpdateAttr({url: the_url, method: 'DELETE', body: JSON.stringify(id_list)});
-        });
-    };
-    function doExport() {};
-    function doUpdate() {};
-    switch(action) {
-        case 'deactive':
-            doDeactive();
-            break;
-        case 'delete':
-            doDelete();
-            break;
-        case 'update':
-            doUpdate();
-            break;
-        case 'export':
-            doExport();
-            break;
-        default:
-            break;
-    };
-})
-</script>
-{% endblock %}
-