reorganize imports with ruff

core/admin.py

@@ -14,12 +14,12 @@
 #
 #
 
-from django.contrib import admin
 from ajax_select import make_ajax_form
-from core.models import User, Page, RealGroup, MetaGroup, SithFile
+from django.contrib import admin
 from django.contrib.auth.models import Group as AuthGroup
 from haystack.admin import SearchModelAdmin
 
+from core.models import MetaGroup, Page, RealGroup, SithFile, User
 
 admin.site.unregister(AuthGroup)
 admin.site.register(MetaGroup)

core/apps.py

@@ -34,8 +34,8 @@ class SithConfig(AppConfig):
     verbose_name = "Core app of the Sith"
 
     def ready(self):
+        import core.signals  # noqa F401
         from forum.models import Forum
-        import core.signals
 
         cache.clear()
 

core/converters.py

@@ -1,6 +1,3 @@
-from core.models import Page
-
-
 class FourDigitYearConverter:
     regex = "[0-9]{4}"
 

core/lookups.py

@@ -14,15 +14,14 @@
 #
 #
 
+from ajax_select import LookupChannel, register
 from django.core.exceptions import PermissionDenied
-from ajax_select import register, LookupChannel
 
-from core.views.site import search_user
-from core.models import User, Group, SithFile
-from club.models import Club
-from counter.models import Product, Counter, Customer
 from accounting.models import ClubAccount, Company
-from eboutic.models import BasketItem
+from club.models import Club
+from core.models import Group, SithFile, User
+from core.views.site import search_user
+from counter.models import Counter, Customer, Product
 
 
 def check_token(request):

core/management/commands/check_fs.py

@@ -23,8 +23,8 @@
 #
 
 import os
+
 from django.core.management.base import BaseCommand
-from django.core.management import call_command
 
 from core.models import SithFile
 

core/management/commands/compilemessages.py

@@ -25,6 +25,7 @@
 
 
 import os
+
 from django.core.management.commands import compilemessages
 
 

core/management/commands/compilestatic.py

@@ -24,9 +24,10 @@
 #
 
 import os
+
 import sass
-from django.core.management.base import BaseCommand
 from django.conf import settings
+from django.core.management.base import BaseCommand
 
 
 class Command(BaseCommand):

core/management/commands/documentation.py

@@ -24,10 +24,9 @@
 #
 
 import os
-import sys
 import signal
-
-from http.server import test, CGIHTTPRequestHandler
+import sys
+from http.server import CGIHTTPRequestHandler, test
 
 from django.core.management.base import BaseCommand
 from django.utils import autoreload

core/management/commands/install_xapian.py

@@ -15,11 +15,12 @@
 #
 
 import os
-import tomli
 import subprocess
-from django.core.management.base import BaseCommand, CommandParser
 from pathlib import Path
 
+import tomli
+from django.core.management.base import BaseCommand, CommandParser
+
 
 class Command(BaseCommand):
     help = "Install xapian"

core/management/commands/markdown.py

@@ -23,6 +23,7 @@
 #
 
 import os
+
 from django.core.management.base import BaseCommand
 
 from core.markdown import markdown

core/management/commands/populate.py

@@ -24,38 +24,37 @@
 
 import os
 from datetime import date, datetime, timedelta
-from io import StringIO, BytesIO
+from io import BytesIO, StringIO
 from pathlib import Path
 
-from django.contrib.auth.models import Permission
-from django.core.management.base import BaseCommand
-from django.core.management import call_command
 from django.conf import settings
-from django.db import connection
+from django.contrib.auth.models import Permission
 from django.contrib.sites.models import Site
+from django.core.management import call_command
+from django.core.management.base import BaseCommand
+from django.db import connection
 from django.utils import timezone
-
 from PIL import Image
 
-from core.models import Group, User, Page, PageRev, SithFile
 from accounting.models import (
-    GeneralJournal,
+    AccountingType,
     BankAccount,
     ClubAccount,
-    Operation,
-    AccountingType,
-    SimplifiedAccountingType,
     Company,
+    GeneralJournal,
+    Operation,
+    SimplifiedAccountingType,
 )
