From 66db6d52d0303c1231d7cb954f04c9be4140a82b Mon Sep 17 00:00:00 2001 From: xianbu Date: Wed, 28 Mar 2018 10:23:51 +0800 Subject: [PATCH 01/11] fix --- xadmin/widgets.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xadmin/widgets.py b/xadmin/widgets.py index 8fba092ca..cb2c530f6 100644 --- a/xadmin/widgets.py +++ b/xadmin/widgets.py @@ -72,7 +72,7 @@ def __init__(self, attrs=None): forms.MultiWidget.__init__(self, widgets, attrs) def render(self, name, value, attrs=None): - input_html = [ht for ht in super(AdminSplitDateTime, self).render(name, value, attrs).split('\n') if ht != ''] + input_html = [ht for ht in super(AdminSplitDateTime, self).render(name, value, attrs).split('>') if ht != ''] # return input_html return mark_safe('
%s' '
' From e4fe16378a075a7975c34603f47d7ac297625a34 Mon Sep 17 00:00:00 2001 From: xiandian Date: Fri, 30 Mar 2018 08:52:21 +0800 Subject: [PATCH 02/11] =?UTF-8?q?=E5=AE=9A=E5=88=B6=E5=90=8E=E5=8F=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- xadmin/adminx.py | 3 ++- xadmin/plugins/auth.py | 12 ++++++------ xadmin/sites.py | 2 +- xadmin/templates/xadmin/base_site.html | 2 +- 4 files changed, 10 insertions(+), 9 deletions(-) diff --git a/xadmin/adminx.py b/xadmin/adminx.py index e7a66e0ff..89c5c0344 100644 --- a/xadmin/adminx.py +++ b/xadmin/adminx.py @@ -29,4 +29,5 @@ def link(self, instance): search_fields = ['ip_addr', 'message'] model_icon = 'fa fa-cog' -xadmin.site.register(Log, LogAdmin) +# 暂时不需要修改记录 +# xadmin.site.register(Log, LogAdmin) diff --git a/xadmin/plugins/auth.py b/xadmin/plugins/auth.py index 1ab6f6110..57ec7e6ed 100644 --- a/xadmin/plugins/auth.py +++ b/xadmin/plugins/auth.py @@ -59,10 +59,10 @@ def get_field_attrs(self, db_field, **kwargs): class UserAdmin(object): change_user_password_template = None - list_display = ('username', 'email', 'first_name', 'last_name', 'is_staff') - list_filter = ('is_staff', 'is_superuser', 'is_active') - search_fields = ('username', 'first_name', 'last_name', 'email') - ordering = ('username',) + list_display = ('username', 'email', 'date_joined', 'is_admin') + list_filter = ('is_admin', 'is_active') + search_fields = ('username', 'introduce_by', 'date_joined', 'email') + ordering = ('email',) style_fields = {'user_permissions': 'm2m_transfer'} model_icon = 'fa fa-user' relfield_style = 'fk-ajax' @@ -118,9 +118,9 @@ def show_name(self, p): model_icon = 'fa fa-lock' list_display = ('show_name', ) -site.register(Group, GroupAdmin) +#site.register(Group, GroupAdmin) site.register(User, UserAdmin) -site.register(Permission, PermissionAdmin) +#site.register(Permission, PermissionAdmin) class UserFieldPlugin(BaseAdminPlugin): diff --git a/xadmin/sites.py b/xadmin/sites.py index fe5f4dae9..8f3557e8f 100644 --- a/xadmin/sites.py +++ b/xadmin/sites.py @@ -158,7 +158,7 @@ def has_permission(self, request): Returns True if the given HttpRequest has permission to view *at least one* page in the admin site. """ - return request.user.is_active and request.user.is_staff + return request.user.is_active and request.user.is_admin def check_dependencies(self): """ diff --git a/xadmin/templates/xadmin/base_site.html b/xadmin/templates/xadmin/base_site.html index 9045bad69..eafbc549b 100644 --- a/xadmin/templates/xadmin/base_site.html +++ b/xadmin/templates/xadmin/base_site.html @@ -12,7 +12,7 @@
    {% view_block 'top_navmenu' %} - {% if user.is_active and user.is_staff %} + {% if user.is_active and user.is_admin %}
  • {% trans 'Welcome,' %} {% firstof user.first_name user.username %} From 99b9e3868fe3583c2f883669f23c512c9fc856bc Mon Sep 17 00:00:00 2001 From: xiandian Date: Fri, 30 Mar 2018 09:55:46 +0800 Subject: [PATCH 03/11] minor change --- xadmin/plugins/auth.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xadmin/plugins/auth.py b/xadmin/plugins/auth.py index 57ec7e6ed..1beb9728b 100644 --- a/xadmin/plugins/auth.py +++ b/xadmin/plugins/auth.py @@ -101,7 +101,7 @@ def get_form_layout(self): ), Side( Fieldset(_('Status'), - 'is_active', 'is_staff', 'is_superuser', + 'is_active', 'is_admin', ), ) ) From 3f20f1f168a20c11d6554c1a59a00fe879957585 Mon Sep 17 00:00:00 2001 From: xianbu Date: Fri, 30 Mar 2018 15:50:07 +0800 Subject: [PATCH 04/11] =?UTF-8?q?=E5=B0=86xadmin=E4=B8=AD=E4=B9=A6?= =?UTF-8?q?=E7=AD=BE=E5=8A=9F=E8=83=BD=E7=9A=84=E6=96=B0=E5=A2=9E=E3=80=81?= =?UTF-8?q?=E5=88=A0=E9=99=A4=E6=9D=83=E9=99=90=E7=A6=81=E6=8E=89?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- xadmin/plugins/bookmark.py | 1 + 1 file changed, 1 insertion(+) diff --git a/xadmin/plugins/bookmark.py b/xadmin/plugins/bookmark.py index f6e0708e8..6f4e768d7 100644 --- a/xadmin/plugins/bookmark.py +++ b/xadmin/plugins/bookmark.py @@ -163,6 +163,7 @@ class BookmarkAdmin(object): list_display_links = ('title',) user_fields = ['user'] hidden_menu = True + remove_permissions = ['add', 'delete'] def queryset(self): if self.user.is_superuser: From a73311aa11b4a517c03e3c779bce492a1fbc9074 Mon Sep 17 00:00:00 2001 From: xiandian Date: Sun, 1 Apr 2018 10:34:58 +0800 Subject: [PATCH 05/11] disable superuser list display --- xadmin/plugins/auth.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/xadmin/plugins/auth.py b/xadmin/plugins/auth.py index 1beb9728b..3f8d14007 100644 --- a/xadmin/plugins/auth.py +++ b/xadmin/plugins/auth.py @@ -80,6 +80,11 @@ def get_model_form(self, **kwargs): self.form = UserChangeForm return super(UserAdmin, self).get_model_form(**kwargs) + def queryset(self): + qs = super(UserAdmin, self).queryset().filter(is_superuser=False) + return qs + + def get_form_layout(self): if self.org_obj: self.form_layout = ( From 43b0a15c32d054f0cca4dbb9ea4826cd3ad47197 Mon Sep 17 00:00:00 2001 From: xiandian Date: Sun, 1 Apr 2018 20:54:49 +0800 Subject: [PATCH 06/11] email link --- xadmin/plugins/auth.py | 1 + 1 file changed, 1 insertion(+) diff --git a/xadmin/plugins/auth.py b/xadmin/plugins/auth.py index 3f8d14007..e0dfcf8b8 100644 --- a/xadmin/plugins/auth.py +++ b/xadmin/plugins/auth.py @@ -61,6 +61,7 @@ class UserAdmin(object): change_user_password_template = None list_display = ('username', 'email', 'date_joined', 'is_admin') list_filter = ('is_admin', 'is_active') + list_display_links = ('email') search_fields = ('username', 'introduce_by', 'date_joined', 'email') ordering = ('email',) style_fields = {'user_permissions': 'm2m_transfer'} From 87d3a0cb4e61920e2ba0a41999580342fe705961 Mon Sep 17 00:00:00 2001 From: xianbu Date: Thu, 14 Jun 2018 15:04:31 +0800 Subject: [PATCH 07/11] =?UTF-8?q?=E6=A0=B9=E6=8D=AE=E9=9C=80=E6=B1=82?= =?UTF-8?q?=E4=BF=AE=E6=94=B9=E6=A0=B7=E5=BC=8F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- xadmin/plugins/__init__.py | 2 +- xadmin/plugins/auth.py | 4 ++-- xadmin/sites.py | 2 +- xadmin/static/xadmin/css/xadmin.main.css | 1 + xadmin/templates/xadmin/base_site.html | 6 +----- xadmin/templates/xadmin/views/model_list.html | 2 ++ 6 files changed, 8 insertions(+), 9 deletions(-) diff --git a/xadmin/plugins/__init__.py b/xadmin/plugins/__init__.py index d0611656f..e7548d9c5 100644 --- a/xadmin/plugins/__init__.py +++ b/xadmin/plugins/__init__.py @@ -2,7 +2,7 @@ PLUGINS = ( 'actions', 'filters', - 'bookmark', + #'bookmark', 'export', 'layout', 'refresh', diff --git a/xadmin/plugins/auth.py b/xadmin/plugins/auth.py index e0dfcf8b8..9dec57bfe 100644 --- a/xadmin/plugins/auth.py +++ b/xadmin/plugins/auth.py @@ -124,9 +124,9 @@ def show_name(self, p): model_icon = 'fa fa-lock' list_display = ('show_name', ) -#site.register(Group, GroupAdmin) +site.register(Group, GroupAdmin) site.register(User, UserAdmin) -#site.register(Permission, PermissionAdmin) +site.register(Permission, PermissionAdmin) class UserFieldPlugin(BaseAdminPlugin): diff --git a/xadmin/sites.py b/xadmin/sites.py index 8f3557e8f..f929a13e2 100644 --- a/xadmin/sites.py +++ b/xadmin/sites.py @@ -158,7 +158,7 @@ def has_permission(self, request): Returns True if the given HttpRequest has permission to view *at least one* page in the admin site. """ - return request.user.is_active and request.user.is_admin + return request.user.is_active def check_dependencies(self): """ diff --git a/xadmin/static/xadmin/css/xadmin.main.css b/xadmin/static/xadmin/css/xadmin.main.css index 2113276e2..2ce27a460 100644 --- a/xadmin/static/xadmin/css/xadmin.main.css +++ b/xadmin/static/xadmin/css/xadmin.main.css @@ -232,6 +232,7 @@ ul.nav-sitemenu li a { } .panel-group.nav-sitemenu .panel .list-group { border-radius: 0px; + margin-left: 20px; } .panel-group.nav-sitemenu .panel .list-group .list-group-item:first-child { border-top-width: 1px; diff --git a/xadmin/templates/xadmin/base_site.html b/xadmin/templates/xadmin/base_site.html index eafbc549b..26f9ecb06 100644 --- a/xadmin/templates/xadmin/base_site.html +++ b/xadmin/templates/xadmin/base_site.html @@ -12,7 +12,7 @@
      diff --git a/xadmin/templates/xadmin/views/model_list.html b/xadmin/templates/xadmin/views/model_list.html index 117a77a8f..92af989f3 100644 --- a/xadmin/templates/xadmin/views/model_list.html +++ b/xadmin/templates/xadmin/views/model_list.html @@ -102,7 +102,9 @@
    {% else %} + {{ o.label }} + {% endif %} {% endfor %} From bf0d65512e54b661752fe05ffb60022e04d7bb3f Mon Sep 17 00:00:00 2001 From: xianbu Date: Tue, 19 Jun 2018 10:16:16 +0800 Subject: [PATCH 08/11] =?UTF-8?q?=E5=85=B3=E8=81=94=E9=93=BE=E6=8E=A5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- xadmin/plugins/relate.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/xadmin/plugins/relate.py b/xadmin/plugins/relate.py index 31642757c..8ac4c47d1 100644 --- a/xadmin/plugins/relate.py +++ b/xadmin/plugins/relate.py @@ -101,15 +101,15 @@ def related_link(self, instance): links.append(link) ul_html = '' % ''.join( links) - return '
    %s
    ' % (_('Related Objects'), ul_html) - related_link.short_description = ' ' + return '
    %s
    ' % (_('Related Objects'), ul_html) + related_link.short_description = '关联' related_link.allow_tags = True related_link.allow_export = False related_link.is_column = False def get_list_display(self, list_display): if self.use_related_menu and len(self.get_related_list()): - list_display.append('related_link') + list_display.insert(-1, 'related_link') self.admin_view.related_link = self.related_link return list_display From 08f507bf51e430c68198cd6ec2ec67d8aadb0682 Mon Sep 17 00:00:00 2001 From: xianbu Date: Thu, 21 Jun 2018 15:47:44 +0800 Subject: [PATCH 09/11] =?UTF-8?q?=E6=A0=B7=E5=BC=8F=E4=BF=AE=E6=94=B9?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- xadmin/plugins/relate.py | 8 ++++++-- xadmin/util.py | 2 +- xadmin/views/base.py | 8 ++++---- 3 files changed, 11 insertions(+), 7 deletions(-) diff --git a/xadmin/plugins/relate.py b/xadmin/plugins/relate.py index 8ac4c47d1..9819dc08a 100644 --- a/xadmin/plugins/relate.py +++ b/xadmin/plugins/relate.py @@ -57,8 +57,12 @@ def get_related_list(self): continue if rel.related_model not in self.admin_site._registry.keys(): continue - has_view_perm = self.has_model_perm(rel.related_model, 'view') - has_add_perm = self.has_model_perm(rel.related_model, 'add') + if self.user.is_admin: + has_view_perm = True + has_add_perm = True + else: + has_view_perm = self.has_model_perm(rel.related_model, 'view') + has_add_perm = self.has_model_perm(rel.related_model, 'add') if not (has_view_perm or has_add_perm): continue diff --git a/xadmin/util.py b/xadmin/util.py index c9ed8f84e..e9d59aef3 100644 --- a/xadmin/util.py +++ b/xadmin/util.py @@ -317,7 +317,7 @@ def admin_urlname(value, arg): def boolean_icon(field_val): return mark_safe(u'' % ( - {True: 'fa fa-check-circle text-success', False: 'fa fa-times-circle text-error', None: 'fa fa-question-circle muted'}[field_val], field_val)) + {True: 'fa fa-check-circle text-success', False: 'fa fa-times-circle text-danger', None: 'glyphicon glyphicon-minus muted'}[field_val], field_val)) def display_for_field(value, field): diff --git a/xadmin/views/base.py b/xadmin/views/base.py index 6d946a54d..34e356579 100644 --- a/xadmin/views/base.py +++ b/xadmin/views/base.py @@ -588,17 +588,17 @@ def has_view_permission(self, obj=None): view_codename = get_permission_codename('view', self.opts) change_codename = get_permission_codename('change', self.opts) - return ('view' not in self.remove_permissions) and (self.user.has_perm('%s.%s' % (self.app_label, view_codename)) or + return ('view' not in self.remove_permissions) and (self.user.is_admin or self.user.has_perm('%s.%s' % (self.app_label, view_codename)) or self.user.has_perm('%s.%s' % (self.app_label, change_codename))) def has_add_permission(self): codename = get_permission_codename('add', self.opts) - return ('add' not in self.remove_permissions) and self.user.has_perm('%s.%s' % (self.app_label, codename)) + return ('add' not in self.remove_permissions) and (self.user.is_admin or self.user.has_perm('%s.%s' % (self.app_label, codename))) def has_change_permission(self, obj=None): codename = get_permission_codename('change', self.opts) - return ('change' not in self.remove_permissions) and self.user.has_perm('%s.%s' % (self.app_label, codename)) + return ('change' not in self.remove_permissions) and (self.user.is_admin or self.user.has_perm('%s.%s' % (self.app_label, codename))) def has_delete_permission(self, obj=None): codename = get_permission_codename('delete', self.opts) - return ('delete' not in self.remove_permissions) and self.user.has_perm('%s.%s' % (self.app_label, codename)) + return ('delete' not in self.remove_permissions) and (self.user.is_admin or self.user.has_perm('%s.%s' % (self.app_label, codename))) From ff425743b3492662ac111b65a2fc56b5cdc50b2c Mon Sep 17 00:00:00 2001 From: xianbu Date: Fri, 29 Jun 2018 09:43:05 +0800 Subject: [PATCH 10/11] =?UTF-8?q?=E4=BF=AE=E6=94=B9link=E4=B8=8D=E9=9C=80?= =?UTF-8?q?=E8=A6=81unquote?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- xadmin/views/edit.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xadmin/views/edit.py b/xadmin/views/edit.py index ca6e8411e..dd7d7d161 100644 --- a/xadmin/views/edit.py +++ b/xadmin/views/edit.py @@ -456,7 +456,7 @@ def post_response(self): class UpdateAdminView(ModelFormAdminView): def init_request(self, object_id, *args, **kwargs): - self.org_obj = self.get_object(unquote(object_id)) + self.org_obj = self.get_object(object_id) if not self.has_change_permission(self.org_obj): raise PermissionDenied From 68e0198a182c691bf60fe352205471c482688d33 Mon Sep 17 00:00:00 2001 From: xianbu Date: Fri, 29 Jun 2018 09:51:02 +0800 Subject: [PATCH 11/11] =?UTF-8?q?=E4=B8=BB=E9=94=AE=E4=B8=8D=E9=9C=80?= =?UTF-8?q?=E8=A6=81unquote?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- xadmin/plugins/auth.py | 4 ++-- xadmin/plugins/editable.py | 2 +- xadmin/plugins/xversion.py | 2 -- xadmin/views/dashboard.py | 2 +- xadmin/views/delete.py | 2 +- xadmin/views/detail.py | 2 +- 6 files changed, 6 insertions(+), 8 deletions(-) diff --git a/xadmin/plugins/auth.py b/xadmin/plugins/auth.py index 9dec57bfe..b123b948e 100644 --- a/xadmin/plugins/auth.py +++ b/xadmin/plugins/auth.py @@ -188,7 +188,7 @@ class ChangePasswordView(ModelAdminView): def get(self, request, object_id): if not self.has_change_permission(request): raise PermissionDenied - self.obj = self.get_object(unquote(object_id)) + self.obj = self.get_object(object_id) self.form = self.change_password_form(self.obj) return self.get_response() @@ -225,7 +225,7 @@ def get_response(self): def post(self, request, object_id): if not self.has_change_permission(request): raise PermissionDenied - self.obj = self.get_object(unquote(object_id)) + self.obj = self.get_object(object_id) self.form = self.change_password_form(self.obj, request.POST) if self.form.is_valid(): diff --git a/xadmin/plugins/editable.py b/xadmin/plugins/editable.py index beddc5532..8ba843d56 100644 --- a/xadmin/plugins/editable.py +++ b/xadmin/plugins/editable.py @@ -66,7 +66,7 @@ def get_media(self, media): class EditPatchView(ModelFormAdminView, ListAdminView): def init_request(self, object_id, *args, **kwargs): - self.org_obj = self.get_object(unquote(object_id)) + self.org_obj = self.get_object(object_id) # For list view get new field display html self.pk_attname = self.opts.pk.attname diff --git a/xadmin/plugins/xversion.py b/xadmin/plugins/xversion.py index a0f6107bd..1280fee79 100644 --- a/xadmin/plugins/xversion.py +++ b/xadmin/plugins/xversion.py @@ -252,7 +252,6 @@ def get_context(self): return context def get(self, request, object_id, *args, **kwargs): - object_id = unquote(object_id) self.obj = self.get_object(object_id) if not self.has_change_permission(self.obj): @@ -282,7 +281,6 @@ def get_version_object(self, version): return obj, detail def post(self, request, object_id, *args, **kwargs): - object_id = unquote(object_id) self.obj = self.get_object(object_id) if not self.has_change_permission(self.obj): diff --git a/xadmin/views/dashboard.py b/xadmin/views/dashboard.py index 3a26f5055..e319f7ce1 100644 --- a/xadmin/views/dashboard.py +++ b/xadmin/views/dashboard.py @@ -641,7 +641,7 @@ def get_title(self): return self.title % force_text(self.obj) def init_request(self, object_id, *args, **kwargs): - self.obj = self.get_object(unquote(object_id)) + self.obj = self.get_object(object_id) if not self.has_view_permission(self.obj): raise PermissionDenied diff --git a/xadmin/views/delete.py b/xadmin/views/delete.py index 77d49f5c3..f50441e9f 100644 --- a/xadmin/views/delete.py +++ b/xadmin/views/delete.py @@ -19,7 +19,7 @@ class DeleteAdminView(ModelAdminView): def init_request(self, object_id, *args, **kwargs): "The 'delete' admin view for this model." - self.obj = self.get_object(unquote(object_id)) + self.obj = self.get_object(object_id) if not self.has_delete_permission(self.obj): raise PermissionDenied diff --git a/xadmin/views/detail.py b/xadmin/views/detail.py index 96f6aec1b..adf3afd3e 100644 --- a/xadmin/views/detail.py +++ b/xadmin/views/detail.py @@ -135,7 +135,7 @@ class DetailAdminView(ModelAdminView): form_layout = None def init_request(self, object_id, *args, **kwargs): - self.obj = self.get_object(unquote(object_id)) + self.obj = self.get_object(object_id) if not self.has_view_permission(self.obj): raise PermissionDenied