Browse Source

Remove unused imports

rafalp 6 years ago
parent
commit
f963d24ed9
38 changed files with 15 additions and 62 deletions
  1. 0 2
      misago/categories/permissions.py
  2. 4 1
      misago/faker/management/commands/createfakeusers.py
  3. 0 2
      misago/markup/parser.py
  4. 0 1
      misago/readtracker/tests/test_categoriestracker.py
  5. 0 1
      misago/readtracker/tests/test_poststracker.py
  6. 0 1
      misago/readtracker/tests/test_threadstracker.py
  7. 0 1
      misago/search/context_processors.py
  8. 0 1
      misago/search/views.py
  9. 0 5
      misago/threads/api/postendpoints/delete.py
  10. 0 3
      misago/threads/api/postendpoints/patch_event.py
  11. 0 2
      misago/threads/api/threadendpoints/merge.py
  12. 0 1
      misago/threads/api/threads.py
  13. 0 1
      misago/threads/permissions/bestanswers.py
  14. 0 1
      misago/threads/tests/test_emailnotification_middleware.py
  15. 0 2
      misago/threads/tests/test_thread_postbulkpatch_api.py
  16. 1 1
      misago/threads/tests/test_thread_postmerge_api.py
  17. 1 1
      misago/threads/tests/test_thread_postpatch_api.py
  18. 1 1
      misago/threads/tests/test_thread_postsplit_api.py
  19. 0 1
      misago/threads/tests/test_threads_api.py
  20. 0 1
      misago/threads/tests/test_threadslists.py
  21. 1 4
      misago/threads/viewmodels/threads.py
  22. 0 1
      misago/users/api/userendpoints/changeemail.py
  23. 0 1
      misago/users/api/userendpoints/create.py
  24. 1 2
      misago/users/management/commands/deletemarkedusers.py
  25. 1 1
      misago/users/management/commands/deleteprofilefield.py
  26. 1 3
      misago/users/models/user.py
  27. 0 2
      misago/users/namechanges.py
  28. 2 3
      misago/users/test.py
  29. 0 1
      misago/users/tests/test_auth_api.py
  30. 0 1
      misago/users/tests/test_auth_backend.py
  31. 0 1
      misago/users/tests/test_bans.py
  32. 1 1
      misago/users/tests/test_datadownloads.py
  33. 0 1
      misago/users/tests/test_mention_api.py
  34. 1 5
      misago/users/tests/test_namechanges.py
  35. 0 1
      misago/users/tests/test_signatures.py
  36. 0 1
      misago/users/tests/test_social_pipeline.py
  37. 0 2
      misago/users/tests/test_user_avatar_api.py
  38. 0 1
      misago/users/tests/test_users_api.py

+ 0 - 2
misago/categories/permissions.py

@@ -1,5 +1,4 @@
 from django import forms
 from django import forms
-from django.contrib.auth import get_user_model
 from django.core.exceptions import PermissionDenied
 from django.core.exceptions import PermissionDenied
 from django.http import Http404
 from django.http import Http404
 from django.utils.translation import gettext_lazy as _
 from django.utils.translation import gettext_lazy as _
@@ -7,7 +6,6 @@ from django.utils.translation import gettext_lazy as _
 from ..acl import algebra
 from ..acl import algebra
 from ..acl.decorators import return_boolean
 from ..acl.decorators import return_boolean
 from ..admin.forms import YesNoSwitch
 from ..admin.forms import YesNoSwitch
-from ..users.models import AnonymousUser
 from .models import Category, CategoryRole, RoleCategoryACL
 from .models import Category, CategoryRole, RoleCategoryACL
 
 
 
 

+ 4 - 1
misago/faker/management/commands/createfakeusers.py

@@ -53,7 +53,10 @@ class Command(BaseCommand):
                     rank=random.choice(ranks),
                     rank=random.choice(ranks),
                 )
                 )
 
 
