PEP8 and code style cleanups.

mayan/apps/acls/tests/test_models.py

@@ -2,7 +2,7 @@ from __future__ import absolute_import, unicode_literals
 
 from django.contrib.auth import get_user_model
 from django.core.exceptions import PermissionDenied
-from django.test import TestCase, override_settings
+from django.test import override_settings
 
 from documents.models import Document, DocumentType
 from documents.permissions import permission_document_view

mayan/apps/authentication/auth/model_auth_backend.py

@@ -1,6 +1,5 @@
 from __future__ import unicode_literals
 
-from django.conf import settings
 from django.contrib.auth import get_user_model
 from django.contrib.auth.backends import ModelBackend
 

mayan/apps/authentication/tests/test_views.py

@@ -3,7 +3,7 @@ from __future__ import absolute_import, unicode_literals
 from django.conf import settings
 from django.contrib.auth import get_user_model
 from django.core.urlresolvers import reverse
-from django.test import TestCase, override_settings
+from django.test import override_settings
 from django.test.client import Client
 
 from organizations.tests.base import OrganizationTestCase

mayan/apps/checkouts/tests/test_models.py

@@ -4,7 +4,7 @@ import datetime
 import time
 
 from django.contrib.auth import get_user_model
-from django.test import TestCase, override_settings
+from django.test import override_settings
 from django.utils.timezone import now
 
 from documents.exceptions import NewDocumentVersionNotAllowed

mayan/apps/common/management/commands/initialsetup.py

@@ -12,9 +12,15 @@ class Command(management.BaseCommand):
     def handle(self, *args, **options):
         management.call_command('createsettings', interactive=False)
         try:
-            result = management.call_command('migrate', interactive=False)
-        except OperationalError as exception:
-            self.stderr.write(self.style.NOTICE('Unable to migrate the database. The initialsetup command is to be used only on new installations. To upgrade existing installations use the performupgrade command.'))
+            management.call_command('migrate', interactive=False)
+        except OperationalError:
+            self.stderr.write(
+                self.style.NOTICE(
+                    'Unable to migrate the database. The initialsetup command '
+                    'is to be used only on new installations. To upgrade '
+                    'existing installations use the performupgrade command.'
+                )
+            )
             raise
         management.call_command('createautoadmin', interactive=False)
         management.call_command('createorganizationadmin', interactive=False)

mayan/apps/common/tests/test_views.py

@@ -5,7 +5,6 @@ from django.contrib.auth import get_user_model
 from django.core.urlresolvers import clear_url_caches, reverse
 from django.http import HttpResponse
 from django.template import Context, Template
-from django.test import TestCase
 
 from organizations.tests.base import OrganizationTestCase
 from permissions import Permission

mayan/apps/document_states/tests/test_organization_views.py

@@ -6,12 +6,7 @@ from organizations.tests.test_organization_views import OrganizationViewTestCase
 
 from ..models import Workflow
 
-from .literals import (
-    TEST_WORKFLOW_LABEL, TEST_WORKFLOW_LABEL_EDITED,
-    TEST_WORKFLOW_INITIAL_STATE_LABEL, TEST_WORKFLOW_INITIAL_STATE_COMPLETION,
-    TEST_WORKFLOW_STATE_LABEL, TEST_WORKFLOW_STATE_COMPLETION,
-    TEST_WORKFLOW_TRANSITION_LABEL
-)
+from .literals import TEST_WORKFLOW_LABEL, TEST_WORKFLOW_LABEL_EDITED
 
 
 @override_settings(OCR_AUTO_OCR=False)

mayan/apps/documents/managers.py

@@ -4,7 +4,6 @@ from datetime import timedelta
 import logging
 
 from django.apps import apps
-from django.conf import settings
 from django.db import models
 from django.utils.timezone import now
 
@@ -182,8 +181,6 @@ class RecentDocumentManager(models.Manager):
 
 class TrashedDocumentManager(models.Manager):
     def get_queryset(self):
-        DocumentType = apps.get_model('documents', 'DocumentType')
-
         return super(
             TrashedDocumentManager, self
         ).get_queryset().filter(in_trash=True)

mayan/apps/documents/tests/test_api.py

@@ -11,9 +11,6 @@ from django.utils.six import BytesIO
 from rest_framework import status
 
 from rest_api.tests import GenericAPITestCase