-from core.utils import resize_image
 from club.models import Club, Membership
-from subscription.models import Subscription
-from counter.models import Customer, ProductType, Product, Counter, Selling, StudentCard
-from com.models import Sith, Weekmail, News, NewsDate
-from election.models import Election, Role, Candidature, ElectionList
+from com.models import News, NewsDate, Sith, Weekmail
+from core.models import Group, Page, PageRev, SithFile, User
+from core.utils import resize_image
+from counter.models import Counter, Customer, Product, ProductType, Selling, StudentCard
+from election.models import Candidature, Election, ElectionList, Role
 from forum.models import Forum, ForumTopic
 from pedagogy.models import UV
-from sas.models import Album, Picture, PeoplePictureRelation
+from sas.models import Album, PeoplePictureRelation, Picture
+from subscription.models import Subscription
 
 
 class Command(BaseCommand):

core/management/commands/repair_fs.py

@@ -23,8 +23,8 @@
 #
 
 import os
+
 from django.core.management.base import BaseCommand
-from django.core.management import call_command
 
 from core.models import SithFile
 

core/management/commands/setup.py

@@ -15,8 +15,9 @@
 #
 
 import os
-from django.core.management.base import BaseCommand
+
 from django.core.management import call_command
+from django.core.management.base import BaseCommand
 
 
 class Command(BaseCommand):

core/markdown.py

@@ -16,8 +16,9 @@
 
 import os
 import re
-from mistune import Renderer, InlineGrammar, InlineLexer, Markdown, escape, escape_link
+
 from django.urls import reverse
+from mistune import InlineGrammar, InlineLexer, Markdown, Renderer, escape, escape_link
 
 
 class SithRenderer(Renderer):

core/middleware.py

@@ -16,12 +16,13 @@
 
 import importlib
 import threading
+
 from django.conf import settings
-from django.utils.functional import SimpleLazyObject
 from django.contrib.auth import get_user
 from django.contrib.auth.middleware import (
     AuthenticationMiddleware as DjangoAuthenticationMiddleware,
 )
+from django.utils.functional import SimpleLazyObject
 
 module, klass = settings.AUTH_ANONYMOUS_MODEL.rsplit(".", 1)
 AnonymousUser = getattr(importlib.import_module(module), klass)

core/migrations/0001_initial.py

@@ -1,14 +1,14 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
 import django.contrib.auth.models
-import django.db.models.deletion
 import django.core.validators
-import core.models
+import django.db.models.deletion
 import phonenumber_field.modelfields
 from django.conf import settings
-import django.db.models.deletion
+from django.db import migrations, models
+
+import core.models
 
 
 class Migration(migrations.Migration):

core/migrations/0003_auto_20160902_1914.py

@@ -1,8 +1,8 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
 import django.core.validators
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0004_user_godfathers.py

@@ -1,8 +1,8 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
 from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0005_auto_20161105_1035.py

@@ -1,9 +1,9 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
-from django.conf import settings
 import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0009_auto_20161120_1155.py

@@ -1,8 +1,9 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
 import django.db.models.deletion
+from django.db import migrations, models
+
 import core.models
 
 

core/migrations/0011_auto_20161124_0848.py

@@ -1,8 +1,9 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
 import django.utils.timezone
+from django.db import migrations, models
+
 import core.models
 
 

core/migrations/0012_notification.py

@@ -1,10 +1,10 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
-from django.conf import settings
-import django.utils.timezone
 import django.db.models.deletion
+import django.utils.timezone
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0015_sithfile_moderator.py

@@ -1,9 +1,9 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
-from django.conf import settings
 import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0016_auto_20161212_1922.py

@@ -1,9 +1,9 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
-from django.conf import settings
 import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0020_auto_20170324_0917.py

@@ -1,8 +1,8 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
 import django.core.validators
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0023_auto_20170902_1226.py

@@ -1,9 +1,9 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
-from django.conf import settings
 import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0025_auto_20170919_1521.py

@@ -1,8 +1,8 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
 import django.core.validators
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0027_gift.py

