Add test case in user preferences for other subscriber accessing and accessing trombi user

Fix crash when admin gets to preferences of an user subscribed to a trombinoscope

.github/actions/setup_project/action.yml

@@ -6,26 +6,20 @@ runs:
     - name: Install apt packages
       uses: awalsh128/cache-apt-pkgs-action@latest
       with:
-        packages: gettext libgraphviz-dev
+        packages: gettext
         version: 1.0  # increment to reset cache
 
-    - name: Install dependencies
-      run: |
-        sudo apt update
-        sudo apt install gettext libgraphviz-dev
-      shell: bash
-
     - name: Set up python
-      uses: actions/setup-python@v4
+      uses: actions/setup-python@v5
       with:
-        python-version: "3.10"
+        python-version: "3.12"
 
     - name: Load cached Poetry installation
       id: cached-poetry
       uses: actions/cache@v3
       with:
         path: ~/.local
-        key: poetry-0  # increment to reset cache
+        key: poetry-3  # increment to reset cache
 
     - name: Install Poetry
       if: steps.cached-poetry.outputs.cache-hit != 'true'

.github/dependabot.yml

@@ -8,11 +8,7 @@ updates:
   - package-ecosystem: "pip" # See documentation for possible values
     directory: "/" # Location of package manifests
     schedule:
-      interval: "daily"
-    # Raise pull requests for version updates
-    # to pip against the `develop` branch
+      interval: "weekly"
     target-branch: "taiste"
-    reviewers:
-      - "ae-utbm/developpers-v3"
     commit-message:
       prefix: "[UPDATE] "

.github/workflows/ci.yml

@@ -1,4 +1,4 @@
-name: Sith 3 CI
+name: Sith CI
 
 on:
   push:
@@ -8,21 +8,23 @@ on:
   workflow_dispatch:
 
 jobs:
-  ruff:
-    name: Ruff lint & format
+  pre-commit:
+    name: Launch pre-commits checks (ruff)
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
-      - name: ruff format
-        uses: chartboost/ruff-action@v1  # format
-        with:
-          args: format --diff
-      - name: ruff check
-        uses: chartboost/ruff-action@v1  # lint
+    - uses: actions/checkout@v4
+    - uses: actions/setup-python@v5
+    - uses: pre-commit/action@v3.0.1
+      with:
+        extra_args: --all-files
 
   tests:
     name: Run tests and generate coverage report
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false  # don't interrupt the other test processes
+      matrix:
+        pytest-mark: [slow, not slow]
     steps:
       - name: Check out repository
         uses: actions/checkout@v4
@@ -30,7 +32,7 @@ jobs:
       - uses: ./.github/actions/setup_xapian
       - uses: ./.github/actions/compile_messages
       - name: Run tests
-        run: poetry run coverage run -m pytest
+        run: poetry run coverage run -m pytest -m "${{ matrix.pytest-mark }}"
       - name: Generate coverage report
         run: |
           poetry run coverage report

.github/workflows/deploy.yml

@@ -14,7 +14,7 @@ jobs:
   
     steps:
     - name: SSH Remote Commands
-      uses: appleboy/ssh-action@dce9d565de8d876c11d93fa4fe677c0285a66d78
+      uses: appleboy/ssh-action@v1.1.0
       with:
         # Proxy
         proxy_host : ${{secrets.PROXY_HOST}}
@@ -31,16 +31,17 @@ jobs:
 
         script_stop: true
 
-        # See https://github.com/ae-utbm/sith3/wiki/GitHub-Actions#deployment-action
+        # See https://github.com/ae-utbm/sith/wiki/GitHub-Actions#deployment-action
         script: |
-          export PATH="/home/sith/.local/bin:$PATH"
-          pushd ${{secrets.SITH_PATH}}
+          cd ${{secrets.SITH_PATH}}
 
-          git pull
-          poetry install
+          git fetch
+          git reset --hard origin/master
+          poetry install --with prod --without docs,tests
+          npm install
+          poetry run ./manage.py install_xapian
           poetry run ./manage.py migrate
-          echo "yes" | poetry run ./manage.py collectstatic
-          poetry run ./manage.py compilestatic
+          poetry run ./manage.py collectstatic --clear --noinput
           poetry run ./manage.py compilemessages
 
           sudo systemctl restart uwsgi
@@ -51,14 +52,14 @@ jobs:
     timeout-minutes: 30
     needs: deployment
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
       - name: Sentry Release
-        uses: getsentry/action-release@v1.2.0
+        uses: getsentry/action-release@v1.7.0
         env:
           SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
           SENTRY_ORG: ${{ secrets.SENTRY_ORG }}
           SENTRY_PROJECT: ${{ secrets.SENTRY_PROJECT }}
           SENTRY_URL: ${{ secrets.SENTRY_URL }}
         with:
-          environment: production
+          environment: production

.github/workflows/deploy_docs.yml

@@ -0,0 +1,21 @@
+name: deploy_docs
+on:
+  push:
+    branches:
+      - master
+permissions:
+  contents: write
+jobs:
+  deploy:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: ./.github/actions/setup_project
+      - run: echo "cache_id=$(date --utc '+%V')" >> $GITHUB_ENV
+      - uses: actions/cache@v3
+        with:
+          key: mkdocs-material-${{ env.cache_id }}
+          path: .cache
+          restore-keys: |
+            mkdocs-material-
+      - run: poetry run mkdocs gh-deploy --force

.github/workflows/taiste.yml

@@ -1,4 +1,4 @@
-name: Sith3 taiste
+name: Sith taiste
 
 on:
   push:
@@ -13,7 +13,7 @@ jobs:
 
     steps:
     - name: SSH Remote Commands
-      uses: appleboy/ssh-action@dce9d565de8d876c11d93fa4fe677c0285a66d78
+      uses: appleboy/ssh-action@v1.1.0
       with:
         # Proxy
         proxy_host : ${{secrets.PROXY_HOST}}
@@ -30,34 +30,17 @@ jobs:
 
         script_stop: true
 
-        # See https://github.com/ae-utbm/sith3/wiki/GitHub-Actions#deployment-action
+        # See https://github.com/ae-utbm/sith/wiki/GitHub-Actions#deployment-action
         script: |
-          export PATH="$HOME/.poetry/bin:$PATH"
-          pushd ${{secrets.SITH_PATH}}
+          cd ${{secrets.SITH_PATH}}
 
-          git pull
-          poetry install
+          git fetch
+          git reset --hard origin/taiste
+          poetry install --with prod --without docs,tests
+          npm install
+          poetry run ./manage.py install_xapian
           poetry run ./manage.py migrate
-          echo "yes" | poetry run ./manage.py collectstatic
-          poetry run ./manage.py compilestatic
+          poetry run ./manage.py collectstatic --clear --noinput
           poetry run ./manage.py compilemessages
 
           sudo systemctl restart uwsgi
-
-  sentry:
-    runs-on: ubuntu-latest
-    environment: taiste
-    timeout-minutes: 30
-    needs: deployment
-    steps:
-      - uses: actions/checkout@v3
-
-      - name: Sentry Release
-        uses: getsentry/action-release@v1.2.0
-        env:
-          SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
-          SENTRY_ORG: ${{ secrets.SENTRY_ORG }}
-          SENTRY_PROJECT: ${{ secrets.SENTRY_PROJECT }}
-          SENTRY_URL: ${{ secrets.SENTRY_URL }}
-        with:
-          environment: taiste

.gitignore

@@ -1,4 +1,4 @@
-db.sqlite3
+*.sqlite3
 *.log
 *.pyc
 *.mo
@@ -17,4 +17,7 @@ sith/settings_custom.py
 sith/search_indexes/
 .coverage
 coverage_report/
-doc/_build
+node_modules/
+
+# compiled documentation
+site/

.pre-commit-config.yaml

@@ -0,0 +1,26 @@
+repos:
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    # Ruff version.
+    rev: v0.8.3
+    hooks:
+      - id: ruff  # just check the code, and print the errors
+      - id: ruff  # actually fix the fixable errors, but print nothing
+        args: ["--fix", "--silent"]
+      # Run the formatter.
+      - id: ruff-format
+  - repo: https://github.com/biomejs/pre-commit
+    rev: "v0.1.0"  # Use the sha / tag you want to point at
+    hooks:
+      - id: biome-check
+        additional_dependencies: ["@biomejs/biome@1.9.3"]
+  - repo: https://github.com/rtts/djhtml
+    rev: 3.0.7
+    hooks:
+      - id: djhtml
+        name: format templates
+        entry: djhtml --tabwidth 2
+        types: ["jinja"]
+      - id: djcss
+        name: format scss files
+        entry: djcss --tabwidth 2
+        types: ["scss"]

.readthedocs.yml

@@ -1,26 +0,0 @@
-# Read the Docs configuration file
-# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
-
-# Required
-version: 2
-
-# Allow installing xapian-bindings in pip
-build:
-  apt_packages:
-    - libxapian-dev
-
-# Build documentation in the doc/ directory with Sphinx
-sphinx:
-  configuration: doc/conf.py
-
-# Optionally build your docs in additional formats such as PDF and ePub
-formats: all
-
-# Optionally set the version of Python and requirements required to build your docs
-python:
-  version: "3.8"
-  install:
-    - method: pip
-      path: .
-      extra_requirements:
-        - docs

CONTRIBUTORS.md

