mirror of
https://github.com/jazzband/django-admin2.git
synced 2026-03-16 22:20:24 +00:00
fixes to review by @andrewsmedina #403
This commit is contained in:
parent
84ae23b903
commit
cde4de20c8
3 changed files with 2 additions and 6 deletions
|
|
@ -123,7 +123,6 @@ class Admin2(object):
|
|||
try:
|
||||
import_module("%s.admin2" % app_name)
|
||||
except ImportError as e:
|
||||
|
||||
if str(e).startswith("No module named") and 'admin2' in str(e):
|
||||
continue
|
||||
raise e
|
||||
|
|
|
|||
|
|
@ -2,7 +2,6 @@
|
|||
from __future__ import division, absolute_import, unicode_literals
|
||||
|
||||
import collections
|
||||
import six
|
||||
|
||||
from itertools import chain
|
||||
|
||||
|
|
@ -12,6 +11,7 @@ from django.utils.html import format_html
|
|||
from django.utils.encoding import force_text, force_bytes
|
||||
from django.utils.safestring import mark_safe
|
||||
from django.forms import widgets as django_widgets
|
||||
from django.utils import six
|
||||
from django.utils.translation import ugettext_lazy
|
||||
|
||||
import django_filters
|
||||
|
|
|
|||
|
|
@ -28,6 +28,7 @@ from . import permissions, utils
|
|||
from .forms import AdminAuthenticationForm
|
||||
from .viewmixins import Admin2Mixin, AdminModel2Mixin, Admin2ModelFormMixin
|
||||
from .filters import build_list_filter, build_date_filter
|
||||
from .models import LogEntry
|
||||
|
||||
class AdminView(object):
|
||||
|
||||
|
|
@ -390,7 +391,6 @@ class ModelEditFormView(AdminModel2Mixin, Admin2ModelFormMixin,
|
|||
|
||||
def forms_valid(self, form, inlines):
|
||||
response = super(ModelEditFormView, self).forms_valid(form, inlines)
|
||||
from .models import LogEntry
|
||||
LogEntry.objects.log_action(
|
||||
self.request.user.id,
|
||||
self.object,
|
||||
|
|
@ -427,7 +427,6 @@ class ModelAddFormView(AdminModel2Mixin, Admin2ModelFormMixin,
|
|||
|
||||
def forms_valid(self, form, inlines):
|
||||
response = super(ModelAddFormView, self).forms_valid(form, inlines)
|
||||
from .models import LogEntry
|
||||
LogEntry.objects.log_action(
|
||||
self.request.user.id,
|
||||
self.object,
|
||||
|
|
@ -471,7 +470,6 @@ class ModelDeleteView(AdminModel2Mixin, generic.DeleteView):
|
|||
return context
|
||||
|
||||
def delete(self, request, *args, **kwargs):
|
||||
from .models import LogEntry
|
||||
LogEntry.objects.log_action(
|
||||
request.user.id,
|
||||
self.get_object(),
|
||||
|
|
@ -509,7 +507,6 @@ class ModelHistoryView(AdminModel2Mixin, generic.ListView):
|
|||
|
||||
def get_queryset(self):
|
||||
content_type = ContentType.objects.get_for_model(self.get_object())
|
||||
from .models import LogEntry
|
||||
return LogEntry.objects.filter(
|
||||
content_type=content_type,
|
||||
object_id=self.get_object().id
|
||||
|
|
|
|||
Loading…
Reference in a new issue