@@ -1,10 +1,10 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
-from django.conf import settings
-import django.utils.timezone
 import django.db.models.deletion
+import django.utils.timezone
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

core/migrations/0029_auto_20180426_2013.py

@@ -1,9 +1,10 @@
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
-from django.db import migrations, models
-import core.models
 import django.db.models.deletion
+from django.db import migrations, models
+
+import core.models
 
 
 class Migration(migrations.Migration):

core/migrations/0034_operationlog.py

@@ -1,8 +1,8 @@
 # Generated by Django 2.2.6 on 2019-11-14 15:10
 
+import django.db.models.deletion
 from django.conf import settings
 from django.db import migrations, models
-import django.db.models.deletion
 
 
 class Migration(migrations.Migration):

core/models.py

@@ -23,36 +23,39 @@
 #
 #
 import importlib
-from typing import Union, Optional, List
+import os
+import unicodedata
+from datetime import date, timedelta
+from typing import List, Optional, Union
 
-from django.core.cache import cache
-from django.core.mail import send_mail
+from django.conf import settings
 from django.contrib.auth.models import (
     AbstractBaseUser,
     UserManager,
-    Group as AuthGroup,
-    GroupManager as AuthGroupManager,
+)
+from django.contrib.auth.models import (
     AnonymousUser as AuthAnonymousUser,
 )
-from django.utils.translation import gettext_lazy as _
-from django.utils import timezone
-from django.core import validators
-from django.core.exceptions import ValidationError, PermissionDenied
-from django.urls import reverse
-from django.conf import settings
-from django.db import models, transaction
+from django.contrib.auth.models import (
+    Group as AuthGroup,
+)
+from django.contrib.auth.models import (
+    GroupManager as AuthGroupManager,
+)
 from django.contrib.staticfiles.storage import staticfiles_storage
-from django.utils.html import escape
+from django.core import validators
+from django.core.cache import cache
+from django.core.exceptions import PermissionDenied, ValidationError
+from django.core.mail import send_mail
+from django.db import models, transaction
+from django.urls import reverse
+from django.utils import timezone
 from django.utils.functional import cached_property
-
-import os
-from core import utils
-
+from django.utils.html import escape
+from django.utils.translation import gettext_lazy as _
 from phonenumber_field.modelfields import PhoneNumberField
 
-from datetime import timedelta, date
-
-import unicodedata
+from core import utils
 
 
 class RealGroupManager(AuthGroupManager):

core/scss/finder.py

@@ -25,6 +25,7 @@
 
 import os
 from collections import OrderedDict
+
 from django.conf import settings
 from django.contrib.staticfiles.finders import FileSystemFinder
 from django.core.files.storage import FileSystemStorage

core/scss/processor.py

@@ -24,12 +24,14 @@
 #
 
 import os
-import sass
 from urllib.parse import urljoin
-from django.utils.encoding import force_bytes, iri_to_uri
+
+import sass
+from django.conf import settings
 from django.core.files.base import ContentFile
 from django.templatetags.static import static
-from django.conf import settings
+from django.utils.encoding import force_bytes, iri_to_uri
+
 from core.scss.storage import ScssFileStorage, find_file
 
 

core/search_indexes.py

@@ -24,7 +24,6 @@
 #
 
 from django.db import models
-
 from haystack import indexes, signals
 
 from core.models import User

core/signals.py

@@ -8,10 +8,10 @@ from core.models import User
 @receiver(m2m_changed, sender=User.groups.through, dispatch_uid="user_groups_changed")
 def user_groups_changed(sender, instance: User, **kwargs):
     """
-    Clear the cached clubs of the user
+    Clear the cached groups of the user
     """
     # As a m2m relationship doesn't live within the model
     # but rather on an intermediary table, there is no
     # model method to override, meaning we must use
-    # a signal to invalidate the cache when a user is removed from a club
-    cache.delete(f"user_{instance.id}_groups")
+    # a signal to invalidate the cache when a user is removed from a group
+    cache.delete(f"user_{instance.pk}_groups")

core/templatetags/renderer.py

@@ -24,15 +24,15 @@
 #
 
 import datetime
