PEP8 cleanups.
This commit is contained in:
@@ -2,7 +2,6 @@ from __future__ import absolute_import, unicode_literals
|
|||||||
|
|
||||||
from django.contrib.contenttypes.models import ContentType
|
from django.contrib.contenttypes.models import ContentType
|
||||||
|
|
||||||
from documents.permissions import permission_document_view
|
|
||||||
from documents.tests.test_views import GenericDocumentViewTestCase
|
from documents.tests.test_views import GenericDocumentViewTestCase
|
||||||
from user_management.tests import (
|
from user_management.tests import (
|
||||||
TEST_USER_USERNAME, TEST_USER_PASSWORD
|
TEST_USER_USERNAME, TEST_USER_PASSWORD
|
||||||
|
|||||||
@@ -3,7 +3,6 @@ from __future__ import absolute_import, unicode_literals
|
|||||||
import itertools
|
import itertools
|
||||||
import logging
|
import logging
|
||||||
|
|
||||||
from django.contrib import messages
|
|
||||||
from django.contrib.contenttypes.models import ContentType
|
from django.contrib.contenttypes.models import ContentType
|
||||||
from django.core.exceptions import PermissionDenied
|
from django.core.exceptions import PermissionDenied
|
||||||
from django.core.urlresolvers import reverse
|
from django.core.urlresolvers import reverse
|
||||||
|
|||||||
@@ -282,7 +282,7 @@ class SingleObjectCreateView(ObjectNameMixin, ViewPermissionCheckMixin, ExtraCon
|
|||||||
messages.error(
|
messages.error(
|
||||||
self.request,
|
self.request,
|
||||||
_('%(object)s not created, error: %(error)s') % {
|
_('%(object)s not created, error: %(error)s') % {
|
||||||
'object': self.get_object_name(),
|
'object': self.get_object_name(context=context),
|
||||||
'error': exception
|
'error': exception
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
@@ -293,7 +293,7 @@ class SingleObjectCreateView(ObjectNameMixin, ViewPermissionCheckMixin, ExtraCon
|
|||||||
self.request,
|
self.request,
|
||||||
_(
|
_(
|
||||||
'%(object)s created successfully.'
|
'%(object)s created successfully.'
|
||||||
) % {'object': self.get_object_name()}
|
) % {'object': self.get_object_name(context=context)}
|
||||||
)
|
)
|
||||||
|
|
||||||
return HttpResponseRedirect(self.get_success_url())
|
return HttpResponseRedirect(self.get_success_url())
|
||||||
|
|||||||
@@ -35,8 +35,7 @@ from .links import (
|
|||||||
)
|
)
|
||||||
from .permissions import (
|
from .permissions import (
|
||||||
permission_document_indexing_create, permission_document_indexing_delete,
|
permission_document_indexing_create, permission_document_indexing_delete,
|
||||||
permission_document_indexing_edit, permission_document_indexing_view,
|
permission_document_indexing_edit, permission_document_indexing_view
|
||||||
permission_document_indexing_rebuild
|
|
||||||
)
|
)
|
||||||
from .widgets import get_instance_link, index_instance_item_link, node_level
|
from .widgets import get_instance_link, index_instance_item_link, node_level
|
||||||
|
|
||||||
|
|||||||
@@ -2,13 +2,11 @@ from __future__ import unicode_literals
|
|||||||
|
|
||||||
from django.utils.translation import ugettext_lazy as _
|
from django.utils.translation import ugettext_lazy as _
|
||||||
|
|
||||||
from documents.permissions import permission_document_view
|
|
||||||
from navigation import Link
|
from navigation import Link
|
||||||
|
|
||||||
from .permissions import (
|
from .permissions import (
|
||||||
permission_document_indexing_create, permission_document_indexing_edit,
|
permission_document_indexing_create, permission_document_indexing_edit,
|
||||||
permission_document_indexing_delete, permission_document_indexing_rebuild,
|
permission_document_indexing_delete, permission_document_indexing_rebuild,
|
||||||
permission_document_indexing_view
|
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -1,6 +1,5 @@
|
|||||||
from __future__ import absolute_import, unicode_literals
|
from __future__ import absolute_import, unicode_literals
|
||||||
|
|
||||||
from documents.permissions import permission_document_view
|
|
||||||
from documents.tests.test_views import GenericDocumentViewTestCase
|
from documents.tests.test_views import GenericDocumentViewTestCase
|
||||||
from user_management.tests import (
|
from user_management.tests import (
|
||||||
TEST_USER_USERNAME, TEST_USER_PASSWORD
|
TEST_USER_USERNAME, TEST_USER_PASSWORD
|
||||||
@@ -9,7 +8,7 @@ from user_management.tests import (
|
|||||||
from ..models import Index
|
from ..models import Index
|
||||||
from ..permissions import (
|
from ..permissions import (
|
||||||
permission_document_indexing_create, permission_document_indexing_delete,
|
permission_document_indexing_create, permission_document_indexing_delete,
|
||||||
permission_document_indexing_edit, permission_document_indexing_view
|
permission_document_indexing_edit
|
||||||
)
|
)
|
||||||
|
|
||||||
TEST_INDEX_LABEL = 'test label'
|
TEST_INDEX_LABEL = 'test label'
|
||||||
|
|||||||
@@ -3,9 +3,7 @@ from __future__ import absolute_import, unicode_literals
|
|||||||
from django.contrib import messages
|
from django.contrib import messages
|
||||||
from django.core.exceptions import PermissionDenied
|
from django.core.exceptions import PermissionDenied
|
||||||
from django.core.urlresolvers import reverse, reverse_lazy
|
from django.core.urlresolvers import reverse, reverse_lazy
|
||||||
from django.http import HttpResponseRedirect
|
from django.shortcuts import get_object_or_404
|
||||||
from django.shortcuts import get_object_or_404, render_to_response
|
|
||||||
from django.template import RequestContext
|
|
||||||
from django.utils.html import mark_safe
|
from django.utils.html import mark_safe
|
||||||
from django.utils.translation import ugettext_lazy as _
|
from django.utils.translation import ugettext_lazy as _
|
||||||
|
|
||||||
@@ -147,6 +145,7 @@ class SetupIndexTreeTemplateListView(SingleObjectListView):
|
|||||||
'title': _('Tree template nodes for index: %s') % self.get_index(),
|
'title': _('Tree template nodes for index: %s') % self.get_index(),
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
class TemplateNodeCreateView(SingleObjectCreateView):
|
class TemplateNodeCreateView(SingleObjectCreateView):
|
||||||
form_class = IndexTemplateNodeForm
|
form_class = IndexTemplateNodeForm
|
||||||
model = IndexTemplateNode
|
model = IndexTemplateNode
|
||||||
|
|||||||
@@ -3,7 +3,6 @@ from __future__ import unicode_literals
|
|||||||
|
|
||||||
from django.apps import apps
|
from django.apps import apps
|
||||||
from django.utils.html import mark_safe, escape
|
from django.utils.html import mark_safe, escape
|
||||||
from django.utils.translation import ugettext
|
|
||||||
|
|
||||||
|
|
||||||
def get_instance_link(index_instance_node):
|
def get_instance_link(index_instance_node):
|
||||||
|
|||||||
Reference in New Issue
Block a user