@@ -1,31 +0,0 @@
-# Contributors
-
-Thanks to everyone who has contributed to this project! We appreciate your time and effort.
-
-## Copyright
-
-> All contributions to this project are subject to the copyright owned by the Association des Étudiants de l'Université de Technologie de Belfort-Montbéliard (AE UTBM), for both past and future years. By making a contribution to this project, you acknowledge and agree that AE UTBM has the exclusive right to use, distribute, and modify your contribution, as well as to license others to do the same, in any way they see fit.
-
-## License
-
-This project was previously released under the MIT license, but it has since been changed to the GPL v3 license. Any contributions made to this project before the switch to GPL v3 are still subject to the MIT license, while contributions made after the switch are subject to the GPL v3 license.
-
-## List of Contributors
-
-- [@Hyask](https://github.com/Hyask) — Florent "Skia" Jacquet <skia@hya.sk>
-- [@klmp200](https://github.com/klmp200) — Antoine "Sli" Bartuccio <antoine@bartuccio.fr>
-- [@nab-os](https://github.com/nab-os) (AKA Gnikwo) — Sasha "Nabos" Ballet <nabos@glargh.fr>
-- [@Krophil](https://github.com/Krophil) — Pierre "Krophil'" Brunet <pierre.brunet@krophil.fr>
-- [@guillaume-renaud](https://github.com/guillaume-renaud) — Guillaume "Lo-J" RENAUD <renaudg779@gmail.com>
-- [@imperosol](https://github.com/imperosol) — Thomas "Maréchal" Girod <thgirod@hotmail.com>
-- [@TheoDurr](https://github.com/TheoDurr) — Théo "Ailé" Durr <git@theodurr.fr>
-- [@RTrioux](https://github.com/RTrioux) — Robin "Vial" Trioux
-- [@TheRolfFR](https://github.com/TheRolfFR) — Yann "Réseau" Le Vaguerès
-- [@Magador](https://github.com/Magador) — Lucie "Magador" Lenglet
-- [@lsacienne](https://github.com/lsacienne) — Alexandre "L'Sacienne" Viala
-- [@Juknum](https://github.com/Juknum) — Julien "Tinople" Constant
-- [@Tartofraise](https://github.com/Tartofraise) — Ryan "Soldat" Hadj-Mebarek
-- [@FrancescoWitz](https://github.com/FrancescoWitz) — Francesco "Och" WITZ
-- [@maxence-leblanc](https://github.com/maxence-leblanc) — Maxence "Juste" LEBLANC
-
-_If you've contributed to this project and your name isn't on the list, please let us know so we can add you. And if you've contributed anonymously, thank you! We appreciate your contributions just as much as those from named contributors._

LICENSE.old

@@ -1,21 +0,0 @@
-The MIT License (MIT)
-
-Copyright (c) 2016 Skia
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.

README.md

@@ -1,40 +1,21 @@
-<p align="center">
-  <a href="#">
-    <img src="https://img.shields.io/badge/Code%20Style-Black-000000?style=for-the-badge">
-  </a>
-  <a href="#">
-    <img src="https://img.shields.io/github/checks-status/ae-utbm/sith3/master?logo=github&style=for-the-badge&label=BUILD">
-  </a>
-  <a href="https://sith-ae.readthedocs.io/">
-    <img src="https://img.shields.io/readthedocs/sith-ae?logo=readthedocs&style=for-the-badge">
-  </a>
-  <a href="https://discord.gg/XK9WfPsUFm">
-    <img src="https://img.shields.io/discord/971448179075731476?label=Discord&logo=discord&style=for-the-badge">
-  </a>
-</p>
+# Sith
 
-<h3 align="center">This is the source code of the UTBM's student association available at https://ae.utbm.fr/.</h3>
+[![Ruff](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/astral-sh/ruff/main/assets/badge/v2.json)](#)
+[![License: GPL v3](https://img.shields.io/badge/License-GPLv3-blue.svg)](https://www.gnu.org/licenses/gpl-3.0)
+[![CI status](https://github.com/ae-utbm/sith/actions/workflows/ci.yml/badge.svg)](#)
+[![Docs status](https://github.com/ae-utbm/sith/actions/workflows/deploy_docs.yml/badge.svg)](https://ae-utbm.github.io/sith)
+[![Built with Material for MkDocs](https://img.shields.io/badge/Material_for_MkDocs-526CFE?style=default&logo=MaterialForMkDocs&logoColor=white)](https://squidfunk.github.io/mkdocs-material/)
+[![Checked with Biome](https://img.shields.io/badge/Checked_with-Biome-60a5fa?style=flat&logo=biome)](https://biomejs.dev)
+[![discord](https://img.shields.io/discord/971448179075731476?label=discord&logo=discord&style=default)](https://discord.gg/xk9wfpsufm)
 
-<p align="justify">All documentation is in the <code>docs</code> directory and online at https://sith-ae.readthedocs.io/. This documentation is written in French because it targets a French audience and it's too much work to maintain two versions. The code and code comments are strictly written in English.</p>
+### This is the source code of the UTBM's student association available at [https://ae.utbm.fr/](https://ae.utbm.fr/).
 
-<h4>If you want to contribute, here's how we recommend to read the docs:</h4>
+All documentation is in the `docs` directory and online at [https://ae-utbm.github.io/sith](https://ae-utbm.github.io/sith). This documentation is written in French because it targets a French audience and it's too much work to maintain two versions. The code and code comments are strictly written in English.
 
-<ul>
-  <li>
-    <p align="justify">
-      First, it's advised to read the about part of the project to understand the goals and the mindset of the current and previous maintainers and know what to expect to learn.
-    </p>
-  </li>
-  <li>
-    <p align="justify">
-      If in the first part you realize that you need more background about what we use, we provide some links to tutorials and documentation at the end of our documentation. Feel free to use it and complete it with what you found helpful.
-    </p>
-  </li>
-  <li>
-    <p align="justify">
-      Keep in mind that this documentation is thought to be read in order.
-    </p>
-  </li>
-</ul>
+#### If you want to contribute, here's how we recommend to read the docs:
+
+* First, it's advised to read the about part of the project to understand the goals and the mindset of the current and previous maintainers and know what to expect to learn.
+* If in the first part you realize that you need more background about what we use, we provide some links to tutorials and documentation at the end of our documentation. Feel free to use it and complete it with what you found helpful.
+* Keep in mind that this documentation is thought to be read in order.
 
 > This project is licensed under GNU GPL, see the LICENSE file at the top of the repository for more details.

accounting/__init__.py

@@ -1,19 +1,14 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #

accounting/admin.py

@@ -1,26 +1,30 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 
 from django.contrib import admin
 
-from accounting.models import *
+from accounting.models import (
+    AccountingType,
+    BankAccount,
+    ClubAccount,
+    Company,
+    GeneralJournal,
+    Label,
+    Operation,
+    SimplifiedAccountingType,
+)
 
 admin.site.register(BankAccount)
 admin.site.register(ClubAccount)

accounting/api.py

@@ -0,0 +1,23 @@
+from typing import Annotated
+
+from annotated_types import MinLen
+from ninja_extra import ControllerBase, api_controller, paginate, route
+from ninja_extra.pagination import PageNumberPaginationExtra
+from ninja_extra.schemas import PaginatedResponseSchema
+
+from accounting.models import ClubAccount, Company
+from accounting.schemas import ClubAccountSchema, CompanySchema
+from core.api_permissions import CanAccessLookup
+
+
+@api_controller("/lookup", permissions=[CanAccessLookup])
+class AccountingController(ControllerBase):
+    @route.get("/club-account", response=PaginatedResponseSchema[ClubAccountSchema])
+    @paginate(PageNumberPaginationExtra, page_size=50)
+    def search_club_account(self, search: Annotated[str, MinLen(1)]):
+        return ClubAccount.objects.filter(name__icontains=search).values()
+
+    @route.get("/company", response=PaginatedResponseSchema[CompanySchema])
+    @paginate(PageNumberPaginationExtra, page_size=50)
+    def search_company(self, search: Annotated[str, MinLen(1)]):
+        return Company.objects.filter(name__icontains=search).values()

accounting/migrations/0001_initial.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 import django.core.validators

accounting/migrations/0002_auto_20160824_2152.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 import django.db.models.deletion
@@ -101,6 +100,6 @@ class Migration(migrations.Migration):
             ),
         ),
         migrations.AlterUniqueTogether(
-            name="operation", unique_together=set([("number", "journal")])
+            name="operation", unique_together={("number", "journal")}
         ),
     ]

accounting/migrations/0003_auto_20160824_2203.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 import phonenumber_field.modelfields

accounting/migrations/0004_auto_20161005_1505.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 import django.db.models.deletion
@@ -46,6 +45,6 @@ class Migration(migrations.Migration):
             ),
         ),
         migrations.AlterUniqueTogether(
-            name="label", unique_together=set([("name", "club_account")])
+            name="label", unique_together={("name", "club_account")}
         ),
     ]

accounting/migrations/0005_auto_20170324_0917.py

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

accounting/models.py

@@ -1,21 +1,16 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 
 from decimal import Decimal
@@ -34,22 +29,32 @@ from core.models import SithFile, User
 
 
 class CurrencyField(models.DecimalField):
-    """
-    This is a custom database field used for currency
-    """
+    """Custom database field used for currency."""
 
     def __init__(self, *args, **kwargs):
         kwargs["max_digits"] = 12
         kwargs["decimal_places"] = 2
-        super(CurrencyField, self).__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
 
     def to_python(self, value):
         try:
-            return super(CurrencyField, self).to_python(value).quantize(Decimal("0.01"))
+            return super().to_python(value).quantize(Decimal("0.01"))
         except AttributeError:
             return None
 
 
+if settings.TESTING:
+    from model_bakery import baker
+
+    baker.generators.add(
+        CurrencyField,
+        lambda: baker.random_gen.gen_decimal(max_digits=8, decimal_places=2),
+    )
+else:  # pragma: no cover
+    # baker is only used in tests, so we don't need coverage for this part
+    pass
+
+
 # Accounting classes
 
 
@@ -66,31 +71,8 @@ class Company(models.Model):
     class Meta:
         verbose_name = _("company")
 
-    def is_owned_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
-        if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
-            return True
-        return False
-
-    def can_be_edited_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
-        for club in user.memberships.filter(end_date=None).all():
-            if club and club.role == settings.SITH_CLUB_ROLES_ID["Treasurer"]:
-                return True
-        return False
-
-    def can_be_viewed_by(self, user):
-        """
-        Method to see if that object can be viewed by the given user
-        """
-        for club in user.memberships.filter(end_date=None).all():
-            if club and club.role >= settings.SITH_CLUB_ROLES_ID["Treasurer"]:
-                return True
-        return False
+    def __str__(self):
+        return self.name
 
     def get_absolute_url(self):
         return reverse("accounting:co_edit", kwargs={"co_id": self.id})
@@ -98,8 +80,21 @@ class Company(models.Model):
     def get_display_name(self):
         return self.name
 
-    def __str__(self):
-        return self.name
+    def is_owned_by(self, user):
+        """Check if that object can be edited by the given user."""
+        return user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID)
+
+    def can_be_edited_by(self, user):
+        """Check if that object can be edited by the given user."""
+        return user.memberships.filter(
+            end_date=None, club__role=settings.SITH_CLUB_ROLES_ID["Treasurer"]
+        ).exists()
+
+    def can_be_viewed_by(self, user):
+        """Check if that object can be viewed by the given user."""
+        return user.memberships.filter(
+            end_date=None, club__role_gte=settings.SITH_CLUB_ROLES_ID["Treasurer"]
+        ).exists()
 
 
 class BankAccount(models.Model):
@@ -117,24 +112,20 @@ class BankAccount(models.Model):
         verbose_name = _("Bank account")
         ordering = ["club", "name"]
 
+    def __str__(self):
+        return self.name
+
+    def get_absolute_url(self):
+        return reverse("accounting:bank_details", kwargs={"b_account_id": self.id})
+
     def is_owned_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
+        """Check if that object can be edited by the given user."""
         if user.is_anonymous:
             return False
         if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
             return True
         m = self.club.get_membership_for(user)
-        if m is not None and m.role >= settings.SITH_CLUB_ROLES_ID["Treasurer"]:
-            return True
-        return False
-
-    def get_absolute_url(self):
-        return reverse("accounting:bank_details", kwargs={"b_account_id": self.id})
-
-    def __str__(self):
-        return self.name
+        return m is not None and m.role >= settings.SITH_CLUB_ROLES_ID["Treasurer"]
 
 
 class ClubAccount(models.Model):
@@ -156,48 +147,33 @@ class ClubAccount(models.Model):
         verbose_name = _("Club account")
         ordering = ["bank_account", "name"]
 
-    def is_owned_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
-        if user.is_anonymous:
-            return False
-        if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
-            return True
-        return False
-
-    def can_be_edited_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
-        m = self.club.get_membership_for(user)
-        if m and m.role == settings.SITH_CLUB_ROLES_ID["Treasurer"]:
-            return True
-        return False
-
-    def can_be_viewed_by(self, user):
-        """
-        Method to see if that object can be viewed by the given user
-        """
-        m = self.club.get_membership_for(user)
-        if m and m.role >= settings.SITH_CLUB_ROLES_ID["Treasurer"]:
-            return True
-        return False
-
-    def has_open_journal(self):
-        for j in self.journals.all():
-            if not j.closed:
-                return True
-        return False
-
-    def get_open_journal(self):
-        return self.journals.filter(closed=False).first()
+    def __str__(self):
+        return self.name
 
     def get_absolute_url(self):
         return reverse("accounting:club_details", kwargs={"c_account_id": self.id})
 
-    def __str__(self):
-        return self.name
+    def is_owned_by(self, user):
+        """Check if that object can be edited by the given user."""
+        if user.is_anonymous:
+            return False
+        return user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID)
+
+    def can_be_edited_by(self, user):
+        """Check if that object can be edited by the given user."""
+        m = self.club.get_membership_for(user)
+        return m and m.role == settings.SITH_CLUB_ROLES_ID["Treasurer"]
+
+    def can_be_viewed_by(self, user):
+        """Check if that object can be viewed by the given user."""
+        m = self.club.get_membership_for(user)
+        return m and m.role >= settings.SITH_CLUB_ROLES_ID["Treasurer"]
+
+    def has_open_journal(self):
+        return self.journals.filter(closed=False).exists()
+
+    def get_open_journal(self):
+        return self.journals.filter(closed=False).first()
 
     def get_display_name(self):
         return _("%(club_account)s on %(bank_account)s") % {
@@ -207,9 +183,7 @@ class ClubAccount(models.Model):
 
 
 class GeneralJournal(models.Model):
-    """
-    Class storing all the operations for a period of time
-    """
+    """Class storing all the operations for a period of time."""
 
     start_date = models.DateField(_("start date"))
     end_date = models.DateField(_("end date"), null=True, blank=True, default=None)
@@ -229,36 +203,28 @@ class GeneralJournal(models.Model):
         verbose_name = _("General journal")
         ordering = ["-start_date"]
 
-    def is_owned_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
-        if user.is_anonymous:
-            return False
-        if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
-            return True
-        if self.club_account.can_be_edited_by(user):
-            return True
-        return False
-
-    def can_be_edited_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
-        if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
-            return True
-        if self.club_account.can_be_edited_by(user):
-            return True
-        return False
-
-    def can_be_viewed_by(self, user):
-        return self.club_account.can_be_viewed_by(user)
+    def __str__(self):
+        return self.name
 
     def get_absolute_url(self):
         return reverse("accounting:journal_details", kwargs={"j_id": self.id})
 
-    def __str__(self):
-        return self.name
+    def is_owned_by(self, user):
+        """Check if that object can be edited by the given user."""
+        if user.is_anonymous:
+            return False
+        if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
+            return True
+        return self.club_account.can_be_edited_by(user)
+
+    def can_be_edited_by(self, user):
+        """Check if that object can be edited by the given user."""
+        if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
+            return True
+        return self.club_account.can_be_edited_by(user)
+
+    def can_be_viewed_by(self, user):
+        return self.club_account.can_be_viewed_by(user)
 
     def update_amounts(self):
         self.amount = 0
@@ -276,9 +242,7 @@ class GeneralJournal(models.Model):
 
 
 class Operation(models.Model):
-    """
-    An operation is a line in the journal, a debit or a credit
-    """
+    """An operation is a line in the journal, a debit or a credit."""
 
     number = models.IntegerField(_("number"))
     journal = models.ForeignKey(
@@ -361,6 +325,18 @@ class Operation(models.Model):
         unique_together = ("number", "journal")
         ordering = ["-number"]
 
+    def __str__(self):
+        return f"{self.amount} € | {self.date} | {self.accounting_type} | {self.done}"
+
+    def save(self, *args, **kwargs):
+        if self.number is None:
+            self.number = self.journal.operations.count() + 1
+        super().save(*args, **kwargs)
+        self.journal.update_amounts()
+
+    def get_absolute_url(self):
+        return reverse("accounting:journal_details", kwargs={"j_id": self.journal.id})
+
     def __getattribute__(self, attr):
         if attr == "target":
             return self.get_target()
@@ -368,7 +344,7 @@ class Operation(models.Model):
             return object.__getattribute__(self, attr)
 
     def clean(self):
-        super(Operation, self).clean()
+        super().clean()
         if self.date is None:
             raise ValidationError(_("The date must be set."))
         elif self.date < self.journal.start_date:
@@ -414,16 +390,8 @@ class Operation(models.Model):
             tar = Company.objects.filter(id=self.target_id).first()
         return tar
 
-    def save(self):
-        if self.number is None:
-            self.number = self.journal.operations.count() + 1
-        super(Operation, self).save()
-        self.journal.update_amounts()
-
     def is_owned_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
+        """Check if that object can be edited by the given user."""
         if user.is_anonymous:
             return False
         if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
@@ -431,40 +399,22 @@ class Operation(models.Model):
         if self.journal.closed:
             return False
         m = self.journal.club_account.club.get_membership_for(user)
-        if m is not None and m.role >= settings.SITH_CLUB_ROLES_ID["Treasurer"]:
-            return True
-        return False
+        return m is not None and m.role >= settings.SITH_CLUB_ROLES_ID["Treasurer"]
 
     def can_be_edited_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
+        """Check if that object can be edited by the given user."""
         if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
             return True
         if self.journal.closed:
             return False
         m = self.journal.club_account.club.get_membership_for(user)
-        if m is not None and m.role == settings.SITH_CLUB_ROLES_ID["Treasurer"]:
-            return True
-        return False
-
-    def get_absolute_url(self):
-        return reverse("accounting:journal_details", kwargs={"j_id": self.journal.id})
-
-    def __str__(self):
-        return "%d € | %s | %s | %s" % (
-            self.amount,
-            self.date,
-            self.accounting_type,
-            self.done,
-        )
+        return m is not None and m.role == settings.SITH_CLUB_ROLES_ID["Treasurer"]
 
 
 class AccountingType(models.Model):
-    """
-    Class describing the accounting types.
+    """Accounting types.
 
-    Thoses are numbers used in accounting to classify operations
+    Those are numbers used in accounting to classify operations
     """
 
     code = models.CharField(
@@ -491,27 +441,21 @@ class AccountingType(models.Model):
         verbose_name = _("accounting type")
         ordering = ["movement_type", "code"]
 
-    def is_owned_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
-        if user.is_anonymous:
-            return False
-        if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID):
-            return True
-        return False
+    def __str__(self):
+        return self.code + " - " + self.get_movement_type_display() + " - " + self.label
 
     def get_absolute_url(self):
         return reverse("accounting:type_list")
 
-    def __str__(self):
-        return self.code + " - " + self.get_movement_type_display() + " - " + self.label
+    def is_owned_by(self, user):
+        """Check if that object can be edited by the given user."""
+        if user.is_anonymous:
+            return False
+        return user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID)
 
 
 class SimplifiedAccountingType(models.Model):
-    """
-    Class describing the simplified accounting types.
-    """
+    """Simplified version of `AccountingType`."""
 
     label = models.CharField(_("label"), max_length=128)
     accounting_type = models.ForeignKey(
@@ -525,6 +469,15 @@ class SimplifiedAccountingType(models.Model):
         verbose_name = _("simplified type")
         ordering = ["accounting_type__movement_type", "accounting_type__code"]
 
+    def __str__(self):
+        return (
+            f"{self.get_movement_type_display()} "
+            f"- {self.accounting_type.code} - {self.label}"
+        )
+
+    def get_absolute_url(self):
+        return reverse("accounting:simple_type_list")
+
     @property
     def movement_type(self):
         return self.accounting_type.movement_type
@@ -532,21 +485,9 @@ class SimplifiedAccountingType(models.Model):
     def get_movement_type_display(self):
         return self.accounting_type.get_movement_type_display()
 
-    def get_absolute_url(self):
-        return reverse("accounting:simple_type_list")
-
-    def __str__(self):
-        return (
-            self.get_movement_type_display()
-            + " - "
-            + self.accounting_type.code
-            + " - "
-            + self.label
-        )
-
 
 class Label(models.Model):
-    """Label allow a club to sort its operations"""
+    """Label allow a club to sort its operations."""
 
     name = models.CharField(_("label"), max_length=64)
     club_account = models.ForeignKey(

accounting/schemas.py

@@ -0,0 +1,15 @@
+from ninja import ModelSchema
+
+from accounting.models import ClubAccount, Company
+
+
+class ClubAccountSchema(ModelSchema):
+    class Meta:
+        model = ClubAccount
+        fields = ["id", "name"]
+
+
+class CompanySchema(ModelSchema):
+    class Meta:
+        model = Company
+        fields = ["id", "name"]

accounting/static/bundled/accounting/components/ajax-select-index.ts

@@ -0,0 +1,60 @@
+import { AjaxSelect } from "#core:core/components/ajax-select-base";
+import { registerComponent } from "#core:utils/web-components";
+import type { TomOption } from "tom-select/dist/types/types";
+import type { escape_html } from "tom-select/dist/types/utils";
+import {
+  type ClubAccountSchema,
+  type CompanySchema,
+  accountingSearchClubAccount,
+  accountingSearchCompany,
+} from "#openapi";
+
+@registerComponent("club-account-ajax-select")
+export class ClubAccountAjaxSelect extends AjaxSelect {
+  protected valueField = "id";
+  protected labelField = "name";
+  protected searchField = ["code", "name"];
+
+  protected async search(query: string): Promise<TomOption[]> {
+    const resp = await accountingSearchClubAccount({ query: { search: query } });
+    if (resp.data) {
+      return resp.data.results;
+    }
+    return [];
+  }
+
+  protected renderOption(item: ClubAccountSchema, sanitize: typeof escape_html) {
+    return `<div class="select-item">
+            <span class="select-item-text">${sanitize(item.name)}</span>
+          </div>`;
+  }
+
+  protected renderItem(item: ClubAccountSchema, sanitize: typeof escape_html) {
+    return `<span>${sanitize(item.name)}</span>`;
+  }
+}
+
+@registerComponent("company-ajax-select")
+export class CompanyAjaxSelect extends AjaxSelect {
+  protected valueField = "id";
+  protected labelField = "name";
+  protected searchField = ["code", "name"];
+
+  protected async search(query: string): Promise<TomOption[]> {
+    const resp = await accountingSearchCompany({ query: { search: query } });
+    if (resp.data) {
+      return resp.data.results;
+    }
+    return [];
+  }
+
+  protected renderOption(item: CompanySchema, sanitize: typeof escape_html) {
+    return `<div class="select-item">
+            <span class="select-item-text">${sanitize(item.name)}</span>
+          </div>`;
+  }
+
+  protected renderItem(item: CompanySchema, sanitize: typeof escape_html) {
+    return `<span>${sanitize(item.name)}</span>`;
+  }
+}

accounting/templates/accounting/accountingtype_list.jinja

@@ -1,27 +1,27 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Accounting type list{% endtrans %}
+  {% trans %}Accounting type list{% endtrans %}
 {% endblock %}
 
 {% block content %}
-    <div id="accounting">
-        <p>
-        <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
-        {% trans %}Accounting types{% endtrans %}
-        </p>
-        <hr>
-        <p><a href="{{ url('accounting:type_new') }}">{% trans %}New accounting type{% endtrans %}</a></p>
-        {% if accountingtype_list %}
-        <h3>{% trans %}Accounting type list{% endtrans %}</h3>
-        <ul>
-            {% for a in accountingtype_list  %}
-            <li><a href="{{ url('accounting:type_edit', type_id=a.id) }}">{{ a }}</a></li>
-            {% endfor %}
-        </ul>
-        {% else %}
-        {% trans %}There is no types in this website.{% endtrans %}
-        {% endif %}
-    </div>
+  <div id="accounting">
+    <p>
+      <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
+      {% trans %}Accounting types{% endtrans %}
+    </p>
+    <hr>
+    <p><a href="{{ url('accounting:type_new') }}">{% trans %}New accounting type{% endtrans %}</a></p>
+    {% if accountingtype_list %}
+      <h3>{% trans %}Accounting type list{% endtrans %}</h3>
+      <ul>
+        {% for a in accountingtype_list  %}
+          <li><a href="{{ url('accounting:type_edit', type_id=a.id) }}">{{ a }}</a></li>
+        {% endfor %}
+      </ul>
+    {% else %}
+      {% trans %}There is no types in this website.{% endtrans %}
+    {% endif %}
+  </div>
 {% endblock %}
 

accounting/templates/accounting/bank_account_details.jinja

@@ -1,37 +1,37 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Bank account: {% endtrans %}{{ object.name }}
+  {% trans %}Bank account: {% endtrans %}{{ object.name }}
 {% endblock %}
 
 {% block content %}
-    <div id="accounting">
-        <p>
-        <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
-        {{ object.name }}
-        </p>
-        <hr>
-        <h2>{% trans %}Bank account: {% endtrans %}{{ object.name }}</h2>
-        {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) and not object.club_accounts.exists() %}
-        <a href="{{ url('accounting:bank_delete', b_account_id=object.id) }}">{% trans %}Delete{% endtrans %}</a>
-        {% endif %}
-        <h4>{% trans %}Infos{% endtrans %}</h4>
-        <ul>
-            <li><strong>{% trans %}IBAN: {% endtrans %}</strong>{{ object.iban }}</li>
-            <li><strong>{% trans %}Number: {% endtrans %}</strong>{{ object.number }}</li>
-        </ul>
-        <p><a href="{{ url('accounting:club_new') }}?parent={{ object.id }}">{% trans %}New club account{% endtrans %}</a></p>
-        <ul>
-        {% for c in object.club_accounts.all() %}
-            <li><a href="{{ url('accounting:club_details', c_account_id=c.id) }}">{{ c }}</a>
-                - <a href="{{ url('accounting:club_edit', c_account_id=c.id) }}">{% trans %}Edit{% endtrans %}</a>
-                {% if c.journals.count() == 0 %}
-                - <a href="{{ url('accounting:club_delete', c_account_id=c.id) }}">{% trans %}Delete{% endtrans %}</a>
-                {% endif %}
-                </li>
-        {% endfor %}
-        </ul>
-    </div>
+  <div id="accounting">
+    <p>
+      <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
+      {{ object.name }}
+    </p>
+    <hr>
+    <h2>{% trans %}Bank account: {% endtrans %}{{ object.name }}</h2>
+    {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) and not object.club_accounts.exists() %}
+      <a href="{{ url('accounting:bank_delete', b_account_id=object.id) }}">{% trans %}Delete{% endtrans %}</a>
+    {% endif %}
+    <h4>{% trans %}Infos{% endtrans %}</h4>
+    <ul>
+      <li><strong>{% trans %}IBAN: {% endtrans %}</strong>{{ object.iban }}</li>
+      <li><strong>{% trans %}Number: {% endtrans %}</strong>{{ object.number }}</li>
+    </ul>
+    <p><a href="{{ url('accounting:club_new') }}?parent={{ object.id }}">{% trans %}New club account{% endtrans %}</a></p>
+    <ul>
+      {% for c in object.club_accounts.all() %}
+        <li><a href="{{ url('accounting:club_details', c_account_id=c.id) }}">{{ c }}</a>
+          - <a href="{{ url('accounting:club_edit', c_account_id=c.id) }}">{% trans %}Edit{% endtrans %}</a>
+          {% if c.journals.count() == 0 %}
+            - <a href="{{ url('accounting:club_delete', c_account_id=c.id) }}">{% trans %}Delete{% endtrans %}</a>
+          {% endif %}
+        </li>
+      {% endfor %}
+    </ul>
+  </div>
 {% endblock %}
 
 

accounting/templates/accounting/bank_account_list.jinja

@@ -1,32 +1,32 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Bank account list{% endtrans %}
+  {% trans %}Bank account list{% endtrans %}
 {% endblock %}
 
 {% block content %}
-    <div id="accounting">
-        <h4>
-        {% trans %}Accounting{% endtrans %}
-        </h4>
-        {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) %}
-        <p><a href="{{ url('accounting:simple_type_list') }}">{% trans %}Manage simplified types{% endtrans %}</a></p>
-        <p><a href="{{ url('accounting:type_list') }}">{% trans %}Manage accounting types{% endtrans %}</a></p>
-        <p><a href="{{ url('accounting:bank_new') }}">{% trans %}New bank account{% endtrans %}</a></p>
-        {% endif %}
-        {% if bankaccount_list %}
-        <h3>{% trans %}Bank account list{% endtrans %}</h3>
-            <ul>
-                {% for a in object_list  %}
-                <li><a href="{{ url('accounting:bank_details', b_account_id=a.id) }}">{{ a }}</a>
-                    - <a href="{{ url('accounting:bank_edit', b_account_id=a.id) }}">{% trans %}Edit{% endtrans %}</a>
-                </li>
-                {% endfor %}
-            </ul>
-        {% else %}
-        {% trans %}There is no accounts in this website.{% endtrans %}
-        {% endif %}
-    </div>
+  <div id="accounting">
+    <h4>
+      {% trans %}Accounting{% endtrans %}
+    </h4>
+    {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) %}
+      <p><a href="{{ url('accounting:simple_type_list') }}">{% trans %}Manage simplified types{% endtrans %}</a></p>
+      <p><a href="{{ url('accounting:type_list') }}">{% trans %}Manage accounting types{% endtrans %}</a></p>
+      <p><a href="{{ url('accounting:bank_new') }}">{% trans %}New bank account{% endtrans %}</a></p>
+    {% endif %}
+    {% if bankaccount_list %}
+      <h3>{% trans %}Bank account list{% endtrans %}</h3>
+      <ul>
+        {% for a in object_list  %}
+          <li><a href="{{ url('accounting:bank_details', b_account_id=a.id) }}">{{ a }}</a>
+            - <a href="{{ url('accounting:bank_edit', b_account_id=a.id) }}">{% trans %}Edit{% endtrans %}</a>
+          </li>
+        {% endfor %}
+      </ul>
+    {% else %}
+      {% trans %}There is no accounts in this website.{% endtrans %}
+    {% endif %}
+  </div>
 {% endblock %}
 
 

accounting/templates/accounting/club_account_details.jinja

@@ -1,68 +1,68 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Club account:{% endtrans %} {{ object.name }}
+  {% trans %}Club account:{% endtrans %} {{ object.name }}
 {% endblock %}
 
 {% block content %}
-    <div id="accounting">
-        <p>
-        <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
-        <a href="{{ url('accounting:bank_details', b_account_id=object.bank_account.id) }}">{{object.bank_account }}</a> >
-        {{ object }}
-        </p>
-        <hr>
-        <h2>{% trans %}Club account:{% endtrans %} {{ object.name }}</h2>
-        {% if user.is_root and not object.journals.exists() %}
-        <a href="{{ url('accounting:club_delete', c_account_id=object.id) }}">{% trans %}Delete{% endtrans %}</a>
-        {% endif %}
-        {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) %}
-        <p><a href="{{ url('accounting:label_new') }}?parent={{ object.id }}">{% trans %}New label{% endtrans %}</a></p>
-        {% endif %}
-        <p><a href="{{ url('accounting:label_list', clubaccount_id=object.id) }}">{% trans %}Label list{% endtrans %}</a></p>
-        {% if not object.has_open_journal() %}
-        <p><a href="{{ url('accounting:journal_new') }}?parent={{ object.id }}">{% trans %}New journal{% endtrans %}</a></p>
-        {% else %}
-        <p>{% trans %}You can not create new journal while you still have one opened{% endtrans %}</p>
-        {% endif %}
-        <table>
-            <thead>
-            <tr>
-                <td>{% trans %}Name{% endtrans %}</td>
-                <td>{% trans %}Start{% endtrans %}</td>
-                <td>{% trans %}End{% endtrans %}</td>
-                <td>{% trans %}Amount{% endtrans %}</td>
-                <td>{% trans %}Effective amount{% endtrans %}</td>
-                <td>{% trans %}Closed{% endtrans %}</td>
-                <td>{% trans %}Actions{% endtrans %}</td>
-            </tr>
-            </thead>
-            <tbody>
-            {% for j in object.journals.all() %}
-            <tr>
-                <td>{{ j.name }}</td>
-                <td>{{ j.start_date }}</td>
-                {% if j.end_date %}
-                <td>{{ j.end_date }}</td>
-                {% else %}
-                <td> - </td>
-                {% endif %}
-                <td>{{ j.amount }} €</td>
-                <td>{{ j.effective_amount }} €</td>
-                {% if j.closed %}
-                <td>{% trans %}Yes{% endtrans %}</td>
-                {% else %}
-                <td>{% trans %}No{% endtrans %}</td>
-                {% endif %}
-                <td> <a href="{{ url('accounting:journal_details', j_id=j.id) }}">{% trans %}View{% endtrans %}</a>
-                    <a href="{{ url('accounting:journal_edit', j_id=j.id) }}">{% trans %}Edit{% endtrans %}</a>
-                    {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) and j.operations.count() == 0 %}
-                        <a href="{{ url('accounting:journal_delete', j_id=j.id) }}">{% trans %}Delete{% endtrans %}</a>
-                    {% endif %}
-                    </td>
-            </tr>
-            {% endfor %}
-            </tbody>
-        </table>
-    </div>
+  <div id="accounting">
+    <p>
+      <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
+      <a href="{{ url('accounting:bank_details', b_account_id=object.bank_account.id) }}">{{object.bank_account }}</a> >
+      {{ object }}
+    </p>
+    <hr>
+    <h2>{% trans %}Club account:{% endtrans %} {{ object.name }}</h2>
+    {% if user.is_root and not object.journals.exists() %}
+      <a href="{{ url('accounting:club_delete', c_account_id=object.id) }}">{% trans %}Delete{% endtrans %}</a>
+    {% endif %}
+    {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) %}
+      <p><a href="{{ url('accounting:label_new') }}?parent={{ object.id }}">{% trans %}New label{% endtrans %}</a></p>
+    {% endif %}
+    <p><a href="{{ url('accounting:label_list', clubaccount_id=object.id) }}">{% trans %}Label list{% endtrans %}</a></p>
+    {% if not object.has_open_journal() %}
+      <p><a href="{{ url('accounting:journal_new') }}?parent={{ object.id }}">{% trans %}New journal{% endtrans %}</a></p>
+    {% else %}
+      <p>{% trans %}You can not create new journal while you still have one opened{% endtrans %}</p>
+    {% endif %}
+    <table>
+      <thead>
+        <tr>
+          <td>{% trans %}Name{% endtrans %}</td>
+          <td>{% trans %}Start{% endtrans %}</td>
+          <td>{% trans %}End{% endtrans %}</td>
+          <td>{% trans %}Amount{% endtrans %}</td>
+          <td>{% trans %}Effective amount{% endtrans %}</td>
+          <td>{% trans %}Closed{% endtrans %}</td>
+          <td>{% trans %}Actions{% endtrans %}</td>
+        </tr>
+      </thead>
+      <tbody>
+        {% for j in object.journals.all() %}
+          <tr>
+            <td>{{ j.name }}</td>
+            <td>{{ j.start_date }}</td>
+            {% if j.end_date %}
+              <td>{{ j.end_date }}</td>
+            {% else %}
+              <td> - </td>
+            {% endif %}
+            <td>{{ j.amount }} €</td>
+            <td>{{ j.effective_amount }} €</td>
+            {% if j.closed %}
+              <td>{% trans %}Yes{% endtrans %}</td>
+            {% else %}
+              <td>{% trans %}No{% endtrans %}</td>
+            {% endif %}
+            <td> <a href="{{ url('accounting:journal_details', j_id=j.id) }}">{% trans %}View{% endtrans %}</a>
+              <a href="{{ url('accounting:journal_edit', j_id=j.id) }}">{% trans %}Edit{% endtrans %}</a>
+              {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) and j.operations.count() == 0 %}
+                <a href="{{ url('accounting:journal_delete', j_id=j.id) }}">{% trans %}Delete{% endtrans %}</a>
+              {% endif %}
+            </td>
+          </tr>
+        {% endfor %}
+      </tbody>
+    </table>
+  </div>
 {% endblock %}

accounting/templates/accounting/co_list.jinja

@@ -1,30 +1,30 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Company list{% endtrans %}
+  {% trans %}Company list{% endtrans %}
 {% endblock %}
 
 {% block content %}
-    <div id="accounting">
-        {% if user.is_root
-           or user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID)
-        %}
-        <p><a href="{{ url('accounting:co_new') }}">{% trans %}Create new company{% endtrans %}</a></p>
-        {% endif %}
-        <br/>
-        <table>
-            <thead>
-            <tr>
-                <td>{% trans %}Companies{% endtrans %}</td>
-            </tr>
-            </thead>
-            <tbody>
-                {% for o in object_list %}
-                    <tr>
-                        <td><a href="{{ url('accounting:co_edit', co_id=o.id) }}">{{ o.get_display_name() }}</a></td>
-                    </tr>
-                {% endfor %}
-            </tbody>
-        </table>
-    </div>
+  <div id="accounting">
+    {% if user.is_root
+    or user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID)
+    %}
+    <p><a href="{{ url('accounting:co_new') }}">{% trans %}Create new company{% endtrans %}</a></p>
+{% endif %}
+<br/>
+<table>
+  <thead>
+    <tr>
+      <td>{% trans %}Companies{% endtrans %}</td>
+    </tr>
+  </thead>
+  <tbody>
+    {% for o in object_list %}
+      <tr>
+        <td><a href="{{ url('accounting:co_edit', co_id=o.id) }}">{{ o.get_display_name() }}</a></td>
+      </tr>
+    {% endfor %}
+  </tbody>
+</table>
+</div>
 {% endblock %}

accounting/templates/accounting/journal_details.jinja

@@ -1,103 +1,103 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}General journal:{% endtrans %} {{ object.name }}
+  {% trans %}General journal:{% endtrans %} {{ object.name }}
 {% endblock %}
 
 {% block content %}
-    <div id="accounting">
-        <p>
-        <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
-        <a href="{{ url('accounting:bank_details', b_account_id=object.club_account.bank_account.id) }}">{{object.club_account.bank_account }}</a> >
-        <a href="{{ url('accounting:club_details', c_account_id=object.club_account.id) }}">{{ object.club_account }}</a> >
-        {{ object.name }}
-        </p>
-        <hr>
-        <h2>{% trans %}General journal:{% endtrans %} {{ object.name }}</h2>
-        <p><a href="{{ url('accounting:label_new') }}?parent={{ object.club_account.id }}">{% trans %}New label{% endtrans %}</a></p>
-        <p><a href="{{ url('accounting:label_list', clubaccount_id=object.club_account.id) }}">{% trans %}Label list{% endtrans %}</a></p>
-        <p><a href="{{ url('accounting:co_list') }}">{% trans %}Company list{% endtrans %}</a></p>
-        <p><strong>{% trans %}Amount: {% endtrans %}</strong>{{ object.amount }} € -
-        <strong>{% trans %}Effective amount: {% endtrans %}</strong>{{ object.effective_amount }} €</p>
-        {% if object.closed %}
-        <p>{% trans %}Journal is closed, you can not create operation{% endtrans %}</p>
-        {% else %}
-        <p><a href="{{ url('accounting:op_new', j_id=object.id) }}">{% trans %}New operation{% endtrans %}</a></p>
-        </br>
-        {% endif %}
-        <div class="journal-table">
-        <table>
-            <thead>
+  <div id="accounting">
+    <p>
+      <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
+      <a href="{{ url('accounting:bank_details', b_account_id=object.club_account.bank_account.id) }}">{{object.club_account.bank_account }}</a> >
+      <a href="{{ url('accounting:club_details', c_account_id=object.club_account.id) }}">{{ object.club_account }}</a> >
+      {{ object.name }}
+    </p>
+    <hr>
+    <h2>{% trans %}General journal:{% endtrans %} {{ object.name }}</h2>
+    <p><a href="{{ url('accounting:label_new') }}?parent={{ object.club_account.id }}">{% trans %}New label{% endtrans %}</a></p>
+    <p><a href="{{ url('accounting:label_list', clubaccount_id=object.club_account.id) }}">{% trans %}Label list{% endtrans %}</a></p>
+    <p><a href="{{ url('accounting:co_list') }}">{% trans %}Company list{% endtrans %}</a></p>
+    <p><strong>{% trans %}Amount: {% endtrans %}</strong>{{ object.amount }} € -
+      <strong>{% trans %}Effective amount: {% endtrans %}</strong>{{ object.effective_amount }} €</p>
+    {% if object.closed %}
+      <p>{% trans %}Journal is closed, you can not create operation{% endtrans %}</p>
+    {% else %}
+      <p><a href="{{ url('accounting:op_new', j_id=object.id) }}">{% trans %}New operation{% endtrans %}</a></p>
+      </br>
+    {% endif %}
+    <div class="journal-table">
+      <table>
+        <thead>
+          <tr>
+            <td>{% trans %}Nb{% endtrans %}</td>
+            <td>{% trans %}Date{% endtrans %}</td>
+            <td>{% trans %}Label{% endtrans %}</td>
+            <td>{% trans %}Amount{% endtrans %}</td>
+            <td>{% trans %}Payment mode{% endtrans %}</td>
+            <td>{% trans %}Target{% endtrans %}</td>
+            <td>{% trans %}Code{% endtrans %}</td>
+            <td>{% trans %}Nature{% endtrans %}</td>
+            <td>{% trans %}Done{% endtrans %}</td>
+            <td>{% trans %}Comment{% endtrans %}</td>
+            <td>{% trans %}File{% endtrans %}</td>
+            <td>{% trans %}Actions{% endtrans %}</td>
+            <td>{% trans %}PDF{% endtrans %}</td>
+          </tr>
+        </thead>
+        <tbody>
+          {% for o in object.operations.all() %}
             <tr>
-                <td>{% trans %}Nb{% endtrans %}</td>
-                <td>{% trans %}Date{% endtrans %}</td>
-                <td>{% trans %}Label{% endtrans %}</td>
-                <td>{% trans %}Amount{% endtrans %}</td>
-                <td>{% trans %}Payment mode{% endtrans %}</td>
-                <td>{% trans %}Target{% endtrans %}</td>
-                <td>{% trans %}Code{% endtrans %}</td>
-                <td>{% trans %}Nature{% endtrans %}</td>
-                <td>{% trans %}Done{% endtrans %}</td>
-                <td>{% trans %}Comment{% endtrans %}</td>
-                <td>{% trans %}File{% endtrans %}</td>
-                <td>{% trans %}Actions{% endtrans %}</td>
-                <td>{% trans %}PDF{% endtrans %}</td>
-            </tr>
-            </thead>
-            <tbody>
-            {% for o in object.operations.all() %}
-            <tr>
-                <td>{{ o.number }}</td>
-                <td>{{ o.date }}</td>
-                <td>{{ o.label or "" }}</td>
-                {% if o.accounting_type.movement_type == "DEBIT" %}
-                    <td class="neg-amount">&nbsp;{{ o.amount }}&nbsp;€</td>
-                {% else %}
-                    <td class="pos-amount">&nbsp;{{ o.amount }}&nbsp;€</td>
-                {% endif %}
-                <td>{{ o.get_mode_display() }}</td>
-                {% if o.target_type == "OTHER" %}
+              <td>{{ o.number }}</td>
+              <td>{{ o.date }}</td>
+              <td>{{ o.label or "" }}</td>
+              {% if o.accounting_type.movement_type == "DEBIT" %}
+                <td class="neg-amount">&nbsp;{{ o.amount }}&nbsp;€</td>
+              {% else %}
+                <td class="pos-amount">&nbsp;{{ o.amount }}&nbsp;€</td>
+              {% endif %}
+              <td>{{ o.get_mode_display() }}</td>
+              {% if o.target_type == "OTHER" %}
                 <td>{{ o.target_label }}</td>
-                {% else %}
+              {% else %}
                 <td><a href="{{ o.target.get_absolute_url() }}">{{ o.target.get_display_name() }}</a></td>
-                {% endif %}
-                <td>{{ o.accounting_type.code }}</td>
-                <td>{{ o.accounting_type.label }}</td>
-                {% if o.done %}
+              {% endif %}
+              <td>{{ o.accounting_type.code }}</td>
+              <td>{{ o.accounting_type.label }}</td>
+              {% if o.done %}
                 <td>{% trans %}Yes{% endtrans %}</td>
-                {% else %}
+              {% else %}
                 <td>{% trans %}No{% endtrans %}</td>
-                {% endif %}
-                <td>{{ o.remark }}
+              {% endif %}
+              <td>{{ o.remark }}
                 {% if not o.linked_operation and o.target_type == "ACCOUNT" and not o.target.has_open_journal() %}
-                    <p><strong>
-    {% trans %}Warning: this operation has no linked operation because the targeted club account has no opened journal.{% endtrans %}
-                </strong></p>
-                <p><strong>
-    {% trans url=o.target.get_absolute_url() %}Open a journal in <a href="{{ url }}">this club account</a>, then save this operation again to make the linked operation.{% endtrans %}
-            </strong></p>
+                  <p><strong>
+                    {% trans %}Warning: this operation has no linked operation because the targeted club account has no opened journal.{% endtrans %}
+                  </strong></p>
+                  <p><strong>
+                    {% trans url=o.target.get_absolute_url() %}Open a journal in <a href="{{ url }}">this club account</a>, then save this operation again to make the linked operation.{% endtrans %}
+                  </strong></p>
                 {% endif %}
-                </td>
-                {% if o.invoice %}
+              </td>
+              {% if o.invoice %}
                 <td><a href="{{ url('core:download', file_id=o.invoice.id) }}">{{ o.invoice.name }}</a></td>
-                {% else %}
+              {% else %}
                 <td>-</td>
+              {% endif %}
+              <td>
+                {%
+                if o.journal.club_account.bank_account.name not in ["AE TI", "TI"]
+                or user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID)
+                %}
+                {% if not o.journal.closed %}
+                  <a href="{{ url('accounting:op_edit', op_id=o.id) }}">{% trans %}Edit{% endtrans %}</a>
                 {% endif %}
-                <td>
-                    {%
-                        if o.journal.club_account.bank_account.name not in ["AE TI", "TI"]
-                        or user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID)
-                    %}
-                        {% if not o.journal.closed %}
-                            <a href="{{ url('accounting:op_edit', op_id=o.id) }}">{% trans %}Edit{% endtrans %}</a>
-                        {% endif %}
-                    {% endif %}
-                </td>
-                <td><a href="{{ url('accounting:op_pdf', op_id=o.id) }}">{% trans %}Generate{% endtrans %}</a></td>
-            </tr>
-            {% endfor %}
-            </tbody>
-        </table>
-        </div>
-    </div>
+          {% endif %}
+        </td>
+        <td><a href="{{ url('accounting:op_pdf', op_id=o.id) }}">{% trans %}Generate{% endtrans %}</a></td>
+      </tr>
+{% endfor %}
+</tbody>
+</table>
+</div>
+</div>
 {% endblock %}

accounting/templates/accounting/journal_statement_accounting.jinja

@@ -1,33 +1,33 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}General journal:{% endtrans %} {{ object.name }}
+  {% trans %}General journal:{% endtrans %} {{ object.name }}
 {% endblock %}
 
 
 {% block content %}
-<div id="accounting">
+  <div id="accounting">
     <h3>{% trans %}Accounting statement: {% endtrans %} {{ object.name }}</h3>
 
     <table>
-        <thead>
-            <tr>
-                <td>{% trans %}Operation type{% endtrans %}</td>
-                <td>{% trans %}Sum{% endtrans %}</td>
-            </tr>
-        </thead>
-        <tbody>
-            {% for k,v in statement.items() %}
-            <tr>
-                <td>{{ k }}</td>
-                <td>{{ "%.2f" % v }}</td>
-            </tr>
-            {% endfor %}
-        </tbody>
+      <thead>
+        <tr>
+          <td>{% trans %}Operation type{% endtrans %}</td>
+          <td>{% trans %}Sum{% endtrans %}</td>
+        </tr>
+      </thead>
+      <tbody>
+        {% for k,v in statement.items() %}
+          <tr>
+            <td>{{ k }}</td>
+            <td>{{ "%.2f" % v }}</td>
+          </tr>
+        {% endfor %}
+      </tbody>
 
     </table>
 
     <p><strong>{% trans %}Amount: {% endtrans %}</strong>{{ "%.2f" % object.amount }} €</p>
     <p><strong>{% trans %}Effective amount: {% endtrans %}</strong>{{ "%.2f" %object.effective_amount }} €</p>
-</div>
+  </div>
 {% endblock %}

accounting/templates/accounting/journal_statement_nature.jinja

@@ -1,57 +1,57 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}General journal:{% endtrans %} {{ object.name }}
+  {% trans %}General journal:{% endtrans %} {{ object.name }}
 {% endblock %}
 
 {% macro display_tables(dict) %}
-<div id="accounting">
+  <div id="accounting">
     <h6>{% trans %}Credit{% endtrans %}</h6>
     <table>
-        <thead>
-            <tr>
-                <td>{% trans %}Nature of operation{% endtrans %}</td>
-                <td>{% trans %}Sum{% endtrans %}</td>
-            </tr>
-        </thead>
-        <tbody>
-            {% for k,v in dict['CREDIT'].items() %}
-            <tr>
-                <td>{{ k }}</td>
-                <td>{{ "%.2f" % v }}</td>
-            </tr>
-            {% endfor %}
-        </tbody>
+      <thead>
+        <tr>
+          <td>{% trans %}Nature of operation{% endtrans %}</td>
+          <td>{% trans %}Sum{% endtrans %}</td>
+        </tr>
+      </thead>
+      <tbody>
+        {% for k,v in dict['CREDIT'].items() %}
+          <tr>
+            <td>{{ k }}</td>
+            <td>{{ "%.2f" % v }}</td>
+          </tr>
+        {% endfor %}
+      </tbody>
     </table>
     {% trans %}Total: {% endtrans %}{{ "%.2f" % dict['CREDIT_sum'] }}
 
     <h6>{% trans %}Debit{% endtrans %}</h6>
     <table>
-        <thead>
-            <tr>
-                <td>{% trans %}Nature of operation{% endtrans %}</td>
-                <td>{% trans %}Sum{% endtrans %}</td>
-            </tr>
-        </thead>
-        <tbody>
-            {% for k,v in dict['DEBIT'].items() %}
-            <tr>
-                <td>{{ k }}</td>
-                <td>{{ "%.2f" % v }}</td>
-            </tr>
-            {% endfor %}
-        </tbody>
+      <thead>
+        <tr>
+          <td>{% trans %}Nature of operation{% endtrans %}</td>
+          <td>{% trans %}Sum{% endtrans %}</td>
+        </tr>
+      </thead>
+      <tbody>
+        {% for k,v in dict['DEBIT'].items() %}
+          <tr>
+            <td>{{ k }}</td>
+            <td>{{ "%.2f" % v }}</td>
+          </tr>
+        {% endfor %}
+      </tbody>
     </table>
     {% trans %}Total: {% endtrans %}{{ "%.2f" % dict['DEBIT_sum'] }}
-    {% endmacro %}
+{% endmacro %}
 
-    {% block content %}
-    <h3>{% trans %}Statement by nature: {% endtrans %} {{ object.name }}</h3>
+{% block content %}
+  <h3>{% trans %}Statement by nature: {% endtrans %} {{ object.name }}</h3>
 
-    {% for k,v in statement.items() %}
-        <h4 style="background: lightblue; padding: 4px;">{{ k }} : {{ "%.2f" % (v['CREDIT_sum'] - v['DEBIT_sum']) }}</h4>
+  {% for k,v in statement.items() %}
+    <h4 style="background: lightblue; padding: 4px;">{{ k }} : {{ "%.2f" % (v['CREDIT_sum'] - v['DEBIT_sum']) }}</h4>
     {{ display_tables(v) }}
     <hr>
-    {% endfor %}
-</div>
+  {% endfor %}
+  </div>
 {% endblock %}

accounting/templates/accounting/journal_statement_person.jinja

@@ -1,37 +1,37 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}General journal:{% endtrans %} {{ object.name }}
+  {% trans %}General journal:{% endtrans %} {{ object.name }}
 {% endblock %}
 
 
 {% block content %}
-<div id="accounting">
+  <div id="accounting">
     <h3>{% trans %}Statement by person: {% endtrans %} {{ object.name }}</h3>
 
     <h4>{% trans %}Credit{% endtrans %}</h4>
 
     <table>
-        <thead>
-            <tr>
-                <td>{% trans %}Target of the operation{% endtrans %}</td>
-                <td>{% trans %}Sum{% endtrans %}</td>
-            </tr>
-        </thead>
-        <tbody>
-            {% for key in credit_statement.keys() %}
-            <tr>
-                {% if key.target_type == "OTHER" %}
-                <td>{{ o.target_label }}</td>
-                {% elif key %}
-                <td><a href="{{ key.get_absolute_url() }}">{{ key.get_display_name() }}</a></td>
-                {% else %}
-                <td></td>
-                {% endif %}
-                <td>{{ "%.2f" % credit_statement[key] }}</td>
-            </tr>
-            {% endfor %}
-        </tbody>
+      <thead>
+        <tr>
+          <td>{% trans %}Target of the operation{% endtrans %}</td>
+          <td>{% trans %}Sum{% endtrans %}</td>
+        </tr>
+      </thead>
+      <tbody>
+        {% for key in credit_statement.keys() %}
+          <tr>
+            {% if key.target_type == "OTHER" %}
+              <td>{{ o.target_label }}</td>
+            {% elif key %}
+              <td><a href="{{ key.get_absolute_url() }}">{{ key.get_display_name() }}</a></td>
+            {% else %}
+              <td></td>
+            {% endif %}
+            <td>{{ "%.2f" % credit_statement[key] }}</td>
+          </tr>
+        {% endfor %}
+      </tbody>
 
     </table>
 
@@ -40,29 +40,29 @@
     <h4>{% trans %}Debit{% endtrans %}</h4>
 
     <table>
-        <thead>
-            <tr>
-                <td>{% trans %}Target of the operation{% endtrans %}</td>
-                <td>{% trans %}Sum{% endtrans %}</td>
-            </tr>
-        </thead>
-        <tbody>
-            {% for key in debit_statement.keys() %}
-            <tr>
-                {% if key.target_type == "OTHER" %}
-                <td>{{ o.target_label }}</td>
-                {% elif key %}
-                <td><a href="{{ key.get_absolute_url() }}">{{ key.get_display_name() }}</a></td>
-                {% else %}
-                <td></td>
-                {% endif %}
-                <td>{{ "%.2f" % debit_statement[key] }}</td>
-            </tr>
-            {% endfor %}
-        </tbody>
+      <thead>
+        <tr>
+          <td>{% trans %}Target of the operation{% endtrans %}</td>
+          <td>{% trans %}Sum{% endtrans %}</td>
+        </tr>
+      </thead>
+      <tbody>
+        {% for key in debit_statement.keys() %}
+          <tr>
+            {% if key.target_type == "OTHER" %}
+              <td>{{ o.target_label }}</td>
+            {% elif key %}
+              <td><a href="{{ key.get_absolute_url() }}">{{ key.get_display_name() }}</a></td>
+            {% else %}
+              <td></td>
+            {% endif %}
+            <td>{{ "%.2f" % debit_statement[key] }}</td>
+          </tr>
+        {% endfor %}
+      </tbody>
 
     </table>
 
     <p>Total : {{ "%.2f" % total_debit }}</p>
-</div>
+  </div>
 {% endblock %}

accounting/templates/accounting/label_list.jinja

@@ -1,36 +1,36 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Label list{% endtrans %}
+  {% trans %}Label list{% endtrans %}
 {% endblock %}
 
 {% block content %}
-    <div id="accounting">
-        <p>
-        <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
-        <a href="{{ url('accounting:bank_details', b_account_id=object.bank_account.id) }}">{{object.bank_account }}</a> >
-        <a href="{{ url('accounting:club_details', c_account_id=object.id) }}">{{ object }}</a>
-        </p>
-        <hr>
-        <p><a href="{{ url('accounting:club_details', c_account_id=object.id) }}">{% trans %}Back to club account{% endtrans %}</a></p>
-        {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) %}
-        <p><a href="{{ url('accounting:label_new') }}?parent={{ object.id }}">{% trans %}New label{% endtrans %}</a></p>
-        {% endif %}
-        {% if object.labels.all() %}
-        <h3>{% trans %}Label list{% endtrans %}</h3>
-        <ul>
-            {% for l in object.labels.all()  %}
-            <li><a href="{{ url('accounting:label_edit', label_id=l.id) }}">{{ l }}</a>
+  <div id="accounting">
+    <p>
+      <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
+      <a href="{{ url('accounting:bank_details', b_account_id=object.bank_account.id) }}">{{object.bank_account }}</a> >
+      <a href="{{ url('accounting:club_details', c_account_id=object.id) }}">{{ object }}</a>
+    </p>
+    <hr>
+    <p><a href="{{ url('accounting:club_details', c_account_id=object.id) }}">{% trans %}Back to club account{% endtrans %}</a></p>
+    {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) %}
+      <p><a href="{{ url('accounting:label_new') }}?parent={{ object.id }}">{% trans %}New label{% endtrans %}</a></p>
+    {% endif %}
+    {% if object.labels.all() %}
+      <h3>{% trans %}Label list{% endtrans %}</h3>
+      <ul>
+        {% for l in object.labels.all()  %}
+          <li><a href="{{ url('accounting:label_edit', label_id=l.id) }}">{{ l }}</a>
             {% if user.is_in_group(pk=settings.SITH_GROUP_ACCOUNTING_ADMIN_ID) %}
-             -
-                <a href="{{ url('accounting:label_delete', label_id=l.id) }}">{% trans %}Delete{% endtrans %}</a>
+              -
+              <a href="{{ url('accounting:label_delete', label_id=l.id) }}">{% trans %}Delete{% endtrans %}</a>
             {% endif %}
-            </li>
-            {% endfor %}
-        </ul>
-        {% else %}
-        {% trans %}There is no label in this club account.{% endtrans %}
-        {% endif %}
-    </div>
+          </li>
+        {% endfor %}
+      </ul>
+    {% else %}
+      {% trans %}There is no label in this club account.{% endtrans %}
+    {% endif %}
+  </div>
 {% endblock %}
 

accounting/templates/accounting/operation_edit.jinja

@@ -1,123 +1,123 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Edit operation{% endtrans %}
+  {% trans %}Edit operation{% endtrans %}
 {% endblock %}
 
 {% block content %}
-<div id="accounting">
+  <div id="accounting">
     <p>
-    <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
-    <a href="{{ url('accounting:bank_details', b_account_id=object.club_account.bank_account.id) }}">{{object.club_account.bank_account }}</a> >
-    <a href="{{ url('accounting:club_details', c_account_id=object.club_account.id) }}">{{ object.club_account }}</a> >
-    <a href="{{ url('accounting:journal_details', j_id=object.id) }}">{{ object.name }}</a> >
-    {% trans %}Edit operation{% endtrans %}
+      <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
+      <a href="{{ url('accounting:bank_details', b_account_id=object.club_account.bank_account.id) }}">{{object.club_account.bank_account }}</a> >
+      <a href="{{ url('accounting:club_details', c_account_id=object.club_account.id) }}">{{ object.club_account }}</a> >
+      <a href="{{ url('accounting:journal_details', j_id=object.id) }}">{{ object.name }}</a> >
+      {% trans %}Edit operation{% endtrans %}
     </p>
     <hr>
     <h2>{% trans %}Edit operation{% endtrans %}</h2>
     <form action="" method="post">
-        {% csrf_token %}
-        {{ form.non_field_errors() }}
-        {{ form.journal }}
-        {{ form.target_id }}
-        <p>{{ form.amount.errors }}<label for="{{ form.amount.name }}">{{ form.amount.label }}</label> {{ form.amount }}</p>
-        <p>{{ form.remark.errors }}<label for="{{ form.remark.name }}">{{ form.remark.label }}</label> {{ form.remark }}</p>
-        <br />
-        <strong>{% trans %}Warning: if you select <em>Account</em>, the opposite operation will be created in the target account. If you don't want that, select <em>Club</em> instead of <em>Account</em>.{% endtrans %}</strong>
-        <p>{{ form.target_type.errors }}<label for="{{ form.target_type.name }}">{{ form.target_type.label }}</label> {{ form.target_type }}</p>
-            {{ form.user }}
-            {{ form.club }}
-            {{ form.club_account }}
-            {{ form.company }}
-            {{ form.target_label }}
-            <span id="id_need_link_full"><label>{{ form.need_link.label }}</label> {{ form.need_link }}</span>
-        <p>{{ form.date.errors }}<label for="{{ form.date.name }}">{{ form.date.label }}</label> {{ form.date }}</p>
-        <p>{{ form.mode.errors }}<label for="{{ form.mode.name }}">{{ form.mode.label }}</label> {{ form.mode }}</p>
-        <p>{{ form.cheque_number.errors }}<label for="{{ form.cheque_number.name }}">{{ form.cheque_number.label }}</label> {{
+      {% csrf_token %}
+      {{ form.non_field_errors() }}
+      {{ form.journal }}
+      {{ form.target_id }}
+      <p>{{ form.amount.errors }}<label for="{{ form.amount.name }}">{{ form.amount.label }}</label> {{ form.amount }}</p>
+      <p>{{ form.remark.errors }}<label for="{{ form.remark.name }}">{{ form.remark.label }}</label> {{ form.remark }}</p>
+      <br />
+      <strong>{% trans %}Warning: if you select <em>Account</em>, the opposite operation will be created in the target account. If you don't want that, select <em>Club</em> instead of <em>Account</em>.{% endtrans %}</strong>
+      <p>{{ form.target_type.errors }}<label for="{{ form.target_type.name }}">{{ form.target_type.label }}</label> {{ form.target_type }}</p>
+      {{ form.user }}
+      {{ form.club }}
+      {{ form.club_account }}
+      {{ form.company }}
+      {{ form.target_label }}
+      <span id="id_need_link_full"><label>{{ form.need_link.label }}</label> {{ form.need_link }}</span>
+      <p>{{ form.date.errors }}<label for="{{ form.date.name }}">{{ form.date.label }}</label> {{ form.date }}</p>
+      <p>{{ form.mode.errors }}<label for="{{ form.mode.name }}">{{ form.mode.label }}</label> {{ form.mode }}</p>
+      <p>{{ form.cheque_number.errors }}<label for="{{ form.cheque_number.name }}">{{ form.cheque_number.label }}</label> {{
         form.cheque_number }}</p>
-        <p>{{ form.invoice.errors }}<label for="{{ form.invoice.name }}">{{ form.invoice.label }}</label> {{ form.invoice }}</p>
-        <p>{{ form.simpleaccounting_type.errors }}<label for="{{ form.simpleaccounting_type.name }}">{{
-            form.simpleaccounting_type.label }}</label> {{ form.simpleaccounting_type }}</p>
-        <p>{{ form.accounting_type.errors }}<label for="{{ form.accounting_type.name }}">{{ form.accounting_type.label }}</label> {{
+      <p>{{ form.invoice.errors }}<label for="{{ form.invoice.name }}">{{ form.invoice.label }}</label> {{ form.invoice }}</p>
+      <p>{{ form.simpleaccounting_type.errors }}<label for="{{ form.simpleaccounting_type.name }}">{{
+        form.simpleaccounting_type.label }}</label> {{ form.simpleaccounting_type }}</p>
+      <p>{{ form.accounting_type.errors }}<label for="{{ form.accounting_type.name }}">{{ form.accounting_type.label }}</label> {{
         form.accounting_type }}</p>
-        <p>{{ form.label.errors }}<label for="{{ form.label.name }}">{{ form.label.label }}</label> {{ form.label }}</p>
-        <p>{{ form.done.errors }}<label for="{{ form.done.name }}">{{ form.done.label }}</label> {{ form.done }}</p>
-        {% if form.instance.linked_operation %}
+      <p>{{ form.label.errors }}<label for="{{ form.label.name }}">{{ form.label.label }}</label> {{ form.label }}</p>
+      <p>{{ form.done.errors }}<label for="{{ form.done.name }}">{{ form.done.label }}</label> {{ form.done }}</p>
+      {% if form.instance.linked_operation %}
         {% set obj = form.instance.linked_operation %}
         <p><strong>{% trans %}Linked operation:{% endtrans %}</strong><br>
-        <a href="{{ url('accounting:bank_details', b_account_id=obj.journal.club_account.bank_account.id) }}">
-                {{obj.journal.club_account.bank_account }}</a> >
-        <a href="{{ url('accounting:club_details', c_account_id=obj.journal.club_account.id) }}">{{ obj.journal.club_account }}</a> >
-        <a href="{{ url('accounting:journal_details', j_id=obj.journal.id) }}">{{ obj.journal }}</a> >
-        n°{{ obj.number }}
+          <a href="{{ url('accounting:bank_details', b_account_id=obj.journal.club_account.bank_account.id) }}">
+            {{obj.journal.club_account.bank_account }}</a> >
+          <a href="{{ url('accounting:club_details', c_account_id=obj.journal.club_account.id) }}">{{ obj.journal.club_account }}</a> >
+          <a href="{{ url('accounting:journal_details', j_id=obj.journal.id) }}">{{ obj.journal }}</a> >
+          n°{{ obj.number }}
         </p>
-        {% endif %}
-        <p><input type="submit" value="{% trans %}Save{% endtrans %}" /></p>
+      {% endif %}
+      <p><input type="submit" value="{% trans %}Save{% endtrans %}" /></p>
     </form>
-    {% endblock %}
+{% endblock %}
 
-    {% block script %}
-        {{ super() }}
-    <script>
+{% block script %}
+  {{ super() }}
+  <script>
     $( function() {
-            var target_type = $('#id_target_type');
-            var user = $('#id_user_wrapper');
-            var club = $('#id_club_wrapper');
-            var club_account = $('#id_club_account_wrapper');
-            var company = $('#id_company_wrapper');
-            var other = $('#id_target_label');
-            var need_link = $('#id_need_link_full');
-            function update_targets () {
-                if (target_type.val() == "USER") {
-                    console.log(user);
-                    user.show();
-                    club.hide();
-                    club_account.hide();
-                    company.hide();
-                    other.hide();
-                    need_link.hide();
-                } else if (target_type.val() == "ACCOUNT") {
-                    club_account.show();
-                    need_link.show();
-                    user.hide();
-                    club.hide();
-                    company.hide();
-                    other.hide();
-                } else if (target_type.val() == "CLUB") {
-                    club.show();
-                    user.hide();
-                    club_account.hide();
-                    company.hide();
-                    other.hide();
-                    need_link.hide();
-                } else if (target_type.val() == "COMPANY") {
-                    company.show();
-                    user.hide();
-                    club_account.hide();
-                    club.hide();
-                    other.hide();
-                    need_link.hide();
-                } else if (target_type.val() == "OTHER") {
-                    other.show();
-                    user.hide();
-                    club.hide();
-                    club_account.hide();
-                    company.hide();
-                    need_link.hide();
-                } else {
-                    company.hide();
-                    user.hide();
-                    club_account.hide();
-                    club.hide();
-                    other.hide();
-                    need_link.hide();
-                }
-            }
-            update_targets();
-            target_type.change(update_targets);
-            } );
-    </script>
-</div>
+      var target_type = $('#id_target_type');
+      var user = $('user-ajax-select');
+      var club = $('club-ajax-select');
+      var club_account = $('club-account-ajax-select');
+      var company = $('company-ajax-select');
+      var other = $('#id_target_label');
+      var need_link = $('#id_need_link_full');
+      function update_targets () {
+        if (target_type.val() == "USER") {
+          console.log(user);
+          user.show();
+          club.hide();
+          club_account.hide();
+          company.hide();
+          other.hide();
+          need_link.hide();
+        } else if (target_type.val() == "ACCOUNT") {
+          club_account.show();
+          need_link.show();
+          user.hide();
+          club.hide();
+          company.hide();
+          other.hide();
+        } else if (target_type.val() == "CLUB") {
+          club.show();
+          user.hide();
+          club_account.hide();
+          company.hide();
+          other.hide();
+          need_link.hide();
+        } else if (target_type.val() == "COMPANY") {
+          company.show();
+          user.hide();
+          club_account.hide();
+          club.hide();
+          other.hide();
+          need_link.hide();
+        } else if (target_type.val() == "OTHER") {
+          other.show();
+          user.hide();
+          club.hide();
+          club_account.hide();
+          company.hide();
+          need_link.hide();
+        } else {
+          company.hide();
+          user.hide();
+          club_account.hide();
+          club.hide();
+          other.hide();
+          need_link.hide();
+        }
+      }
+      update_targets();
+      target_type.change(update_targets);
+    } );
+  </script>
+  </div>
 {% endblock %}
 
 

accounting/templates/accounting/refound_account.jinja

@@ -1,16 +1,16 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Refound account{% endtrans %}
+  {% trans %}Refound account{% endtrans %}
 {% endblock %}
 
 {% block content %}
-	<div id="accounting">
-	    <h3>{% trans %}Refound account{% endtrans %}</h3>
-	    <form action="" method="post">
-	        {% csrf_token %}
-	        {{ form.as_p() }}
-	        <p><input type="submit" value="{% trans %}Refound{% endtrans %}" /></p>
-	    </form>
-	</div>
+  <div id="accounting">
+    <h3>{% trans %}Refound account{% endtrans %}</h3>
+    <form action="" method="post">
+      {% csrf_token %}
+      {{ form.as_p() }}
+      <p><input type="submit" value="{% trans %}Refound{% endtrans %}" /></p>
+    </form>
+  </div>
 {% endblock %}

accounting/templates/accounting/simplifiedaccountingtype_list.jinja

@@ -1,27 +1,27 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Simplified type list{% endtrans %}
+  {% trans %}Simplified type list{% endtrans %}
 {% endblock %}
 
 {% block content %}
-    <div id="accounting">
-        <p>
-        <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
-        {% trans %}Simplified types{% endtrans %}
-        </p>
-        <hr>
-        <p><a href="{{ url('accounting:simple_type_new') }}">{% trans %}New simplified type{% endtrans %}</a></p>
-        {% if simplifiedaccountingtype_list %}
-        <h3>{% trans %}Simplified type list{% endtrans %}</h3>
-        <ul>
-            {% for a in simplifiedaccountingtype_list  %}
-            <li><a href="{{ url('accounting:simple_type_edit', type_id=a.id) }}">{{ a }}</a></li>
-            {% endfor %}
-        </ul>
-        {% else %}
-        {% trans %}There is no types in this website.{% endtrans %}
-        {% endif %}
-    </div>
+  <div id="accounting">
+    <p>
+      <a href="{{ url('accounting:bank_list') }}">{% trans %}Accounting{% endtrans %}</a> >
+      {% trans %}Simplified types{% endtrans %}
+    </p>
+    <hr>
+    <p><a href="{{ url('accounting:simple_type_new') }}">{% trans %}New simplified type{% endtrans %}</a></p>
+    {% if simplifiedaccountingtype_list %}
+      <h3>{% trans %}Simplified type list{% endtrans %}</h3>
+      <ul>
+        {% for a in simplifiedaccountingtype_list  %}
+          <li><a href="{{ url('accounting:simple_type_edit', type_id=a.id) }}">{{ a }}</a></li>
+        {% endfor %}
+      </ul>
+    {% else %}
+      {% trans %}There is no types in this website.{% endtrans %}
+    {% endif %}
+  </div>
 {% endblock %}
 

accounting/tests.py

@@ -1,21 +1,16 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 
 from datetime import date, timedelta
@@ -33,7 +28,7 @@ from accounting.models import (
 from core.models import User
 
 
-class RefoundAccountTest(TestCase):
+class TestRefoundAccount(TestCase):
     @classmethod
     def setUpTestData(cls):
         cls.skia = User.objects.get(username="skia")
@@ -72,7 +67,7 @@ class RefoundAccountTest(TestCase):
         assert self.skia.customer.amount == 0
 
 
-class JournalTest(TestCase):
+class TestJournal(TestCase):
     @classmethod
     def setUpTestData(cls):
         cls.journal = GeneralJournal.objects.get(id=1)
@@ -96,7 +91,7 @@ class JournalTest(TestCase):
         assert "<td>M\xc3\xa9thode de paiement</td>" not in str(response_get.content)
 
 
-class OperationTest(TestCase):
+class TestOperation(TestCase):
     def setUp(self):
         self.tomorrow_formatted = (date.today() + timedelta(days=1)).strftime(
             "%d/%m/%Y"
@@ -107,7 +102,7 @@ class OperationTest(TestCase):
             code="443", label="Ce code n'existe pas", movement_type="CREDIT"
         )
         at.save()
-        l = Label.objects.create(club_account=self.journal.club_account, name="bob")
+        label = Label.objects.create(club_account=self.journal.club_account, name="bob")
         self.client.force_login(User.objects.get(username="comptable"))
         self.op1 = Operation(
             journal=self.journal,
@@ -116,7 +111,7 @@ class OperationTest(TestCase):
             remark="Test bilan",
             mode="CASH",
             done=True,
-            label=l,
+            label=label,
             accounting_type=at,
             target_type="USER",
             target_id=self.skia.id,
@@ -129,7 +124,7 @@ class OperationTest(TestCase):
             remark="Test bilan",
             mode="CASH",
             done=True,
-            label=l,
+            label=label,
             accounting_type=at,
             target_type="USER",
             target_id=self.skia.id,
@@ -269,33 +264,26 @@ class OperationTest(TestCase):
         )
         self.assertContains(response, "Total : 5575.72", status_code=200)
         self.assertContains(response, "Total : 71.42")
-        self.assertContains(
-            response,
-            """
-                <td><a href="/user/1/">S&#39; Kia</a></td>
-                
-                <td>3.00</td>""",
+        content = response.content.decode()
+        self.assertInHTML(
+            """<td><a href="/user/1/">S&#39; Kia</a></td><td>3.00</td>""", content
         )
-        self.assertContains(
-            response,
-            """
-                <td><a href="/user/1/">S&#39; Kia</a></td>
-                
-                <td>823.00</td>""",
+        self.assertInHTML(
+            """<td><a href="/user/1/">S&#39; Kia</a></td><td>823.00</td>""", content
         )
 
     def test_accounting_statement(self):
         response = self.client.get(
             reverse("accounting:journal_accounting_statement", args=[self.journal.id])
         )
-        self.assertContains(
-            response,
+        assert response.status_code == 200
+        self.assertInHTML(
             """
             <tr>
                 <td>443 - Crédit - Ce code n&#39;existe pas</td>
                 <td>3.00</td>
             </tr>""",
-            status_code=200,
+            response.content.decode(),
         )
         self.assertContains(
             response,

accounting/urls.py

@@ -1,26 +1,55 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 
 from django.urls import path
 
-from accounting.views import *
+from accounting.views import (
+    AccountingTypeCreateView,
+    AccountingTypeEditView,
+    AccountingTypeListView,
+    BankAccountCreateView,
+    BankAccountDeleteView,
+    BankAccountDetailView,
+    BankAccountEditView,
+    BankAccountListView,
+    ClubAccountCreateView,
+    ClubAccountDeleteView,
+    ClubAccountDetailView,
+    ClubAccountEditView,
+    CompanyCreateView,
+    CompanyEditView,
+    CompanyListView,
+    JournalAccountingStatementView,
+    JournalCreateView,
+    JournalDeleteView,
+    JournalDetailView,
+    JournalEditView,
+    JournalNatureStatementView,
+    JournalPersonStatementView,
+    LabelCreateView,
+    LabelDeleteView,
+    LabelEditView,
+    LabelListView,
+    OperationCreateView,
+    OperationEditView,
+    OperationPDFView,
+    RefoundAccountView,
+    SimplifiedAccountingTypeCreateView,
+    SimplifiedAccountingTypeEditView,
+    SimplifiedAccountingTypeListView,
+)
 
 urlpatterns = [
     # Accounting types

accounting/views.py

@@ -1,26 +1,20 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 
 import collections
 
-from ajax_select.fields import AutoCompleteSelectField
 from django import forms
 from django.conf import settings
 from django.core.exceptions import PermissionDenied, ValidationError
@@ -44,6 +38,13 @@ from accounting.models import (
     Operation,
     SimplifiedAccountingType,
 )
+from accounting.widgets.select import (
+    AutoCompleteSelectClubAccount,
+    AutoCompleteSelectCompany,
+)
+from club.models import Club
+from club.widgets.select import AutoCompleteSelectClub
+from core.models import User
 from core.views import (
     CanCreateMixin,
     CanEditMixin,
@@ -52,15 +53,14 @@ from core.views import (
     TabedViewMixin,
 )
 from core.views.forms import SelectDate, SelectFile
+from core.views.widgets.select import AutoCompleteSelectUser
 from counter.models import Counter, Product, Selling
 
 # Main accounting view
 
 
 class BankAccountListView(CanViewMixin, ListView):
-    """
-    A list view for the admins
-    """
+    """A list view for the admins."""
 
     model = BankAccount
     template_name = "accounting/bank_account_list.jinja"
@@ -71,18 +71,14 @@ class BankAccountListView(CanViewMixin, ListView):
 
 
 class SimplifiedAccountingTypeListView(CanViewMixin, ListView):
-    """
-    A list view for the admins
-    """
+    """A list view for the admins."""
 
     model = SimplifiedAccountingType
     template_name = "accounting/simplifiedaccountingtype_list.jinja"
 
 
 class SimplifiedAccountingTypeEditView(CanViewMixin, UpdateView):
-    """
-    An edit view for the admins
-    """
+    """An edit view for the admins."""
 
     model = SimplifiedAccountingType
     pk_url_kwarg = "type_id"
@@ -91,9 +87,7 @@ class SimplifiedAccountingTypeEditView(CanViewMixin, UpdateView):
 
 
 class SimplifiedAccountingTypeCreateView(CanCreateMixin, CreateView):
-    """
-    Create an accounting type (for the admins)
-    """
+    """Create an accounting type (for the admins)."""
 
     model = SimplifiedAccountingType
     fields = ["label", "accounting_type"]
@@ -104,18 +98,14 @@ class SimplifiedAccountingTypeCreateView(CanCreateMixin, CreateView):
 
 
 class AccountingTypeListView(CanViewMixin, ListView):
-    """
-    A list view for the admins
-    """
+    """A list view for the admins."""
 
     model = AccountingType
     template_name = "accounting/accountingtype_list.jinja"
 
 
 class AccountingTypeEditView(CanViewMixin, UpdateView):
-    """
-    An edit view for the admins
-    """
+    """An edit view for the admins."""
 
     model = AccountingType
     pk_url_kwarg = "type_id"
@@ -124,9 +114,7 @@ class AccountingTypeEditView(CanViewMixin, UpdateView):
 
 
 class AccountingTypeCreateView(CanCreateMixin, CreateView):
-    """
-    Create an accounting type (for the admins)
-    """
+    """Create an accounting type (for the admins)."""
 
     model = AccountingType
     fields = ["code", "label", "movement_type"]
@@ -137,9 +125,7 @@ class AccountingTypeCreateView(CanCreateMixin, CreateView):
 
 
 class BankAccountEditView(CanViewMixin, UpdateView):
-    """
-    An edit view for the admins
-    """
+    """An edit view for the admins."""
 
     model = BankAccount
     pk_url_kwarg = "b_account_id"
@@ -148,9 +134,7 @@ class BankAccountEditView(CanViewMixin, UpdateView):
 
 
 class BankAccountDetailView(CanViewMixin, DetailView):
-    """
-    A detail view, listing every club account
-    """
+    """A detail view, listing every club account."""
 
     model = BankAccount
     pk_url_kwarg = "b_account_id"
@@ -158,9 +142,7 @@ class BankAccountDetailView(CanViewMixin, DetailView):
 
 
 class BankAccountCreateView(CanCreateMixin, CreateView):
-    """
-    Create a bank account (for the admins)
-    """
+    """Create a bank account (for the admins)."""
 
     model = BankAccount
     fields = ["name", "club", "iban", "number"]
@@ -170,9 +152,7 @@ class BankAccountCreateView(CanCreateMixin, CreateView):
 class BankAccountDeleteView(
     CanEditPropMixin, DeleteView
 ):  # TODO change Delete to Close
-    """
-    Delete a bank account (for the admins)
-    """
+    """Delete a bank account (for the admins)."""
 
     model = BankAccount
     pk_url_kwarg = "b_account_id"
@@ -184,9 +164,7 @@ class BankAccountDeleteView(
 
 
 class ClubAccountEditView(CanViewMixin, UpdateView):
-    """
-    An edit view for the admins
-    """
+    """An edit view for the admins."""
 
     model = ClubAccount
     pk_url_kwarg = "c_account_id"
@@ -195,9 +173,7 @@ class ClubAccountEditView(CanViewMixin, UpdateView):
 
 
 class ClubAccountDetailView(CanViewMixin, DetailView):
-    """
-    A detail view, listing every journal
-    """
+    """A detail view, listing every journal."""
 
     model = ClubAccount
     pk_url_kwarg = "c_account_id"
@@ -205,17 +181,15 @@ class ClubAccountDetailView(CanViewMixin, DetailView):
 
 
 class ClubAccountCreateView(CanCreateMixin, CreateView):
-    """
-    Create a club account (for the admins)
-    """
+    """Create a club account (for the admins)."""
 
     model = ClubAccount
     fields = ["name", "club", "bank_account"]
     template_name = "core/create.jinja"
 
     def get_initial(self):
-        ret = super(ClubAccountCreateView, self).get_initial()
-        if "parent" in self.request.GET.keys():
+        ret = super().get_initial()
+        if "parent" in self.request.GET:
             obj = BankAccount.objects.filter(id=int(self.request.GET["parent"])).first()
             if obj is not None:
                 ret["bank_account"] = obj.id
@@ -225,9 +199,7 @@ class ClubAccountCreateView(CanCreateMixin, CreateView):
 class ClubAccountDeleteView(
     CanEditPropMixin, DeleteView
 ):  # TODO change Delete to Close
-    """
-    Delete a club account (for the admins)
-    """
+    """Delete a club account (for the admins)."""
 
     model = ClubAccount
     pk_url_kwarg = "c_account_id"
@@ -287,9 +259,7 @@ class JournalTabsMixin(TabedViewMixin):
 
 
 class JournalCreateView(CanCreateMixin, CreateView):
-    """
-    Create a general journal
-    """
+    """Create a general journal."""
 
     model = GeneralJournal
     form_class = modelform_factory(
@@ -300,8 +270,8 @@ class JournalCreateView(CanCreateMixin, CreateView):
     template_name = "core/create.jinja"
 
     def get_initial(self):
-        ret = super(JournalCreateView, self).get_initial()
-        if "parent" in self.request.GET.keys():
+        ret = super().get_initial()
+        if "parent" in self.request.GET:
             obj = ClubAccount.objects.filter(id=int(self.request.GET["parent"])).first()
             if obj is not None:
                 ret["club_account"] = obj.id
@@ -309,9 +279,7 @@ class JournalCreateView(CanCreateMixin, CreateView):
 
 
 class JournalDetailView(JournalTabsMixin, CanViewMixin, DetailView):
-    """
-    A detail view, listing every operation
-    """
+    """A detail view, listing every operation."""
 
     model = GeneralJournal
     pk_url_kwarg = "j_id"
@@ -320,9 +288,7 @@ class JournalDetailView(JournalTabsMixin, CanViewMixin, DetailView):
 
 
 class JournalEditView(CanEditMixin, UpdateView):
-    """
-    Update a general journal
-    """
+    """Update a general journal."""
 
     model = GeneralJournal
     pk_url_kwarg = "j_id"
@@ -331,9 +297,7 @@ class JournalEditView(CanEditMixin, UpdateView):
 
 
 class JournalDeleteView(CanEditPropMixin, DeleteView):
-    """
-    Delete a club account (for the admins)
-    """
+    """Delete a club account (for the admins)."""
 
     model = GeneralJournal
     pk_url_kwarg = "j_id"
@@ -343,7 +307,7 @@ class JournalDeleteView(CanEditPropMixin, DeleteView):
     def dispatch(self, request, *args, **kwargs):
         self.object = self.get_object()
         if self.object.operations.count() == 0:
-            return super(JournalDeleteView, self).dispatch(request, *args, **kwargs)
+            return super().dispatch(request, *args, **kwargs)
         else:
             raise PermissionDenied
 
@@ -377,12 +341,30 @@ class OperationForm(forms.ModelForm):
             "invoice": SelectFile,
         }
 
-    user = AutoCompleteSelectField("users", help_text=None, required=False)
-    club_account = AutoCompleteSelectField(
-        "club_accounts", help_text=None, required=False
+    user = forms.ModelChoiceField(
+        help_text=None,
+        required=False,
+        widget=AutoCompleteSelectUser,
+        queryset=User.objects.all(),
+    )
+    club_account = forms.ModelChoiceField(
+        help_text=None,
+        required=False,
+        widget=AutoCompleteSelectClubAccount,
+        queryset=ClubAccount.objects.all(),
+    )
+    club = forms.ModelChoiceField(
+        help_text=None,
+        required=False,
+        widget=AutoCompleteSelectClub,
+        queryset=Club.objects.all(),
+    )
+    company = forms.ModelChoiceField(
+        help_text=None,
+        required=False,
+        widget=AutoCompleteSelectCompany,
+        queryset=Company.objects.all(),
     )
-    club = AutoCompleteSelectField("clubs", help_text=None, required=False)
-    company = AutoCompleteSelectField("companies", help_text=None, required=False)
     need_link = forms.BooleanField(
         label=_("Link this operation to the target account"),
         required=False,
@@ -391,7 +373,7 @@ class OperationForm(forms.ModelForm):
 
     def __init__(self, *args, **kwargs):
         club_account = kwargs.pop("club_account", None)
-        super(OperationForm, self).__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
         if club_account:
             self.fields["label"].queryset = club_account.labels.order_by("name").all()
         if self.instance.target_type == "USER":
@@ -404,8 +386,8 @@ class OperationForm(forms.ModelForm):
             self.fields["company"].initial = self.instance.target_id
 
     def clean(self):
-        self.cleaned_data = super(OperationForm, self).clean()
-        if "target_type" in self.cleaned_data.keys():
+        self.cleaned_data = super().clean()
+        if "target_type" in self.cleaned_data:
             if (
                 self.cleaned_data.get("user") is None
                 and self.cleaned_data.get("club") is None
@@ -434,7 +416,7 @@ class OperationForm(forms.ModelForm):
         return self.cleaned_data
 
     def save(self):
-        ret = super(OperationForm, self).save()
+        ret = super().save()
         if (
             self.instance.target_type == "ACCOUNT"
             and not self.instance.linked_operation
@@ -472,9 +454,7 @@ class OperationForm(forms.ModelForm):
 
 
 class OperationCreateView(CanCreateMixin, CreateView):
-    """
-    Create an operation
-    """
+    """Create an operation."""
 
     model = Operation
     form_class = OperationForm
@@ -486,23 +466,21 @@ class OperationCreateView(CanCreateMixin, CreateView):
         return self.form_class(club_account=ca, **self.get_form_kwargs())
 
     def get_initial(self):
-        ret = super(OperationCreateView, self).get_initial()
+        ret = super().get_initial()
         if self.journal is not None:
             ret["journal"] = self.journal.id
         return ret
 
     def get_context_data(self, **kwargs):
-        """Add journal to the context"""
-        kwargs = super(OperationCreateView, self).get_context_data(**kwargs)
+        """Add journal to the context."""
+        kwargs = super().get_context_data(**kwargs)
         if self.journal:
             kwargs["object"] = self.journal
         return kwargs
 
 
 class OperationEditView(CanEditMixin, UpdateView):
-    """
-    An edit view, working as detail for the moment
-    """
+    """An edit view, working as detail for the moment."""
 
     model = Operation
     pk_url_kwarg = "op_id"
@@ -510,16 +488,14 @@ class OperationEditView(CanEditMixin, UpdateView):
     template_name = "accounting/operation_edit.jinja"
 
     def get_context_data(self, **kwargs):
-        """Add journal to the context"""
-        kwargs = super(OperationEditView, self).get_context_data(**kwargs)
+        """Add journal to the context."""
+        kwargs = super().get_context_data(**kwargs)
         kwargs["object"] = self.object.journal
         return kwargs
 
 
 class OperationPDFView(CanViewMixin, DetailView):
-    """
-    Display the PDF of a given operation
-    """
+    """Display the PDF of a given operation."""
 
     model = Operation
     pk_url_kwarg = "op_id"
@@ -671,9 +647,7 @@ class OperationPDFView(CanViewMixin, DetailView):
 
 
 class JournalNatureStatementView(JournalTabsMixin, CanViewMixin, DetailView):
-    """
-    Display a statement sorted by labels
-    """
+    """Display a statement sorted by labels."""
 
     model = GeneralJournal
     pk_url_kwarg = "j_id"
@@ -684,19 +658,17 @@ class JournalNatureStatementView(JournalTabsMixin, CanViewMixin, DetailView):
         ret = collections.OrderedDict()
         statement = collections.OrderedDict()
         total_sum = 0
-        for sat in [None] + list(
-            SimplifiedAccountingType.objects.order_by("label").all()
-        ):
-            sum = queryset.filter(
+        for sat in [
+            None,
+            *list(SimplifiedAccountingType.objects.order_by("label")),
+        ]:
+            amount = queryset.filter(
                 accounting_type__movement_type=movement_type, simpleaccounting_type=sat
             ).aggregate(amount_sum=Sum("amount"))["amount_sum"]
-            if sat:
-                sat = sat.label
-            else:
-                sat = ""
-            if sum:
-                total_sum += sum
-                statement[sat] = sum
+            label = sat.label if sat is not None else ""
+            if amount:
+                total_sum += amount
+                statement[label] = amount
         ret[movement_type] = statement
         ret[movement_type + "_sum"] = total_sum
         return ret
@@ -719,28 +691,23 @@ class JournalNatureStatementView(JournalTabsMixin, CanViewMixin, DetailView):
             self.statement(self.object.operations.filter(label=None).all(), "DEBIT")
         )
         statement[_("No label operations")] = no_label_statement
-        for l in labels:
+        for label in labels:
             l_stmt = collections.OrderedDict()
-            l_stmt.update(
-                self.statement(self.object.operations.filter(label=l).all(), "CREDIT")
-            )
-            l_stmt.update(
-                self.statement(self.object.operations.filter(label=l).all(), "DEBIT")
-            )
-            statement[l] = l_stmt
+            journals = self.object.operations.filter(label=label).all()
+            l_stmt.update(self.statement(journals, "CREDIT"))
+            l_stmt.update(self.statement(journals, "DEBIT"))
+            statement[label] = l_stmt
         return statement
 
     def get_context_data(self, **kwargs):
-        """Add infos to the context"""
-        kwargs = super(JournalNatureStatementView, self).get_context_data(**kwargs)
+        """Add infos to the context."""
+        kwargs = super().get_context_data(**kwargs)
         kwargs["statement"] = self.big_statement()
         return kwargs
 
 
 class JournalPersonStatementView(JournalTabsMixin, CanViewMixin, DetailView):
-    """
-    Calculate a dictionary with operation target and sum of operations
-    """
+    """Calculate a dictionary with operation target and sum of operations."""
 
     model = GeneralJournal
     pk_url_kwarg = "j_id"
@@ -770,8 +737,8 @@ class JournalPersonStatementView(JournalTabsMixin, CanViewMixin, DetailView):
         return sum(self.statement(movement_type).values())
 
     def get_context_data(self, **kwargs):
-        """Add journal to the context"""
-        kwargs = super(JournalPersonStatementView, self).get_context_data(**kwargs)
+        """Add journal to the context."""
+        kwargs = super().get_context_data(**kwargs)
         kwargs["credit_statement"] = self.statement("CREDIT")
         kwargs["debit_statement"] = self.statement("DEBIT")
         kwargs["total_credit"] = self.total("CREDIT")
@@ -780,9 +747,7 @@ class JournalPersonStatementView(JournalTabsMixin, CanViewMixin, DetailView):
 
 
 class JournalAccountingStatementView(JournalTabsMixin, CanViewMixin, DetailView):
-    """
-    Calculate a dictionary with operation type and sum of operations
-    """
+    """Calculate a dictionary with operation type and sum of operations."""
 
     model = GeneralJournal
     pk_url_kwarg = "j_id"
@@ -800,8 +765,8 @@ class JournalAccountingStatementView(JournalTabsMixin, CanViewMixin, DetailView)
         return statement
 
     def get_context_data(self, **kwargs):
-        """Add journal to the context"""
-        kwargs = super(JournalAccountingStatementView, self).get_context_data(**kwargs)
+        """Add journal to the context."""
+        kwargs = super().get_context_data(**kwargs)
         kwargs["statement"] = self.statement()
         return kwargs
 
@@ -815,9 +780,7 @@ class CompanyListView(CanViewMixin, ListView):
 
 
 class CompanyCreateView(CanCreateMixin, CreateView):
-    """
-    Create a company
-    """
+    """Create a company."""
 
     model = Company
     fields = ["name"]
@@ -826,9 +789,7 @@ class CompanyCreateView(CanCreateMixin, CreateView):
 
 
 class CompanyEditView(CanCreateMixin, UpdateView):
-    """
-    Edit a company
-    """
+    """Edit a company."""
 
     model = Company
     pk_url_kwarg = "co_id"
@@ -856,8 +817,8 @@ class LabelCreateView(
     template_name = "core/create.jinja"
 
     def get_initial(self):
-        ret = super(LabelCreateView, self).get_initial()
-        if "parent" in self.request.GET.keys():
+        ret = super().get_initial()
+        if "parent" in self.request.GET:
             obj = ClubAccount.objects.filter(id=int(self.request.GET["parent"])).first()
             if obj is not None:
                 ret["club_account"] = obj.id
@@ -881,15 +842,17 @@ class LabelDeleteView(CanEditMixin, DeleteView):
 
 
 class CloseCustomerAccountForm(forms.Form):
-    user = AutoCompleteSelectField(
-        "users", label=_("Refound this account"), help_text=None, required=True
+    user = forms.ModelChoiceField(
+        label=_("Refound this account"),
+        help_text=None,
+        required=True,
+        widget=AutoCompleteSelectUser,
+        queryset=User.objects.all(),
     )
 
 
 class RefoundAccountView(FormView):
-    """
-    Create a selling with the same amount than the current user money
-    """
+    """Create a selling with the same amount than the current user money."""
 
     template_name = "accounting/refound_account.jinja"
     form_class = CloseCustomerAccountForm
@@ -901,19 +864,19 @@ class RefoundAccountView(FormView):
             raise PermissionDenied
 
     def dispatch(self, request, *arg, **kwargs):
-        res = super(RefoundAccountView, self).dispatch(request, *arg, **kwargs)
+        res = super().dispatch(request, *arg, **kwargs)
         if self.permission(request.user):
             return res
 
     def post(self, request, *arg, **kwargs):
         self.operator = request.user
         if self.permission(request.user):
-            return super(RefoundAccountView, self).post(self, request, *arg, **kwargs)
+            return super().post(self, request, *arg, **kwargs)
 
     def form_valid(self, form):
         self.customer = form.cleaned_data["user"]
         self.create_selling()
-        return super(RefoundAccountView, self).form_valid(form)
+        return super().form_valid(form)
 
     def get_success_url(self):
         return reverse("accounting:refound_account")

accounting/widgets/select.py

@@ -0,0 +1,39 @@
+from pydantic import TypeAdapter
+
+from accounting.models import ClubAccount, Company
+from accounting.schemas import ClubAccountSchema, CompanySchema
+from core.views.widgets.select import AutoCompleteSelect, AutoCompleteSelectMultiple
+
+_js = ["bundled/accounting/components/ajax-select-index.ts"]
+
+
+class AutoCompleteSelectClubAccount(AutoCompleteSelect):
+    component_name = "club-account-ajax-select"
+    model = ClubAccount
+    adapter = TypeAdapter(list[ClubAccountSchema])
+
+    js = _js
+
+
+class AutoCompleteSelectMultipleClubAccount(AutoCompleteSelectMultiple):
+    component_name = "club-account-ajax-select"
+    model = ClubAccount
+    adapter = TypeAdapter(list[ClubAccountSchema])
+
+    js = _js
+
+
+class AutoCompleteSelectCompany(AutoCompleteSelect):
+    component_name = "company-ajax-select"
+    model = Company
+    adapter = TypeAdapter(list[CompanySchema])
+
+    js = _js
+
+
+class AutoCompleteSelectMultipleCompany(AutoCompleteSelectMultiple):
+    component_name = "company-ajax-select"
+    model = Company
+    adapter = TypeAdapter(list[CompanySchema])
+
+    js = _js

antispam/admin.py

@@ -0,0 +1,10 @@
+from django.contrib import admin
+
+from antispam.models import ToxicDomain
+
+
+@admin.register(ToxicDomain)
+class ToxicDomainAdmin(admin.ModelAdmin):
+    list_display = ("domain", "is_externally_managed", "created")
+    search_fields = ("domain", "is_externally_managed", "created")
+    list_filter = ("is_externally_managed",)

antispam/apps.py

@@ -0,0 +1,7 @@
+from django.apps import AppConfig
+
+
+class AntispamConfig(AppConfig):
+    default_auto_field = "django.db.models.BigAutoField"
+    verbose_name = "antispam"
+    name = "antispam"

antispam/forms.py

@@ -0,0 +1,18 @@
+import re
+
+from django import forms
+from django.core.validators import EmailValidator
+from django.utils.translation import gettext_lazy as _
+
+from antispam.models import ToxicDomain
+
+
+class AntiSpamEmailField(forms.EmailField):
+    """An email field that email addresses with a known toxic domain."""
+
+    def run_validators(self, value: str):
+        super().run_validators(value)
+        # Domain part should exist since email validation is guaranteed to run first
+        domain = re.search(EmailValidator.domain_regex, value)
+        if ToxicDomain.objects.filter(domain=domain[0]).exists():
+            raise forms.ValidationError(_("Email domain is not allowed."))

antispam/management/commands/update_spam_database.py

@@ -0,0 +1,69 @@
+import requests
+from django.conf import settings
+from django.core.management import BaseCommand
+from django.db.models import Max
+from django.utils import timezone
+
+from antispam.models import ToxicDomain
+
+
+class Command(BaseCommand):
+    """Update blocked ips/mails database"""
+
+    help = "Update blocked ips/mails database"
+
+    def add_arguments(self, parser):
+        parser.add_argument(
+            "--force", action="store_true", help="Force re-creation even if up to date"
+        )
+
+    def _should_update(self, *, force: bool = False) -> bool:
+        if force:
+            return True
+        oldest = ToxicDomain.objects.filter(is_externally_managed=True).aggregate(
+            res=Max("created")
+        )["res"]
+        return not (oldest and timezone.now() < (oldest + timezone.timedelta(days=1)))
+
+    def _download_domains(self, providers: list[str]) -> set[str]:
+        domains = set()
+        for provider in providers:
+            res = requests.get(provider)
+            if not res.ok:
+                self.stderr.write(
+                    f"Source {provider} responded with code {res.status_code}"
+                )
+                continue
+            domains |= set(res.content.decode().splitlines())
+        return domains
+
+    def _update_domains(self, domains: set[str]):
+        # Cleanup database
+        ToxicDomain.objects.filter(is_externally_managed=True).delete()
+
+        # Create database
+        ToxicDomain.objects.bulk_create(
+            [
+                ToxicDomain(domain=domain, is_externally_managed=True)
+                for domain in domains
+            ],
+            ignore_conflicts=True,
+        )
+        self.stdout.write("Domain database updated")
+
+    def handle(self, *args, **options):
+        if not self._should_update(force=options["force"]):
+            self.stdout.write("Domain database is up to date")
+            return
+        self.stdout.write("Updating domain database")
+
+        domains = self._download_domains(settings.TOXIC_DOMAINS_PROVIDERS)
+
+        if not domains:
+            self.stderr.write(
+                "No domains could be fetched from settings.TOXIC_DOMAINS_PROVIDERS. "
+                "Please, have a look at your settings."
+            )
+            return
+
+        self._update_domains(domains)

antispam/migrations/0001_initial.py

@@ -0,0 +1,35 @@
+# Generated by Django 4.2.14 on 2024-08-03 23:05
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    initial = True
+
+    dependencies = []
+
+    operations = [
+        migrations.CreateModel(
+            name="ToxicDomain",
+            fields=[
+                (
+                    "domain",
+                    models.URLField(
+                        max_length=253,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="domain",
+                    ),
+                ),
+                ("created", models.DateTimeField(auto_now_add=True)),
+                (
+                    "is_externally_managed",
+                    models.BooleanField(
+                        default=False,
+                        help_text="True if kept up-to-date using external toxic domain providers, else False",
+                        verbose_name="is externally managed",
+                    ),
+                ),
+            ],
+        ),
+    ]

antispam/models.py

@@ -0,0 +1,19 @@
+from django.db import models
+from django.utils.translation import gettext_lazy as _
+
+
+class ToxicDomain(models.Model):
+    """Domain marked as spam in public databases"""
+
+    domain = models.URLField(_("domain"), max_length=253, primary_key=True)
+    created = models.DateTimeField(auto_now_add=True)
+    is_externally_managed = models.BooleanField(
+        _("is externally managed"),
+        default=False,
+        help_text=_(
+            "True if kept up-to-date using external toxic domain providers, else False"
+        ),
+    )
+
+    def __str__(self) -> str:
+        return self.domain

api/__init__.py

@@ -1,19 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#

api/admin.py

@@ -1,21 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-# Register your models here.

api/models.py

@@ -1,21 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-# Create your models here.

api/tests.py

@@ -1,21 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-# Create your tests here.

api/urls.py

@@ -1,54 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-from django.urls import include, path, re_path
-from rest_framework import routers
-
-from api.views import *
-
-# Router config
-router = routers.DefaultRouter()
-router.register(r"counter", CounterViewSet, basename="api_counter")
-router.register(r"user", UserViewSet, basename="api_user")
-router.register(r"club", ClubViewSet, basename="api_club")
-router.register(r"group", GroupViewSet, basename="api_group")
-
-# Launderette
-router.register(
-    r"launderette/place", LaunderettePlaceViewSet, basename="api_launderette_place"
-)
-router.register(
-    r"launderette/machine",
-    LaunderetteMachineViewSet,
-    basename="api_launderette_machine",
-)
-router.register(
-    r"launderette/token", LaunderetteTokenViewSet, basename="api_launderette_token"
-)
-
-urlpatterns = [
-    # API
-    re_path(r"^", include(router.urls)),
-    re_path(r"^login/", include("rest_framework.urls", namespace="rest_framework")),
-    re_path(r"^markdown$", RenderMarkdown, name="api_markdown"),
-    re_path(r"^mailings$", FetchMailingLists, name="mailings_fetch"),
-    re_path(r"^uv$", uv_endpoint, name="uv_endpoint"),
-    path("sas/<int:user>", all_pictures_of_user_endpoint, name="all_pictures_of_user"),
-]

api/views/__init__.py

@@ -1,77 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-from django.core.exceptions import PermissionDenied
-from django.db.models.query import QuerySet
-from rest_framework import viewsets
-from rest_framework.decorators import action
-from rest_framework.response import Response
-
-from core.views import can_edit, can_view
-
-
-def check_if(obj, user, test):
-    """
-    Detect if it's a single object or a queryset
-    aply a given test on individual object and return global permission
-    """
-    if isinstance(obj, QuerySet):
-        for o in obj:
-            if test(o, user) is False:
-                return False
-        return True
-    else:
-        return test(obj, user)
-
-
-class ManageModelMixin:
-    @action(detail=True)
-    def id(self, request, pk=None):
-        """
-        Get by id (api/v1/router/{pk}/id/)
-        """
-        self.queryset = get_object_or_404(self.queryset.filter(id=pk))
-        serializer = self.get_serializer(self.queryset)
-        return Response(serializer.data)
-
-
-class RightModelViewSet(ManageModelMixin, viewsets.ModelViewSet):
-    def dispatch(self, request, *arg, **kwargs):
-        res = super(RightModelViewSet, self).dispatch(request, *arg, **kwargs)
-        obj = self.queryset
-        user = self.request.user
-        try:
-            if request.method == "GET" and check_if(obj, user, can_view):
-                return res
-            if request.method != "GET" and check_if(obj, user, can_edit):
-                return res
-        except:
-            pass  # To prevent bug with Anonymous user
-        raise PermissionDenied
-
-
-from .api import *
-from .club import *
-from .counter import *
-from .group import *
-from .launderette import *
-from .sas import *
-from .user import *
-from .uv import *

api/views/api.py

@@ -1,38 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-from rest_framework.decorators import api_view, renderer_classes
-from rest_framework.renderers import StaticHTMLRenderer
-from rest_framework.response import Response
-
-from core.templatetags.renderer import markdown
-
-
-@api_view(["POST"])
-@renderer_classes((StaticHTMLRenderer,))
-def RenderMarkdown(request):
-    """
-    Render Markdown
-    """
-    try:
-        data = markdown(request.POST["text"])
-    except:
-        data = "Error"
-    return Response(data)

api/views/club.py

@@ -1,58 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-from django.conf import settings
-from django.core.exceptions import PermissionDenied
-from rest_framework import serializers
-from rest_framework.decorators import api_view, renderer_classes
-from rest_framework.renderers import StaticHTMLRenderer
-from rest_framework.response import Response
-
-from api.views import RightModelViewSet
-from club.models import Club, Mailing
-
-
-class ClubSerializer(serializers.ModelSerializer):
-    class Meta:
-        model = Club
-        fields = ("id", "name", "unix_name", "address", "members")
-
-
-class ClubViewSet(RightModelViewSet):
-    """
-    Manage Clubs (api/v1/club/)
-    """
-
-    serializer_class = ClubSerializer
-    queryset = Club.objects.all()
-
-
-@api_view(["GET"])
-@renderer_classes((StaticHTMLRenderer,))
-def FetchMailingLists(request):
-    key = request.GET.get("key", "")
-    if key != settings.SITH_MAILING_FETCH_KEY:
-        raise PermissionDenied
-    data = ""
-    for mailing in Mailing.objects.filter(
-        is_moderated=True, club__is_active=True
-    ).all():
-        data += mailing.fetch_format() + "\n"
-    return Response(data)

api/views/counter.py

@@ -1,55 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-from rest_framework import serializers
-from rest_framework.decorators import action
-from rest_framework.response import Response
-
-from api.views import RightModelViewSet
-from counter.models import Counter
-
-
-class CounterSerializer(serializers.ModelSerializer):
-    is_open = serializers.BooleanField(read_only=True)
-    barman_list = serializers.ListField(
-        child=serializers.IntegerField(), read_only=True
-    )
-
-    class Meta:
-        model = Counter
-        fields = ("id", "name", "type", "club", "products", "is_open", "barman_list")
-
-
-class CounterViewSet(RightModelViewSet):
-    """
-    Manage Counters (api/v1/counter/)
-    """
-
-    serializer_class = CounterSerializer
-    queryset = Counter.objects.all()
-
-    @action(detail=False)
-    def bar(self, request):
-        """
-        Return all bars (api/v1/counter/bar/)
-        """
-        self.queryset = self.queryset.filter(type="BAR")
-        serializer = self.get_serializer(self.queryset, many=True)
-        return Response(serializer.data)

api/views/group.py

@@ -1,38 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-from rest_framework import serializers
-
-from api.views import RightModelViewSet
-from core.models import RealGroup
-
-
-class GroupSerializer(serializers.ModelSerializer):
-    class Meta:
-        model = RealGroup
-
-
-class GroupViewSet(RightModelViewSet):
-    """
-    Manage Groups (api/v1/group/)
-    """
-
-    serializer_class = GroupSerializer
-    queryset = RealGroup.objects.all()

api/views/launderette.py

@@ -1,131 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-from rest_framework import serializers
-from rest_framework.decorators import action
-from rest_framework.response import Response
-
-from api.views import RightModelViewSet
-from launderette.models import Launderette, Machine, Token
-
-
-class LaunderettePlaceSerializer(serializers.ModelSerializer):
-    machine_list = serializers.ListField(
-        child=serializers.IntegerField(), read_only=True
-    )
-    token_list = serializers.ListField(child=serializers.IntegerField(), read_only=True)
-
-    class Meta:
-        model = Launderette
-        fields = (
-            "id",
-            "name",
-            "counter",
-            "machine_list",
-            "token_list",
-            "get_absolute_url",
-        )
-
-
-class LaunderetteMachineSerializer(serializers.ModelSerializer):
-    class Meta:
-        model = Machine
-        fields = ("id", "name", "type", "is_working", "launderette")
-
-
-class LaunderetteTokenSerializer(serializers.ModelSerializer):
-    class Meta:
-        model = Token
-        fields = (
-            "id",
-            "name",
-            "type",
-            "launderette",
-            "borrow_date",
-            "user",
-            "is_avaliable",
-        )
-
-
-class LaunderettePlaceViewSet(RightModelViewSet):
-    """
-    Manage Launderette (api/v1/launderette/place/)
-    """
-
-    serializer_class = LaunderettePlaceSerializer
-    queryset = Launderette.objects.all()
-
-
-class LaunderetteMachineViewSet(RightModelViewSet):
-    """
-    Manage Washing Machines (api/v1/launderette/machine/)
-    """
-
-    serializer_class = LaunderetteMachineSerializer
-    queryset = Machine.objects.all()
-
-
-class LaunderetteTokenViewSet(RightModelViewSet):
-    """
-    Manage Launderette's tokens (api/v1/launderette/token/)
-    """
-
-    serializer_class = LaunderetteTokenSerializer
-    queryset = Token.objects.all()
-
-    @action(detail=False)
-    def washing(self, request):
-        """
-        Return all washing tokens (api/v1/launderette/token/washing)
-        """
-        self.queryset = self.queryset.filter(type="WASHING")
-        serializer = self.get_serializer(self.queryset, many=True)
-        return Response(serializer.data)
-
-    @action(detail=False)
-    def drying(self, request):
-        """
-        Return all drying tokens (api/v1/launderette/token/drying)
-        """
-        self.queryset = self.queryset.filter(type="DRYING")
-        serializer = self.get_serializer(self.queryset, many=True)
-        return Response(serializer.data)
-
-    @action(detail=False)
-    def avaliable(self, request):
-        """
-        Return all avaliable tokens (api/v1/launderette/token/avaliable)
-        """
-        self.queryset = self.queryset.filter(
-            borrow_date__isnull=True, user__isnull=True
-        )
-        serializer = self.get_serializer(self.queryset, many=True)
-        return Response(serializer.data)
-
-    @action(detail=False)
-    def unavaliable(self, request):
-        """
-        Return all unavaliable tokens (api/v1/launderette/token/unavaliable)
-        """
-        self.queryset = self.queryset.filter(
-            borrow_date__isnull=False, user__isnull=False
-        )
-        serializer = self.get_serializer(self.queryset, many=True)
-        return Response(serializer.data)

api/views/sas.py

@@ -1,43 +0,0 @@
-from typing import List
-
-from rest_framework.decorators import api_view, renderer_classes
-from rest_framework.exceptions import PermissionDenied
-from rest_framework.generics import get_object_or_404
-from rest_framework.renderers import JSONRenderer
-from rest_framework.request import Request
-from rest_framework.response import Response
-
-from core.models import User
-from core.views import can_edit
-from sas.models import Picture
-
-
-def all_pictures_of_user(user: User) -> List[Picture]:
-    return [
-        relation.picture
-        for relation in user.pictures.exclude(picture=None)
-        .order_by("-picture__parent__date", "id")
-        .select_related("picture__parent")
-    ]
-
-
-@api_view(["GET"])
-@renderer_classes((JSONRenderer,))
-def all_pictures_of_user_endpoint(request: Request, user: int):
-    requested_user: User = get_object_or_404(User, pk=user)
-    if not can_edit(requested_user, request.user):
-        raise PermissionDenied
-
-    return Response(
-        [
-            {
-                "name": f"{picture.parent.name} - {picture.name}",
-                "date": picture.date,
-                "author": str(picture.owner),
-                "full_size_url": picture.get_download_url(),
-                "compressed_url": picture.get_download_compressed_url(),
-                "thumb_url": picture.get_download_thumb_url(),
-            }
-            for picture in all_pictures_of_user(requested_user)
-        ]
-    )

api/views/user.py

@@ -1,63 +0,0 @@
-# -*- coding:utf-8 -*-
-#
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
-#
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
-#
-# You can find the whole source code at https://github.com/ae-utbm/sith3
-#
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
-#
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
-#
-
-import datetime
-
-from rest_framework import serializers
-from rest_framework.decorators import action
-from rest_framework.response import Response
-
-from api.views import RightModelViewSet
-from core.models import User
-
-
-class UserSerializer(serializers.ModelSerializer):
-    class Meta:
-        model = User
-        fields = (
-            "id",
-            "first_name",
-            "last_name",
-            "email",
-            "date_of_birth",
-            "nick_name",
-            "is_active",
-            "date_joined",
-        )
-
-
-class UserViewSet(RightModelViewSet):
-    """
-    Manage Users (api/v1/user/)
-    Only show active users
-    """
-
-    serializer_class = UserSerializer
-    queryset = User.objects.filter(is_active=True)
-
-    @action(detail=False)
-    def birthday(self, request):
-        """
-        Return all users born today (api/v1/user/birstdays)
-        """
-        date = datetime.datetime.today()
-        self.queryset = self.queryset.filter(date_of_birth=date)
-        serializer = self.get_serializer(self.queryset, many=True)
-        return Response(serializer.data)

api/views/uv.py

@@ -1,128 +0,0 @@
-import json
-import urllib.request
-
-from django.conf import settings
-from django.core.exceptions import PermissionDenied
-from rest_framework import serializers
-from rest_framework.decorators import api_view, renderer_classes
-from rest_framework.renderers import JSONRenderer
-from rest_framework.response import Response
-
-from pedagogy.views import CanCreateUVFunctionMixin
-
-
-@api_view(["GET"])
-@renderer_classes((JSONRenderer,))
-def uv_endpoint(request):
-    if not CanCreateUVFunctionMixin.can_create_uv(request.user):
-        raise PermissionDenied
-
-    params = request.query_params
-    if "year" not in params or "code" not in params:
-        raise serializers.ValidationError("Missing query parameter")
-
-    short_uv, full_uv = find_uv("fr", params["year"], params["code"])
-    if short_uv is None or full_uv is None:
-        return Response(status=204)
-
-    return Response(make_clean_uv(short_uv, full_uv))
-
-
-def find_uv(lang, year, code):
-    """
-    Uses the UTBM API to find an UV.
-    short_uv is the UV entry in the UV list. It is returned as it contains
-    information which are not in full_uv.
-    full_uv is the detailed representation of an UV.
-    """
-    # query the UV list
-    uvs_url = settings.SITH_PEDAGOGY_UTBM_API + "/uvs/{}/{}".format(lang, year)
-    response = urllib.request.urlopen(uvs_url)
-    uvs = json.loads(response.read().decode("utf-8"))
-
-    try:
-        # find the first UV which matches the code
-        short_uv = next(uv for uv in uvs if uv["code"] == code)
-    except StopIteration:
-        return (None, None)
-
-    # get detailed information about the UV
-    uv_url = settings.SITH_PEDAGOGY_UTBM_API + "/uv/{}/{}/{}/{}".format(
-        lang, year, code, short_uv["codeFormation"]
-    )
-    response = urllib.request.urlopen(uv_url)
-    full_uv = json.loads(response.read().decode("utf-8"))
-
-    return (short_uv, full_uv)
-
-
-def make_clean_uv(short_uv, full_uv):
-    """
-    Cleans the data up so that it corresponds to our data representation.
-    """
-    res = {}
-
-    res["credit_type"] = short_uv["codeCategorie"]
-
-    # probably wrong on a few UVs as we pick the first UV we find but
-    # availability depends on the formation
-    semesters = {
-        (True, True): "AUTUMN_AND_SPRING",
-        (True, False): "AUTUMN",
-        (False, True): "SPRING",
-    }
-    res["semester"] = semesters.get(
-        (short_uv["ouvertAutomne"], short_uv["ouvertPrintemps"]), "CLOSED"
-    )
-
-    langs = {"es": "SP", "en": "EN", "de": "DE"}
-    res["language"] = langs.get(full_uv["codeLangue"], "FR")
-
-    if full_uv["departement"] == "Pôle Humanités":
-        res["department"] = "HUMA"
-    else:
-        departments = {
-            "AL": "IMSI",
-            "AE": "EE",
-            "GI": "GI",
-            "GC": "EE",
-            "GM": "MC",
-            "TC": "TC",
-            "GP": "IMSI",
-            "ED": "EDIM",
-            "AI": "GI",
-            "AM": "MC",
-        }
-        res["department"] = departments.get(full_uv["codeFormation"], "NA")
-
-    res["credits"] = full_uv["creditsEcts"]
-
-    activities = ("CM", "TD", "TP", "THE", "TE")
-    for activity in activities:
-        res["hours_{}".format(activity)] = 0
-    for activity in full_uv["activites"]:
-        if activity["code"] in activities:
-            res["hours_{}".format(activity["code"])] += activity["nbh"] // 60
-
-    # wrong if the manager changes depending on the semester
-    semester = full_uv.get("automne", None)
-    if not semester:
-        semester = full_uv.get("printemps", {})
-    res["manager"] = semester.get("responsable", "")
-
-    res["title"] = full_uv["libelle"]
-
-    descriptions = {
-        "objectives": "objectifs",
-        "program": "programme",
-        "skills": "acquisitionCompetences",
-        "key_concepts": "acquisitionNotions",
-    }
-
-    for res_key, full_uv_key in descriptions.items():
-        res[res_key] = full_uv[full_uv_key]
-        # if not found or the API did not return a string
-        if type(res[res_key]) != str:
-            res[res_key] = ""
-
-    return res

biome.json

@@ -0,0 +1,29 @@
+{
+  "$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
+  "vcs": {
+    "enabled": true,
+    "clientKind": "git",
+    "useIgnoreFile": true
+  },
+  "files": {
+    "ignoreUnknown": false,
+    "ignore": ["*.min.*", "staticfiles/generated"]
+  },
+  "formatter": {
+    "enabled": true,
+    "indentStyle": "space",
+    "lineWidth": 88
+  },
+  "organizeImports": {
+    "enabled": true
+  },
+  "linter": {
+    "enabled": true,
+    "rules": {
+      "all": true
+    }
+  },
+  "javascript": {
+    "globals": ["Alpine", "$", "jQuery", "gettext", "interpolate"]
+  }
+}

club/__init__.py

@@ -1,19 +1,14 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #

club/admin.py

@@ -1,23 +1,17 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
-from ajax_select import make_ajax_form
 from django.contrib import admin
 
 from club.models import Club, Membership
@@ -37,4 +31,4 @@ class MembershipAdmin(admin.ModelAdmin):
         "user__last_name",
         "club__name",
     )
-    form = make_ajax_form(Membership, {"user": "users"})
+    autocomplete_fields = ("user",)

club/api.py

@@ -0,0 +1,22 @@
+from typing import Annotated
+
+from annotated_types import MinLen
+from ninja_extra import ControllerBase, api_controller, paginate, route
+from ninja_extra.pagination import PageNumberPaginationExtra
+from ninja_extra.schemas import PaginatedResponseSchema
+
+from club.models import Club
+from club.schemas import ClubSchema
+from core.api_permissions import CanAccessLookup
+
+
+@api_controller("/club")
+class ClubController(ControllerBase):
+    @route.get(
+        "/search",
+        response=PaginatedResponseSchema[ClubSchema],
+        permissions=[CanAccessLookup],
+    )
+    @paginate(PageNumberPaginationExtra, page_size=50)
+    def search_club(self, search: Annotated[str, MinLen(1)]):
+        return Club.objects.filter(name__icontains=search).values()

club/forms.py

@@ -1,31 +1,35 @@
-# -*- coding:utf-8 -*-
 #
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
+# Copyright 2016,2017
+# - Skia <skia@libskia.so>
+# - Sli <antoine@bartuccio.fr>
 #
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
+# Ce fichier fait partie du site de l'Association des Étudiants de l'UTBM,
+# http://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# This program is free software; you can redistribute it and/or modify it under
+# the terms of the GNU General Public License a published by the Free Software
+# Foundation; either version 3 of the License, or (at your option) any later
+# version.
 #
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
+# This program is distributed in the hope that it will be useful, but WITHOUT
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+# FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+# details.
+#
+# You should have received a copy of the GNU General Public License along with
+# this program; if not, write to the Free Sofware Foundation, Inc., 59 Temple
+# Place - Suite 330, Boston, MA 02111-1307, USA.
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 
-from ajax_select.fields import AutoCompleteSelectMultipleField
 from django import forms
 from django.conf import settings
 from django.utils.translation import gettext_lazy as _
 
 from club.models import Club, Mailing, MailingSubscription, Membership
 from core.models import User
-from core.views.forms import SelectDate, TzAwareDateTimeField
+from core.views.forms import SelectDate, SelectDateTime
+from core.views.widgets.select import AutoCompleteSelectMultipleUser
 from counter.models import Counter
 
 
@@ -35,28 +39,27 @@ class ClubEditForm(forms.ModelForm):
         fields = ["address", "logo", "short_description"]
 
     def __init__(self, *args, **kwargs):
-        super(ClubEditForm, self).__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
         self.fields["short_description"].widget = forms.Textarea()
 
 
 class MailingForm(forms.Form):
-    """
-    Form handling mailing lists right
-    """
+    """Form handling mailing lists right."""
 
     ACTION_NEW_MAILING = 1
     ACTION_NEW_SUBSCRIPTION = 2
     ACTION_REMOVE_SUBSCRIPTION = 3
 
-    subscription_users = AutoCompleteSelectMultipleField(
-        "users",
+    subscription_users = forms.ModelMultipleChoiceField(
         label=_("Users to add"),
         help_text=_("Search users to add (one or more)."),
         required=False,
+        widget=AutoCompleteSelectMultipleUser,
+        queryset=User.objects.all(),
     )
 
     def __init__(self, club_id, user_id, mailings, *args, **kwargs):
-        super(MailingForm, self).__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
 
         self.fields["action"] = forms.TypedChoiceField(
             choices=(
@@ -101,24 +104,15 @@ class MailingForm(forms.Form):
         )
 
     def check_required(self, cleaned_data, field):
-        """
-        If the given field doesn't exist or has no value, add a required error on it
-        """
+        """If the given field doesn't exist or has no value, add a required error on it."""
         if not cleaned_data.get(field, None):
             self.add_error(field, _("This field is required"))
 
     def clean_subscription_users(self):
-        """
-        Convert given users into real users and check their validity
-        """
-        cleaned_data = super(MailingForm, self).clean()
+        """Convert given users into real users and check their validity."""
+        cleaned_data = super().clean()
         users = []
         for user in cleaned_data["subscription_users"]:
-            user = User.objects.filter(id=user).first()
-            if not user:
-                raise forms.ValidationError(
-                    _("One of the selected users doesn't exist"), code="invalid"
-                )
             if not user.email:
                 raise forms.ValidationError(
                     _("One of the selected users doesn't have an email address"),
@@ -128,9 +122,9 @@ class MailingForm(forms.Form):
         return users
 
     def clean(self):
-        cleaned_data = super(MailingForm, self).clean()
+        cleaned_data = super().clean()
 
-        if not "action" in cleaned_data:
+        if "action" not in cleaned_data:
             # If there is no action provided, we can stop here
             raise forms.ValidationError(_("An action is required"), code="invalid")
 
@@ -151,15 +145,19 @@ class MailingForm(forms.Form):
 
 
 class SellingsForm(forms.Form):
-    begin_date = TzAwareDateTimeField(label=_("Begin date"), required=False)
-    end_date = TzAwareDateTimeField(label=_("End date"), required=False)
+    begin_date = forms.DateTimeField(
+        label=_("Begin date"), widget=SelectDateTime, required=False
+    )
+    end_date = forms.DateTimeField(
+        label=_("End date"), widget=SelectDateTime, required=False
+    )
 
     counters = forms.ModelMultipleChoiceField(
         Counter.objects.order_by("name").all(), label=_("Counter"), required=False
     )
 
     def __init__(self, club, *args, **kwargs):
-        super(SellingsForm, self).__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
         self.fields["products"] = forms.ModelMultipleChoiceField(
             club.products.order_by("name").filter(archived=False).all(),
             label=_("Products"),
@@ -173,18 +171,17 @@ class SellingsForm(forms.Form):
 
 
 class ClubMemberForm(forms.Form):
-    """
-    Form handling the members of a club
-    """
+    """Form handling the members of a club."""
 
     error_css_class = "error"
     required_css_class = "required"
 
-    users = AutoCompleteSelectMultipleField(
-        "users",
+    users = forms.ModelMultipleChoiceField(
         label=_("Users to add"),
         help_text=_("Search users to add (one or more)."),
         required=False,
+        widget=AutoCompleteSelectMultipleUser,
+        queryset=User.objects.all(),
     )
 
     def __init__(self, *args, **kwargs):
@@ -196,7 +193,7 @@ class ClubMemberForm(forms.Form):
                 self.club.members.filter(end_date=None).order_by("-role").all()
             )
         self.request_user_membership = self.club.get_membership_for(self.request_user)
-        super(ClubMemberForm, self).__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
 
         # Using a ModelForm binds too much the form with the model and we don't want that
         # We want the view to process the model creation since they are multiple users
@@ -232,18 +229,13 @@ class ClubMemberForm(forms.Form):
             self.fields.pop("start_date")
 
     def clean_users(self):
+        """Check that the user is not trying to add an user already in the club.
+
+        Also check that the user is valid and has a valid subscription.
         """
-        Check that the user is not trying to add an user already in the club
-        Also check that the user is valid and has a valid subscription
-        """
-        cleaned_data = super(ClubMemberForm, self).clean()
+        cleaned_data = super().clean()
         users = []
-        for user_id in cleaned_data["users"]:
-            user = User.objects.filter(id=user_id).first()
-            if not user:
-                raise forms.ValidationError(
-                    _("One of the selected users doesn't exist"), code="invalid"
-                )
+        for user in cleaned_data["users"]:
             if not user.is_subscribed:
                 raise forms.ValidationError(
                     _("User must be subscriber to take part to a club"), code="invalid"
@@ -256,10 +248,8 @@ class ClubMemberForm(forms.Form):
         return users
 
     def clean(self):
-        """
-        Check user rights for adding an user
-        """
-        cleaned_data = super(ClubMemberForm, self).clean()
+        """Check user rights for adding an user."""
+        cleaned_data = super().clean()
 
         if "start_date" in cleaned_data and not cleaned_data["start_date"]:
             # Drop start_date if allowed to edition but not specified

club/migrations/0001_initial.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 import django.core.validators

club/migrations/0002_auto_20160824_2152.py

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

club/migrations/0003_auto_20160902_2042.py

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

club/migrations/0004_auto_20160915_1057.py

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

club/migrations/0005_auto_20161120_1149.py

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

club/migrations/0006_auto_20161229_0040.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 import django.utils.timezone

club/migrations/0007_auto_20170324_0917.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 from django.db import migrations

club/migrations/0008_auto_20170515_2214.py

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

club/migrations/0009_auto_20170822_2232.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 import re
@@ -110,6 +109,6 @@ class Migration(migrations.Migration):
         ),
         migrations.AlterUniqueTogether(
             name="mailingsubscription",
-            unique_together=set([("user", "email", "mailing")]),
+            unique_together={("user", "email", "mailing")},
         ),
     ]

club/migrations/0010_auto_20170912_2028.py

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

club/migrations/0010_club_logo.py

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

club/migrations/0011_auto_20180426_2013.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import unicode_literals
 
 import django.db.models.deletion
@@ -16,7 +15,7 @@ class Migration(migrations.Migration):
             name="owner_group",
             field=models.ForeignKey(
                 on_delete=django.db.models.deletion.CASCADE,
-                default=club.models.Club.get_default_owner_group,
+                default=club.models.get_default_owner_group,
                 related_name="owned_club",
                 to="core.Group",
             ),

club/models.py

@@ -1,23 +1,29 @@
-# -*- coding:utf-8 -*-
 #
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
+# Copyright 2016,2017
+# - Skia <skia@libskia.so>
+# - Sli <antoine@bartuccio.fr>
 #
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
+# Ce fichier fait partie du site de l'Association des Étudiants de l'UTBM,
+# http://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# This program is free software; you can redistribute it and/or modify it under
+# the terms of the GNU General Public License a published by the Free Software
+# Foundation; either version 3 of the License, or (at your option) any later
+# version.
 #
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
+# This program is distributed in the hope that it will be useful, but WITHOUT
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+# FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+# details.
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
+# You should have received a copy of the GNU General Public License along with
+# this program; if not, write to the Free Sofware Foundation, Inc., 59 Temple
+# Place - Suite 330, Boston, MA 02111-1307, USA.
 #
-from typing import Optional
+#
+from __future__ import annotations
+
+from typing import Self
 
 from django.conf import settings
 from django.core import validators
@@ -29,6 +35,7 @@ from django.db.models import Q
 from django.urls import reverse
 from django.utils import timezone
 from django.utils.functional import cached_property
+from django.utils.timezone import localdate
 from django.utils.translation import gettext_lazy as _
 
 from core.models import Group, MetaGroup, Notification, Page, RealGroup, SithFile, User
@@ -36,10 +43,13 @@ from core.models import Group, MetaGroup, Notification, Page, RealGroup, SithFil
 # Create your models here.
 
 
+# This function prevents generating migration upon settings change
+def get_default_owner_group():
+    return settings.SITH_GROUP_ROOT_ID
+
+
 class Club(models.Model):
-    """
-    The Club class, made as a tree to allow nice tidy organization
-    """
+    """The Club class, made as a tree to allow nice tidy organization."""
 
     id = models.AutoField(primary_key=True, db_index=True)
     name = models.CharField(_("name"), max_length=64)
@@ -70,10 +80,6 @@ class Club(models.Model):
     )
     address = models.CharField(_("address"), max_length=254)
 
-    # This function prevents generating migration upon settings change
-    def get_default_owner_group():
-        return settings.SITH_GROUP_ROOT_ID
-
     owner_group = models.ForeignKey(
         Group,
         related_name="owned_club",
@@ -101,6 +107,34 @@ class Club(models.Model):
     class Meta:
         ordering = ["name", "unix_name"]
 
+    def __str__(self):
+        return self.name
+
+    @transaction.atomic()
+    def save(self, *args, **kwargs):
+        old = Club.objects.filter(id=self.id).first()
+        creation = old is None
+        if not creation and old.unix_name != self.unix_name:
+            self._change_unixname(self.unix_name)
+        super().save(*args, **kwargs)
+        if creation:
+            board = MetaGroup(name=self.unix_name + settings.SITH_BOARD_SUFFIX)
+            board.save()
+            member = MetaGroup(name=self.unix_name + settings.SITH_MEMBER_SUFFIX)
+            member.save()
+            subscribers = Group.objects.filter(
+                name=settings.SITH_MAIN_MEMBERS_GROUP
+            ).first()
+            self.make_home()
+            self.home.edit_groups.set([board])
+            self.home.view_groups.set([member, subscribers])
+            self.home.save()
+        self.make_page()
+        cache.set(f"sith_club_{self.unix_name}", self)
+
+    def get_absolute_url(self):
+        return reverse("club:club_view", kwargs={"club_id": self.id})
+
     @cached_property
     def president(self):
         return self.members.filter(
@@ -108,7 +142,7 @@ class Club(models.Model):
         ).first()
 
     def check_loop(self):
-        """Raise a validation error when a loop is found within the parent list"""
+        """Raise a validation error when a loop is found within the parent list."""
         objs = []
         cur = self
         while cur.parent is not None:
@@ -179,48 +213,18 @@ class Club(models.Model):
             self.page.parent = self.parent.page
             self.page.save(force_lock=True)
 
-    @transaction.atomic()
-    def save(self, *args, **kwargs):
-        old = Club.objects.filter(id=self.id).first()
-        creation = old is None
-        if not creation and old.unix_name != self.unix_name:
-            self._change_unixname(self.unix_name)
-        super(Club, self).save(*args, **kwargs)
-        if creation:
-            board = MetaGroup(name=self.unix_name + settings.SITH_BOARD_SUFFIX)
-            board.save()
-            member = MetaGroup(name=self.unix_name + settings.SITH_MEMBER_SUFFIX)
-            member.save()
-            subscribers = Group.objects.filter(
-                name=settings.SITH_MAIN_MEMBERS_GROUP
-            ).first()
-            self.make_home()
-            self.home.edit_groups.set([board])
-            self.home.view_groups.set([member, subscribers])
-            self.home.save()
-        self.make_page()
-        cache.set(f"sith_club_{self.unix_name}", self)
-
     def delete(self, *args, **kwargs):
-        super().delete(*args, **kwargs)
         # Invalidate the cache of this club and of its memberships
         for membership in self.members.ongoing().select_related("user"):
             cache.delete(f"membership_{self.id}_{membership.user.id}")
         cache.delete(f"sith_club_{self.unix_name}")
-
-    def __str__(self):
-        return self.name
-
-    def get_absolute_url(self):
-        return reverse("club:club_view", kwargs={"club_id": self.id})
+        super().delete(*args, **kwargs)
 
     def get_display_name(self):
         return self.name
 
     def is_owned_by(self, user):
-        """
-        Method to see if that object can be super edited by the given user
-        """
+        """Method to see if that object can be super edited by the given user."""
         if user.is_anonymous:
             return False
         return user.is_board_member
@@ -229,24 +233,21 @@ class Club(models.Model):
         return "https://%s%s" % (settings.SITH_URL, self.logo.url)
 
     def can_be_edited_by(self, user):
-        """
-        Method to see if that object can be edited by the given user
-        """
+        """Method to see if that object can be edited by the given user."""
         return self.has_rights_in_club(user)
 
     def can_be_viewed_by(self, user):
-        """
-        Method to see if that object can be seen by the given user
-        """
+        """Method to see if that object can be seen by the given user."""
         sub = User.objects.filter(pk=user.pk).first()
         if sub is None:
             return False
         return sub.was_subscribed
 
-    def get_membership_for(self, user: User) -> Optional["Membership"]:
-        """
-        Return the current membership the given user.
-        The result is cached.
+    def get_membership_for(self, user: User) -> Membership | None:
+        """Return the current membership the given user.
+
+        Note:
+            The result is cached.
         """
         if user.is_anonymous:
             return None
@@ -267,16 +268,12 @@ class Club(models.Model):
 
 
 class MembershipQuerySet(models.QuerySet):
-    def ongoing(self) -> "MembershipQuerySet":
-        """
-        Filter all memberships which are not finished yet
-        """
-        # noinspection PyTypeChecker
-        return self.filter(Q(end_date=None) | Q(end_date__gte=timezone.now()))
+    def ongoing(self) -> Self:
+        """Filter all memberships which are not finished yet."""
+        return self.filter(Q(end_date=None) | Q(end_date__gt=localdate()))
 
-    def board(self) -> "MembershipQuerySet":
-        """
-        Filter all memberships where the user is/was in the board.
+    def board(self) -> Self:
+        """Filter all memberships where the user is/was in the board.
 
         Be aware that users who were in the board in the past
         are included, even if there are no more members.
@@ -284,13 +281,12 @@ class MembershipQuerySet(models.QuerySet):
         If you want to get the users who are currently in the board,
         mind combining this with the :meth:`ongoing` queryset method
         """
-        # noinspection PyTypeChecker
         return self.filter(role__gt=settings.SITH_MAXIMUM_FREE_ROLE)
 
     def update(self, **kwargs):
-        """
-        Work just like the default Django's update() method,
-        but add a cache refresh for the elements of the queryset.
+        """Refresh the cache for the elements of the queryset.
+
+        Besides that, does the same job as a regular update method.
 
         Be aware that this adds a db query to retrieve the updated objects
         """
@@ -310,8 +306,7 @@ class MembershipQuerySet(models.QuerySet):
                     )
 
     def delete(self):
-        """
-        Work just like the default Django's delete() method,
+        """Work just like the default Django's delete() method,
         but add a cache invalidation for the elements of the queryset
         before the deletion.
 
@@ -327,8 +322,7 @@ class MembershipQuerySet(models.QuerySet):
 
 
 class Membership(models.Model):
-    """
-    The Membership class makes the connection between User and Clubs
+    """The Membership class makes the connection between User and Clubs.
 
     Both Users and Clubs can have many Membership objects:
        - a user can be a member of many clubs at a time
@@ -369,36 +363,11 @@ class Membership(models.Model):
 
     def __str__(self):
         return (
-            self.club.name
-            + " - "
-            + self.user.username
-            + " - "
-            + str(settings.SITH_CLUB_ROLES[self.role])
-            + str(" - " + str(_("past member")) if self.end_date is not None else "")
+            f"{self.club.name} - {self.user.username} "
+            f"- {settings.SITH_CLUB_ROLES[self.role]} "
+            f"- {str(_('past member')) if self.end_date is not None else ''}"
         )
 
-    def is_owned_by(self, user):
-        """
-        Method to see if that object can be super edited by the given user
-        """
-        if user.is_anonymous:
-            return False
-        return user.is_board_member
-
-    def can_be_edited_by(self, user: User) -> bool:
-        """
-        Check if that object can be edited by the given user
-        """
-        if user.is_root or user.is_board_member:
-            return True
-        membership = self.club.get_membership_for(user)
-        if membership is not None and membership.role >= self.role:
-            return True
-        return False
-
-    def get_absolute_url(self):
-        return reverse("club:club_members", kwargs={"club_id": self.club_id})
-
     def save(self, *args, **kwargs):
         super().save(*args, **kwargs)
         if self.end_date is None:
@@ -406,15 +375,32 @@ class Membership(models.Model):
         else:
             cache.set(f"membership_{self.club_id}_{self.user_id}", "not_member")
 
+    def get_absolute_url(self):
+        return reverse("club:club_members", kwargs={"club_id": self.club_id})
+
+    def is_owned_by(self, user):
+        """Method to see if that object can be super edited by the given user."""
+        if user.is_anonymous:
+            return False
+        return user.is_board_member
+
+    def can_be_edited_by(self, user: User) -> bool:
+        """Check if that object can be edited by the given user."""
+        if user.is_root or user.is_board_member:
+            return True
+        membership = self.club.get_membership_for(user)
+        return membership is not None and membership.role >= self.role
+
     def delete(self, *args, **kwargs):
         super().delete(*args, **kwargs)
         cache.delete(f"membership_{self.club_id}_{self.user_id}")
 
 
 class Mailing(models.Model):
-    """
-    This class correspond to a mailing list
-    Remember that mailing lists should be validated by UTBM
+    """A Mailing list for a club.
+
+    Warning:
+        Remember that mailing lists should be validated by UTBM.
     """
 
     club = models.ForeignKey(
@@ -447,6 +433,26 @@ class Mailing(models.Model):
         on_delete=models.CASCADE,
     )
 
+    def __str__(self):
+        return "%s - %s" % (self.club, self.email_full)
+
+    def save(self, *args, **kwargs):
+        if not self.is_moderated:
+            for user in (
+                RealGroup.objects.filter(id=settings.SITH_GROUP_COM_ADMIN_ID)
+                .first()
+                .users.all()
+            ):
+                if not user.notifications.filter(
+                    type="MAILING_MODERATION", viewed=False
+                ).exists():
+                    Notification(
+                        user=user,
+                        url=reverse("com:mailing_admin"),
+                        type="MAILING_MODERATION",
+                    ).save(*args, **kwargs)
+        super().save(*args, **kwargs)
+
     def clean(self):
         if Mailing.objects.filter(email=self.email).exists():
             raise ValidationError(_("This mailing list already exists."))
@@ -454,7 +460,7 @@ class Mailing(models.Model):
             self.is_moderated = True
         else:
             self.moderator = None
-        super(Mailing, self).clean()
+        super().clean()
 
     @property
     def email_full(self):
@@ -476,39 +482,15 @@ class Mailing(models.Model):
 
     def delete(self, *args, **kwargs):
         self.subscriptions.all().delete()
-        super(Mailing, self).delete()
+        super().delete()
 
     def fetch_format(self):
-        resp = self.email + ": "
-        for sub in self.subscriptions.all():
-            resp += sub.fetch_format()
-        return resp
-
-    def save(self):
-        if not self.is_moderated:
-            for user in (
-                RealGroup.objects.filter(id=settings.SITH_GROUP_COM_ADMIN_ID)
-                .first()
-                .users.all()
-            ):
-                if not user.notifications.filter(
-                    type="MAILING_MODERATION", viewed=False
-                ).exists():
-                    Notification(
-                        user=user,
-                        url=reverse("com:mailing_admin"),
-                        type="MAILING_MODERATION",
-                    ).save()
-        super(Mailing, self).save()
-
-    def __str__(self):
-        return "%s - %s" % (self.club, self.email_full)
+        destination = "".join(s.fetch_format() for s in self.subscriptions.all())
+        return f"{self.email}: {destination}"
 
 
 class MailingSubscription(models.Model):
-    """
-    This class makes the link between user and mailing list
-    """
+    """Link between user and mailing list."""
 
     mailing = models.ForeignKey(
         Mailing,
@@ -531,6 +513,9 @@ class MailingSubscription(models.Model):
     class Meta:
         unique_together = (("user", "email", "mailing"),)
 
+    def __str__(self):
+        return "(%s) - %s : %s" % (self.mailing, self.get_username, self.email)
+
     def clean(self):
         if not self.user and not self.email:
             raise ValidationError(_("At least user or email is required"))
@@ -545,7 +530,7 @@ class MailingSubscription(models.Model):
                     )
         except ObjectDoesNotExist:
             pass
-        super(MailingSubscription, self).clean()
+        super().clean()
 
     def is_owned_by(self, user):
         if user.is_anonymous:
@@ -573,6 +558,3 @@ class MailingSubscription(models.Model):
 
     def fetch_format(self):
         return self.get_email + " "
-
-    def __str__(self):
-        return "(%s) - %s : %s" % (self.mailing, self.get_username, self.email)

club/schemas.py

@@ -0,0 +1,9 @@
+from ninja import ModelSchema
+
+from club.models import Club
+
+
+class ClubSchema(ModelSchema):
+    class Meta:
+        model = Club
+        fields = ["id", "name"]

club/static/bundled/club/components/ajax-select-index.ts

@@ -0,0 +1,30 @@
+import { AjaxSelect } from "#core:core/components/ajax-select-base";
+import { registerComponent } from "#core:utils/web-components";
+import type { TomOption } from "tom-select/dist/types/types";
+import type { escape_html } from "tom-select/dist/types/utils";
+import { type ClubSchema, clubSearchClub } from "#openapi";
+
+@registerComponent("club-ajax-select")
+export class ClubAjaxSelect extends AjaxSelect {
+  protected valueField = "id";
+  protected labelField = "name";
+  protected searchField = ["code", "name"];
+
+  protected async search(query: string): Promise<TomOption[]> {
+    const resp = await clubSearchClub({ query: { search: query } });
+    if (resp.data) {
+      return resp.data.results;
+    }
+    return [];
+  }
+
+  protected renderOption(item: ClubSchema, sanitize: typeof escape_html) {
+    return `<div class="select-item">
+            <span class="select-item-text">${sanitize(item.name)}</span>
+          </div>`;
+  }
+
+  protected renderItem(item: ClubSchema, sanitize: typeof escape_html) {
+    return `<span>${sanitize(item.name)}</span>`;
+  }
+}

club/templates/club/club_detail.jinja

@@ -2,16 +2,16 @@
 {% from 'core/macros.jinja' import user_profile_link %}
 
 {% block content %}
-	<div id="club_detail">
-		{% if club.logo %}
-			<div class="club_logo"><img src="{{ club.logo.url }}" alt="{{ club.unix_name }}"></div>
-		{% endif %}
-		{% if page_revision %}
-			{{ page_revision|markdown }}
- 		{% else %}
- 		<h3>{% trans %}Club{% endtrans %}</h3>
-	    {% endif %}
-    </div>
+  <div id="club_detail">
+    {% if club.logo %}
+      <div class="club_logo"><img src="{{ club.logo.url }}" alt="{{ club.unix_name }}"></div>
+    {% endif %}
+    {% if page_revision %}
+      {{ page_revision|markdown }}
+    {% else %}
+      <h3>{% trans %}Club{% endtrans %}</h3>
+    {% endif %}
+  </div>
 {% endblock %}
 
 

club/templates/club/club_list.jinja

@@ -1,48 +1,48 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Club list{% endtrans %}
+  {% trans %}Club list{% endtrans %}
 {% endblock %}
 
 {% macro display_club(club) -%}
 
-        {% if club.is_active or user.is_root %}
-        
-        <li><a href="{{ url('club:club_view', club_id=club.id) }}">{{ club.name }}</a>
-        
-        {% if not club.is_active %}
-            ({% trans %}inactive{% endtrans %})
-        {% endif %}
+  {% if club.is_active or user.is_root %}
 
-        {% if club.president %} - <a href="{{ url('core:user_profile', user_id=club.president.user.id) }}">{{ club.president.user }}</a>{% endif %}
-        {% if club.short_description %}<p>{{ club.short_description|markdown }}</p>{% endif %}
-            
-        {% endif %}
+    <li><a href="{{ url('club:club_view', club_id=club.id) }}">{{ club.name }}</a>
 
-        {%- if club.children.all()|length != 0 %}
-        <ul>
-            {%- for c in club.children.order_by('name') %}
-                {{ display_club(c) }}
-            {%- endfor %}
-        </ul>
-        {%- endif -%}
-    </li>
+      {% if not club.is_active %}
+        ({% trans %}inactive{% endtrans %})
+      {% endif %}
+
+      {% if club.president %} - <a href="{{ url('core:user_profile', user_id=club.president.user.id) }}">{{ club.president.user }}</a>{% endif %}
+      {% if club.short_description %}<p>{{ club.short_description|markdown }}</p>{% endif %}
+
+  {% endif %}
+
+  {%- if club.children.all()|length != 0 %}
+    <ul>
+      {%- for c in club.children.order_by('name') %}
+        {{ display_club(c) }}
+      {%- endfor %}
+    </ul>
+  {%- endif -%}
+  </li>
 {%- endmacro %}
 
 {% block content %}
-    {% if user.is_root %}
+  {% if user.is_root %}
     <p><a href="{{ url('club:club_new') }}">{% trans %}New club{% endtrans %}</a></p>
-    {% endif %}
-    {% if club_list %}
+  {% endif %}
+  {% if club_list %}
     <h3>{% trans %}Club list{% endtrans %}</h3>
     <ul>
-        {%- for c in club_list.all().order_by('name') if c.parent is none %}
+      {%- for c in club_list.all().order_by('name') if c.parent is none %}
         {{ display_club(c) }}
-        {%- endfor %}
+      {%- endfor %}
     </ul>
-    {% else %}
+  {% else %}
     {% trans %}There is no club in this website.{% endtrans %}
-    {% endif %}
+  {% endif %}
 {% endblock %}
 
 

club/templates/club/club_members.jinja

@@ -2,81 +2,81 @@
 {% from 'core/macros.jinja' import user_profile_link, select_all_checkbox %}
 
 {% block content %}
-    <h2>{% trans %}Club members{% endtrans %}</h2>
-    {% if members %}
+  <h2>{% trans %}Club members{% endtrans %}</h2>
+  {% if members %}
     <form action="{{ url('club:club_members', club_id=club.id) }}" id="users_old" method="post">
-        {% csrf_token %}
-        {% set users_old = dict(form.users_old | groupby("choice_label")) %}
-        {% if users_old %}
-            {{ select_all_checkbox("users_old") }}
-            <p></p>
-        {% endif %}
-        <table>
-            <thead>
-                <tr>
-                    <td>{% trans %}User{% endtrans %}</td>
-                    <td>{% trans %}Role{% endtrans %}</td>
-                    <td>{% trans %}Description{% endtrans %}</td>
-                    <td>{% trans %}Since{% endtrans %}</td>
-                    {% if users_old %}
-                        <td>{% trans %}Mark as old{% endtrans %}</td>
-                    {% endif %}
-                </tr>
-            </thead>
-            <tbody>
-            {% for m in members %}
-                <tr>
-                    <td>{{ user_profile_link(m.user) }}</td>
-                    <td>{{ settings.SITH_CLUB_ROLES[m.role] }}</td>
-                    <td>{{ m.description }}</td>
-                    <td>{{ m.start_date }}</td>
-                    {% if users_old %}
-                        <td>
-                        {% set user_old = users_old[m.user.get_display_name()] %}
-                        {% if user_old %}
-                            {{ user_old[0].tag() }}
-                        {% endif %}
-                        </td>
-                    {% endif %}
-                </tr>
-            {% endfor %}
-            </tbody>
-        </table>
-        {{ form.users_old.errors }}
-        {% if users_old %}
-            <p></p>
-            <input type="submit" name="submit" value="{% trans %}Mark as old{% endtrans %}">
-        {% endif %}
+      {% csrf_token %}
+      {% set users_old = dict(form.users_old | groupby("choice_label")) %}
+      {% if users_old %}
+        {{ select_all_checkbox("users_old") }}
+        <p></p>
+      {% endif %}
+      <table>
+        <thead>
+          <tr>
+            <td>{% trans %}User{% endtrans %}</td>
+            <td>{% trans %}Role{% endtrans %}</td>
+            <td>{% trans %}Description{% endtrans %}</td>
+            <td>{% trans %}Since{% endtrans %}</td>
+            {% if users_old %}
+              <td>{% trans %}Mark as old{% endtrans %}</td>
+            {% endif %}
+          </tr>
+        </thead>
+        <tbody>
+          {% for m in members %}
+            <tr>
+              <td>{{ user_profile_link(m.user) }}</td>
+              <td>{{ settings.SITH_CLUB_ROLES[m.role] }}</td>
+              <td>{{ m.description }}</td>
+              <td>{{ m.start_date }}</td>
+              {% if users_old %}
+                <td>
+                  {% set user_old = users_old[m.user.get_display_name()] %}
+                  {% if user_old %}
+                    {{ user_old[0].tag() }}
+                  {% endif %}
+                </td>
+              {% endif %}
+            </tr>
+          {% endfor %}
+        </tbody>
+      </table>
+      {{ form.users_old.errors }}
+      {% if users_old %}
+        <p></p>
+        <input type="submit" name="submit" value="{% trans %}Mark as old{% endtrans %}">
+      {% endif %}
     </form>
-    {% else %}
+  {% else %}
     <p>{% trans %}There are no members in this club.{% endtrans %}</p>
+  {% endif %}
+  <form action="{{ url('club:club_members', club_id=club.id) }}" id="add_users" method="post">
+    {% csrf_token %}
+    {{ form.non_field_errors() }}
+    <p>
+      {{ form.users.errors }}
+      <label for="{{ form.users.id_for_label }}">{{ form.users.label }} :</label>
+      {{ form.users }}
+      <span class="helptext">{{ form.users.help_text }}</span>
+    </p>
+    <p>
+      {{ form.role.errors }}
+      <label for="{{ form.role.id_for_label }}">{{ form.role.label }} :</label>
+      {{ form.role }}
+    </p>
+    {% if form.start_date %}
+      <p>
+        {{ form.start_date.errors }}
+        <label for="{{ form.start_date.id_for_label }}">{{ form.start_date.label }} :</label>
+        {{ form.start_date }}
+      </p>
     {% endif %}
-    <form action="{{ url('club:club_members', club_id=club.id) }}" id="add_users" method="post">
-        {% csrf_token %}
-        {{ form.non_field_errors() }}
-        <p>
-            {{ form.users.errors }}
-            <label for="{{ form.users.id_for_label }}">{{ form.users.label }} :</label>
-            {{ form.users }}
-            <span class="helptext">{{ form.users.help_text }}</span>
-        </p>
-        <p>
-            {{ form.role.errors }}
-            <label for="{{ form.role.id_for_label }}">{{ form.role.label }} :</label>
-            {{ form.role }}
-        </p>
-        {% if form.start_date %}
-        <p>
-            {{ form.start_date.errors }}
-            <label for="{{ form.start_date.id_for_label }}">{{ form.start_date.label }} :</label>
-            {{ form.start_date }}
-        </p>
-        {% endif %}
-        <p>
-            {{ form.description.errors }}
-            <label for="{{ form.description.id_for_label }}">{{ form.description.label }} :</label>
-            {{ form.description }}
-        </p>
-        <p><input type="submit" value="{% trans %}Add{% endtrans %}" /></p>
-    </form>
+    <p>
+      {{ form.description.errors }}
+      <label for="{{ form.description.id_for_label }}">{{ form.description.label }} :</label>
+      {{ form.description }}
+    </p>
+    <p><input type="submit" value="{% trans %}Add{% endtrans %}" /></p>
+  </form>
 {% endblock %}

club/templates/club/club_old_members.jinja

@@ -2,27 +2,27 @@
 {% from 'core/macros.jinja' import user_profile_link %}
 
 {% block content %}
-    <h2>{% trans %}Club old members{% endtrans %}</h2>
-    <table>
-        <thead>
-            <td>{% trans %}User{% endtrans %}</td>
-            <td>{% trans %}Role{% endtrans %}</td>
-            <td>{% trans %}Description{% endtrans %}</td>
-            <td>{% trans %}From{% endtrans %}</td>
-            <td>{% trans %}To{% endtrans %}</td>
-        </thead>
-        <tbody>
-        {% for m in club.members.exclude(end_date=None).order_by('-role', 'description', '-end_date').all() %}
-            <tr>
-                <td>{{ user_profile_link(m.user) }}</td>
-                <td>{{ settings.SITH_CLUB_ROLES[m.role] }}</td>
-                <td>{{ m.description }}</td>
-                <td>{{ m.start_date }}</td>
-                <td>{{ m.end_date }}</td>
-            </tr>
-        {% endfor %}
-        </tbody>
-    </table>
+  <h2>{% trans %}Club old members{% endtrans %}</h2>
+  <table>
+    <thead>
+      <td>{% trans %}User{% endtrans %}</td>
+      <td>{% trans %}Role{% endtrans %}</td>
+      <td>{% trans %}Description{% endtrans %}</td>
+      <td>{% trans %}From{% endtrans %}</td>
+      <td>{% trans %}To{% endtrans %}</td>
+    </thead>
+    <tbody>
+      {% for m in club.members.exclude(end_date=None).order_by('-role', 'description', '-end_date').all() %}
+        <tr>
+          <td>{{ user_profile_link(m.user) }}</td>
+          <td>{{ settings.SITH_CLUB_ROLES[m.role] }}</td>
+          <td>{{ m.description }}</td>
+          <td>{{ m.start_date }}</td>
+          <td>{{ m.end_date }}</td>
+        </tr>
+      {% endfor %}
+    </tbody>
+  </table>
 {% endblock %}
 
 

club/templates/club/club_sellings.jinja

@@ -1,66 +1,94 @@
 {% extends "core/base.jinja" %}
-{% from 'core/macros.jinja' import user_profile_link, paginate %}
+{% from 'core/macros.jinja' import user_profile_link %}
+
+{# This page uses a custom macro instead of the core `paginate_jinja` and `paginate_alpine`
+because it works with a somewhat dynamic form,
+but was written before Alpine was introduced in the project.
+TODO : rewrite the pagination used in this template an Alpine one
+#}
+{% macro paginate(page_obj, paginator, js_action) %}
+  {% set js = js_action|default('') %}
+  {% if page_obj.has_previous() or page_obj.has_next() %}
+    {% if page_obj.has_previous() %}
+      <a {% if js %} type="submit" onclick="{{ js }}" {% endif %} href="?page={{ page_obj.previous_page_number() }}">{% trans %}Previous{% endtrans %}</a>
+    {% else %}
+      <span class="disabled">{% trans %}Previous{% endtrans %}</span>
+    {% endif %}
+    {% for i in paginator.page_range %}
+      {% if page_obj.number == i %}
+        <span class="active">{{ i }} <span class="sr-only">({% trans %}current{% endtrans %})</span></span>
+      {% else %}
+        <a {% if js %} type="submit" onclick="{{ js }}" {% endif %} href="?page={{ i }}">{{ i }}</a>
+      {% endif %}
+    {% endfor %}
+    {% if page_obj.has_next() %}
+      <a {% if js %} type="submit" onclick="{{ js }}" {% endif %} href="?page={{ page_obj.next_page_number() }}">{% trans %}Next{% endtrans %}</a>
+    {% else %}
+      <span class="disabled">{% trans %}Next{% endtrans %}</span>
+    {% endif %}
+  {% endif %}
+{% endmacro %}
 
 {% block content %}
-<h3>{% trans %}Sales{% endtrans %}</h3>
-<form id="form" action="?page=1" method="post">
+  <h3>{% trans %}Sales{% endtrans %}</h3>
+  <form id="form" action="?page=1" method="post">
     {% csrf_token %}
     {{ form }}
     <p><input type="submit" value="{% trans %}Show{% endtrans %}" /></p>
     <p><input type="submit" value="{% trans %}Download as cvs{% endtrans %}" formaction="{{ url('club:sellings_csv', club_id=object.id) }}"/></p>
-</form>
-<p>
-{% trans %}Quantity: {% endtrans %}{{ total_quantity }} {% trans %}units{% endtrans %}<br/>
-{% trans %}Total: {% endtrans %}{{ total }} €<br/>
-{% trans %}Benefit: {% endtrans %}{{ benefit }} €
-</p>
-<table>
+  </form>
+  <p>
+    {% trans %}Quantity: {% endtrans %}{{ total_quantity }} {% trans %}units{% endtrans %}<br/>
+    {% trans %}Total: {% endtrans %}{{ total }} €<br/>
+    {% trans %}Benefit: {% endtrans %}{{ benefit }} €
+  </p>
+  <table>
     <thead>
-        <tr>
-            <td>{% trans %}Date{% endtrans %}</td>
-            <td>{% trans %}Counter{% endtrans %}</td>
-            <td>{% trans %}Barman{% endtrans %}</td>
-            <td>{% trans %}Customer{% endtrans %}</td>
-            <td>{% trans %}Label{% endtrans %}</td>
-            <td>{% trans %}Quantity{% endtrans %}</td>
-            <td>{% trans %}Total{% endtrans %}</td>
-            <td>{% trans %}Payment method{% endtrans %}</td>
-        </tr>
+      <tr>
+        <td>{% trans %}Date{% endtrans %}</td>
+        <td>{% trans %}Counter{% endtrans %}</td>
+        <td>{% trans %}Barman{% endtrans %}</td>
+        <td>{% trans %}Customer{% endtrans %}</td>
+        <td>{% trans %}Label{% endtrans %}</td>
+        <td>{% trans %}Quantity{% endtrans %}</td>
+        <td>{% trans %}Total{% endtrans %}</td>
+        <td>{% trans %}Payment method{% endtrans %}</td>
+      </tr>
     </thead>
     <tbody>
-    {% for s in paginated_result %}
+      {% for s in paginated_result %}
         <tr>
-            <td>{{ s.date|localtime|date(DATETIME_FORMAT) }} {{ s.date|localtime|time(DATETIME_FORMAT) }}</td>
-            <td>{{ s.counter }}</td>
-            {% if s.seller %}
+          <td>{{ s.date|localtime|date(DATETIME_FORMAT) }} {{ s.date|localtime|time(DATETIME_FORMAT) }}</td>
+          <td>{{ s.counter }}</td>
+          {% if s.seller %}
             <td><a href="{{ s.seller.get_absolute_url() }}">{{ s.seller.get_display_name() }}</a></td>
-            {% else %}
+          {% else %}
             <td></td>
-            {% endif %}
-            {% if s.customer %}
+          {% endif %}
+          {% if s.customer %}
             <td><a href="{{ s.customer.user.get_absolute_url() }}">{{ s.customer.user.get_display_name() }}</a></td>
-            {% else %}
+          {% else %}
             <td></td>
-            {% endif %}
-            <td>{{ s.label }}</td>
-            <td>{{ s.quantity }}</td>
-            <td>{{ s.quantity * s.unit_price }} €</td>
-            <td>{{ s.get_payment_method_display() }}</td>
-            {% if s.is_owned_by(user) %}
-                <td><a href="{{ url('counter:selling_delete', selling_id=s.id) }}">{% trans %}Delete{% endtrans %}</a></td>
-            {% endif %}
+          {% endif %}
+          <td>{{ s.label }}</td>
+          <td>{{ s.quantity }}</td>
+          <td>{{ s.quantity * s.unit_price }} €</td>
+          <td>{{ s.get_payment_method_display() }}</td>
+          {% if s.is_owned_by(user) %}
+            <td><a href="{{ url('counter:selling_delete', selling_id=s.id) }}">{% trans %}Delete{% endtrans %}</a></td>
+          {% endif %}
         </tr>
-    {% endfor %}
+      {% endfor %}
     </tbody>
-</table>
-<script type="text/javascript">
+  </table>
+  <script type="text/javascript">
     function formPagination(link){
-        $("form").attr("action", link.href);
-        link.href = "javascript:void(0)"; // block link action
-        $("form").submit();
+      $("form").attr("action", link.href);
+      link.href = "javascript:void(0)"; // block link action
+      $("form").submit();
     }
-</script>
-{{ paginate(paginated_result, paginator, "formPagination(this)") }}
+  </script>
+  {{ paginate(paginated_result, paginator, "formPagination(this)") }}
 {% endblock %}
 
 

club/templates/club/club_tools.jinja

@@ -1,46 +1,46 @@
 {% extends "core/base.jinja" %}
 
 {% block content %}
-<h3>{% trans %}Club tools{% endtrans %}</h3>
-<div>
+  <h3>{% trans %}Club tools{% endtrans %}</h3>
+  <div>
     <h4>{% trans %}Communication:{% endtrans %}</h4>
     <ul>
-        <li> <a href="{{ url('com:news_new') }}?club={{ object.id }}">{% trans %}Create a news{% endtrans %}</a></li>
-        <li> <a href="{{ url('com:weekmail_article') }}?club={{ object.id }}">{% trans %}Post in the Weekmail{% endtrans %}</a></li>
-        {% if object.trombi %}
+      <li> <a href="{{ url('com:news_new') }}?club={{ object.id }}">{% trans %}Create a news{% endtrans %}</a></li>
+      <li> <a href="{{ url('com:weekmail_article') }}?club={{ object.id }}">{% trans %}Post in the Weekmail{% endtrans %}</a></li>
+      {% if object.trombi %}
         <li> <a href="{{ url('trombi:detail', trombi_id=object.trombi.id) }}">{% trans %}Edit Trombi{% endtrans %}</a></li>
-        {% else %}
+      {% else %}
         <li> <a href="{{ url('trombi:create', club_id=object.id) }}">{% trans %}New Trombi{% endtrans %}</a></li>
         <li> <a href="{{ url('club:poster_list', club_id=object.id) }}">{% trans %}Posters{% endtrans %}</a></li>
-        {% endif %}
+      {% endif %}
     </ul>
     <h4>{% trans %}Counters:{% endtrans %}</h4>
     <ul>
-    {% if object.unix_name == settings.SITH_LAUNDERETTE_MANAGER['unix_name'] %}
-    {% for l in Launderette.objects.all() %}
-        <li><a href="{{ url('launderette:main_click', launderette_id=l.id) }}">{{ l }}</a></li>
-    {% endfor %}
-    {% elif object.counters.filter(type="OFFICE")|count > 0 %}
+      {% if object.unix_name == settings.SITH_LAUNDERETTE_MANAGER['unix_name'] %}
+        {% for l in Launderette.objects.all() %}
+          <li><a href="{{ url('launderette:main_click', launderette_id=l.id) }}">{{ l }}</a></li>
+        {% endfor %}
+      {% elif object.counters.filter(type="OFFICE")|count > 0 %}
         {% for c in object.counters.filter(type="OFFICE") %}
-        <li>{{ c }}:
+          <li>{{ c }}:
             <a href="{{ url('counter:details', counter_id=c.id) }}">View</a>
             <a href="{{ url('counter:admin', counter_id=c.id) }}">Edit</a>
-        </li>
+          </li>
         {% endfor %}
-    {% endif %}
+      {% endif %}
     </ul>
     {% if object.club_account.exists() %}
-        <h4>{% trans %}Accounting: {% endtrans %}</h4>
-        <ul>
+      <h4>{% trans %}Accounting: {% endtrans %}</h4>
+      <ul>
         {% for ca in object.club_account.all() %}
-            <li><a href="{{ url('accounting:club_details', c_account_id=ca.id) }}">{{ ca.get_display_name() }}</a></li>
+          <li><a href="{{ url('accounting:club_details', c_account_id=ca.id) }}">{{ ca.get_display_name() }}</a></li>
         {% endfor %}
-        </ul>
+      </ul>
     {% endif %}
     {% if object.unix_name == settings.SITH_LAUNDERETTE_MANAGER['unix_name'] %}
-    <li><a href="{{ url('launderette:launderette_list') }}">{% trans %}Manage launderettes{% endtrans %}</a></li>
+      <li><a href="{{ url('launderette:launderette_list') }}">{% trans %}Manage launderettes{% endtrans %}</a></li>
     {% endif %}
-</div>
+  </div>
 {% endblock %}
 
 

club/templates/club/mailing.jinja

@@ -2,107 +2,107 @@
 {% from 'core/macros.jinja' import select_all_checkbox %}
 
 {% block title %}
-{% trans %}Mailing lists{% endtrans %}
+  {% trans %}Mailing lists{% endtrans %}
 {% endblock %}
 
 {% block content %}
 
-    <b>{% trans %}Remember : mailing lists need to be moderated, if your new created list is not shown wait until moderation takes action{% endtrans %}</b>
+  <b>{% trans %}Remember : mailing lists need to be moderated, if your new created list is not shown wait until moderation takes action{% endtrans %}</b>
 
-    {% if mailings_not_moderated %}
-        <p>{% trans %}Mailing lists waiting for moderation{% endtrans %}</p>
-        <ul>
-            {% for mailing in mailings_not_moderated %}
-                <li>{{ mailing.email_full }}<a href="{{ url('club:mailing_delete', mailing_id=mailing.id) }}"> - {% trans %}Delete{% endtrans %}</a></li>
-            {% endfor %}
-        </ul>
-    {% endif %}
+  {% if mailings_not_moderated %}
+    <p>{% trans %}Mailing lists waiting for moderation{% endtrans %}</p>
+    <ul>
+      {% for mailing in mailings_not_moderated %}
+        <li>{{ mailing.email_full }}<a href="{{ url('club:mailing_delete', mailing_id=mailing.id) }}"> - {% trans %}Delete{% endtrans %}</a></li>
+      {% endfor %}
+    </ul>
+  {% endif %}
 
-    {% if mailings_moderated %}
+  {% if mailings_moderated %}
     {% for mailing in mailings_moderated %}
-    <h2>{% trans %}Mailing{% endtrans %} {{ mailing.email_full }}
+      <h2>{% trans %}Mailing{% endtrans %} {{ mailing.email_full }}
         {%- if user.is_owner(mailing) -%}
-        <a href="{{ url('club:mailing_delete', mailing_id=mailing.id) }}"> - {% trans %}Delete{% endtrans %}</a>
+          <a href="{{ url('club:mailing_delete', mailing_id=mailing.id) }}"> - {% trans %}Delete{% endtrans %}</a>
         {%- endif -%}
-    </h2>
-    <form method="GET" action="{{ url('club:mailing_generate', mailing_id=mailing.id) }}" style="display:inline-block;">
+      </h2>
+      <form method="GET" action="{{ url('club:mailing_generate', mailing_id=mailing.id) }}" style="display:inline-block;">
         <input type="submit" name="generateMalingList" value="{% trans %}Generate mailing list{% endtrans %}">
-    </form>
-    {% set form_mailing_removal = form["removal_" + mailing.id|string] %}
-    {% if form_mailing_removal.field.choices %}
-    {% set ms = dict(mailing.subscriptions.all() | groupby('id')) %}
-    <form action="{{ url('club:mailing', club_id=club.id) }}" id="{{ form_mailing_removal.auto_id }}" method="post" enctype="multipart/form-data">
-        <p style="margin-bottom: 1em;">{{ select_all_checkbox(form_mailing_removal.auto_id) }}</p>
-        {% csrf_token %}
-        <input hidden type="number" name="{{ form.action.name }}" value="{{ form_actions.REMOVE_SUBSCRIPTION }}" />
-        <table>
+      </form>
+      {% set form_mailing_removal = form["removal_" + mailing.id|string] %}
+      {% if form_mailing_removal.field.choices %}
+        {% set ms = dict(mailing.subscriptions.all() | groupby('id')) %}
+        <form action="{{ url('club:mailing', club_id=club.id) }}" id="{{ form_mailing_removal.auto_id }}" method="post" enctype="multipart/form-data">
+          <p style="margin-bottom: 1em;">{{ select_all_checkbox(form_mailing_removal.auto_id) }}</p>
+          {% csrf_token %}
+          <input hidden type="number" name="{{ form.action.name }}" value="{{ form_actions.REMOVE_SUBSCRIPTION }}" />
+          <table>
             <thead>
-                <tr>
-                    <td>{% trans %}User{% endtrans %}</td>
-                    <td>{% trans %}Email{% endtrans %}</td>
-                    <td>{% trans %}Delete{% endtrans %}</td>
-                </tr>
+              <tr>
+                <td>{% trans %}User{% endtrans %}</td>
+                <td>{% trans %}Email{% endtrans %}</td>
+                <td>{% trans %}Delete{% endtrans %}</td>
+              </tr>
             </thead>
             <tbody>
-                {% for widget in form_mailing_removal.subwidgets %}
+              {% for widget in form_mailing_removal.subwidgets %}
                 {% set user = ms[widget.data.value.value][0] %}
                 <tr>
-                    <td>{{ user.get_username }}</td>
-                    <td>{{ user.get_email }}</td>
-                    <td>{{ widget.tag() }}</td>
+                  <td>{{ user.get_username }}</td>
+                  <td>{{ user.get_email }}</td>
+                  <td>{{ widget.tag() }}</td>
                 </tr>
-                {% endfor %}
+              {% endfor %}
             </tbody>
-        </table>
-        {{ form_mailing_removal.errors }}
-        <p><input type="submit" value="{% trans %}Remove from mailing list{% endtrans %}" /></p>
-    </form>
+          </table>
+          {{ form_mailing_removal.errors }}
+          <p><input type="submit" value="{% trans %}Remove from mailing list{% endtrans %}" /></p>
+        </form>
 
-    {% else %}
+      {% else %}
         <p><b>{% trans %}There is no subscriber for this mailing list{% endtrans %}</b></p>
-    {% endif %}
+      {% endif %}
     {% endfor %}
 
-    {% else %}
+  {% else %}
     <p>{% trans %}No mailing list existing for this club{% endtrans %}</p>
-    {% endif %}
+  {% endif %}
 
-    <p>{{ form.non_field_errors() }}</p>
-    {% if mailings_moderated %}
-        <h2>{% trans %}New member{% endtrans %}</h2>
-        <form action="{{ url('club:mailing', club_id=club.id) }}" method="post" enctype="multipart/form-data">
-            {% csrf_token %}
-            <p>
-                {{ form.subscription_mailing.errors }}
-                <label for="{{ form.subscription_mailing.id_for_label }}">{{ form.subscription_mailing.label }}</label>
-                {{ form.subscription_mailing }}
-            </p>
-            <p>
-                {{ form.subscription_users.errors }}
-                <label for="{{ form.subscription_users.id_for_label }}">{{ form.subscription_users.label }}</label>
-                {{ form.subscription_users }}
-                <span class="helptext">{{ form.subscription_users.help_text }}</span>
-            </p>
-            <p>
-                {{ form.subscription_email.errors }}
-                <label for="{{ form.subscription_email.id_for_label }}">{{ form.subscription_email.label }}</label>
-                {{ form.subscription_email }}
-            </p>
-            <input hidden type="number" name="{{ form.action.name }}" value="{{ form_actions.NEW_SUBSCRIPTION }}" />
-            <p><input type="submit" value="{% trans %}Add to mailing list{% endtrans %}" /></p>
-        </form>
-    {% endif %}
-
-    <h2>{% trans %}New mailing{% endtrans %}</h2>
+  <p>{{ form.non_field_errors() }}</p>
+  {% if mailings_moderated %}
+    <h2>{% trans %}New member{% endtrans %}</h2>
     <form action="{{ url('club:mailing', club_id=club.id) }}" method="post" enctype="multipart/form-data">
-        {% csrf_token %}
-        <p>
-            {{ form.mailing_email.errors }}
-            <label for="{{ form.mailing_email.id_for_label }}">{{ form.mailing_email.label }}</label>
-            {{ form.mailing_email }}
-        </p>
-        <input hidden type="number" name="{{ form.action.name }}" value="{{ form_actions.NEW_MALING }}" />
-        <p><input type="submit" value="{% trans %}Create mailing list{% endtrans %}" /></p>
+      {% csrf_token %}
+      <p>
+        {{ form.subscription_mailing.errors }}
+        <label for="{{ form.subscription_mailing.id_for_label }}">{{ form.subscription_mailing.label }}</label>
+        {{ form.subscription_mailing }}
+      </p>
+      <p>
+        {{ form.subscription_users.errors }}
+        <label for="{{ form.subscription_users.id_for_label }}">{{ form.subscription_users.label }}</label>
+        {{ form.subscription_users }}
+        <span class="helptext">{{ form.subscription_users.help_text }}</span>
+      </p>
+      <p>
+        {{ form.subscription_email.errors }}
+        <label for="{{ form.subscription_email.id_for_label }}">{{ form.subscription_email.label }}</label>
+        {{ form.subscription_email }}
+      </p>
+      <input hidden type="number" name="{{ form.action.name }}" value="{{ form_actions.NEW_SUBSCRIPTION }}" />
+      <p><input type="submit" value="{% trans %}Add to mailing list{% endtrans %}" /></p>
     </form>
+  {% endif %}
+
+  <h2>{% trans %}New mailing{% endtrans %}</h2>
+  <form action="{{ url('club:mailing', club_id=club.id) }}" method="post" enctype="multipart/form-data">
+    {% csrf_token %}
+    <p>
+      {{ form.mailing_email.errors }}
+      <label for="{{ form.mailing_email.id_for_label }}">{{ form.mailing_email.label }}</label>
+      {{ form.mailing_email }}
+    </p>
+    <input hidden type="number" name="{{ form.action.name }}" value="{{ form_actions.NEW_MALING }}" />
+    <p><input type="submit" value="{% trans %}Create mailing list{% endtrans %}" /></p>
+  </form>
 
 {% endblock %}

club/templates/club/page_history.jinja

@@ -2,11 +2,11 @@
 {% from 'core/macros_pages.jinja' import page_history %}
 
 {% block content %}
-	{% if club.page %}
-	{{ page_history(club.page) }}
-	{% else %}
-	    {% trans %}No page existing for this club{% endtrans %}
-	{% endif %}
+  {% if club.page %}
+    {{ page_history(club.page) }}
+  {% else %}
+    {% trans %}No page existing for this club{% endtrans %}
+  {% endif %}
 {% endblock %}
 
 

club/templates/club/pagerev_edit.jinja

@@ -2,7 +2,7 @@
 {% from 'core/macros_pages.jinja' import page_edit_form %}
 
 {% block content %}
-{{ page_edit_form(page, form, url('club:club_edit_page', club_id=page.club.id), csrf_token) }}
+  {{ page_edit_form(page, form, url('club:club_edit_page', club_id=page.club.id), csrf_token) }}
 {% endblock %}
 
 

club/templates/club/stats.jinja

@@ -1,48 +1,48 @@
 {% extends "core/base.jinja" %}
 
 {% block title %}
-{% trans %}Club stats{% endtrans %}
+  {% trans %}Club stats{% endtrans %}
 {% endblock %}
 
 {% block content %}
-    {% if club_list %}
+  {% if club_list %}
     <h3>{% trans %}Club stats{% endtrans %}</h3>
     <form action="" method="GET">
-        {% csrf_token %}
-        <p>
+      {% csrf_token %}
+      <p>
         <select name="branch">
-            {% for b in settings.SITH_PROFILE_DEPARTMENTS %}
-                <option value="{{ b[0] }}">{{ b[0] }}</option>
-            {% endfor %}
+          {% for b in settings.SITH_PROFILE_DEPARTMENTS %}
+            <option value="{{ b[0] }}">{{ b[0] }}</option>
+          {% endfor %}
         </select>
-        </p>
-        <p><input type="submit" value="{% trans %}Show{% endtrans %}" /></p>
+      </p>
+      <p><input type="submit" value="{% trans %}Show{% endtrans %}" /></p>
     </form>
     <table>
-        <thead>
-            <tr>
-                <td>Club</td>
-                <td>Member number</td>
-                <td>Old member number</td>
-            </tr>
-        </thead>
-        <tbody>
+      <thead>
+        <tr>
+          <td>Club</td>
+          <td>Member number</td>
+          <td>Old member number</td>
+        </tr>
+      </thead>
+      <tbody>
         {% for c in club_list.order_by('id') %}
-            {% set members = c.members.all() %}
-            {% if request.GET['branch'] %}
+          {% set members = c.members.all() %}
+          {% if request.GET['branch'] %}
             {% set members = members.filter(user__department=request.GET['branch']) %}
-            {% endif %}
-            <tr>
-                <td>{{ c.get_display_name() }}</td>
-                <td>{{ members.filter(end_date=None, role__gt=settings.SITH_MAXIMUM_FREE_ROLE).count() }}</td>
-                <td>{{ members.exclude(end_date=None, role__gt=settings.SITH_MAXIMUM_FREE_ROLE).count() }}</td>
-            </tr>
+          {% endif %}
+          <tr>
+            <td>{{ c.get_display_name() }}</td>
+            <td>{{ members.filter(end_date=None, role__gt=settings.SITH_MAXIMUM_FREE_ROLE).count() }}</td>
+            <td>{{ members.exclude(end_date=None, role__gt=settings.SITH_MAXIMUM_FREE_ROLE).count() }}</td>
+          </tr>
         {% endfor %}
-        </tbody>
+      </tbody>
     </table>
-    {% else %}
+  {% else %}
     {% trans %}There is no club in this website.{% endtrans %}
-    {% endif %}
+  {% endif %}
 {% endblock %}
 
 

club/tests.py

@@ -1,21 +1,16 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 from datetime import timedelta
 
@@ -24,18 +19,19 @@ from django.core.cache import cache
 from django.test import TestCase
 from django.urls import reverse
 from django.utils import timezone
-from django.utils.timezone import localtime, now
+from django.utils.timezone import localdate, localtime, now
 from django.utils.translation import gettext as _
 
 from club.forms import MailingForm
 from club.models import Club, Mailing, Membership
+from core.baker_recipes import subscriber_user
 from core.models import AnonymousUser, User
 from sith.settings import SITH_BAR_MANAGER, SITH_MAIN_CLUB_ID
 
 
-class ClubTest(TestCase):
-    """
-    Set up data for test cases related to clubs and membership
+class TestClub(TestCase):
+    """Set up data for test cases related to clubs and membership.
+
     The generated dataset is the one created by the populate command,
     plus the following modifications :
 
@@ -50,6 +46,8 @@ class ClubTest(TestCase):
     def setUpTestData(cls):
         # subscribed users - initial members
         cls.skia = User.objects.get(username="skia")
+        # by default, Skia is in the AE, which creates side effect
+        cls.skia.memberships.all().delete()
         cls.richard = User.objects.get(username="rbatsbak")
         cls.comptable = User.objects.get(username="comptable")
         cls.sli = User.objects.get(username="sli")
@@ -66,45 +64,38 @@ class ClubTest(TestCase):
         cls.public = User.objects.get(username="public")
 
         cls.ae = Club.objects.filter(pk=SITH_MAIN_CLUB_ID)[0]
-
-    def setUp(self):
-        # by default, Skia is in the AE, which creates side effect
-        self.skia.memberships.all().delete()
-
-        # create a fake club
-        self.club = Club.objects.create(
+        cls.club = Club.objects.create(
             name="Fake Club",
             unix_name="fake-club",
             address="5 rue de la République, 90000 Belfort",
         )
-        self.members_url = reverse(
-            "club:club_members", kwargs={"club_id": self.club.id}
-        )
+        cls.members_url = reverse("club:club_members", kwargs={"club_id": cls.club.id})
         a_month_ago = now() - timedelta(days=30)
         yesterday = now() - timedelta(days=1)
         Membership.objects.create(
-            club=self.club, user=self.skia, start_date=a_month_ago, role=3
+            club=cls.club, user=cls.skia, start_date=a_month_ago, role=3
         )
-        Membership.objects.create(club=self.club, user=self.richard, role=1)
+        Membership.objects.create(club=cls.club, user=cls.richard, role=1)
         Membership.objects.create(
-            club=self.club, user=self.comptable, start_date=a_month_ago, role=10
+            club=cls.club, user=cls.comptable, start_date=a_month_ago, role=10
         )
 
         # sli was a member but isn't anymore
         Membership.objects.create(
-            club=self.club,
-            user=self.sli,
+            club=cls.club,
+            user=cls.sli,
             start_date=a_month_ago,
             end_date=yesterday,
             role=2,
         )
+
+    def setUp(self):
         cache.clear()
 
 
-class MembershipQuerySetTest(ClubTest):
+class TestMembershipQuerySet(TestClub):
     def test_ongoing(self):
-        """
-        Test that the ongoing queryset method returns the memberships that
+        """Test that the ongoing queryset method returns the memberships that
         are not ended.
         """
         current_members = list(self.club.members.ongoing().order_by("id"))
@@ -116,10 +107,21 @@ class MembershipQuerySetTest(ClubTest):
         expected.sort(key=lambda i: i.id)
         assert current_members == expected
 
+    def test_ongoing_with_membership_ending_today(self):
+        """Test that a membership ending the present day is considered as ended."""
+        today = localdate()
+        self.richard.memberships.filter(club=self.club).update(end_date=today)
+        current_members = list(self.club.members.ongoing().order_by("id"))
+        expected = [
+            self.skia.memberships.get(club=self.club),
+            self.comptable.memberships.get(club=self.club),
+        ]
+        expected.sort(key=lambda i: i.id)
+        assert current_members == expected
+
     def test_board(self):
-        """
-        Test that the board queryset method returns the memberships
-        of user in the club board
+        """Test that the board queryset method returns the memberships
+        of user in the club board.
         """
         board_members = list(self.club.members.board().order_by("id"))
         expected = [
@@ -132,9 +134,8 @@ class MembershipQuerySetTest(ClubTest):
         assert board_members == expected
 
     def test_ongoing_board(self):
-        """
-        Test that combining ongoing and board returns users
-        who are currently board members of the club
+        """Test that combining ongoing and board returns users
+        who are currently board members of the club.
         """
         members = list(self.club.members.ongoing().board().order_by("id"))
         expected = [
@@ -145,9 +146,7 @@ class MembershipQuerySetTest(ClubTest):
         assert members == expected
 
     def test_update_invalidate_cache(self):
-        """
-        Test that the `update` queryset method properly invalidate cache
-        """
+        """Test that the `update` queryset method properly invalidate cache."""
         mem_skia = self.skia.memberships.get(club=self.club)
         cache.set(f"membership_{mem_skia.club_id}_{mem_skia.user_id}", mem_skia)
         self.skia.memberships.update(end_date=localtime(now()).date())
@@ -166,10 +165,7 @@ class MembershipQuerySetTest(ClubTest):
         assert new_mem.role == 5
 
     def test_delete_invalidate_cache(self):
-        """
-        Test that the `delete` queryset properly invalidate cache
-        """
-
+        """Test that the `delete` queryset properly invalidate cache."""
         mem_skia = self.skia.memberships.get(club=self.club)
         mem_comptable = self.comptable.memberships.get(club=self.club)
         cache.set(f"membership_{mem_skia.club_id}_{mem_skia.user_id}", mem_skia)
@@ -180,21 +176,16 @@ class MembershipQuerySetTest(ClubTest):
         # should delete the subscriptions of skia and comptable
         self.club.members.ongoing().board().delete()
 
-        assert (
-            cache.get(f"membership_{mem_skia.club_id}_{mem_skia.user_id}")
-            == "not_member"
-        )
-        assert (
-            cache.get(f"membership_{mem_comptable.club_id}_{mem_comptable.user_id}")
-            == "not_member",
-        )
+        for membership in (mem_skia, mem_comptable):
+            cached_mem = cache.get(
+                f"membership_{membership.club_id}_{membership.user_id}"
+            )
+            assert cached_mem == "not_member"
 
 
-class ClubModelTest(ClubTest):
+class TestClubModel(TestClub):
     def assert_membership_started_today(self, user: User, role: int):
-        """
-        Assert that the given membership is active and started today
-        """
+        """Assert that the given membership is active and started today."""
         membership = user.memberships.ongoing().filter(club=self.club).first()
         assert membership is not None
         assert localtime(now()).date() == membership.start_date
@@ -207,17 +198,14 @@ class ClubModelTest(ClubTest):
         assert user.is_in_group(name=board_group)
 
     def assert_membership_ended_today(self, user: User):
-        """
-        Assert that the given user have a membership which ended today
-        """
+        """Assert that the given user have a membership which ended today."""
         today = localtime(now()).date()
         assert user.memberships.filter(club=self.club, end_date=today).exists()
         assert self.club.get_membership_for(user) is None
 
     def test_access_unauthorized(self):
-        """
-        Test that users who never subscribed and anonymous users
-        cannot see the page
+        """Test that users who never subscribed and anonymous users
+        cannot see the page.
         """
         response = self.client.post(self.members_url)
         assert response.status_code == 403
@@ -227,8 +215,7 @@ class ClubModelTest(ClubTest):
         assert response.status_code == 403
 
     def test_display(self):
-        """
-        Test that a GET request return a page where the requested
+        """Test that a GET request return a page where the requested
         information are displayed.
         """
         self.client.force_login(self.skia)
@@ -263,27 +250,23 @@ class ClubModelTest(ClubTest):
         self.assertInHTML(expected_html, response.content.decode())
 
     def test_root_add_one_club_member(self):
-        """
-        Test that root users can add members to clubs, one at a time
-        """
+        """Test that root users can add members to clubs, one at a time."""
         self.client.force_login(self.root)
         response = self.client.post(
             self.members_url,
-            {"users": self.subscriber.id, "role": 3},
+            {"users": [self.subscriber.id], "role": 3},
         )
         self.assertRedirects(response, self.members_url)
         self.subscriber.refresh_from_db()
         self.assert_membership_started_today(self.subscriber, role=3)
 
     def test_root_add_multiple_club_member(self):
-        """
-        Test that root users can add multiple members at once to clubs
-        """
+        """Test that root users can add multiple members at once to clubs."""
         self.client.force_login(self.root)
         response = self.client.post(
             self.members_url,
             {
-                "users": f"|{self.subscriber.id}|{self.krophil.id}|",
+                "users": (self.subscriber.id, self.krophil.id),
                 "role": 3,
             },
         )
@@ -293,8 +276,7 @@ class ClubModelTest(ClubTest):
         self.assert_membership_started_today(self.krophil, role=3)
 
     def test_add_unauthorized_members(self):
-        """
-        Test that users who are not currently subscribed
+        """Test that users who are not currently subscribed
         cannot be members of clubs.
         """
         self.client.force_login(self.root)
@@ -314,9 +296,8 @@ class ClubModelTest(ClubTest):
         assert '<ul class="errorlist"><li>' in response.content.decode()
 
     def test_add_members_already_members(self):
-        """
-        Test that users who are already members of a club
-        cannot be added again to this club
+        """Test that users who are already members of a club
+        cannot be added again to this club.
         """
         self.client.force_login(self.root)
         current_membership = self.skia.memberships.ongoing().get(club=self.club)
@@ -332,8 +313,7 @@ class ClubModelTest(ClubTest):
         assert self.club.get_membership_for(self.skia) == new_membership
 
     def test_add_not_existing_users(self):
-        """
-        Test that not existing users cannot be added in clubs.
+        """Test that not existing users cannot be added in clubs.
         If one user in the request is invalid, no membership creation at all
         can take place.
         """
@@ -350,7 +330,7 @@ class ClubModelTest(ClubTest):
         response = self.client.post(
             self.members_url,
             {
-                "users": f"|{self.subscriber.id}|{9999}|",
+                "users": (self.subscriber.id, 9999),
                 "start_date": "12/06/2016",
                 "role": 3,
             },
@@ -361,9 +341,7 @@ class ClubModelTest(ClubTest):
         assert self.club.members.count() == nb_memberships
 
     def test_president_add_members(self):
-        """
-        Test that the president of the club can add members
-        """
+        """Test that the president of the club can add members."""
         president = self.club.members.get(role=10).user
         nb_club_membership = self.club.members.count()
         nb_subscriber_memberships = self.subscriber.memberships.count()
@@ -380,8 +358,7 @@ class ClubModelTest(ClubTest):
         self.assert_membership_started_today(self.subscriber, role=9)
 
     def test_add_member_greater_role(self):
-        """
-        Test that a member of the club member cannot create
+        """Test that a member of the club member cannot create
         a membership with a greater role than its own.
         """
         self.client.force_login(self.skia)
@@ -400,9 +377,7 @@ class ClubModelTest(ClubTest):
         assert not self.subscriber.memberships.filter(club=self.club).exists()
 
     def test_add_member_without_role(self):
-        """
-        Test that trying to add members without specifying their role fails
-        """
+        """Test that trying to add members without specifying their role fails."""
         self.client.force_login(self.root)
         response = self.client.post(
             self.members_url,
@@ -414,9 +389,7 @@ class ClubModelTest(ClubTest):
         )
 
     def test_end_membership_self(self):
-        """
-        Test that a member can end its own membership
-        """
+        """Test that a member can end its own membership."""
         self.client.force_login(self.skia)
         self.client.post(
             self.members_url,
@@ -426,9 +399,8 @@ class ClubModelTest(ClubTest):
         self.assert_membership_ended_today(self.skia)
 
     def test_end_membership_lower_role(self):
-        """
-        Test that board members of the club can end memberships
-        of users with lower roles
+        """Test that board members of the club can end memberships
+        of users with lower roles.
         """
         # remainder : skia has role 3, comptable has role 10, richard has role 1
         self.client.force_login(self.skia)
@@ -441,9 +413,8 @@ class ClubModelTest(ClubTest):
         self.assert_membership_ended_today(self.richard)
 
     def test_end_membership_higher_role(self):
-        """
-        Test that board members of the club cannot end memberships
-        of users with higher roles
+        """Test that board members of the club cannot end memberships
+        of users with higher roles.
         """
         membership = self.comptable.memberships.filter(club=self.club).first()
         self.client.force_login(self.skia)
@@ -460,16 +431,15 @@ class ClubModelTest(ClubTest):
         assert membership.end_date is None
 
     def test_end_membership_as_main_club_board(self):
-        """
-        Test that board members of the main club can end the membership
-        of anyone
+        """Test that board members of the main club can end the membership
+        of anyone.
         """
         # make subscriber a board member
-        self.subscriber.memberships.all().delete()
-        Membership.objects.create(club=self.ae, user=self.subscriber, role=3)
+        subscriber = subscriber_user.make()
+        Membership.objects.create(club=self.ae, user=subscriber, role=3)
 
-        nb_memberships = self.club.members.count()
-        self.client.force_login(self.subscriber)
+        nb_memberships = self.club.members.ongoing().count()
+        self.client.force_login(subscriber)
         response = self.client.post(
             self.members_url,
             {"users_old": self.comptable.id},
@@ -479,10 +449,8 @@ class ClubModelTest(ClubTest):
         assert self.club.members.ongoing().count() == nb_memberships - 1
 
     def test_end_membership_as_root(self):
-        """
-        Test that root users can end the membership of anyone
-        """
-        nb_memberships = self.club.members.count()
+        """Test that root users can end the membership of anyone."""
+        nb_memberships = self.club.members.ongoing().count()
         self.client.force_login(self.root)
         response = self.client.post(
             self.members_url,
@@ -491,12 +459,9 @@ class ClubModelTest(ClubTest):
         self.assertRedirects(response, self.members_url)
         self.assert_membership_ended_today(self.comptable)
         assert self.club.members.ongoing().count() == nb_memberships - 1
-        assert self.club.members.count() == nb_memberships
 
     def test_end_membership_as_foreigner(self):
-        """
-        Test that users who are not in this club cannot end its memberships
-        """
+        """Test that users who are not in this club cannot end its memberships."""
         nb_memberships = self.club.members.count()
         membership = self.richard.memberships.filter(club=self.club).first()
         self.client.force_login(self.subscriber)
@@ -510,9 +475,8 @@ class ClubModelTest(ClubTest):
         assert membership == new_mem
 
     def test_delete_remove_from_meta_group(self):
-        """
-        Test that when a club is deleted, all its members are removed from the
-        associated metagroup
+        """Test that when a club is deleted, all its members are removed from the
+        associated metagroup.
         """
         memberships = self.club.members.select_related("user")
         users = [membership.user for membership in memberships]
@@ -523,9 +487,7 @@ class ClubModelTest(ClubTest):
             assert not user.is_in_group(name=meta_group)
 
     def test_add_to_meta_group(self):
-        """
-        Test that when a membership begins, the user is added to the meta group
-        """
+        """Test that when a membership begins, the user is added to the meta group."""
         group_members = self.club.unix_name + settings.SITH_MEMBER_SUFFIX
         board_members = self.club.unix_name + settings.SITH_BOARD_SUFFIX
         assert not self.subscriber.is_in_group(name=group_members)
@@ -535,9 +497,7 @@ class ClubModelTest(ClubTest):
         assert self.subscriber.is_in_group(name=board_members)
 
     def test_remove_from_meta_group(self):
-        """
-        Test that when a membership ends, the user is removed from meta group
-        """
+        """Test that when a membership ends, the user is removed from meta group."""
         group_members = self.club.unix_name + settings.SITH_MEMBER_SUFFIX
         board_members = self.club.unix_name + settings.SITH_BOARD_SUFFIX
         assert self.comptable.is_in_group(name=group_members)
@@ -547,9 +507,7 @@ class ClubModelTest(ClubTest):
         assert not self.comptable.is_in_group(name=board_members)
 
     def test_club_owner(self):
-        """
-        Test that a club is owned only by board members of the main club
-        """
+        """Test that a club is owned only by board members of the main club."""
         anonymous = AnonymousUser()
         assert not self.club.is_owned_by(anonymous)
         assert not self.club.is_owned_by(self.subscriber)
@@ -560,8 +518,8 @@ class ClubModelTest(ClubTest):
         assert self.club.is_owned_by(self.sli)
 
 
-class MailingFormTest(TestCase):
-    """Perform validation tests for MailingForm"""
+class TestMailingForm(TestCase):
+    """Perform validation tests for MailingForm."""
 
     @classmethod
     def setUpTestData(cls):
@@ -671,7 +629,7 @@ class MailingFormTest(TestCase):
             self.mail_url,
             {
                 "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
-                "subscription_users": "|%s|%s|" % (self.comunity.id, self.rbatsbak.id),
+                "subscription_users": (self.comunity.id, self.rbatsbak.id),
                 "subscription_mailing": Mailing.objects.get(email="mde").id,
             },
         )
@@ -757,16 +715,17 @@ class MailingFormTest(TestCase):
             self.mail_url,
             {
                 "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
-                "subscription_users": "|789|",
+                "subscription_users": [789],
                 "subscription_mailing": Mailing.objects.get(email="mde").id,
             },
         )
         assert response.status_code == 200
         self.assertInHTML(
-            _("One of the selected users doesn't exist"), response.content.decode()
+            _("You must specify at least an user or an email address"),
+            response.content.decode(),
         )
 
-        # An user has no email adress
+        # An user has no email address
         self.krophil.email = ""
         self.krophil.save()
 
@@ -824,8 +783,11 @@ class MailingFormTest(TestCase):
             self.mail_url,
             {
                 "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
-                "subscription_users": "|%s|%s|%s|"
-                % (self.comunity.id, self.rbatsbak.id, self.krophil.id),
+                "subscription_users": (
+                    self.comunity.id,
+                    self.rbatsbak.id,
+                    self.krophil.id,
+                ),
                 "subscription_mailing": mde.id,
             },
         )
@@ -876,10 +838,8 @@ class MailingFormTest(TestCase):
         assert "krophil@git.an" not in content
 
 
-class ClubSellingViewTest(TestCase):
-    """
-    Perform basics tests to ensure that the page is available
-    """
+class TestClubSellingView(TestCase):
+    """Perform basics tests to ensure that the page is available."""
 
     @classmethod
     def setUpTestData(cls):
@@ -887,9 +847,7 @@ class ClubSellingViewTest(TestCase):
         cls.skia = User.objects.get(username="skia")
 
     def test_page_not_internal_error(self):
-        """
-        Test that the page does not return and internal error
-        """
+        """Test that the page does not return and internal error."""
         self.client.force_login(self.skia)
         response = self.client.get(
             reverse("club:club_sellings", kwargs={"club_id": self.ae.id})

club/urls.py

@@ -1,26 +1,55 @@
-# -*- coding:utf-8 -*-
 #
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
+# Copyright 2016,2017
+# - Skia <skia@libskia.so>
+# - Sli <antoine@bartuccio.fr>
 #
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
+# Ce fichier fait partie du site de l'Association des Étudiants de l'UTBM,
+# http://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# This program is free software; you can redistribute it and/or modify it under
+# the terms of the GNU General Public License a published by the Free Software
+# Foundation; either version 3 of the License, or (at your option) any later
+# version.
 #
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
+# This program is distributed in the hope that it will be useful, but WITHOUT
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+# FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+# details.
+#
+# You should have received a copy of the GNU General Public License along with
+# this program; if not, write to the Free Sofware Foundation, Inc., 59 Temple
+# Place - Suite 330, Boston, MA 02111-1307, USA.
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 
 from django.urls import path
 
-from club.views import *
+from club.views import (
+    ClubCreateView,
+    ClubEditPropView,
+    ClubEditView,
+    ClubListView,
+    ClubMailingView,
+    ClubMembersView,
+    ClubOldMembersView,
+    ClubPageEditView,
+    ClubPageHistView,
+    ClubRevView,
+    ClubSellingCSVView,
+    ClubSellingView,
+    ClubStatView,
+    ClubToolsView,
+    ClubView,
+    MailingAutoGenerationView,
+    MailingDeleteView,
+    MailingSubscriptionDeleteView,
+    MembershipDeleteView,
+    MembershipSetOldView,
+    PosterCreateView,
+    PosterDeleteView,
+    PosterEditView,
+    PosterListView,
+)
 
 urlpatterns = [
     path("", ClubListView.as_view(), name="club_list"),
@@ -28,32 +57,20 @@ urlpatterns = [
     path("stats/", ClubStatView.as_view(), name="club_stats"),
     path("<int:club_id>/", ClubView.as_view(), name="club_view"),
     path(
-        "<int:club_id>/rev/<int:rev_id>/",
-        ClubRevView.as_view(),
-        name="club_view_rev",
+        "<int:club_id>/rev/<int:rev_id>/", ClubRevView.as_view(), name="club_view_rev"
     ),
     path("<int:club_id>/hist/", ClubPageHistView.as_view(), name="club_hist"),
     path("<int:club_id>/edit/", ClubEditView.as_view(), name="club_edit"),
-    path(
-        "<int:club_id>/edit/page/",
-        ClubPageEditView.as_view(),
-        name="club_edit_page",
-    ),
+    path("<int:club_id>/edit/page/", ClubPageEditView.as_view(), name="club_edit_page"),
     path("<int:club_id>/members/", ClubMembersView.as_view(), name="club_members"),
     path(
         "<int:club_id>/elderlies/",
         ClubOldMembersView.as_view(),
         name="club_old_members",
     ),
+    path("<int:club_id>/sellings/", ClubSellingView.as_view(), name="club_sellings"),
     path(
-        "<int:club_id>/sellings/",
-        ClubSellingView.as_view(),
-        name="club_sellings",
-    ),
-    path(
-        "<int:club_id>/sellings/csv/",
-        ClubSellingCSVView.as_view(),
-        name="sellings_csv",
+        "<int:club_id>/sellings/csv/", ClubSellingCSVView.as_view(), name="sellings_csv"
     ),
     path("<int:club_id>/prop/", ClubEditPropView.as_view(), name="club_prop"),
     path("<int:club_id>/tools/", ClubToolsView.as_view(), name="tools"),
@@ -85,9 +102,7 @@ urlpatterns = [
     ),
     path("<int:club_id>/poster/", PosterListView.as_view(), name="poster_list"),
     path(
-        "<int:club_id>/poster/create/",
-        PosterCreateView.as_view(),
-        name="poster_create",
+        "<int:club_id>/poster/create/", PosterCreateView.as_view(), name="poster_create"
     ),
     path(
         "<int:club_id>/poster/<int:poster_id>/edit/",

club/views.py

@@ -1,21 +1,25 @@
-# -*- coding:utf-8 -*-
 #
-# Copyright 2023 © AE UTBM
-# ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
+# Copyright 2016,2017
+# - Skia <skia@libskia.so>
+# - Sli <antoine@bartuccio.fr>
 #
-# This file is part of the website of the UTBM Student Association (AE UTBM),
-# https://ae.utbm.fr.
+# Ce fichier fait partie du site de l'Association des Étudiants de l'UTBM,
+# http://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# This program is free software; you can redistribute it and/or modify it under
+# the terms of the GNU General Public License a published by the Free Software
+# Foundation; either version 3 of the License, or (at your option) any later
+# version.
 #
-# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
-# OR WITHIN THE LOCAL FILE "LICENSE"
+# This program is distributed in the hope that it will be useful, but WITHOUT
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+# FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+# details.
+#
+# You should have received a copy of the GNU General Public License along with
+# this program; if not, write to the Free Sofware Foundation, Inc., 59 Temple
+# Place - Suite 330, Boston, MA 02111-1307, USA.
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
 
 import csv
@@ -171,18 +175,14 @@ class ClubTabsMixin(TabedViewMixin):
 
 
 class ClubListView(ListView):
-    """
-    List the Clubs
-    """
+    """List the Clubs."""
 
     model = Club
     template_name = "club/club_list.jinja"
 
 
 class ClubView(ClubTabsMixin, DetailView):
-    """
-    Front page of a Club
-    """
+    """Front page of a Club."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -190,24 +190,22 @@ class ClubView(ClubTabsMixin, DetailView):
     current_tab = "infos"
 
     def get_context_data(self, **kwargs):
-        kwargs = super(ClubView, self).get_context_data(**kwargs)
+        kwargs = super().get_context_data(**kwargs)
         if self.object.page and self.object.page.revisions.exists():
             kwargs["page_revision"] = self.object.page.revisions.last().content
         return kwargs
 
 
 class ClubRevView(ClubView):
-    """
-    Display a specific page revision
-    """
+    """Display a specific page revision."""
 
     def dispatch(self, request, *args, **kwargs):
         obj = self.get_object()
         self.revision = get_object_or_404(PageRev, pk=kwargs["rev_id"], page__club=obj)
-        return super(ClubRevView, self).dispatch(request, *args, **kwargs)
+        return super().dispatch(request, *args, **kwargs)
 
     def get_context_data(self, **kwargs):
-        kwargs = super(ClubRevView, self).get_context_data(**kwargs)
+        kwargs = super().get_context_data(**kwargs)
         kwargs["page_revision"] = self.revision.content
         return kwargs
 
@@ -220,7 +218,7 @@ class ClubPageEditView(ClubTabsMixin, PageEditViewBase):
         self.club = get_object_or_404(Club, pk=kwargs["club_id"])
         if not self.club.page:
             raise Http404
-        return super(ClubPageEditView, self).dispatch(request, *args, **kwargs)
+        return super().dispatch(request, *args, **kwargs)
 
     def get_object(self):
         self.page = self.club.page
@@ -231,9 +229,7 @@ class ClubPageEditView(ClubTabsMixin, PageEditViewBase):
 
 
 class ClubPageHistView(ClubTabsMixin, CanViewMixin, DetailView):
-    """
-    Modification hostory of the page
-    """
+    """Modification hostory of the page."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -242,9 +238,7 @@ class ClubPageHistView(ClubTabsMixin, CanViewMixin, DetailView):
 
 
 class ClubToolsView(ClubTabsMixin, CanEditMixin, DetailView):
-    """
-    Tools page of a Club
-    """
+    """Tools page of a Club."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -253,9 +247,7 @@ class ClubToolsView(ClubTabsMixin, CanEditMixin, DetailView):
 
 
 class ClubMembersView(ClubTabsMixin, CanViewMixin, DetailFormView):
-    """
-    View of a club's members
-    """
+    """View of a club's members."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -264,22 +256,20 @@ class ClubMembersView(ClubTabsMixin, CanViewMixin, DetailFormView):
     current_tab = "members"
 
     def get_form_kwargs(self):
-        kwargs = super(ClubMembersView, self).get_form_kwargs()
+        kwargs = super().get_form_kwargs()
         kwargs["request_user"] = self.request.user
         kwargs["club"] = self.get_object()
         kwargs["club_members"] = self.members
         return kwargs
 
     def get_context_data(self, *args, **kwargs):
-        kwargs = super(ClubMembersView, self).get_context_data(*args, **kwargs)
+        kwargs = super().get_context_data(*args, **kwargs)
         kwargs["members"] = self.members
         return kwargs
 
     def form_valid(self, form):
-        """
-        Check user rights
-        """
-        resp = super(ClubMembersView, self).form_valid(form)
+        """Check user rights."""
+        resp = super().form_valid(form)
 
         data = form.clean()
         users = data.pop("users", [])
@@ -294,7 +284,7 @@ class ClubMembersView(ClubTabsMixin, CanViewMixin, DetailFormView):
 
     def dispatch(self, request, *args, **kwargs):
         self.members = self.get_object().members.ongoing().order_by("-role")
-        return super(ClubMembersView, self).dispatch(request, *args, **kwargs)
+        return super().dispatch(request, *args, **kwargs)
 
     def get_success_url(self, **kwargs):
         return reverse_lazy(
@@ -303,9 +293,7 @@ class ClubMembersView(ClubTabsMixin, CanViewMixin, DetailFormView):
 
 
 class ClubOldMembersView(ClubTabsMixin, CanViewMixin, DetailView):
-    """
-    Old members of a club
-    """
+    """Old members of a club."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -314,9 +302,7 @@ class ClubOldMembersView(ClubTabsMixin, CanViewMixin, DetailView):
 
 
 class ClubSellingView(ClubTabsMixin, CanEditMixin, DetailFormView):
-    """
-    Sellings of a club
-    """
+    """Sellings of a club."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -328,12 +314,12 @@ class ClubSellingView(ClubTabsMixin, CanEditMixin, DetailFormView):
     def dispatch(self, request, *args, **kwargs):
         try:
             self.asked_page = int(request.GET.get("page", 1))
-        except ValueError:
-            raise Http404
-        return super(ClubSellingView, self).dispatch(request, *args, **kwargs)
+        except ValueError as e:
+            raise Http404 from e
+        return super().dispatch(request, *args, **kwargs)
 
     def get_form_kwargs(self):
-        kwargs = super(ClubSellingView, self).get_form_kwargs()
+        kwargs = super().get_form_kwargs()
         kwargs["club"] = self.object
         return kwargs
 
@@ -341,7 +327,7 @@ class ClubSellingView(ClubTabsMixin, CanEditMixin, DetailFormView):
         return self.get(request, *args, **kwargs)
 
     def get_context_data(self, **kwargs):
-        kwargs = super(ClubSellingView, self).get_context_data(**kwargs)
+        kwargs = super().get_context_data(**kwargs)
         qs = Selling.objects.filter(club=self.object)
 
         kwargs["result"] = qs[:0]
@@ -385,19 +371,17 @@ class ClubSellingView(ClubTabsMixin, CanEditMixin, DetailFormView):
         kwargs["paginator"] = Paginator(kwargs["result"], self.paginate_by)
         try:
             kwargs["paginated_result"] = kwargs["paginator"].page(self.asked_page)
-        except InvalidPage:
-            raise Http404
+        except InvalidPage as e:
+            raise Http404 from e
 
         return kwargs
 
 
 class ClubSellingCSVView(ClubSellingView):
-    """
-    Generate sellings in csv for a given period
-    """
+    """Generate sellings in csv for a given period."""
 
     class StreamWriter:
-        """Implements a file-like interface for streaming the CSV"""
+        """Implements a file-like interface for streaming the CSV."""
 
         def write(self, value):
             """Write the value by returning it, instead of storing in a buffer."""
@@ -413,7 +397,8 @@ class ClubSellingCSVView(ClubSellingView):
             row.append(selling.customer.user.get_display_name())
         else:
             row.append("")
-        row = row + [
+        row = [
+            *row,
             selling.label,
             selling.quantity,
             selling.quantity * selling.unit_price,
@@ -424,7 +409,7 @@ class ClubSellingCSVView(ClubSellingView):
             row.append(selling.product.purchase_price)
             row.append(selling.product.selling_price - selling.product.purchase_price)
         else:
-            row = row + ["", "", ""]
+            row = [*row, "", "", ""]
         return row
 
     def get(self, request, *args, **kwargs):
@@ -471,9 +456,7 @@ class ClubSellingCSVView(ClubSellingView):
 
 
 class ClubEditView(ClubTabsMixin, CanEditMixin, UpdateView):
-    """
-    Edit a Club's main informations (for the club's members)
-    """
+    """Edit a Club's main informations (for the club's members)."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -483,9 +466,7 @@ class ClubEditView(ClubTabsMixin, CanEditMixin, UpdateView):
 
 
 class ClubEditPropView(ClubTabsMixin, CanEditPropMixin, UpdateView):
-    """
-    Edit the properties of a Club object (for the Sith admins)
-    """
+    """Edit the properties of a Club object (for the Sith admins)."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -495,9 +476,7 @@ class ClubEditPropView(ClubTabsMixin, CanEditPropMixin, UpdateView):
 
 
 class ClubCreateView(CanCreateMixin, CreateView):
-    """
-    Create a club (for the Sith admin)
-    """
+    """Create a club (for the Sith admin)."""
 
     model = Club
     pk_url_kwarg = "club_id"
@@ -506,9 +485,7 @@ class ClubCreateView(CanCreateMixin, CreateView):
 
 
 class MembershipSetOldView(CanEditMixin, DetailView):
-    """
-    Set a membership as beeing old
-    """
+    """Set a membership as beeing old."""
 
     model = Membership
     pk_url_kwarg = "membership_id"
@@ -537,9 +514,7 @@ class MembershipSetOldView(CanEditMixin, DetailView):
 
 
 class MembershipDeleteView(UserIsRootMixin, DeleteView):
-    """
-    Delete a membership (for admins only)
-    """
+    """Delete a membership (for admins only)."""
 
     model = Membership
     pk_url_kwarg = "membership_id"
@@ -553,15 +528,13 @@ class ClubStatView(TemplateView):
     template_name = "club/stats.jinja"
 
     def get_context_data(self, **kwargs):
-        kwargs = super(ClubStatView, self).get_context_data(**kwargs)
+        kwargs = super().get_context_data(**kwargs)
         kwargs["club_list"] = Club.objects.all()
         return kwargs
 
 
 class ClubMailingView(ClubTabsMixin, CanEditMixin, DetailFormView):
-    """
-    A list of mailing for a given club
-    """
+    """A list of mailing for a given club."""
 
     model = Club
     form_class = MailingForm
@@ -570,7 +543,7 @@ class ClubMailingView(ClubTabsMixin, CanEditMixin, DetailFormView):
     current_tab = "mailing"
 
     def get_form_kwargs(self):
-        kwargs = super(ClubMailingView, self).get_form_kwargs()
+        kwargs = super().get_form_kwargs()
         kwargs["club_id"] = self.get_object().id
         kwargs["user_id"] = self.request.user.id
         kwargs["mailings"] = self.mailings
@@ -578,10 +551,10 @@ class ClubMailingView(ClubTabsMixin, CanEditMixin, DetailFormView):
 
     def dispatch(self, request, *args, **kwargs):
         self.mailings = Mailing.objects.filter(club_id=self.get_object().id).all()
-        return super(ClubMailingView, self).dispatch(request, *args, **kwargs)
+        return super().dispatch(request, *args, **kwargs)
 
     def get_context_data(self, **kwargs):
-        kwargs = super(ClubMailingView, self).get_context_data(**kwargs)
+        kwargs = super().get_context_data(**kwargs)
         kwargs["club"] = self.get_object()
         kwargs["user"] = self.request.user
         kwargs["mailings"] = self.mailings
@@ -599,9 +572,7 @@ class ClubMailingView(ClubTabsMixin, CanEditMixin, DetailFormView):
         return kwargs
 
     def add_new_mailing(self, cleaned_data) -> ValidationError | None:
-        """
-        Create a new mailing list from the form
-        """
+        """Create a new mailing list from the form."""
         mailing = Mailing(
             club=self.get_object(),
             email=cleaned_data["mailing_email"],
@@ -616,9 +587,7 @@ class ClubMailingView(ClubTabsMixin, CanEditMixin, DetailFormView):
         return None
 
     def add_new_subscription(self, cleaned_data) -> ValidationError | None:
-        """
-        Add mailing subscriptions for each user given and/or for the specified email in form
-        """
+        """Add mailing subscriptions for each user given and/or for the specified email in form."""
         users_to_save = []
 
         for user in cleaned_data["subscription_users"]:
@@ -652,20 +621,16 @@ class ClubMailingView(ClubTabsMixin, CanEditMixin, DetailFormView):
         return None
 
     def remove_subscription(self, cleaned_data):
-        """
-        Remove specified users from a mailing list
-        """
+        """Remove specified users from a mailing list."""
         fields = [
-            cleaned_data[key]
-            for key in cleaned_data.keys()
-            if key.startswith("removal_")
+            val for key, val in cleaned_data.items() if key.startswith("removal_")
         ]
         for field in fields:
             for sub in field:
                 sub.delete()
 
     def form_valid(self, form):
-        resp = super(ClubMailingView, self).form_valid(form)
+        resp = super().form_valid(form)
 
         cleaned_data = form.clean()
         error = None
@@ -697,7 +662,7 @@ class MailingDeleteView(CanEditMixin, DeleteView):
 
     def dispatch(self, request, *args, **kwargs):
         self.club_id = self.get_object().club.id
-        return super(MailingDeleteView, self).dispatch(request, *args, **kwargs)
+        return super().dispatch(request, *args, **kwargs)
 
     def get_success_url(self, **kwargs):
         if self.redirect_page:
@@ -713,9 +678,7 @@ class MailingSubscriptionDeleteView(CanEditMixin, DeleteView):
 
     def dispatch(self, request, *args, **kwargs):
         self.club_id = self.get_object().mailing.club.id
-        return super(MailingSubscriptionDeleteView, self).dispatch(
-            request, *args, **kwargs
-        )
+        return super().dispatch(request, *args, **kwargs)
 
     def get_success_url(self, **kwargs):
         return reverse_lazy("club:mailing", kwargs={"club_id": self.club_id})
@@ -726,7 +689,7 @@ class MailingAutoGenerationView(View):
         self.mailing = get_object_or_404(Mailing, pk=kwargs["mailing_id"])
         if not request.user.can_edit(self.mailing):
             raise PermissionDenied
-        return super(MailingAutoGenerationView, self).dispatch(request, *args, **kwargs)
+        return super().dispatch(request, *args, **kwargs)
 
     def get(self, request, *args, **kwargs):
         club = self.mailing.club
@@ -740,25 +703,25 @@ class MailingAutoGenerationView(View):
 
 
 class PosterListView(ClubTabsMixin, PosterListBaseView, CanViewMixin):
-    """List communication posters"""
+    """List communication posters."""
 
     def get_object(self):
         return self.club
 
     def get_context_data(self, **kwargs):
-        kwargs = super(PosterListView, self).get_context_data(**kwargs)
+        kwargs = super().get_context_data(**kwargs)
         kwargs["app"] = "club"
         kwargs["club"] = self.club
         return kwargs
 
 
 class PosterCreateView(PosterCreateBaseView, CanCreateMixin):
-    """Create communication poster"""
+    """Create communication poster."""
 
     pk_url_kwarg = "club_id"
 
     def get_object(self):
-        obj = super(PosterCreateView, self).get_object()
+        obj = super().get_object()
         if not obj:
             return self.club
         return obj
@@ -768,19 +731,19 @@ class PosterCreateView(PosterCreateBaseView, CanCreateMixin):
 
 
 class PosterEditView(ClubTabsMixin, PosterEditBaseView, CanEditMixin):
-    """Edit communication poster"""
+    """Edit communication poster."""
 
     def get_success_url(self):
         return reverse_lazy("club:poster_list", kwargs={"club_id": self.club.id})
 
     def get_context_data(self, **kwargs):
-        kwargs = super(PosterEditView, self).get_context_data(**kwargs)
+        kwargs = super().get_context_data(**kwargs)
         kwargs["app"] = "club"
         return kwargs
 
 
 class PosterDeleteView(PosterDeleteBaseView, ClubTabsMixin, CanEditMixin):
-    """Delete communication poster"""
+    """Delete communication poster."""
 
     def get_success_url(self):
         return reverse_lazy("club:poster_list", kwargs={"club_id": self.club.id})

club/widgets/select.py

@@ -0,0 +1,23 @@
+from pydantic import TypeAdapter
+
+from club.models import Club
+from club.schemas import ClubSchema
+from core.views.widgets.select import AutoCompleteSelect, AutoCompleteSelectMultiple
+
+_js = ["bundled/club/components/ajax-select-index.ts"]
+
+
+class AutoCompleteSelectClub(AutoCompleteSelect):
+    component_name = "club-ajax-select"
+    model = Club
+    adapter = TypeAdapter(list[ClubSchema])
+
+    js = _js
+
+
+class AutoCompleteSelectMultipleClub(AutoCompleteSelectMultiple):
+    component_name = "club-ajax-select"
+    model = Club
+    adapter = TypeAdapter(list[ClubSchema])
+
+    js = _js

com/__init__.py

@@ -1,19 +1,14 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #

com/admin.py

@@ -1,46 +1,34 @@
-# -*- coding:utf-8 -*-
 #
 # Copyright 2023 © AE UTBM
 # ae@utbm.fr / ae.info@utbm.fr
-# All contributors are listed in the CONTRIBUTORS file.
 #
 # This file is part of the website of the UTBM Student Association (AE UTBM),
 # https://ae.utbm.fr.
 #
-# You can find the whole source code at https://github.com/ae-utbm/sith3
+# You can find the source code of the website at https://github.com/ae-utbm/sith
 #
 # LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
+# SEE : https://raw.githubusercontent.com/ae-utbm/sith/master/LICENSE
 # OR WITHIN THE LOCAL FILE "LICENSE"
 #
-# PREVIOUSLY LICENSED UNDER THE MIT LICENSE,
-# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE.old
-# OR WITHIN THE LOCAL FILE "LICENSE.old"
 #
-from ajax_select import make_ajax_form
 from django.contrib import admin
 from haystack.admin import SearchModelAdmin
 
-from com.models import *
+from com.models import News, Poster, Screen, Sith, Weekmail
 
 
 @admin.register(News)
 class NewsAdmin(SearchModelAdmin):
     list_display = ("title", "type", "club", "author")
     search_fields = ("title", "summary", "content")
-    form = make_ajax_form(
-        News,
-        {
-            "author": "users",
-            "moderator": "users",
-        },
-    )
+    autocomplete_fields = ("author", "moderator")
 
 
 @admin.register(Poster)
 class PosterAdmin(SearchModelAdmin):
     list_display = ("name", "club", "date_begin", "date_end", "moderator")
-    form = make_ajax_form(Poster, {"moderator": "users"})
+    autocomplete_fields = ("moderator",)
 
 
 @admin.register(Weekmail)

com/migrations/0001_initial.py

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