-import phonenumbers
 
+import phonenumbers
 from django import template
 from django.template.defaultfilters import stringfilter
 from django.utils.safestring import mark_safe
 from django.utils.translation import ngettext
-from core.scss.processor import ScssProcessor
 
 from core.markdown import markdown as md
+from core.scss.processor import ScssProcessor
 
 register = template.Library()
 

core/templatetags/search_helpers.py

@@ -1,6 +1,6 @@
-from django.template.exceptions import TemplateSyntaxError
 from django import template
 from django.template.defaultfilters import stringfilter
+from django.template.exceptions import TemplateSyntaxError
 
 register = template.Library()
 

core/urls.py

@@ -25,12 +25,12 @@
 
 from django.urls import path, re_path, register_converter
 
-from core.views import *
 from core.converters import (
+    BooleanStringConverter,
     FourDigitYearConverter,
     TwoDigitMonthConverter,
-    BooleanStringConverter,
 )
+from core.views import *
 
 register_converter(FourDigitYearConverter, "yyyy")
 register_converter(TwoDigitMonthConverter, "mm")

core/utils.py

@@ -26,8 +26,8 @@ from typing import Optional
 import PIL
 from django.conf import settings
 from django.core.files.base import ContentFile
-from PIL import ExifTags
 from django.utils import timezone
+from PIL import ExifTags
 
 
 def get_git_revision_short_hash() -> str:

core/views/__init__.py

@@ -25,26 +25,21 @@
 
 import types
 
-from sentry_sdk import last_event_id
-from django.shortcuts import render
+from django.core.exceptions import (
+    ImproperlyConfigured,
+    PermissionDenied,
+)
 from django.http import (
     HttpResponseForbidden,
     HttpResponseNotFound,
     HttpResponseServerError,
 )
-from django.template import RequestContext
-from django.core.exceptions import (
-    PermissionDenied,
-    ObjectDoesNotExist,
-    ImproperlyConfigured,
-)
-from django.views.generic.base import View
-from django.views.generic.edit import FormView
-from django.views.generic.detail import SingleObjectMixin
 from django.utils.functional import cached_property
-from django.db.models import Count
+from django.views.generic.base import View
+from django.views.generic.detail import SingleObjectMixin
+from django.views.generic.edit import FormView
+from sentry_sdk import last_event_id
 
-from core.models import Group
 from core.views.forms import LoginForm
 
 
@@ -361,8 +356,8 @@ class DetailFormView(SingleObjectMixin, FormView):
         return super(DetailFormView, self).get_object()
 
 
-from .user import *
-from .page import *
 from .files import *
-from .site import *
 from .group import *
+from .page import *
+from .site import *
+from .user import *

core/views/files.py

@@ -15,29 +15,28 @@
 #
 
 # This file contains all the views that concern the page model
-from django.shortcuts import redirect, get_object_or_404
-from django.utils.http import http_date
-from django.views.generic import ListView, DetailView, TemplateView
-from django.views.generic.edit import UpdateView, FormMixin, DeleteView
-from django.views.generic.detail import SingleObjectMixin
-from django.forms.models import modelform_factory
-from django.conf import settings
-from django.utils.translation import gettext_lazy as _
-from django.http import Http404, HttpResponse
-from wsgiref.util import FileWrapper
-from django.urls import reverse
-from django.core.exceptions import PermissionDenied
-from django import forms
-
 import os
+from wsgiref.util import FileWrapper
 
 from ajax_select import make_ajax_field
+from django import forms
+from django.conf import settings
+from django.core.exceptions import PermissionDenied
+from django.forms.models import modelform_factory
+from django.http import Http404, HttpResponse
+from django.shortcuts import get_object_or_404, redirect
+from django.urls import reverse
+from django.utils.http import http_date
+from django.utils.translation import gettext_lazy as _
+from django.views.generic import DetailView, ListView, TemplateView
+from django.views.generic.detail import SingleObjectMixin
+from django.views.generic.edit import DeleteView, FormMixin, UpdateView
 