-                dynamic.set_avatar(user)
+                if random.randint(0, 100) < 80:
+                    gallery.set_random_avatar(user)
+                else:
+                    dynamic.set_avatar(user)
                 user.save(update_fields=["avatars"])
                 user.save(update_fields=["avatars"])
             except (ValidationError, IntegrityError):
             except (ValidationError, IntegrityError):
                 pass
                 pass

+ 0 - 2
misago/markup/parser.py

@@ -1,5 +1,3 @@
-import warnings
-
 import bleach
 import bleach
 import markdown
 import markdown
 from bs4 import BeautifulSoup
 from bs4 import BeautifulSoup

+ 0 - 1
misago/readtracker/tests/test_categoriestracker.py

@@ -10,7 +10,6 @@ from ...conf import settings
 from ...conftest import get_cache_versions
 from ...conftest import get_cache_versions
 from ...threads import test
 from ...threads import test
 from ...users.test import create_test_user
 from ...users.test import create_test_user
-from ..models import PostRead
 
 
 cache_versions = get_cache_versions()
 cache_versions = get_cache_versions()
 
 

+ 0 - 1
misago/readtracker/tests/test_poststracker.py

@@ -7,7 +7,6 @@ from ...categories.models import Category
 from ...conf import settings
 from ...conf import settings
 from ...threads import test
 from ...threads import test
 from ...users.test import create_test_user
 from ...users.test import create_test_user
-from ..models import PostRead
 from ..poststracker import make_read_aware, save_read
 from ..poststracker import make_read_aware, save_read
 
 
 
 

+ 0 - 1
misago/readtracker/tests/test_threadstracker.py

@@ -11,7 +11,6 @@ from ...conf import settings
 from ...conftest import get_cache_versions
 from ...conftest import get_cache_versions
 from ...threads import test
 from ...threads import test
 from ...users.test import create_test_user
 from ...users.test import create_test_user
-from ..models import PostRead
 
 
 cache_versions = get_cache_versions()
 cache_versions = get_cache_versions()
 
 

+ 0 - 1
misago/search/context_processors.py

@@ -1,4 +1,3 @@
-from django.core.exceptions import PermissionDenied
 from django.urls import reverse
 from django.urls import reverse
 
 
 from .searchproviders import searchproviders
 from .searchproviders import searchproviders

+ 0 - 1
misago/search/views.py

@@ -1,7 +1,6 @@
 from django.core.exceptions import PermissionDenied
 from django.core.exceptions import PermissionDenied
 from django.http import Http404
 from django.http import Http404
 from django.shortcuts import redirect, render
 from django.shortcuts import redirect, render
-from django.urls import reverse
 from django.utils.translation import gettext as _
 from django.utils.translation import gettext as _
 
 
 from .searchproviders import searchproviders
 from .searchproviders import searchproviders

+ 0 - 5
misago/threads/api/postendpoints/delete.py

@@ -1,16 +1,11 @@
-from django.core.exceptions import PermissionDenied
-from django.utils.translation import gettext as _
-from django.utils.translation import ngettext
 from rest_framework.response import Response
 from rest_framework.response import Response
 
 
 from ....conf import settings
 from ....conf import settings
-from ....core.utils import clean_ids_list
 from ...moderation import posts as moderation
 from ...moderation import posts as moderation
 from ...permissions import (
 from ...permissions import (
     allow_delete_best_answer,
     allow_delete_best_answer,
     allow_delete_event,
     allow_delete_event,
     allow_delete_post,
     allow_delete_post,
-    exclude_invisible_posts,
 )
 )
 from ...serializers import DeletePostsSerializer
 from ...serializers import DeletePostsSerializer
 
 

+ 0 - 3
misago/threads/api/postendpoints/patch_event.py

@@ -1,6 +1,3 @@
-from django.core.exceptions import PermissionDenied
-from django.utils.translation import gettext as _
-
 from ....acl.objectacl import add_acl_to_obj
 from ....acl.objectacl import add_acl_to_obj
 from ....core.apipatch import ApiPatch
 from ....core.apipatch import ApiPatch
 from ...moderation import posts as moderation
 from ...moderation import posts as moderation