-from user_management.tests.literals import (
-    TEST_ADMIN_EMAIL, TEST_ADMIN_PASSWORD, TEST_ADMIN_USERNAME
-)
 
 from .literals import (
     TEST_DOCUMENT_FILENAME, TEST_DOCUMENT_PATH, TEST_DOCUMENT_TYPE,

mayan/apps/documents/tests/test_models.py

@@ -3,8 +3,7 @@ from __future__ import unicode_literals
 from datetime import timedelta
 import time
 
-from common.tests import BaseTestCase
-from django.test import TestCase, override_settings
+from django.test import override_settings
 
 from organizations.tests.base import OrganizationTestCase
 

mayan/apps/documents/tests/test_organization_views.py

@@ -2,12 +2,10 @@ from __future__ import unicode_literals
 
 from organizations.tests.test_organization_views import OrganizationViewTestCase
 
-from ..models import Document,DocumentType
+from ..models import DocumentType
 
 from .literals import (
-    TEST_DOCUMENT_TYPE, TEST_DOCUMENT_TYPE_2_LABEL,
-    TEST_DOCUMENT_TYPE_QUICK_LABEL, TEST_SMALL_DOCUMENT_CHECKSUM,
-    TEST_SMALL_DOCUMENT_PATH
+    TEST_DOCUMENT_TYPE, TEST_DOCUMENT_TYPE_2_LABEL, TEST_SMALL_DOCUMENT_PATH
 )
 
 

mayan/apps/documents/tests/test_views.py

@@ -367,7 +367,6 @@ class DocumentsViewsTestCase(GenericDocumentViewTestCase):
         self.assertContains(response, text='queued', status_code=200)
         self.assertEqual(self.document.pages.count(), page_count)
 
-
     def test_document_multiple_update_page_count_view_no_permission(self):
         self.login(username=TEST_USER_USERNAME, password=TEST_USER_PASSWORD)
 

mayan/apps/folders/tests/test_api.py

@@ -1,15 +1,11 @@
 from __future__ import unicode_literals
 
-from django.contrib.auth import get_user_model
 from django.core.urlresolvers import reverse
 from django.test import override_settings
 
 from documents.models import DocumentType
 from documents.tests import TEST_DOCUMENT_TYPE, TEST_SMALL_DOCUMENT_PATH
 from rest_api.tests import GenericAPITestCase
-from user_management.tests.literals import (
-    TEST_ADMIN_EMAIL, TEST_ADMIN_PASSWORD, TEST_ADMIN_USERNAME
-)
 
 from ..models import Folder
 

mayan/apps/folders/tests/test_organization_views.py

@@ -11,7 +11,7 @@ class FolderOrganizationViewTestCase(OrganizationViewTestCase):
     def test_folder_create_view(self):
         # Create a folder for organization A
         with self.settings(ORGANIZATION_ID=self.organization_a.pk):
-            response = self.post(
+            self.post(
                 'folders:folder_create', data={
                     'label': TEST_FOLDER_LABEL
                 }

mayan/apps/metadata/tests/test_api.py

@@ -1,15 +1,11 @@
 from __future__ import unicode_literals
 
-from django.contrib.auth import get_user_model
 from django.core.urlresolvers import reverse
 from django.test import override_settings
 
 from documents.models import DocumentType
 from documents.tests import TEST_DOCUMENT_TYPE, TEST_SMALL_DOCUMENT_PATH
 from rest_api.tests import GenericAPITestCase
-from user_management.tests.literals import (
-    TEST_ADMIN_EMAIL, TEST_ADMIN_PASSWORD, TEST_ADMIN_USERNAME
-)
 
 from ..models import DocumentMetadata, DocumentTypeMetadataType, MetadataType
 

mayan/apps/metadata/tests/test_organization_views.py

@@ -5,9 +5,8 @@ from organizations.tests.test_organization_views import OrganizationViewTestCase
 from ..models import MetadataType
 
 from .literals import (
-    TEST_DOCUMENT_METADATA_VALUE_2, TEST_METADATA_TYPE_LABEL,
-    TEST_METADATA_TYPE_LABEL_2, TEST_METADATA_TYPE_NAME,
-    TEST_METADATA_TYPE_NAME_2
+    TEST_METADATA_TYPE_LABEL, TEST_METADATA_TYPE_LABEL_2,
+    TEST_METADATA_TYPE_NAME,
 )
 
 

mayan/apps/ocr/parsers.py

@@ -11,7 +11,6 @@ import subprocess
 
 from django.utils.translation import ugettext_lazy as _
 
-from common.settings import setting_temporary_directory
 from common.utils import copyfile, fs_cleanup, mkstemp
 
 from .exceptions import ParserError, NoMIMETypeMatch

mayan/apps/ocr/tests/test_models.py

@@ -2,8 +2,6 @@
 
 from __future__ import unicode_literals
 
-from django.core.files.base import File
-
 from documents.models import DocumentType
 from documents.settings import setting_language_choices
 from documents.tests import (

mayan/apps/ocr/tests/test_parsers.py

@@ -1,10 +1,7 @@
 from __future__ import unicode_literals
 
-from django.core.files.base import File
 from django.test import override_settings
 
-from common.settings import setting_temporary_directory
-from common.tests import BaseTestCase
 from documents.models import DocumentType
 from documents.tests import (
     TEST_DOCUMENT_PATH, TEST_DOCUMENT_TYPE, TEST_HYBRID_DOCUMENT_PATH

mayan/apps/organizations/apps.py

@@ -1,12 +1,10 @@
 from __future__ import unicode_literals
 
-from django.apps import AppConfig
-
 from django.utils.translation import ugettext_lazy as _
 
 from common.apps import MayanAppConfig
 
 
-class OrganizationApp(AppConfig):
+class OrganizationApp(MayanAppConfig):
     name = 'organizations'
     verbose_name = _('Organizations')

mayan/apps/organizations/management/commands/createorganizationadmin.py

@@ -1,15 +1,10 @@
 from __future__ import unicode_literals
 
-import os
-
-from django.conf import settings
 from django.core import management
-from django.utils.crypto import get_random_string
 
 from ...models import Organization
 
 
-
 class Command(management.BaseCommand):
     help = 'Creates an organization admin user with a secure random password and all permissions.'
 

mayan/apps/organizations/migrations/0002_add_data_default_organization.py

@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
+from django.db import migrations
 
 from ..literals import DEFAULT_ORGANIZATION_LABEL
 
@@ -9,7 +9,7 @@ from ..literals import DEFAULT_ORGANIZATION_LABEL
 def default_data(apps, schema_editor):
     # We can't import the Organization model directly as it may be a newer
     # version than this migration expects. We use the historical version.
-    Organization = apps.get_model("organizations", "Organization")
+    Organization = apps.get_model('organizations', 'Organization')
     default_organization = Organization(label=DEFAULT_ORGANIZATION_LABEL)
     default_organization.save()
 

mayan/apps/organizations/models.py

@@ -1,16 +1,13 @@
 from __future__ import absolute_import, unicode_literals
 
 import logging
-import string
-import warnings
 
 from django.apps import apps
 from django.conf import settings
 from django.contrib.auth import get_user_model
-from django.core.exceptions import ImproperlyConfigured, ValidationError
+from django.core.exceptions import ImproperlyConfigured
 from django.db import models
 from django.db.models.signals import pre_save, pre_delete
-from django.utils.deprecation import RemovedInDjango19Warning
 from django.utils.encoding import python_2_unicode_compatible
 from django.utils.translation import ugettext_lazy as _
 
@@ -110,7 +107,7 @@ class Organization(models.Model):
             )
 
             account = UserModel.objects.get(
-                **{UserModel.USERNAME_FIELD: username, 'organization': self,}
+                **{UserModel.USERNAME_FIELD: username, 'organization': self}
             )
             account.set_password(raw_password=password_value)
             account.save()

mayan/apps/organizations/settings.py

@@ -1,7 +1,5 @@
 from __future__ import unicode_literals
 
-import tempfile
-
 from django.utils.translation import ugettext_lazy as _
 
 from smart_settings import Namespace

mayan/apps/organizations/shortcuts.py

@@ -4,5 +4,5 @@ from django.apps import apps
 
 
 def get_current_organization():
-    from .models import Organization
+    Organization = apps.get_model('organizations', 'Organizations')
     return Organization.objects.get_current().pk

mayan/apps/organizations/tests/literals.py

@@ -1,4 +1,3 @@
 from __future__ import absolute_import, unicode_literals
 
 TEST_ORGANIZATION_LABEL = 'test organization label'
-TEST_ORGANIZATION_EDITED_LABEL = 'test organization edited label'

mayan/apps/organizations/tests/test_middleware.py

@@ -4,7 +4,7 @@ import unittest
 
 from django.apps import apps
 from django.conf import settings
-from django.core.exceptions import ObjectDoesNotExist, ValidationError
+from django.core.exceptions import ObjectDoesNotExist
 from django.db import connections, router
 from django.http import HttpRequest
 from django.test import TestCase, modify_settings, override_settings

mayan/apps/organizations/tests/test_models.py

@@ -4,7 +4,7 @@ from django.test import TestCase
 
 from ..models import Organization
 
-from .literals import TEST_ORGANIZATION_LABEL, TEST_ORGANIZATION_EDITED_LABEL
+from .literals import TEST_ORGANIZATION_LABEL
 
 
 class OrganizationModelTestCase(TestCase):

mayan/apps/organizations/utils.py

@@ -3,7 +3,6 @@ from __future__ import unicode_literals
 from django.apps import apps
 from django.core.management.color import no_style
 from django.db import DEFAULT_DB_ALIAS, connections, router
-from django.db.models import signals
 
 from .literals import DEFAULT_ORGANIZATION_LABEL
 
@@ -28,12 +27,14 @@ def create_default_organization(verbosity=2, interactive=True, using=DEFAULT_DB_
         # the next id will be 1, so we coerce it. See #15573 and #16353. This
         # can also crop up outside of tests - see #15346.
         if verbosity >= 2:
-            print("Creating default Organization object")
+            print('Creating default Organization object')
         Organization(pk=1, label=DEFAULT_ORGANIZATION_LABEL).save(using=using)
 
         # We set an explicit pk instead of relying on auto-incrementation,
         # so we need to reset the database sequence. See #17415.
-        sequence_sql = connections[using].ops.sequence_reset_sql(no_style(), [Organization])
+        sequence_sql = connections[using].ops.sequence_reset_sql(
+            no_style(), [Organization]
+        )
         if sequence_sql:
             if verbosity >= 2:
                 print('Resetting sequence')

mayan/apps/permissions/tests/test_models.py

@@ -9,7 +9,7 @@ from user_management.models import MayanGroup
 from user_management.tests import TEST_GROUP, TEST_USER_USERNAME
 
 from ..classes import Permission
-from ..models import Role, StoredPermission
+from ..models import Role
 from ..permissions import permission_role_view
 
 from .literals import TEST_ROLE_LABEL

mayan/apps/rest_api/tests/__init__.py

@@ -1 +1 @@
-from .base import GenericAPITestCase  #NOQA
+from .base import GenericAPITestCase  # NOQA

mayan/apps/rest_api/tests/base.py

@@ -1,8 +1,5 @@
 from __future__ import unicode_literals
 
-from django.contrib.auth import get_user_model
-from django.test import override_settings
-
 from rest_framework.test import APITestCase
 
 from organizations.models import Organization

mayan/apps/tags/tests/test_api.py

@@ -1,15 +1,11 @@
 from __future__ import unicode_literals
 
-from django.contrib.auth import get_user_model
 from django.core.urlresolvers import reverse
 from django.test import override_settings
 
 from documents.models import DocumentType
 from documents.tests import TEST_DOCUMENT_TYPE, TEST_SMALL_DOCUMENT_PATH
 from rest_api.tests.base import GenericAPITestCase
-from user_management.tests.literals import (
-    TEST_ADMIN_EMAIL, TEST_ADMIN_PASSWORD, TEST_ADMIN_USERNAME
-)
 
 from ..models import Tag
 

mayan/apps/tags/tests/test_models.py

@@ -1,7 +1,7 @@
 from __future__ import unicode_literals
 
 from django.core.files.base import File
-from django.test import TestCase, override_settings
+from django.test import override_settings
 
 from documents.models import DocumentType
 from documents.tests import TEST_DOCUMENT_TYPE, TEST_SMALL_DOCUMENT_PATH

mayan/apps/user_management/admin.py

@@ -2,7 +2,6 @@ from __future__ import unicode_literals
 
 from django.contrib import admin
 from django.contrib.auth.admin import UserAdmin, GroupAdmin
-from django.utils.translation import ugettext_lazy as _
 
 from organizations.admin import OrganizationAdminMixin