-from core.models import SithFile, RealGroup, Notification
+from core.models import Notification, RealGroup, SithFile
 from core.views import (
-    CanViewMixin,
     CanEditMixin,
     CanEditPropMixin,
+    CanViewMixin,
     can_view,
 )
 from counter.models import Counter

core/views/forms.py

@@ -21,40 +21,37 @@
 # Place - Suite 330, Boston, MA 02111-1307, USA.
 #
 #
+import datetime
+import re
+from io import BytesIO
+
+from ajax_select import make_ajax_field
+from ajax_select.fields import AutoCompleteSelectField
 from captcha.fields import CaptchaField
-from django.contrib.auth.forms import UserCreationForm, AuthenticationForm
 from django import forms
 from django.conf import settings
-from django.db import transaction
-from django.templatetags.static import static
-from django.urls import reverse
+from django.contrib.auth.forms import AuthenticationForm, UserCreationForm
 from django.core.exceptions import ValidationError
+from django.db import transaction
 from django.forms import (
     CheckboxSelectMultiple,
-    Select,
     DateInput,
-    TextInput,
     DateTimeInput,
     Textarea,
+    TextInput,
 )
-from django.utils.translation import gettext_lazy as _
-from django.utils.translation import gettext
-from phonenumber_field.widgets import PhoneNumberInternationalFallbackWidget
-from ajax_select.fields import AutoCompleteSelectField
-from ajax_select import make_ajax_field
-from django.utils.dateparse import parse_datetime
-from django.utils import timezone
-import datetime
 from django.forms.utils import to_current_timezone
-
-import re
-
-from core.models import User, Page, SithFile, Gift
-
-from core.utils import resize_image
-from io import BytesIO
+from django.templatetags.static import static
+from django.urls import reverse
+from django.utils import timezone
+from django.utils.dateparse import parse_datetime
+from django.utils.translation import gettext
+from django.utils.translation import gettext_lazy as _
+from phonenumber_field.widgets import PhoneNumberInternationalFallbackWidget
 from PIL import Image
 
+from core.models import Gift, Page, SithFile, User
+from core.utils import resize_image
 
 # Widgets
 

core/views/group.py