+ 0 - 2
misago/threads/api/threadendpoints/merge.py

@@ -1,6 +1,4 @@
 from django.core.exceptions import PermissionDenied
 from django.core.exceptions import PermissionDenied
-from django.utils.translation import gettext as _
-from rest_framework.exceptions import ValidationError
 from rest_framework.response import Response
 from rest_framework.response import Response
 
 
 from ....acl.objectacl import add_acl_to_obj
 from ....acl.objectacl import add_acl_to_obj

+ 0 - 1
misago/threads/api/threads.py

@@ -8,7 +8,6 @@ from rest_framework.response import Response
 from ...categories import PRIVATE_THREADS_ROOT_NAME, THREADS_ROOT_NAME
 from ...categories import PRIVATE_THREADS_ROOT_NAME, THREADS_ROOT_NAME
 from ...core.shortcuts import get_int_or_404
 from ...core.shortcuts import get_int_or_404
 from ..models import Post, Thread
 from ..models import Post, Thread
-from ..moderation import threads as moderation
 from ..permissions import allow_use_private_threads
 from ..permissions import allow_use_private_threads
 from ..viewmodels import (
 from ..viewmodels import (
     ForumThread,
     ForumThread,

+ 0 - 1
misago/threads/permissions/bestanswers.py

@@ -6,7 +6,6 @@ from django.utils.translation import ngettext
 
 
 from ...acl import algebra
 from ...acl import algebra
 from ...acl.decorators import return_boolean
 from ...acl.decorators import return_boolean
-from ...admin.forms import YesNoSwitch
 from ...categories.models import Category, CategoryRole
 from ...categories.models import Category, CategoryRole
 from ...categories.permissions import get_categories_roles
 from ...categories.permissions import get_categories_roles
 from ..models import Post, Thread
 from ..models import Post, Thread

+ 0 - 1
misago/threads/tests/test_emailnotification_middleware.py

@@ -1,4 +1,3 @@
-from copy import deepcopy
 from datetime import timedelta
 from datetime import timedelta
 
 
 from django.core import mail
 from django.core import mail

+ 0 - 2
misago/threads/tests/test_thread_postbulkpatch_api.py

@@ -1,8 +1,6 @@
 import json
 import json
-from datetime import timedelta
 
 
 from django.urls import reverse
 from django.urls import reverse
-from django.utils import timezone
 
 
 from .. import test
 from .. import test
 from ...categories.models import Category
 from ...categories.models import Category

+ 1 - 1
misago/threads/tests/test_thread_postmerge_api.py

@@ -6,7 +6,7 @@ from .. import test
 from ...categories.models import Category
 from ...categories.models import Category
 from ...readtracker import poststracker
 from ...readtracker import poststracker
 from ...users.test import AuthenticatedUserTestCase
 from ...users.test import AuthenticatedUserTestCase
-from ..models import Post, Thread
+from ..models import Post
 from ..serializers.moderation import POSTS_LIMIT
 from ..serializers.moderation import POSTS_LIMIT
 from ..test import patch_category_acl
 from ..test import patch_category_acl
 
 

+ 1 - 1
misago/threads/tests/test_thread_postpatch_api.py

@@ -7,7 +7,7 @@ from django.utils import timezone
 from .. import test
 from .. import test
 from ...categories.models import Category
 from ...categories.models import Category
 from ...users.test import AuthenticatedUserTestCase
 from ...users.test import AuthenticatedUserTestCase
-from ..models import Post, Thread
+from ..models import Post
 from ..test import patch_category_acl
 from ..test import patch_category_acl
 
 
 
 

+ 1 - 1
misago/threads/tests/test_thread_postsplit_api.py

@@ -6,7 +6,7 @@ from .. import test
 from ...categories.models import Category
 from ...categories.models import Category
 from ...readtracker import poststracker
 from ...readtracker import poststracker
 from ...users.test import AuthenticatedUserTestCase
 from ...users.test import AuthenticatedUserTestCase
-from ..models import Post, Thread
+from ..models import Post
 from ..serializers.moderation import POSTS_LIMIT
 from ..serializers.moderation import POSTS_LIMIT
 from ..test import patch_category_acl, patch_other_category_acl
 from ..test import patch_category_acl, patch_other_category_acl
 
 

+ 0 - 1
misago/threads/tests/test_threads_api.py

@@ -1,6 +1,5 @@
 from datetime import timedelta
 from datetime import timedelta
 
 
-from django.urls import reverse
 from django.utils import timezone
 from django.utils import timezone
 
 
 from .. import test
 from .. import test

+ 0 - 1
misago/threads/tests/test_threadslists.py

@@ -9,7 +9,6 @@ from ...acl.test import patch_user_acl
 from ...categories.models import Category
 from ...categories.models import Category
 from ...conf import settings
 from ...conf import settings
 from ...readtracker import poststracker
 from ...readtracker import poststracker
-from ...users.models import AnonymousUser
 from ...users.test import AuthenticatedUserTestCase
 from ...users.test import AuthenticatedUserTestCase
 
 
 LISTS_URLS = ("", "my/", "new/", "unread/", "subscribed/")
 LISTS_URLS = ("", "my/", "new/", "unread/", "subscribed/")

+ 1 - 4
misago/threads/viewmodels/threads.py

@@ -1,9 +1,6 @@
-from datetime import timedelta
-
 from django.core.exceptions import PermissionDenied
 from django.core.exceptions import PermissionDenied
-from django.db.models import F, Q
+from django.db.models import Q
 from django.http import Http404
 from django.http import Http404
-from django.utils import timezone
 from django.utils.translation import gettext as _
 from django.utils.translation import gettext as _
 from django.utils.translation import gettext_lazy
 from django.utils.translation import gettext_lazy
 
 

+ 0 - 1
misago/users/api/userendpoints/changeemail.py

@@ -2,7 +2,6 @@ from django.utils.translation import gettext as _
 from rest_framework import status
 from rest_framework import status
 from rest_framework.response import Response
 from rest_framework.response import Response
 
 
-from ....conf import settings
 from ....core.mail import mail_user
 from ....core.mail import mail_user
 from ...credentialchange import store_new_credential
 from ...credentialchange import store_new_credential
 from ...serializers import ChangeEmailSerializer
 from ...serializers import ChangeEmailSerializer

+ 0 - 1
misago/users/api/userendpoints/create.py

@@ -7,7 +7,6 @@ from rest_framework import status
 from rest_framework.response import Response
 from rest_framework.response import Response
 
 
 from ... import captcha
 from ... import captcha
-from ....conf import settings
 from ....legal.models import Agreement
 from ....legal.models import Agreement
 from ...forms.register import RegisterForm
 from ...forms.register import RegisterForm
 from ...registration import (
 from ...registration import (

+ 1 - 2
misago/users/management/commands/deletemarkedusers.py

@@ -1,7 +1,6 @@
 from django.contrib.auth import get_user_model
 from django.contrib.auth import get_user_model
-from django.core.management.base import BaseCommand, CommandError
+from django.core.management.base import BaseCommand
 
 
-from ....conf import settings
 from ....core.pgutils import chunk_queryset
 from ....core.pgutils import chunk_queryset
 from ...permissions import can_delete_own_account
 from ...permissions import can_delete_own_account
 
 

+ 1 - 1
misago/users/management/commands/deleteprofilefield.py

@@ -1,5 +1,5 @@
 from django.contrib.auth import get_user_model
 from django.contrib.auth import get_user_model
-from django.core.management.base import BaseCommand, CommandError
+from django.core.management.base import BaseCommand
 
 
 from ....core.pgutils import chunk_queryset
 from ....core.pgutils import chunk_queryset
 
 

+ 1 - 3
misago/users/models/user.py

@@ -4,10 +4,9 @@ from django.contrib.auth.models import AbstractBaseUser
 from django.contrib.auth.models import AnonymousUser as DjangoAnonymousUser
 from django.contrib.auth.models import AnonymousUser as DjangoAnonymousUser
 from django.contrib.auth.models import PermissionsMixin
 from django.contrib.auth.models import PermissionsMixin
 from django.contrib.auth.models import UserManager as BaseUserManager
 from django.contrib.auth.models import UserManager as BaseUserManager
-from django.contrib.auth.password_validation import validate_password
 from django.contrib.postgres.fields import ArrayField, HStoreField, JSONField
 from django.contrib.postgres.fields import ArrayField, HStoreField, JSONField
 from django.core.mail import send_mail
 from django.core.mail import send_mail
-from django.db import IntegrityError, models, transaction
+from django.db import models
 from django.urls import reverse
 from django.urls import reverse
 from django.utils import timezone
 from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 from django.utils.translation import gettext_lazy as _
@@ -17,7 +16,6 @@ from ...acl.models import Role
 from ...conf import settings
 from ...conf import settings
 from ...core.pgutils import PgPartialIndex
 from ...core.pgutils import PgPartialIndex
 from ...core.utils import slugify
 from ...core.utils import slugify
-from ..audittrail import create_user_audit_trail
 from ..signatures import is_user_signature_valid
 from ..signatures import is_user_signature_valid
 from ..utils import hash_email
 from ..utils import hash_email
 from .online import Online
 from .online import Online

+ 0 - 2
misago/users/namechanges.py

@@ -5,8 +5,6 @@ from datetime import timedelta
 
 
 from django.utils import timezone
 from django.utils import timezone
 
 
-from .models import UsernameChange
-
 
 
 def get_username_options(settings, user, user_acl):
 def get_username_options(settings, user, user_acl):
     changes_left = get_left_namechanges(user, user_acl)
     changes_left = get_left_namechanges(user, user_acl)

+ 2 - 3
misago/users/test.py

@@ -2,7 +2,6 @@ from django.contrib.auth import get_user_model
 from django.test import TestCase
 from django.test import TestCase
 
 
 from .models import AnonymousUser, Online
 from .models import AnonymousUser, Online
-from .setupnewuser import setup_new_user
 
 
 User = get_user_model()
 User = get_user_model()
 
 
@@ -59,7 +58,7 @@ class SuperUserTestCase(AuthenticatedUserTestCase):
 
 
 
 
 def create_test_user(username, email, password=None, **extra_fields):
 def create_test_user(username, email, password=None, **extra_fields):
-    """Faster counterpart of regular `create_user` followed by `setup_new_user`"""
+    """Faster alternative to regular `create_user` followed by `setup_new_user`"""
     if "avatars" not in extra_fields:
     if "avatars" not in extra_fields:
         extra_fields["avatars"] = user_placeholder_avatars
         extra_fields["avatars"] = user_placeholder_avatars
 
 
@@ -67,7 +66,7 @@ def create_test_user(username, email, password=None, **extra_fields):
 
 
 
 
 def create_test_superuser(username, email, password=None, **extra_fields):
 def create_test_superuser(username, email, password=None, **extra_fields):
-    """Faster counterpart of regular `create_superuser` followed by `setup_new_user`"""
+    """Faster alternative to regular `create_superuser` followed by `setup_new_user`"""
     if "avatars" not in extra_fields:
     if "avatars" not in extra_fields:
         extra_fields["avatars"] = user_placeholder_avatars
         extra_fields["avatars"] = user_placeholder_avatars
 
 

+ 0 - 1
misago/users/tests/test_auth_api.py

@@ -1,4 +1,3 @@
-from django.contrib.auth import get_user_model
 from django.core import mail
 from django.core import mail
 from django.test import TestCase
 from django.test import TestCase
 
 

+ 0 - 1
misago/users/tests/test_auth_backend.py

@@ -1,4 +1,3 @@
-from django.contrib.auth import get_user_model
 from django.test import TestCase
 from django.test import TestCase
 
 
 from ..authbackends import MisagoBackend
 from ..authbackends import MisagoBackend

+ 0 - 1
misago/users/tests/test_bans.py

@@ -4,7 +4,6 @@ from django.test import TestCase
 from django.utils import timezone
 from django.utils import timezone
 
 
 from ...conftest import get_cache_versions
 from ...conftest import get_cache_versions
-from ...users import BANS_CACHE
 from ..bans import (
 from ..bans import (
     ban_ip,
     ban_ip,
     ban_user,
     ban_user,

+ 1 - 1
misago/users/tests/test_datadownloads.py

@@ -3,7 +3,7 @@ import os
 from django.core.files import File
 from django.core.files import File
 
 
 from ...categories.models import Category
 from ...categories.models import Category
-from ...threads.models import Attachment, AttachmentType
+from ...threads.models import AttachmentType
 from ...threads.test import post_poll, post_thread
 from ...threads.test import post_poll, post_thread
 from ..audittrail import create_user_audit_trail
 from ..audittrail import create_user_audit_trail
 from ..datadownloads import (
 from ..datadownloads import (

+ 0 - 1
misago/users/tests/test_mention_api.py

@@ -1,7 +1,6 @@
 from django.test import TestCase
 from django.test import TestCase
 from django.urls import reverse
 from django.urls import reverse
 
 
-from ...conf import settings
 from ..test import create_test_user
 from ..test import create_test_user
 
 
 
 

+ 1 - 5
misago/users/tests/test_namechanges.py

@@ -1,10 +1,6 @@
 from datetime import timedelta
 from datetime import timedelta
 
 
-from ..namechanges import (
-    get_left_namechanges,
-    get_next_available_namechange,
-    get_username_options,
-)
+from ..namechanges import get_left_namechanges, get_next_available_namechange
 
 
 
 
 def test_user_without_permission_to_change_name_has_no_changes_left(user):
 def test_user_without_permission_to_change_name_has_no_changes_left(user):

+ 0 - 1
misago/users/tests/test_signatures.py

@@ -2,7 +2,6 @@ from unittest.mock import Mock
 
 
 import pytest
 import pytest
 
 
-from ...acl.useracl import get_user_acl
 from ...users import signatures
 from ...users import signatures
 
 
 
 

+ 0 - 1
misago/users/tests/test_social_pipeline.py

@@ -6,7 +6,6 @@ from django.test import RequestFactory
 from social_core.backends.github import GithubOAuth2
 from social_core.backends.github import GithubOAuth2
 from social_django.utils import load_strategy
 from social_django.utils import load_strategy
 
 
-from ...acl import ACL_CACHE
 from ...acl.useracl import get_user_acl
 from ...acl.useracl import get_user_acl
 from ...conf.dynamicsettings import DynamicSettings
 from ...conf.dynamicsettings import DynamicSettings
 from ...conf.test import override_dynamic_settings
 from ...conf.test import override_dynamic_settings

+ 0 - 2
misago/users/tests/test_user_avatar_api.py

@@ -2,8 +2,6 @@ import json
 import os
 import os
 from pathlib import Path
 from pathlib import Path
 
 
-from django.contrib.auth import get_user_model
-
 from ...acl.test import patch_user_acl
 from ...acl.test import patch_user_acl
 from ...conf import settings
 from ...conf import settings
 from ...conf.test import override_dynamic_settings
 from ...conf.test import override_dynamic_settings

+ 0 - 1
misago/users/tests/test_users_api.py

@@ -4,7 +4,6 @@ from datetime import timedelta
 from django.contrib.auth import get_user_model
 from django.contrib.auth import get_user_model
 from django.test import override_settings
 from django.test import override_settings
 from django.urls import reverse
 from django.urls import reverse
-from django.utils.encoding import smart_str
 
 
 from ...acl.test import patch_user_acl
 from ...acl.test import patch_user_acl
 from ...categories.models import Category
 from ...categories.models import Category