@@ -18,15 +18,12 @@
 This module contains views to manage Groups
 """
 
-from django.views.generic.edit import UpdateView, CreateView, DeleteView
-from django.views.generic import ListView
-from django.views.generic.edit import FormView
-from django.urls import reverse_lazy
-from django.shortcuts import get_object_or_404
-from django.utils.translation import gettext_lazy as _
-from django import forms
-
 from ajax_select.fields import AutoCompleteSelectMultipleField
+from django import forms
+from django.urls import reverse_lazy
+from django.utils.translation import gettext_lazy as _
+from django.views.generic import ListView
+from django.views.generic.edit import CreateView, DeleteView, UpdateView
 
 from core.models import RealGroup, User
 from core.views import CanCreateMixin, CanEditMixin, DetailFormView

core/views/page.py

@@ -15,16 +15,16 @@
 #
 
 # This file contains all the views that concern the page model
-from django.urls import reverse_lazy
-from django.views.generic import ListView, DetailView
-from django.views.generic.edit import UpdateView, CreateView, DeleteView
 from django.forms.models import modelform_factory
 from django.http import Http404
 from django.shortcuts import redirect
+from django.urls import reverse_lazy
+from django.views.generic import DetailView, ListView
+from django.views.generic.edit import CreateView, DeleteView, UpdateView
 
-from core.models import Page, PageRev, LockError
+from core.models import LockError, Page, PageRev
+from core.views import CanCreateMixin, CanEditMixin, CanEditPropMixin, CanViewMixin
 from core.views.forms import MarkdownInput, PageForm, PagePropForm
-from core.views import CanViewMixin, CanEditMixin, CanEditPropMixin, CanCreateMixin
 
 
 class CanEditPagePropMixin(CanEditPropMixin):

core/views/site.py

@@ -23,23 +23,22 @@
 #
 #
 
-from django.shortcuts import render, redirect
-from django.http import JsonResponse
-from django.core import serializers
-from django.contrib.auth.decorators import login_required
-from django.utils import html
-from django.views.generic import ListView, TemplateView
-from django.conf import settings
-from django.utils.text import slugify
-from django.db.models.query import QuerySet
-
 import json
 
+from django.conf import settings
+from django.contrib.auth.decorators import login_required
+from django.core import serializers
+from django.db.models.query import QuerySet
+from django.http import JsonResponse
+from django.shortcuts import redirect, render
+from django.utils import html
+from django.utils.text import slugify
+from django.views.generic import ListView, TemplateView
 from haystack.query import SearchQuerySet
 
-from core.models import User, Notification
-from core.utils import doku_to_markdown, bbcode_to_markdown
 from club.models import Club
+from core.models import Notification, User
+from core.utils import bbcode_to_markdown, doku_to_markdown
 
 
 def index(request, context=None):

core/views/user.py

@@ -24,50 +24,49 @@
 #
 
 # This file contains all the views that concern the user model
-from django.shortcuts import render, redirect, get_object_or_404
+import logging
+from datetime import date, timedelta
+
+from django.conf import settings
 from django.contrib.auth import views
 from django.contrib.auth.forms import PasswordChangeForm
-from django.utils.translation import gettext as _
-from django.urls import reverse
 from django.core.exceptions import PermissionDenied, ValidationError
+from django.forms import CheckboxSelectMultiple
+from django.forms.models import modelform_factory
 from django.http import Http404, HttpResponse
-from django.views.generic.edit import UpdateView
+from django.shortcuts import get_object_or_404, redirect, render
+from django.template.response import TemplateResponse
+from django.urls import reverse, reverse_lazy
+from django.utils.translation import gettext as _
 from django.views.generic import (
-    ListView,
-    DetailView,
-    TemplateView,
     CreateView,
     DeleteView,
+    DetailView,
+    ListView,
+    TemplateView,
 )
-from django.forms.models import modelform_factory
-from django.forms import CheckboxSelectMultiple
-from django.urls import reverse_lazy
-from django.template.response import TemplateResponse
-from django.conf import settings
-from django.views.generic.dates import YearMixin, MonthMixin
+from django.views.generic.dates import MonthMixin, YearMixin
+from django.views.generic.edit import UpdateView
 
-from datetime import timedelta, date
-import logging
 from api.views.sas import all_pictures_of_user
-
+from core.models import Gift, Preferences, SithFile, User
 from core.views import (
-    CanViewMixin,
     CanEditMixin,
     CanEditPropMixin,
-    UserIsLoggedMixin,
-    TabedViewMixin,
+    CanViewMixin,
     QuickNotifMixin,
+    TabedViewMixin,
+    UserIsLoggedMixin,
 )
 from core.views.forms import (
-    RegisteringForm,
-    UserProfileForm,
-    LoginForm,
-    UserGodfathersForm,
     GiftForm,
+    LoginForm,
+    RegisteringForm,
+    UserGodfathersForm,
+    UserProfileForm,
 )
-from core.models import User, SithFile, Preferences, Gift
-from subscription.models import Subscription
 from counter.forms import StudentCardForm
+from subscription.models import Subscription
 from trombi.views import UserTrombiForm
 
 
@@ -501,9 +500,10 @@ class UserStatsView(UserTabsMixin, CanViewMixin, DetailView):
 
     def get_context_data(self, **kwargs):
         kwargs = super(UserStatsView, self).get_context_data(**kwargs)
-        from counter.models import Counter
         from django.db.models import Sum
 
+        from counter.models import Counter
+
         foyer = Counter.objects.filter(name="Foyer").first()
         mde = Counter.objects.filter(name="MDE").first()
         gommette = Counter.objects.filter(name="La Gommette").first()
@@ -601,10 +601,12 @@ class UserUploadProfilePictView(CanEditMixin, DetailView):
     template_name = "core/user_edit.jinja"
 
     def post(self, request, *args, **kwargs):
-        from core.utils import resize_image
         from io import BytesIO
+
         from PIL import Image
 
+        from core.utils import resize_image
+
         self.object = self.get_object()
         if self.object.profile_pict:
             raise ValidationError(_("User already has a profile picture"))