chore(bda): update mail attribution

fix #868

.credentials/HCAPTCHA_SECRET

@@ -0,0 +1 @@
+0x0000000000000000000000000000000000000000

.credentials/HCAPTCHA_SITEKEY

@@ -0,0 +1 @@
+10000000-ffff-ffff-ffff-000000000001

.credentials/KFETOPEN_TOKEN

@@ -0,0 +1 @@
+k-feste_token

.credentials/SECRET_KEY

@@ -0,0 +1 @@
+insecure-key

.credentials/SYMPA_PASSWORD

@@ -0,0 +1 @@
+toto

.credentials/SYMPA_USERNAME

@@ -0,0 +1 @@
+sympa

.envrc

@@ -0,0 +1 @@
+use nix

.gitignore

@@ -5,13 +5,20 @@ cof/settings.py
 settings.py
 *~
 venv/
+.venv/
 .vagrant
 /src
 media/
 *.log
+.sass-cache/
 *.sqlite3
 .coverage
 
 # PyCharm
 .idea
 .cache
+
+# VSCode
+.vscode/
+.direnv
+.static

.gitlab-ci.yml

@@ -1,12 +1,7 @@
-image: "python:3.5"
+image: "python:3.7"
-
-services:
-    - postgres:latest
-    - redis:latest
 
 variables:
     # GestioCOF settings
-    DJANGO_SETTINGS_MODULE: "cof.settings.prod"
     DBHOST: "postgres"
     REDIS_HOST: "redis"
     REDIS_PASSWD: "dummy"
@@ -22,21 +17,24 @@ variables:
     # psql password authentication
     PGPASSWORD: $POSTGRES_PASSWORD
 
-test:
+    # apps to check migrations for
-    stage: test
+    MIGRATION_APPS: "bda bds cofcms clubs events gestioncof kfet kfetauth kfetcms open petitscours shared"
+
+.test_template:
     before_script:
         - mkdir -p vendor/{pip,apt}
-        - apt-get update -q && apt-get -o dir::cache::archives="vendor/apt" install -yqq postgresql-client
+        - apt-get update -q && apt-get -o dir::cache::archives="vendor/apt" install -yqq postgresql-client libldap2-dev libsasl2-dev
-        - sed -E 's/^REDIS_HOST.*/REDIS_HOST = "redis"/' cof/settings/secret_example.py > cof/settings/secret.py
+        - sed -E 's/^REDIS_HOST.*/REDIS_HOST = "redis"/' gestioasso/settings/secret_example.py > gestioasso/settings/secret.py
-        - sed -i.bak -E 's;^REDIS_PASSWD = .*$;REDIS_PASSWD = "";' cof/settings/secret.py
+        - sed -i.bak -E 's;^REDIS_PASSWD = .*$;REDIS_PASSWD = "";' gestioasso/settings/secret.py
         # Remove the old test database if it has not been done yet
         - psql --username=$POSTGRES_USER --host=$DBHOST -c "DROP DATABASE IF EXISTS test_$POSTGRES_DB"
-        - pip install --upgrade -r requirements.txt coverage
+        - pip install --upgrade -r requirements-prod.txt coverage tblib
         - python --version
-    script:
-        - coverage run manage.py test
     after_script:
         - coverage report
+    services:
+        - postgres:11.7
+        - redis:latest
     cache:
         key: test
         paths:
@@ -45,18 +43,61 @@ test:
     # Keep this disabled for now, as it may kill GitLab...
     # coverage: '/TOTAL.*\s(\d+\.\d+)\%$/'
 
+kfettest:
+    stage: test
+    extends: .test_template
+    variables:
+        DJANGO_SETTINGS_MODULE: "gestioasso.settings.cof_prod"
+    script:
+        - coverage run manage.py test kfet
+
+coftest:
+    stage: test
+    extends: .test_template
+    variables:
+        DJANGO_SETTINGS_MODULE: "gestioasso.settings.cof_prod"
+    script:
+        - coverage run manage.py test gestioncof bda petitscours shared --parallel
+
+bdstest:
+    stage: test
+    extends: .test_template
+    variables:
+        DJANGO_SETTINGS_MODULE: "gestioasso.settings.bds_prod"
+    script:
+        - coverage run manage.py test bds clubs events --parallel
+
 linters:
-    image: python:3.6
     stage: test
     before_script:
         - mkdir -p vendor/pip
         - pip install --upgrade black isort flake8
     script:
         - black --check .
-        - isort --recursive --check-only --diff bda cof gestioncof kfet provisioning shared utils
+        - isort --check --diff .
         # Print errors only
-        - flake8 --exit-zero bda cof gestioncof kfet provisioning shared utils
+        - flake8 --exit-zero bda bds clubs gestioasso events gestioncof kfet petitscours provisioning shared
     cache:
         key: linters
         paths:
             - vendor/
+
+# Check whether there are some missing migrations.
+migration_checks:
+    stage: test
+    variables:
+        DJANGO_SETTINGS_MODULE: "gestioasso.settings.local"
+    before_script:
+        - mkdir -p vendor/{pip,apt}
+        - apt-get update -q && apt-get -o dir::cache::archives="vendor/apt" install -yqq postgresql-client libldap2-dev libsasl2-dev
+        - cp gestioasso/settings/secret_example.py gestioasso/settings/secret.py
+        - pip install --upgrade -r requirements-devel.txt
+        - python --version
+    script: python manage.py makemigrations --dry-run --check $MIGRATION_APPS
+    services:
+        # this should not be necessary…
+        - postgres:11.7
+    cache:
+        key: migration_checks
+        paths:
+            - vendor/

.pre-commit.sh

@@ -48,7 +48,7 @@ if type isort &>/dev/null; then
         ISORT_OUTPUT="/tmp/gc-isort-output.log"
         touch $ISORT_OUTPUT
 
-        if ! echo "$STAGED_PYTHON_FILES" | xargs -d'\n' isort --check-only &>$ISORT_OUTPUT; then
+        if ! echo "$STAGED_PYTHON_FILES" | xargs -d'\n' isort --check &>$ISORT_OUTPUT; then
             echo "$STAGED_PYTHON_FILES" | xargs -d'\n' isort &>$ISORT_OUTPUT
             printf "Reformatted.\n"
             formatter_updated=1

CHANGELOG.md

@@ -0,0 +1,303 @@
+# Changelog
+
+Liste des changements notables dans GestioCOF depuis la version 0.1 (septembre
+2018).
+
+## Le FUTUR ! (pas prêt pour la prod)
+
+### Nouveau module de gestion des événements
+
+- Désormais complet niveau modèles
+- Export des participants implémenté
+
+#### TODO
+
+- Vue de création d'événements ergonomique
+- Vue d'inscription à un événement **ou** intégration propre dans la vue
+  "inscription d'un nouveau membre"
+
+### Nouveau module de gestion des clubs
+
+Uniquement un modèle simple de clubs avec des respos. Aucune gestion des
+adhérents ni des cotisations.
+
+## TODO Prod
+
+- Lancer `python manage.py update_translation_fields` après la migration
+- Mettre à jour les units systemd `daphne.service` et `worker.service`
+
+- Créer un compte hCaptcha (https://www.hcaptcha.com/), au COF, et remplacer les secrets associés
+
+## Version ??? - ??/??/????
+
+## Version 0.15.1 - 15/06/2023
+
+### K-Fêt
+
+- Rattrape les erreurs d'envoi de mail de négatif
+- Utilise l'adresse chefs pour les envois de négatifs
+
+## Version 0.15 - 22/05/2023
+
+### K-Fêt
+
+- Rajoute un formulaire de contact
+- Rajoute un formulaire de demande de soirée
+- Désactive les mails d'envoi de négatifs sur les comptes gelés
+
+## Version 0.14 - 19/05/2023
+
+- Répare les dépendances en spécifiant toutes les versions
+
+### K-Fêt
+
+- Répare la gestion des changement d'heure via moment.js
+
+## Version 0.13 - 19/02/2023
+
+### K-Fêt
+
+- Rajoute la valeur des inventaires
+- Résout les problèmes de négatif ne disparaissant pas
+- Affiche son surnom s'il y en a un
+- Bugfixes
+
+## Version 0.12.1 - 03/10/2022
+
+### K-Fêt
+
+- Fixe un problème de rendu causé par l'agrandissement du menu
+
+- Mise à jour vers Channels 3.x et Django 3.2
+
+## Version 0.12 - 17/06/2022
+
+### K-Fêt
+
+- Ajoute une exception à la limite d'historique pour les comptes `LIQ` et `#13`
+- Répare le problème des étiquettes LIQ/Comptes K-Fêt inversées dans les stats des articles K-Fêt
+
+## Version 0.11 - 26/10/2021
+
+### COF
+
+- Répare un problème de rendu sur le wagtail du COF
+
+### K-Fêt
+
+- Ajoute de mails de rappels pour les comptes en négatif
+- La recherche de comptes sur K-Psul remarche normalement
+- Le pointeur de la souris change de forme quand on survole un item d'autocomplétion
+- Modification du gel de compte:
+    - on ne peut plus geler/dégeler son compte soi-même (il faut la permission "Gérer les permissions K-Fêt")
+    - on ne peut rien compter sur un compte gelé (aucune override possible), et les K-Fêteux·ses dont le compte est gelé perdent tout accès à K-Psul
+    - les comptes actuellement gelés (sur l'ancien système) sont dégelés automatiquement
+- Modification du fonctionnement des négatifs
+    - impossible d'avoir des négatifs inférieurs à `kfet_config.overdraft_amount`
+    - il n'y a plus de limite de temps sur les négatifs
+    - supression des autorisations de négatif
+    - il n'est plus possible de réinitialiser la durée d'un négatif en faisant puis en annulant une charge
+- La gestion des erreurs passe du client au serveur, ce qui permet d'avoir des messages plus explicites
+- La supression d'opérations anciennes est réparée
+
+## Version 0.10 - 18/04/2021
+
+### K-Fêt
+
+- On fait sauter la limite qui empêchait de vendre plus de 24 unités d'un item à
+  la fois.
+- L'interface indique plus clairement quand on fait une erreur en modifiant un
+  compte.
+- On supprime la fonction "décalage de balance".
+- L'accès à l'historique est maintenant limité à 7 jours pour raison de
+  confidentialité. Les chefs/trez peuvent disposer d'une permission
+  supplémentaire pour accéder à jusqu'à 30 jours en cas de problème de compta.
+  L'accès à son historique personnel n'est pas limité. Les durées sont
+  configurables dans `settings/cof_prod.py`.
+
+### COF
+
+- Le Captcha sur la page de demande de petits cours utilise maintenant hCaptcha
+  au lieu de ReCaptcha, pour mieux respecter la vie privée des utilisateur·ices
+
+## Version 0.9 - 06/02/2020
+
+### COF / BdA
+
+- Le COF peut remettre à zéro la liste de ses adhérents en août (sans passer par
+  KDE).
+- La page d'accueil affiche la date de fermeture des tirages BdA.
+- On peut revendre une place dès qu'on l'a payée, plus besoin de payer toutes
+  ses places pour pouvoir revendre.
+- On s'assure que l'email fourni lors d'une demande de petit cours est valide.
+
+### BDS
+
+- Le burô peut maintenant accorder ou révoquer le statut de membre du Burô
+  en modifiant le profil d'un membre du BDS.
+- Le burô peut exporter la liste de ses membres avec email au format CSV depuis
+  la page d'accueil.
+
+### K-Fêt
+
+- On affiche les articles actuellement en vente en premier lors des inventaires
+  et des commandes.
+- On peut supprimer un inventaire. Seuls les articles dont c'est le dernier
+  inventaire sont affectés.
+
+## Version 0.8 - 03/12/2020
+
+### COF
+
+- La page "Mes places" dans la section BdA indique quelles places sont sur
+  listing.
+- ergonomie de l'interface admin du BdA : moins d'options inutiles lors de
+  la sélection de participants.
+- les tirages sont maintenant archivables pour éviter d'avoir encore d'autres
+  options inutiles.
+- l'autocomplétion dans l'admin BdA est réparée.
+- Les icones de la page de gestion des petits cours sont (à nouveau) réparées.
+- On a supprimé la possibilité de modifier les mails automatiques depuis
+  l'interface admin car trop problématique. Faute de mieux, envoyer un mail à
+  KDE pour modifier ces mails.
+- corrige un crash sporadique sur la page d'inscription au système de petits
+  cours
+
+### K-Fêt
+
+- (fix partiel) Empêche la K-Fêt de modifier des données COF (e.g. nom, prénom,
+  username) lors de la création d'un nouveau compte.
+- Les statistiques de conso globales montrent deux courbes COF / non-COF au
+  lieu de LIQ / sur compte.
+- Un bug empêchait de fermer manuellement la K-Fêt depuis un compte non
+  privilégié en tapant un mot de passe. C'est corrigé.
+
+## Version 0.7.2 - 08/09/2020
+
+- Nouvelle page 404
+- Correction de bug en K-Fêt : le lien pour créer un nouveau compte exté apparaît
+  à nouveau dans l'autocomplétion
+
+## Version 0.7.1 - 05/09/2020
+
+Petits ajustements sur le site du COF :
+
+- Possibilité d'ajouter des champs d'infos supplémentaires en plus de l'email et
+  de la page web dans les annuaires (clubs et partenaires).
+- Corrige un bug d'affichage des adresses emails de clubs
+
+## Version 0.7 - 29/08/2020
+
+### GestioBDS
+
+- Ajout d'un bouton pour supprimer un compte
+- Le nombre d'adhérent⋅es est affiché sur la page d'accueil
+- le groupe BDS a les bonnes permissions
+
+### Site du COF
+
+- Captcha fonctionnel pour les mailing-listes
+
+### K-Fêt
+
+- L'autocomplétion pour la création de compte K-Fêt se lance à 3 caractères seulement,
+donc est plus rapide.
+
+## Version 0.6 - 27/07/2020
+
+Arrivée du BDS !
+GestioCOF et GestioBDS ont du code en commun mais tournent de façon séparée, les
+deux bases de données sont distinctes.
+
+## Version 0.5 - 11/07/2020
+
+### Problèmes corrigés
+
+- La recherche d'utilisateurices (COF + K-Fêt) fonctionne de nouveau
+- Bug d'affichage quand on a beaucoup de clubs dans le cadre "Accès rapide" sur
+  la page des clubs (nouveau site du COF)
+- Version mobile plus ergonimique sur le nouveau site du COF
+- Cliquer sur "visualiser" sur les pages de clubs dans wagtail ne provoque plus
+  d'erreurs 500 (nouveau site du COF)
+- L'historique des ventes des articles K-Fêt fonctionne à nouveau
+- Les montants en K-Fêt sont à nouveau affichés en UKF (et non en €).
+- Les boutons "afficher/cacher" des mails et noms des participant⋅e⋅s à un
+  spectacle BdA fonctionnent à nouveau.
+- on ne peut plus compter de consos sur ☠☠☠, ni éditer les comptes spéciaux
+(LIQ, GNR, ☠☠☠, #13).
+
+### Nouvelles fonctionnalités
+
+- On n'affiche que 4 articles sur la pages "nouveautés" (nouveau site du COF)
+- Plus de traductions sur le nouveau site du COF
+- Les transferts apparaissent maintenant dans l'historique K-Fêt et l'historique
+  personnel.
+- les statistiques K-Fêt remontent à plus d'un an (et le code est simplifié)
+
+## Version 0.4.1 - 17/01/2020
+
+- Corrige un bug sur K-Psul lorsqu'un trigramme contient des caractères réservés
+  aux urls (\#, /...)
+
+## Version 0.4 - 15/01/2020
+
+- Corrige un bug d'affichage d'images sur l'interface des petits cours
+- La page des transferts permet de créer un nombre illimité de transferts en
+  une fois.
+- Nouveau site du COF : les liens sont optionnels dans les descriptions de clubs
+- Mise à jour du lien vers le calendire de la K-Fêt sur la page d'accueil
+- Certaines opérations sont à nouveau accessibles depuis la session partagée
+  K-Fêt.
+- Le bouton "déconnexion" déconnecte vraiment du CAS pour les comptes clipper
+- Corrige un crash sur la page des reventes pour les nouveaux participants.
+- Corrige un bug d'affichage pour les trigrammes avec caractères spéciaux
+
+## Version 0.3.3 - 30/11/2019
+
+- Corrige un problème de redirection lors de la déconnexion (CAS seulement)
+- Les catégories d'articles K-Fêt peuvent être exemptées de subvention COF
+- Corrige un bug d'affichage dans K-Psul quand on annule une transaction sur LIQ
+- Corrige une privilege escalation liée aux sessions partagées en K-Fêt
+  https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/issues/240
+
+## Version 0.3.2 - 04/11/2019
+
+- Bugfix: modifier un compte K-Fêt ne supprime plus nom/prénom
+
+## Version 0.3.1 - 19/10/2019
+
+- Bugfix: l'historique des utilisateurices s'affiche à nouveau
+
+## Version 0.3 - 16/10/2019
+
+- Comptes extés: lien pour changer son mot de passe sur la page d'accueil
+- Les utilisateurices non-COF peuvent éditer leur profil
+- Un peu de pub pour KDEns sur la page d'accueil
+- Fix erreur 500 sur /bda/revente/<tirage_id>/manage
+- Si on essaie d'accéder au compte que qqn d'autre on a une 404 (et plus une 403)
+- On ne peut plus modifier des comptes COF depuis l'interface K-Fêt
+- Le champ de paiement BdA se fait au niveau des attributions
+- Affiche un message d'erreur plutôt que de crasher si échec de l'envoi du mail
+  de bienvenue aux nouveaux membres
+- On peut supprimer des comptes et des articles K-Fêt
+- Passage à Django2
+- Dev : on peut désactiver la barre de debug avec une variable shell
+- Remplace les CSS de Google par des polices de proximité
+- Passage du site du COF et de la K-Fêt en Wagtail 2.3 et Wagtail-modeltranslation 0.9
+- Ajoute un lien vers l'administration générale depuis les petits cours
+- Abandon de l'ancien catalogue BdA (déjà plus utilisé depuis longtemps)
+- Force l'unicité des logins clipper
+- Nouveau site du COF en wagtail
+- Meilleurs affichage des longues listes de spectacles à cocher dans BdA-Revente
+- Bugfix : les pages de la revente ne sont plus accessibles qu'aux membres du
+  COF
+
+## Version 0.2 - 07/11/2018
+
+- Corrections de bugs d'interface dans l'inscription aux tirages BdA
+- On peut annuler une revente à tout moment
+- Pleiiiiin de tests
+
+## Version 0.1 - 09/09/2018
+
+Début de la numérotation des versions, début du changelog

README.md

@@ -1,4 +1,4 @@
-# GestioCOF
+# GestioCOF / GestioBDS
 
 [![pipeline status](https://git.eleves.ens.fr/cof-geek/gestioCOF/badges/master/pipeline.svg)](https://git.eleves.ens.fr/cof-geek/gestioCOF/commits/master)
 [![coverage report](https://git.eleves.ens.fr/cof-geek/gestioCOF/badges/master/coverage.svg)](https://git.eleves.ens.fr/cof-geek/gestioCOF/commits/master)
@@ -18,7 +18,7 @@ Il vous faudra installer pip, les librairies de développement de python ainsi
 que sqlite3, un moteur de base de données léger et simple d'utilisation. Sous
 Debian et dérivées (Ubuntu, ...) :
 
-    sudo apt-get install python3-pip python3-dev python3-venv sqlite3
+    sudo apt-get install python3-pip python3-dev python3-venv sqlite3 libsasl2-dev python-dev-is-python3 libldap2-dev libssl-dev
 
 Si vous décidez d'utiliser un environnement virtuel Python (virtualenv;
 fortement conseillé), déplacez-vous dans le dossier où est installé GestioCOF
@@ -30,7 +30,15 @@ Pour l'activer, il faut taper
 
     . venv/bin/activate
 
-depuis le même dossier.
+depuis le même dossier. Pour préparer l'environnement à l'utilisation de `./manage.py`
+(qui permet de faire des tests en local), il faut également taper
+
+    export CREDENTIALS_DIRECTORY=$(realpath .credentials)
+    export DJANGO_SETTINGS_MODULE=gestioasso.settings.local
+    export GESTIOCOF_DEBUG=true
+    export GESTIOCOF_STATIC_ROOT=$(realpath .static)
+    export GESTIOBDS_DEBUG=true
+    export GESTIOBDS_STATIC_ROOT=$(realpath .static)
 
 Vous pouvez maintenant installer les dépendances Python depuis le fichier
 `requirements-devel.txt` :
@@ -38,11 +46,11 @@ Vous pouvez maintenant installer les dépendances Python depuis le fichier
     pip install -U pip  # parfois nécessaire la première fois
     pip install -r requirements-devel.txt
 
-Pour terminer, copier le fichier `cof/settings/secret_example.py` vers
+Pour terminer, copier le fichier `gestioasso/settings/secret_example.py` vers
-`cof/settings/secret.py`. Sous Linux ou Mac, préférez plutôt un lien symbolique
+`gestioasso/settings/secret.py`. Sous Linux ou Mac, préférez plutôt un lien symbolique
 pour profiter de façon transparente des mises à jour du fichier:
 
-    ln -s secret_example.py cof/settings/secret.py
+    ln -s secret_example.py gestioasso/settings/secret.py
 
 Nous avons un git hook de pre-commit pour formatter et vérifier que votre code
 vérifie nos conventions. Pour bénéficier des mises à jour du hook, préférez
@@ -186,6 +194,18 @@ Pour lancer les tests :
 python manage.py test
 ```
 
+### Astuces
+
+- En développement on utilise la django debug toolbar parce que c'est utile pour
+  débuguer les templates ou les requêtes SQL mais des fois c'est pénible parce
+  ça fait ramer GestioCOF (surtout dans wagtail).
+  Vous pouvez la désactiver temporairement en définissant la variable
+  d'environnement `DJANGO_NO_DDT` dans votre shell : par exemple dans
+  bash/zsh/…:
+  ```
+  $ export DJANGO_NO_DDT=1
+  ```
+
 
 ## Documentation utilisateur
 

TODO_PROD.md

@@ -1 +0,0 @@
-- Changer les urls dans les mails "bda-revente" et "bda-shotgun"

Vagrantfile

@@ -1,47 +1,19 @@
 # -*- mode: ruby -*-
 # vi: set ft=ruby :
 
-# All Vagrant configuration is done below. The "2" in Vagrant.configure
+# Configuration de base pour GestioCOF.
-# configures the configuration version (we support older styles for
+# Voir https://docs.vagrantup.com pour plus d'informations.
-# backwards compatibility). Please don't change it unless you know what
-# you're doing.
 Vagrant.configure(2) do |config|
-  # The most common configuration options are documented and commented below.
+  # On se base sur Debian 10 (Buster) pour avoir le même environnement qu'en
-  # For a complete reference, please see the online documentation at
+  # production.
-  # https://docs.vagrantup.com.
+  config.vm.box = "debian/contrib-buster64"
-
-  config.vm.box = "ubuntu/xenial64"
 
   # On associe le port 80 dans la machine virtuelle avec le port 8080 de notre
   # ordinateur, et le port 8000 avec le port 8000.
   config.vm.network :forwarded_port, guest: 80, host: 8080
   config.vm.network :forwarded_port, guest: 8000, host: 8000
 
-  # Create a private network, which allows host-only access to the machine
+  # Le restes de la configuration (installation de paquets, etc) est géré un
-  # using a specific IP.
+  # script shell.
-  # config.vm.network "private_network", ip: "192.168.33.10"
-
-  # Provider-specific configuration so you can fine-tune various
-  # backing providers for Vagrant. These expose provider-specific options.
-  # Example for VirtualBox:
-  #
-  # config.vm.provider "virtualbox" do |vb|
-  #   # Display the VirtualBox GUI when booting the machine
-  #   vb.gui = true
-  #
-  #   # Customize the amount of memory on the VM:
-  #   vb.memory = "1024"
-  # end
-  #
-  # View the documentation for the provider you are using for more
-  # information on available options.
-
-  # Enable provisioning with a shell script. Additional provisioners such as
-  # Puppet, Chef, Ansible, Salt, and Docker are also available. Please see the
-  # documentation for more information about their specific syntax and use.
-  # config.vm.provision "shell", inline: <<-SHELL
-  #   sudo apt-get update
-  #   sudo apt-get install -y apache2
-  # SHELL
   config.vm.provision :shell, path: "provisioning/bootstrap.sh"
 end

bda/admin.py

@@ -1,10 +1,11 @@
 from datetime import timedelta
 
-from custommail.shortcuts import send_mass_custom_mail
 from dal.autocomplete import ModelSelect2
 from django import forms
 from django.contrib import admin
-from django.db.models import Count, Sum
+from django.core.mail import send_mass_mail
+from django.db.models import Count, Q, Sum
+from django.template import loader
 from django.template.defaultfilters import pluralize
 from django.utils import timezone
 
@@ -32,20 +33,6 @@ class ReadOnlyMixin(object):
             return readonly_fields + self.readonly_fields_update
 
 
-class ChoixSpectacleAdminForm(forms.ModelForm):
-    class Meta:
-        widgets = {
-            "participant": ModelSelect2(url="bda-participant-autocomplete"),
-            "spectacle": ModelSelect2(url="bda-spectacle-autocomplete"),
-        }
-
-
-class ChoixSpectacleInline(admin.TabularInline):
-    model = ChoixSpectacle
-    form = ChoixSpectacleAdminForm
-    sortable_field_name = "priority"
-
-
 class AttributionTabularAdminForm(forms.ModelForm):
     listing = None
 
@@ -81,27 +68,51 @@ class WithListingAttributionInline(AttributionInline):
     exclude = ("given",)
     form = WithListingAttributionTabularAdminForm
     listing = True
+    verbose_name_plural = "Attributions sur listing"
 
 
 class WithoutListingAttributionInline(AttributionInline):
     form = WithoutListingAttributionTabularAdminForm
     listing = False
+    verbose_name_plural = "Attributions hors listing"
 
 
 class ParticipantAdminForm(forms.ModelForm):
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields["choicesrevente"].queryset = Spectacle.objects.select_related(
+        queryset = Spectacle.objects.select_related("location")
-            "location"
+
-        )
+        if self.instance.pk is not None:
+            queryset = queryset.filter(tirage=self.instance.tirage)
+
+        self.fields["choicesrevente"].queryset = queryset
+
+
+class ParticipantPaidFilter(admin.SimpleListFilter):
+    """
+    Permet de filtrer les participants sur s'ils ont payé leurs places ou pas
+    """
+
+    title = "A payé"
+    parameter_name = "paid"
+
+    def lookups(self, request, model_admin):
+        return ((True, "Oui"), (False, "Non"))
+
+    def queryset(self, request, queryset):
+        return queryset.filter(paid=self.value())
 
 
 class ParticipantAdmin(ReadOnlyMixin, admin.ModelAdmin):
     inlines = [WithListingAttributionInline, WithoutListingAttributionInline]
 
     def get_queryset(self, request):
-        return Participant.objects.annotate(
+        return self.model.objects.annotate_paid().annotate(
-            nb_places=Count("attributions"), total=Sum("attributions__price")
+            nb_places=Count("attributions"),
+            remain=Sum(
+                "attribution__spectacle__price", filter=Q(attribution__paid=False)
+            ),
+            total=Sum("attributions__price"),
         )
 
     def nb_places(self, obj):
@@ -110,6 +121,13 @@ class ParticipantAdmin(ReadOnlyMixin, admin.ModelAdmin):
     nb_places.admin_order_field = "nb_places"
     nb_places.short_description = "Nombre de places"
 
+    def paid(self, obj):
+        return obj.paid
+
+    paid.short_description = "A payé"
+    paid.boolean = True
+    paid.admin_order_field = "paid"
+
     def total(self, obj):
         tot = obj.total
         if tot:
@@ -118,31 +136,46 @@ class ParticipantAdmin(ReadOnlyMixin, admin.ModelAdmin):
             return "0 €"
 
     total.admin_order_field = "total"
-    total.short_description = "Total à payer"
+    total.short_description = "Total des places"
-    list_display = ("user", "nb_places", "total", "paid", "paymenttype", "tirage")
+
-    list_filter = ("paid", "tirage")
+    def remain(self, obj):
+        rem = obj.remain
+        if rem:
+            return "%.02f €" % rem
+        else:
+            return "0 €"
+
+    remain.admin_order_field = "remain"
+    remain.short_description = "Reste à payer"
+
+    list_display = ("user", "nb_places", "total", "paid", "remain", "tirage")
+    list_filter = (ParticipantPaidFilter, "tirage")
     search_fields = ("user__username", "user__first_name", "user__last_name")
     actions = ["send_attribs"]
     actions_on_bottom = True
     list_per_page = 400
-    readonly_fields = ("total",)
+    readonly_fields = ("total", "paid")
     readonly_fields_update = ("user", "tirage")
     form = ParticipantAdminForm
 
     def send_attribs(self, request, queryset):
-        datatuple = []
+        emails = []
         for member in queryset.all():
+            subject = "Résultats du tirage au sort"
             attribs = member.attributions.all()
             context = {"member": member.user}
-            shortname = ""
+
+            template_name = ""
             if len(attribs) == 0:
-                shortname = "bda-attributions-decus"
+                template_name = "bda/mails/attributions-decus.txt"
             else:
-                shortname = "bda-attributions"
+                template_name = "bda/mails/attributions.txt"
                 context["places"] = attribs
-            print(context)
+
-            datatuple.append((shortname, context, "bda@ens.fr", [member.user.email]))
+            message = loader.render_to_string(template_name, context)
-        send_mass_custom_mail(datatuple)
+            emails.append((subject, message, "bda@ens.fr", [member.user.email]))
+
+        send_mass_mail(emails)
         count = len(queryset.all())
         if count == 1:
             message_bit = "1 membre a"
@@ -158,17 +191,6 @@ class ParticipantAdmin(ReadOnlyMixin, admin.ModelAdmin):
 
 
 class AttributionAdminForm(forms.ModelForm):
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        if "spectacle" in self.fields:
-            self.fields["spectacle"].queryset = Spectacle.objects.select_related(
-                "location"
-            )
-        if "participant" in self.fields:
-            self.fields["participant"].queryset = Participant.objects.select_related(
-                "user", "tirage"
-            )
-
     def clean(self):
         cleaned_data = super().clean()
         participant = cleaned_data.get("participant")
@@ -181,13 +203,14 @@ class AttributionAdminForm(forms.ModelForm):
                 )
         return cleaned_data
 
+    class Meta:
+        widgets = {
+            "participant": ModelSelect2(url="bda-participant-autocomplete"),
+            "spectacle": ModelSelect2(url="bda-spectacle-autocomplete"),
+        }
+
 
 class AttributionAdmin(ReadOnlyMixin, admin.ModelAdmin):
-    def paid(self, obj):
-        return obj.participant.paid
-
-    paid.short_description = "A payé"
-    paid.boolean = True
     list_display = ("id", "spectacle", "participant", "given", "paid")
     search_fields = (
         "spectacle__title",
@@ -200,7 +223,7 @@ class AttributionAdmin(ReadOnlyMixin, admin.ModelAdmin):
 
 
 class ChoixSpectacleAdmin(admin.ModelAdmin):
-    form = ChoixSpectacleAdminForm
+    autocomplete_fields = ["participant", "spectacle"]
 
     def tirage(self, obj):
         return obj.participant.tirage
@@ -244,15 +267,14 @@ class SalleAdmin(admin.ModelAdmin):
 class SpectacleReventeAdminForm(forms.ModelForm):
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields["confirmed_entry"].queryset = Participant.objects.select_related(
+        qset = Participant.objects.select_related("user", "tirage")
-            "user", "tirage"
+
-        )
+        if self.instance.pk is not None:
-        self.fields["seller"].queryset = Participant.objects.select_related(
+            qset = qset.filter(tirage=self.instance.seller.tirage)
-            "user", "tirage"
+
-        )
+        self.fields["confirmed_entry"].queryset = qset
-        self.fields["soldTo"].queryset = Participant.objects.select_related(
+        self.fields["seller"].queryset = qset
-            "user", "tirage"
+        self.fields["soldTo"].queryset = qset
-        )
 
 
 class SpectacleReventeAdmin(admin.ModelAdmin):

bda/algorithm.py

@@ -2,7 +2,6 @@ import random
 
 
 class Algorithm(object):
-
     shows = None
     ranks = None
     origranks = None

bda/forms.py

@@ -1,8 +1,9 @@
 from django import forms
 from django.forms.models import BaseInlineFormSet
+from django.template import loader
 from django.utils import timezone
 
-from bda.models import Attribution, Spectacle, SpectacleRevente
+from bda.models import SpectacleRevente
 
 
 class InscriptionInlineFormSet(BaseInlineFormSet):
@@ -38,142 +39,146 @@ class TokenForm(forms.Form):
     token = forms.CharField(widget=forms.widgets.Textarea())
 
 
-class AttributionModelMultipleChoiceField(forms.ModelMultipleChoiceField):
+class TemplateLabelField(forms.ModelMultipleChoiceField):
-    def label_from_instance(self, obj):
+    """
-        return str(obj.spectacle)
+    Extends ModelMultipleChoiceField to offer two more customization options :
+    - `label_from_instance` can be used with a template file
+    - the widget rendering template can be specified with `option_template_name`
+    """
 
-
+    def __init__(
-class ReventeModelMultipleChoiceField(forms.ModelMultipleChoiceField):
+        self,
-    def __init__(self, *args, own=True, **kwargs):
+        label_template_name=None,
+        context_object_name="obj",
+        option_template_name=None,
+        *args,
+        **kwargs
+    ):
         super().__init__(*args, **kwargs)
-        self.own = own
+        self.label_template_name = label_template_name
+        self.context_object_name = context_object_name
+        if option_template_name is not None:
+            self.widget.option_template_name = option_template_name
 
     def label_from_instance(self, obj):
-        label = "{show}{suffix}"
+        if self.label_template_name is None:
-        suffix = ""
+            return super().label_from_instance(obj)
-        if self.own:
-            # C'est notre propre revente : informations sur le statut
-            if obj.soldTo is not None:
-                suffix = " -- Vendue à {firstname} {lastname}".format(
-                    firstname=obj.soldTo.user.first_name,
-                    lastname=obj.soldTo.user.last_name,
-                )
-            elif obj.shotgun:
-                suffix = " -- Tirage infructueux"
-            elif obj.notif_sent:
-                suffix = " -- Inscriptions au tirage en cours"
         else:
-            # Ce n'est pas à nous : on ne voit jamais l'acheteur
+            return loader.render_to_string(
-            suffix = " -- Vendue par {firstname} {lastname}".format(
+                self.label_template_name, context={self.context_object_name: obj}
-                firstname=obj.seller.user.first_name, lastname=obj.seller.user.last_name
             )
 
-        return label.format(show=str(obj.attribution.spectacle), suffix=suffix)
+
+# Formulaires pour revente_manage
 
 
 class ResellForm(forms.Form):
-    attributions = AttributionModelMultipleChoiceField(
-        label="",
-        queryset=Attribution.objects.none(),
-        widget=forms.CheckboxSelectMultiple,
-        required=False,
-    )
-
     def __init__(self, participant, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields["attributions"].queryset = (
+        self.fields["attributions"] = TemplateLabelField(
-            participant.attribution_set.filter(spectacle__date__gte=timezone.now())
+            queryset=participant.attribution_set.filter(
+                spectacle__date__gte=timezone.now(), paid=True
+            )
             .exclude(revente__seller=participant)
-            .select_related("spectacle", "spectacle__location", "participant__user")
+            .select_related("spectacle", "spectacle__location", "participant__user"),
+            widget=forms.CheckboxSelectMultiple,
+            required=False,
+            label_template_name="bda/forms/attribution_label_table.html",
+            option_template_name="bda/forms/checkbox_table.html",
+            context_object_name="attribution",
         )
 
 
 class AnnulForm(forms.Form):
-    reventes = ReventeModelMultipleChoiceField(
-        own=True,
-        label="",
-        queryset=Attribution.objects.none(),
-        widget=forms.CheckboxSelectMultiple,
-        required=False,
-    )
-
     def __init__(self, participant, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields["reventes"].queryset = (
+        self.fields["reventes"] = TemplateLabelField(
-            participant.original_shows.filter(
+            label="",
+            queryset=participant.original_shows.filter(
                 attribution__spectacle__date__gte=timezone.now(), soldTo__isnull=True
             )
             .select_related(
                 "attribution__spectacle", "attribution__spectacle__location"
             )
-            .order_by("-date")
+            .order_by("-date"),
-        )
-
-
-class InscriptionReventeForm(forms.Form):
-    spectacles = forms.ModelMultipleChoiceField(
-        queryset=Spectacle.objects.none(),
             widget=forms.CheckboxSelectMultiple,
             required=False,
-    )
+            label_template_name="bda/forms/revente_self_label_table.html",
-
+            option_template_name="bda/forms/checkbox_table.html",
-    def __init__(self, tirage, *args, **kwargs):
+            context_object_name="revente",
-        super().__init__(*args, **kwargs)
-        self.fields["spectacles"].queryset = tirage.spectacle_set.select_related(
-            "location"
-        ).filter(date__gte=timezone.now())
-
-
-class ReventeTirageAnnulForm(forms.Form):
-    reventes = ReventeModelMultipleChoiceField(
-        own=False,
-        label="",
-        queryset=SpectacleRevente.objects.none(),
-        widget=forms.CheckboxSelectMultiple,
-        required=False,
-    )
-
-    def __init__(self, participant, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        self.fields["reventes"].queryset = participant.entered.filter(
-            soldTo__isnull=True
-        ).select_related("attribution__spectacle", "seller__user")
-
-
-class ReventeTirageForm(forms.Form):
-    reventes = ReventeModelMultipleChoiceField(
-        own=False,
-        label="",
-        queryset=SpectacleRevente.objects.none(),
-        widget=forms.CheckboxSelectMultiple,
-        required=False,
-    )
-
-    def __init__(self, participant, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        self.fields["reventes"].queryset = (
-            SpectacleRevente.objects.filter(
-                notif_sent=True, shotgun=False, tirage_done=False
-            )
-            .exclude(confirmed_entry=participant)
-            .select_related("attribution__spectacle")
         )
 
 
 class SoldForm(forms.Form):
-    reventes = ReventeModelMultipleChoiceField(
-        own=True,
-        label="",
-        queryset=Attribution.objects.none(),
-        widget=forms.CheckboxSelectMultiple,
-    )
-
     def __init__(self, participant, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields["reventes"].queryset = (
+        self.fields["reventes"] = TemplateLabelField(
-            participant.original_shows.filter(soldTo__isnull=False)
+            queryset=participant.original_shows.filter(soldTo__isnull=False)
             .exclude(soldTo=participant)
             .select_related(
                 "attribution__spectacle", "attribution__spectacle__location"
+            ),
+            widget=forms.CheckboxSelectMultiple,
+            label_template_name="bda/forms/revente_sold_label_table.html",
+            option_template_name="bda/forms/checkbox_table.html",
+            context_object_name="revente",
         )
+
+
+# Formulaire pour revente_subscribe
+
+
+class InscriptionReventeForm(forms.Form):
+    def __init__(self, tirage, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.fields["spectacles"] = TemplateLabelField(
+            queryset=tirage.spectacle_set.select_related("location").filter(
+                date__gte=timezone.now()
+            ),
+            widget=forms.CheckboxSelectMultiple,
+            required=False,
+            label_template_name="bda/forms/spectacle_label_table.html",
+            option_template_name="bda/forms/checkbox_table.html",
+            context_object_name="spectacle",
+        )
+
+
+# Formulaires pour revente_tirages
+
+
+class ReventeTirageAnnulForm(forms.Form):
+    def __init__(self, participant, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.fields["reventes"] = TemplateLabelField(
+            queryset=participant.entered.filter(soldTo__isnull=True).select_related(
+                "attribution__spectacle", "seller__user"
+            ),
+            widget=forms.CheckboxSelectMultiple,
+            required=False,
+            label_template_name="bda/forms/revente_other_label_table.html",
+            option_template_name="bda/forms/checkbox_table.html",
+            context_object_name="revente",
+        )
+
+
+class ReventeTirageForm(forms.Form):
+    def __init__(self, participant, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+
+        self.fields["reventes"] = TemplateLabelField(
+            queryset=(
+                SpectacleRevente.objects.filter(
+                    notif_sent=True,
+                    shotgun=False,
+                    tirage_done=False,
+                    attribution__spectacle__tirage=participant.tirage,
+                )
+                .exclude(confirmed_entry=participant)
+                .select_related("attribution__spectacle")
+            ),
+            widget=forms.CheckboxSelectMultiple,
+            required=False,
+            label_template_name="bda/forms/revente_other_label_table.html",
+            option_template_name="bda/forms/checkbox_table.html",
+            context_object_name="revente",
         )

bda/management/commands/loadbdadevdata.py

@@ -81,7 +81,7 @@ class Command(MyBaseCommand):
                 shows = random.sample(
                     list(tirage.spectacle_set.all()), tirage.spectacle_set.count() // 2
                 )
-                for (rank, show) in enumerate(shows):
+                for rank, show in enumerate(shows):
                     choices.append(
                         ChoixSpectacle(
                             participant=part,

bda/migrations/0001_initial.py

@@ -6,7 +6,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [migrations.swappable_dependency(settings.AUTH_USER_MODEL)]
 
     operations = [

bda/migrations/0002_add_tirage.py

@@ -35,7 +35,6 @@ def fill_tirage_fields(apps, schema_editor):
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0001_initial")]
 
     operations = [

bda/migrations/0003_update_tirage_and_spectacle.py

@@ -5,7 +5,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0002_add_tirage")]
 
     operations = [

bda/migrations/0004_mails-rappel.py

@@ -5,7 +5,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0003_update_tirage_and_spectacle")]
 
     operations = [

bda/migrations/0005_encoding.py

@@ -5,7 +5,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0004_mails-rappel")]
 
     operations = [

bda/migrations/0006_add_tirage_switch.py

@@ -18,7 +18,6 @@ def forwards_func(apps, schema_editor):
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0005_encoding")]
 
     operations = [

bda/migrations/0007_extends_spectacle.py

@@ -5,7 +5,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0006_add_tirage_switch")]
 
     operations = [

bda/migrations/0008_py3.py

@@ -5,7 +5,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0007_extends_spectacle")]
 
     operations = [

bda/migrations/0009_revente.py

@@ -6,7 +6,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0008_py3")]
 
     operations = [

bda/migrations/0010_spectaclerevente_shotgun.py

@@ -12,15 +12,15 @@ def forwards_func(apps, schema_editor):
 
     for revente in SpectacleRevente.objects.all():
         is_expired = timezone.now() > revente.date_tirage()
-        is_direct = revente.attribution.spectacle.date >= revente.date and timezone.now() > revente.date + timedelta(
+        is_direct = (
-            minutes=15
+            revente.attribution.spectacle.date >= revente.date
+            and timezone.now() > revente.date + timedelta(minutes=15)
         )
         revente.shotgun = is_expired or is_direct
         revente.save()
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0009_revente")]
 
     operations = [

bda/migrations/0011_tirage_appear_catalogue.py

@@ -5,7 +5,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0010_spectaclerevente_shotgun")]
 
     operations = [

bda/migrations/0012_notif_time.py

@@ -5,7 +5,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0011_tirage_appear_catalogue")]
 
     operations = [

bda/migrations/0012_swap_double_choice.py

@@ -13,7 +13,6 @@ def swap_double_choice(apps, schema_editor):
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0011_tirage_appear_catalogue")]
 
     operations = [

bda/migrations/0013_merge_20180524_2123.py

@@ -6,7 +6,6 @@ from django.db import migrations
 
 
 class Migration(migrations.Migration):
-
     dependencies = [("bda", "0012_notif_time"), ("bda", "0012_swap_double_choice")]
 
     operations = []

bda/migrations/0014_attribution_paid_field.py

@@ -0,0 +1,30 @@
+# Generated by Django 2.2 on 2019-06-03 19:13
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [("bda", "0013_merge_20180524_2123")]
+
+    operations = [
+        migrations.AddField(
+            model_name="attribution",
+            name="paid",
+            field=models.BooleanField(default=False, verbose_name="Payée"),
+        ),
+        migrations.AddField(
+            model_name="attribution",
+            name="paymenttype",
+            field=models.CharField(
+                blank=True,
+                choices=[
+                    ("cash", "Cash"),
+                    ("cb", "CB"),
+                    ("cheque", "Chèque"),
+                    ("autre", "Autre"),
+                ],
+                max_length=6,
+                verbose_name="Moyen de paiement",
+            ),
+        ),
+    ]

bda/migrations/0015_move_bda_payment.py

@@ -0,0 +1,36 @@
+# Generated by Django 2.2 on 2019-06-03 19:30
+
+from django.core.exceptions import MultipleObjectsReturned, ObjectDoesNotExist
+from django.db import migrations
+
+
+def set_attr_payment(apps, schema_editor):
+    Attribution = apps.get_model("bda", "Attribution")
+    for attr in Attribution.objects.all():
+        attr.paid = attr.participant.paid
+        attr.paymenttype = attr.participant.paymenttype
+        attr.save()
+
+
+def set_participant_payment(apps, schema_editor):
+    Participant = apps.get_model("bda", "Participant")
+    for part in Participant.objects.all():
+        attr_set = part.attribution_set
+        part.paid = attr_set.exists() and not attr_set.filter(paid=False).exists()
+        try:
+            # S'il n'y a qu'un seul type de paiement, on le set
+            part.paymenttype = (
+                attr_set.values_list("paymenttype", flat=True).distinct().get()
+            )
+        # Sinon, whatever
+        except (ObjectDoesNotExist, MultipleObjectsReturned):
+            pass
+        part.save()
+
+
+class Migration(migrations.Migration):
+    dependencies = [("bda", "0014_attribution_paid_field")]
+
+    operations = [
+        migrations.RunPython(set_attr_payment, set_participant_payment, atomic=True)
+    ]

bda/migrations/0016_delete_participant_paid.py

@@ -0,0 +1,12 @@
+# Generated by Django 2.2 on 2019-06-03 19:30
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+    dependencies = [("bda", "0015_move_bda_payment")]
+
+    operations = [
+        migrations.RemoveField(model_name="participant", name="paid"),
+        migrations.RemoveField(model_name="participant", name="paymenttype"),
+    ]

bda/migrations/0017_participant_accepte_charte.py

@@ -0,0 +1,17 @@
+# Generated by Django 2.2 on 2019-09-18 16:34
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [("bda", "0016_delete_participant_paid")]
+
+    operations = [
+        migrations.AddField(
+            model_name="participant",
+            name="accepte_charte",
+            field=models.BooleanField(
+                default=False, verbose_name="A accepté la charte BdA"
+            ),
+        )
+    ]

bda/migrations/0018_auto_20201021_1818.py

@@ -0,0 +1,37 @@
+# Generated by Django 2.2.12 on 2020-10-21 16:18
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("bda", "0017_participant_accepte_charte"),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name="participant",
+            options={"ordering": ("-tirage", "user__last_name", "user__first_name")},
+        ),
+        migrations.AddField(
+            model_name="tirage",
+            name="archived",
+            field=models.BooleanField(default=False, verbose_name="Archivé"),
+        ),
+        migrations.AlterField(
+            model_name="participant",
+            name="tirage",
+            field=models.ForeignKey(
+                limit_choices_to={"archived": False},
+                on_delete=django.db.models.deletion.CASCADE,
+                to="bda.Tirage",
+            ),
+        ),
+        migrations.AddConstraint(
+            model_name="participant",
+            constraint=models.UniqueConstraint(
+                fields=("tirage", "user"), name="unique_tirage"
+            ),
+        ),
+    ]

bda/migrations/0019_auto_20220630_1245.py

@@ -0,0 +1,23 @@
+# Generated by Django 3.2.13 on 2022-06-30 10:45
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("bda", "0018_auto_20201021_1818"),
+    ]
+
+    operations = [
+        migrations.AlterUniqueTogether(
+            name="choixspectacle",
+            unique_together=set(),
+        ),
+        migrations.AddConstraint(
+            model_name="choixspectacle",
+            constraint=models.UniqueConstraint(
+                fields=("participant", "spectacle"), name="unique_participation"
+            ),
+        ),
+    ]

bda/migrations/0019_auto_20240707_1359.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.2.28 on 2024-07-07 11:59
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('bda', '0018_auto_20201021_1818'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='attribution',
+            name='paymenttype',
+            field=models.CharField(blank=True, choices=[('cash', 'Cash'), ('cb', 'CB'), ('cheque', 'Chèque'), ('virement', 'Virement'), ('autre', 'Autre')], max_length=8, verbose_name='Moyen de paiement'),
+        ),
+    ]

bda/migrations/0020_merge_0019_auto_20220630_1245_0019_auto_20240707_1359.py

@@ -0,0 +1,13 @@
+# Generated by Django 4.2.16 on 2025-02-26 08:23
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("bda", "0019_auto_20220630_1245"),
+        ("bda", "0019_auto_20240707_1359"),
+    ]
+
+    operations = []

bda/models.py

@@ -2,14 +2,14 @@ import calendar
 import random
 from datetime import timedelta
 
-from custommail.models import CustomMail
-from custommail.shortcuts import send_mass_custom_mail
 from django.conf import settings
 from django.contrib.auth.models import User
 from django.contrib.sites.models import Site
 from django.core import mail
+from django.core.mail import EmailMessage, send_mass_mail
 from django.db import models
-from django.db.models import Count
+from django.db.models import Count, Exists
+from django.template import loader
 from django.utils import formats, timezone
 
 
@@ -31,6 +31,7 @@ class Tirage(models.Model):
         "Tirage à afficher dans le catalogue", default=False
     )
     enable_do_tirage = models.BooleanField("Le tirage peut être lancé", default=False)
+    archived = models.BooleanField("Archivé", default=False)
 
     def __str__(self):
         return "%s - %s" % (
@@ -116,16 +117,19 @@ class Spectacle(models.Model):
         bda_generic.nb_attr = 1
         members.append(bda_generic)
         # On écrit un mail personnalisé à chaque participant
-        datatuple = [
+        mails = [
             (
-                "bda-rappel",
+                str(self),
-                {"member": member, "nb_attr": member.nb_attr, "show": self},
+                loader.render_to_string(
+                    "bda/mails/rappel.txt",
+                    context={"member": member, "nb_attr": member.nb_attr, "show": self},
+                ),
                 settings.MAIL_DATA["rappels"]["FROM"],
                 [member.email],
             )
             for member in members
         ]
-        send_mass_custom_mail(datatuple)
+        send_mass_mail(mails)
         # On enregistre le fait que l'envoi a bien eu lieu
         self.rappel_sent = timezone.now()
         self.save()
@@ -147,10 +151,46 @@ PAYMENT_TYPES = (
     ("cash", "Cash"),
     ("cb", "CB"),
     ("cheque", "Chèque"),
+    ("virement", "Virement"),
     ("autre", "Autre"),
 )
 
 
+class Attribution(models.Model):
+    participant = models.ForeignKey("Participant", on_delete=models.CASCADE)
+    spectacle = models.ForeignKey(
+        Spectacle, on_delete=models.CASCADE, related_name="attribues"
+    )
+    given = models.BooleanField("Donnée", default=False)
+    paid = models.BooleanField("Payée", default=False)
+    paymenttype = models.CharField(
+        "Moyen de paiement", max_length=8, choices=PAYMENT_TYPES, blank=True
+    )
+
+    def __str__(self):
+        return "%s -- %s, %s" % (
+            self.participant.user,
+            self.spectacle.title,
+            self.spectacle.date,
+        )
+
+
+class ParticipantPaidQueryset(models.QuerySet):
+    """
+    Un manager qui annote le queryset avec un champ `paid`,
+    indiquant si un participant a payé toutes ses attributions.
+    """
+
+    def annotate_paid(self):
+        # OuterRef permet de se référer à un champ d'un modèle non encore fixé
+        # Voir:
+        # https://docs.djangoproject.com/en/2.2/ref/models/expressions/#django.db.models.OuterRef
+        unpaid = Attribution.objects.filter(
+            participant=models.OuterRef("pk"), paid=False
+        )
+        return self.annotate(paid=~Exists(unpaid))
+
+
 class Participant(models.Model):
     user = models.ForeignKey(User, on_delete=models.CASCADE)
     choices = models.ManyToManyField(
@@ -159,18 +199,25 @@ class Participant(models.Model):
     attributions = models.ManyToManyField(
         Spectacle, through="Attribution", related_name="attributed_to"
     )
-    paid = models.BooleanField("A payé", default=False)
+    tirage = models.ForeignKey(
-    paymenttype = models.CharField(
+        Tirage, on_delete=models.CASCADE, limit_choices_to={"archived": False}
-        "Moyen de paiement", max_length=6, choices=PAYMENT_TYPES, blank=True
     )
-    tirage = models.ForeignKey(Tirage, on_delete=models.CASCADE)
+    accepte_charte = models.BooleanField("A accepté la charte BdA", default=False)
     choicesrevente = models.ManyToManyField(
         Spectacle, related_name="subscribed", blank=True
     )
 
+    objects = ParticipantPaidQueryset.as_manager()
+
     def __str__(self):
         return "%s - %s" % (self.user, self.tirage.title)
 
+    class Meta:
+        ordering = ("-tirage", "user__last_name", "user__first_name")
+        constraints = [
+            models.UniqueConstraint(fields=("tirage", "user"), name="unique_tirage"),
+        ]
+
 
 DOUBLE_CHOICES = (
     ("1", "1 place"),
@@ -207,26 +254,15 @@ class ChoixSpectacle(models.Model):
 
     class Meta:
         ordering = ("priority",)
-        unique_together = (("participant", "spectacle"),)
+        constraints = [
+            models.UniqueConstraint(
+                fields=["participant", "spectacle"], name="unique_participation"
+            )
+        ]
         verbose_name = "voeu"
         verbose_name_plural = "voeux"
 
 
-class Attribution(models.Model):
-    participant = models.ForeignKey(Participant, on_delete=models.CASCADE)
-    spectacle = models.ForeignKey(
-        Spectacle, on_delete=models.CASCADE, related_name="attribues"
-    )
-    given = models.BooleanField("Donnée", default=False)
-
-    def __str__(self):
-        return "%s -- %s, %s" % (
-            self.participant.user,
-            self.spectacle.title,
-            self.spectacle.date,
-        )
-
-
 class SpectacleRevente(models.Model):
     attribution = models.OneToOneField(
         Attribution, on_delete=models.CASCADE, related_name="revente"
@@ -329,21 +365,24 @@ class SpectacleRevente(models.Model):
         BdA-Revente à tous les intéressés.
         """
         inscrits = self.attribution.spectacle.subscribed.select_related("user")
-        datatuple = [
+        mails = [
             (
-                "bda-revente",
+                "BdA-Revente : {}".format(self.attribution.spectacle.title),
-                {
+                loader.render_to_string(
+                    "bda/mails/revente-new.txt",
+                    context={
                         "member": participant.user,
                         "show": self.attribution.spectacle,
                         "revente": self,
                         "site": Site.objects.get_current(),
                     },
+                ),
                 settings.MAIL_DATA["revente"]["FROM"],
                 [participant.user.email],
             )
             for participant in inscrits
         ]
-        send_mass_custom_mail(datatuple)
+        send_mass_mail(mails)
         self.notif_sent = True
         self.notif_time = timezone.now()
         self.save()
@@ -354,20 +393,23 @@ class SpectacleRevente(models.Model):
         leur indiquer qu'il est désormais disponible au shotgun.
         """
         inscrits = self.attribution.spectacle.subscribed.select_related("user")
-        datatuple = [
+        mails = [
             (
-                "bda-shotgun",
+                "BdA-Revente : {}".format(self.attribution.spectacle.title),
-                {
+                loader.render_to_string(
+                    "bda/mails/revente-shotgun.txt",
+                    context={
                         "member": participant.user,
                         "show": self.attribution.spectacle,
                         "site": Site.objects.get_current(),
                     },
+                ),
                 settings.MAIL_DATA["revente"]["FROM"],
                 [participant.user.email],
             )
             for participant in inscrits
         ]
-        send_mass_custom_mail(datatuple)
+        send_mass_mail(mails)
         self.notif_sent = True
         self.notif_time = timezone.now()
         # Flag inutile, sauf si l'horloge interne merde
@@ -399,31 +441,30 @@ class SpectacleRevente(models.Model):
                     "show": spectacle,
                 }
 
-                c_mails_qs = CustomMail.objects.filter(
+                subject = "BdA-Revente : {}".format(spectacle.title)
-                    shortname__in=[
-                        "bda-revente-winner",
-                        "bda-revente-loser",
-                        "bda-revente-seller",
-                    ]
-                )
-
-                c_mails = {cm.shortname: cm for cm in c_mails_qs}
 
                 mails.append(
-                    c_mails["bda-revente-winner"].get_message(
+                    EmailMessage(
-                        context,
+                        subject=subject,
+                        body=loader.render_to_string(
+                            "bda/mails/revente-tirage-winner.txt",
+                            context=context,
+                        ),
                         from_email=settings.MAIL_DATA["revente"]["FROM"],
                         to=[winner.user.email],
                     )
                 )
-
                 mails.append(
-                    c_mails["bda-revente-seller"].get_message(
+                    EmailMessage(
-                        context,
+                        subject=subject,
+                        body=loader.render_to_string(
+                            "bda/mails/revente-tirage-seller.txt",
+                            context=context,
+                        ),
                         from_email=settings.MAIL_DATA["revente"]["FROM"],
                         to=[seller.user.email],
                         reply_to=[winner.user.email],
-                    )
+                    ),
                 )
 
                 # Envoie un mail aux perdants
@@ -433,11 +474,15 @@ class SpectacleRevente(models.Model):
                         new_context["acheteur"] = inscrit.user
 
                         mails.append(
-                            c_mails["bda-revente-loser"].get_message(
+                            EmailMessage(
-                                new_context,
+                                subject=subject,
+                                body=loader.render_to_string(
+                                    "bda/mails/revente-tirage-loser.txt",
+                                    context=new_context,
+                                ),
                                 from_email=settings.MAIL_DATA["revente"]["FROM"],
                                 to=[inscrit.user.email],
-                            )
+                            ),
                         )
 
                 mail_conn = mail.get_connection()

bda/static/bda/css/bda.css

@@ -0,0 +1,125 @@
+form#tokenform {
+    text-align: center; 
+    font-size: 2em;
+}
+
+label {
+    margin-right: 10px; 
+    vertical-align: top;
+}
+
+form#tokenform textarea {
+    font-size: 2em; 
+    width: 350px; 
+    height: 200px; 
+    font-family: 'Droif Serif', serif;
+}
+
+/* wft ?
+input {
+    width: 400px; 
+    font-size: 2em;
+}*/
+
+ul.losers {
+    display: inline; 
+    margin: 0; 
+    padding: 0;
+}
+
+ul.losers li {
+    display: inline;
+}
+
+span.details {
+    font-size: 0.7em;
+}
+
+td {
+    border: 0px solid black; 
+    padding: 2px;
+}
+.attribresult {
+    margin: 10px 0px;
+}
+
+.spectacle-passe {
+    opacity:0.5;
+}
+
+/** JQuery-Confirm box **/
+
+.jconfirm .jconfirm-bg {
+    background-color: rgb(0,0,0,0.6) !important;
+}
+
+.jconfirm .jconfirm-box {
+    padding:0;
+    border-radius:0 !important;
+    font-family:Roboto;
+}
+
+.jconfirm .jconfirm-box .content-pane {
+    border-bottom:1px solid #ddd;
+    margin: 0px !important;
+}
+
+.jconfirm .jconfirm-box .content {
+    padding: 5px;
+}
+
+.jconfirm .jconfirm-box .content-pane {
+    border-bottom:1px solid #ddd;
+    margin: 0px !important;
+}
+
+.jconfirm .jconfirm-box .content {
+    padding: 10px;
+}
+
+.jconfirm .jconfirm-box .content a,
+.jconfirm .jconfirm-box .content a:hover {
+    color: #D81138;
+    font-weight: bold;
+}
+
+.jconfirm .jconfirm-box .buttons {
+    margin-top:-6px; /* j'arrive pas à voir pk y'a un espace au dessus sinon... */
+    padding:0;
+    height:40px;
+}
+
+.jconfirm .jconfirm-box .buttons button {
+    min-width:40px;
+    height:100%;
+    margin:0;
+    margin:0 !important;
+    border-radius: 0 !important;
+}
+
+.jconfirm .jconfirm-box .buttons button:first-child:focus,
+.jconfirm .jconfirm-box .buttons button:first-child:hover {
+    color:#FFF !important;
+    background:forestgreen !important;
+}
+
+.jconfirm .jconfirm-box .buttons button:nth-child(2):focus,
+.jconfirm .jconfirm-box .buttons button:nth-child(2):hover {
+    color:#FFF !important;
+    background:#D93A32 !important;
+}
+
+.jconfirm .jconfirm-box div.title-c .title {
+    display: block;
+
+    padding:0 15px;
+    height:40px;
+    line-height:40px;
+
+    font-family:Dosis;
+    font-size:20px;
+    font-weight:bold;
+
+    color:#FFF;
+    background-color:rgb(222, 130, 107);
+}

bda/static/css/bda.css

@@ -1,48 +0,0 @@
-form#tokenform {
-    text-align: center; 
-    font-size: 2em;
-}
-
-label {
-    margin-right: 10px; 
-    vertical-align: top;
-}
-
-form#tokenform textarea {
-    font-size: 2em; 
-    width: 350px; 
-    height: 200px; 
-    font-family: 'Droif Serif', serif;
-}
-
-/* wft ?
-input {
-    width: 400px; 
-    font-size: 2em;
-}*/
-
-ul.losers {
-    display: inline; 
-    margin: 0; 
-    padding: 0;
-}
-
-ul.losers li {
-    display: inline;
-}
-
-span.details {
-    font-size: 0.7em;
-}
-
-td {
-    border: 0px solid black; 
-    padding: 2px;
-}
-.attribresult {
-    margin: 10px 0px;
-}
-
-.spectacle-passe {
-    opacity:0.5;
-}

bda/templates/bda-attrib.html

@@ -1,8 +1,8 @@
 {% extends "base_title.html" %}
-{% load staticfiles %}
+{% load static %}
 
 {% block extra_head %}
-	<link type="text/css" rel="stylesheet" href="{% static "css/bda.css" %}" />
+	<link type="text/css" rel="stylesheet" href="{% static "bda/css/bda.css" %}" />
 {% endblock %}
 
 {% block realcontent %}

bda/templates/bda/etat-places.html

@@ -1,5 +1,5 @@
 {% extends "base_title.html" %}
-{% load staticfiles %}
+{% load static %}
 
 {% block realcontent %}
   <h2>État des inscriptions BdA</h2>
@@ -42,9 +42,6 @@
       Total&nbsp;: {{ total }} place{{ total|pluralize }} demandée{{ total|pluralize }}
       sur {{ proposed }} place{{ proposed|pluralize }} proposée{{ proposed|pluralize }}
     </span>
-  <script type="text/javascript"
-          src="{% static "js/joequery-Stupid-Table-Plugin/stupidtable.js" %}">
-  </script>
   <script type="text/javascript">
     $(function(){
     $("table.etat-bda").stupidtable();

bda/templates/bda/forms/attribution_label_table.html

@@ -0,0 +1 @@
+{% include 'bda/forms/spectacle_label_table.html' with spectacle=attribution.spectacle %}

bda/templates/bda/forms/checkbox_table.html

@@ -0,0 +1,4 @@
+<tr>
+    <td><input type="{{ widget.type }}" name="{{ widget.name }}" {% if widget.value != None %} value="{{ widget.value|stringformat:'s' }}"{% endif %}{% include "django/forms/widgets/attrs.html" %}></td>
+    {{ widget.label }}
+</tr>

bda/templates/bda/forms/date_tirage.html

@@ -0,0 +1 @@
+<td data-sort-value="{{ revente.date_tirage | date:"U" }}">{{ revente.date_tirage }}</td>

bda/templates/bda/forms/revente_other_label_table.html

@@ -0,0 +1,3 @@
+{% include 'bda/forms/spectacle_label_table.html' with spectacle=revente.attribution.spectacle %}
+{% with user=revente.seller.user %} <td>{{user.first_name}} {{user.last_name}}</td> {% endwith%}
+{% include 'bda/forms/date_tirage.html' %}

bda/templates/bda/forms/revente_self_label_table.html

@@ -0,0 +1,2 @@
+{% include 'bda/forms/spectacle_label_table.html' with spectacle=revente.attribution.spectacle %}
+{% include 'bda/forms/date_tirage.html' %}

bda/templates/bda/forms/revente_sold_label_table.html

@@ -0,0 +1,4 @@
+{% include 'bda/forms/spectacle_label_table.html' with spectacle=revente.attribution.spectacle %}
+{% with user=revente.soldTo.user %}
+<td><a href="mailto:{{ user.email }}">{{user.first_name}} {{user.last_name}}</a></td>
+{% endwith %}

bda/templates/bda/forms/spectacle_label_table.html

@@ -0,0 +1,4 @@
+<td>{{ spectacle.title }}</td>
+<td data-sort-value="{{ spectacle.timestamp }}">{{ spectacle.date }}</td>
+<td data-sort-value="{{ spectacle.location }}">{{ spectacle.location }}</td>
+<td data-sort-value="{{ spectacle.price |stringformat:".3f" }}">{{ spectacle.price |floatformat }}€</td>

bda/templates/bda/inscription-tirage.html

@@ -1,11 +1,13 @@
 {% extends "base_title.html" %}
-{% load staticfiles %}
+{% load static %}
 
 {% block extra_head %}
-<script src="{% static 'js/jquery-ui.min.js' %}" type="text/javascript"></script>
+<script type="text/javascript" src="{% static 'vendor/jquery/jquery-ui.min.js' %}" ></script>
-<script src="{% static "js/jquery.ui.touch-punch.min.js" %}" type="text/javascript"></script>
+<script type="text/javascript" src="{% static "vendor/jquery/jquery-confirm.js" %}"></script>
-<link type="text/css" rel="stylesheet" href="{% static "css/jquery-ui.min.css" %}" />
+<script type="text/javascript" src="{% static 'gestioncof/vendor/jquery.ui.touch-punch.min.js' %}" ></script>
-<link type="text/css" rel="stylesheet" href="{% static "css/bda.css" %}" />
+<link type="text/css" rel="stylesheet" href="{% static 'vendor/jquery/jquery-confirm.css' %}">
+<link type="text/css" rel="stylesheet" href="{% static 'vendor/jquery/jquery-ui.min.css' %}" />
+<link type="text/css" rel="stylesheet" href="{% static 'bda/css/bda.css' %}" />
 {% endblock %}
 
 {% block realcontent %}
@@ -52,6 +54,11 @@ var django = {
                 } else {
                     deleteInput.attr("checked", true);
                 }
+            } else {
+                // Reset the default values
+                var selects = $(form).find("select");
+                $(selects[0]).val("");
+                $(selects[1]).val("1");
             }
             // callback
         });
@@ -113,11 +120,50 @@ var django = {
                             });
                         </script>
                         <input type="hidden" name="dbstate" value="{{ dbstate }}" />
-                        <input type="submit" class="btn btn-primary" value="Enregistrer" />
+                        <input type="submit" class="btn btn-primary" id="bda-inscr" value="Enregistrer" />
                     </div>
                     <p class="footnotes">
                         <sup>1</sup>: cette liste de v&oelig;ux est ordonnée (du plus important au moins important), pour ajuster la priorité vous pouvez déplacer chaque v&oelig;u.<br />
                     </p>
                 </div>
             </form>
+
+{% if not charte %}
+<script>
+(function ($) {
+    var charte_ok = false ;
+    function link_charte() {
+        $.confirm({
+            title: 'Charte du BdA',
+            columnClass: 'col-md-6 col-md-offset-3',
+            content: `
+            <div>
+                En vous inscrivant à ce tirage du Bureau des Arts, vous vous engagez à \
+                respecter la charte du BdA:</br> \
+                <a target="_blank" href='https://bda.ens.fr/lequipe/charte-bda/'>https://bda.ens.fr/lequipe/charte-bda/</a>
+            </div>`,
+            backgroundDismiss: true,
+            opacity: 1,
+            animation:'top',
+            closeAnimation:'bottom',
+            keyboardEnabled: true,
+            confirmButton: '<span class="glyphicon glyphicon-ok"></span>',
+            cancelButton: '<span class="glyphicon glyphicon-remove"></span>',
+            confirm: function() {
+                charte_ok = true ;
+                $("#bda_form").submit();
+            },
+        });
+    }
+    $(document).ready(function($) {
+        $("#bda_form").submit(function(e) {
+            if (!charte_ok) {
+                e.preventDefault();
+                link_charte();
+            }
+        })
+    })
+})(django.jQuery);
+</script>
+{% endif %}
 {% endblock %}

bda/templates/bda/mails-rappel.html

@@ -26,13 +26,6 @@
     
     <hr \>
 
-    <p>
-      <em>Note :</em> le template de ce mail peut être modifié à
-      <a href="{% url 'admin:custommail_custommail_change' custommail.pk %}">cette adresse</a>
-    </p>
-
-    <hr \>
-
     <h3>Forme des mails</h3>
 
     <h4>Une seule place</h4>

bda/templates/bda/mails/attributions-decus.txt

@@ -0,0 +1,10 @@
+Cher-e {{ member.first_name }},
+
+Tu t'es inscrit-e pour le tirage au sort du BdA. Malheureusement, tu n'as
+obtenu aucune place.
+
+Nous proposons cependant de nombreuses offres hors-tirage tout au long de
+l'année, et nous t'invitons à nous contacter si l'une d'entre elles
+t'intéresse !
+-- 
+Le Bureau des Arts

bda/templates/bda/mails/attributions.txt

@@ -0,0 +1,29 @@
+Cher-e {{ member.first_name }},
+
+Tu t'es inscrit-e pour le tirage au sort du BdA. Tu as été sélectionné-e
+pour les spectacles suivants :
+{% for place in places %}
+- 1 place pour {{ place }}{% endfor %}
+
+*Paiement*
+Au burô :
+L'intégralité de ces places de spectacles est à régler dès maintenant, au bureau du COF pendant les heures de permanences (lundi, mardi, jeudi entre 12h et 14h et entre 18h30 et 19h30, mercredi entre 18h30 et 19h30, vendredi entre 12h et 14h). Les places sont à régler AVANT les représentations. Si vous êtes en vacances, vous pourrez venir les régler dès votre retour. Il est demandé à chacun·e de prendre garde à honorer l’ensemble des places qui lui sont attribuées et de s'engager de fait à payer la ou les place(s) qui lui sont attribuées.
+
+Par virements :
+L'intégralité de ces places de spectacles est à régler dès maintenant par virements. Il vous sera demandé d'envoyer une confirmation de l'envoi de virement à bda@ens.fr.
+IBAN AEENS : FR76 4255 9100 0008 0263 8331 927
+Motif de virements : AVR25(ou MAI25)-tirageprintemps-NOM-prénom
+
+Les places sont à régler AVANT les représentations. Il est demandé à chacun·e de prendre garde à honorer l’ensemble des places qui lui sont attribuées et de s'engager de fait à payer la ou les place(s) qui lui sont attribuées.
+Des facilités de paiement sont bien évidemment possibles : nous pouvons ne pas encaisser le chèque immédiatement, ou bien découper votre paiement en deux fois. Pour ceux qui ne pourraient pas venir payer au bureau, merci de nous contacter par mail.
+
+*Mode de retrait des places*
+Au moment du paiement, certaines places vous seront remises directement, d'autres seront à récupérer au cours de l'année, d'autres encore seront nominatives et à retirer le soir même dans les théâtres correspondants. Pour chaque spectacle, vous recevrez un mail quelques jours avant la représentation vous indiquant le mode de retrait.
+Nous vous rappelons que l'obtention de places du BdA vous engage à respecter les règles de fonctionnement :
+https://bda.ens.fr/lequipe/charte-bda/
+
+Un système de revente des places via les mails BdA-revente est disponible directement sur votre compte GestioCOF. Pour pouvoir l'utiliser, il faut que vous ayez payé vos places en amont.
+
+En vous souhaitant de très beaux spectacles tout au long de l'année,
+-- 
+Le Bureau des Arts

bda/templates/bda/mails/rappel.txt

@@ -0,0 +1,23 @@
+Bonjour {{ member.first_name }},
+
+Nous te rappellons que tu as eu la chance d'obtenir {{ nb_attr|pluralize:"une place,deux places" }}
+pour {{ show.title }}, le {{ show.date }} au {{ show.location }}. N'oublie pas de t'y rendre !
+{% if nb_attr == 2 %}
+Tu as obtenu deux places pour ce spectacle. Nous te rappelons que
+ces places sont strictement réservées aux personnes de moins de 28 ans.
+{% endif %}
+{% if show.listing %}Pour ce spectacle, tu as reçu {{ nb_attr|pluralize:"une place,des places" }} sur
+listing. Il te faudra donc te rendre 15 minutes en avance sur les lieux de la représentation
+pour {{ nb_attr|pluralize:"la,les" }} retirer.
+{% else %}Pour assister à ce spectacle, tu dois présenter les billets qui ont
+été distribués au burô.
+{% endif %}
+
+Si tu ne peux plus assister à cette représentation, tu peux
+revendre ta place via BdA-revente, accessible directement sur
+GestioCOF (lien "revendre une place du premier tirage" sur la page
+d'accueil https://www.cof.ens.fr/gestion/).
+
+En te souhaitant un excellent spectacle,
+-- 
+Le Bureau des Arts

bda/templates/bda/mails/revente-new.txt

@@ -0,0 +1,12 @@
+Bonjour {{ member.first_name }}
+
+Une place pour le spectacle {{ show.title }} ({{ show.date }})
+a été postée sur BdA-Revente.
+
+Si ce spectacle t'intéresse toujours, merci de nous le signaler en cliquant
+sur ce lien : https://{{ site }}{% url "bda-revente-confirm" revente.id %}.
+Dans le cas où plusieurs personnes seraient intéressées, nous procèderons à
+un tirage au sort le {{ revente.date_tirage|date:"DATE_FORMAT" }}.
+
+Chaleureusement,
+Le BdA

bda/templates/bda/mails/revente-seller.txt

@@ -0,0 +1,13 @@
+Bonjour {{ vendeur.first_name }},
+
+Tu t’es bien inscrit·e pour revendre une place pour {{ show.title }}.
+
+{% with revente.date_tirage as time %}
+Le tirage au sort entre tout·e·s les racheteuse·eur·s potentiel·le·s aura lieu
+le {{ time|date:"DATE_FORMAT" }} à {{ time|time:"TIME_FORMAT" }} (dans {{time|timeuntil }}).
+Si personne ne s’est inscrit pour racheter la place, celle-ci apparaîtra parmi
+les « Places disponibles immédiatement à la revente » sur GestioCOF.
+{% endwith %}
+
+Bonne revente !
+Le Bureau des Arts

bda/templates/bda/mails/revente-shotgun-seller.txt

@@ -0,0 +1,6 @@
+Bonjour {{ vendeur.first_name }} !
+
+Je souhaiterais racheter ta place pour {{ show.title }} le {{ show.date }} ({{ show.location }}) à {{ show.price|floatformat:2 }}€.
+Contacte-moi si tu es toujours intéressé·e !
+
+{{ acheteur.get_full_name }} ({{ acheteur.email }})

bda/templates/bda/mails/revente-shotgun.txt

@@ -0,0 +1,11 @@
+Bonjour {{ member.first_name }}
+
+Une place pour le spectacle {{ show.title }} ({{ show.date }})
+a été postée sur BdA-Revente.
+
+Puisque ce spectacle a lieu dans moins de 24h, il n'y a pas de tirage au sort pour
+cette place : elle est disponible immédiatement à l'adresse
+https://{{ site }}{% url "bda-revente-buy" show.id %}, à la disposition de tous.
+
+Chaleureusement,
+Le BdA

bda/templates/bda/mails/revente-tirage-loser.txt

@@ -0,0 +1,9 @@
+Bonjour {{ acheteur.first_name }},
+
+Tu t'étais inscrit·e pour la revente de la place de {{ vendeur.get_full_name }}
+pour {{ show.title }}.
+Malheureusement, une autre personne a été tirée au sort pour racheter la place.
+Tu pourras certainement retenter ta chance pour une autre revente !
+
+À très bientôt,
+Le Bureau des Arts

bda/templates/bda/mails/revente-tirage-seller.txt

@@ -0,0 +1,7 @@
+Bonjour {{ vendeur.first_name }},
+
+La personne tirée au sort pour racheter ta place pour {{ show.title }} est {{ acheteur.get_full_name }}.
+Tu peux le/la contacter à l'adresse {{ acheteur.email }}, ou en répondant à ce mail.
+
+Chaleureusement,
+Le BdA

bda/templates/bda/mails/revente-tirage-winner.txt

@@ -0,0 +1,7 @@
+Bonjour {{ acheteur.first_name }},
+
+Tu as été tiré·e au sort pour racheter une place pour {{ show.title }} le {{ show.date }} ({{ show.location }}) à {{ show.price|floatformat:2 }}€.
+Tu peux contacter le/la vendeur·se à l'adresse {{ vendeur.email }}.
+
+Chaleureusement,
+Le BdA

bda/templates/bda/participants.html

@@ -1,5 +1,5 @@
 {% extends "base_title.html" %}
-{% load staticfiles %}
+{% load static %}
 
 {% block realcontent %}
     <h2>{{ spectacle }}</h2>
@@ -16,7 +16,7 @@
         <tbody>
         {% for participant in participants %}
             <tr>
-                <td data-sort-value="{{ participan.name}}">{{participant.name}}</td>
+                <td data-sort-value="{{ participant.name}}">{{participant.name}}</td>
                 <td data-sort-value="{{participant.nb_places}}">{{participant.nb_places}} place{{participant.nb_places|pluralize}}</td>
                 <td data-sort-value="{{participant.email}}">{{participant.email}}</td>
                 <td data-sort-value="{{ participant.paid}}" class={%if participant.paid %}"greenratio"{%else%}"redratio"{%endif%}>
@@ -38,7 +38,7 @@
     <h3><a href="{% url "admin:bda_attribution_add" %}?spectacle={{spectacle.id}}"><span class="glyphicon glyphicon-plus-sign"></span> Ajouter une attribution</a></h3>
     <div>
       <div>
-        <button class="btn btn-default" type="button" onclick="toggle('export-mails')">Afficher/Cacher mails participants</button>
+        <button class="btn btn-default" type="button" onclick="toggle('export-mails')">Afficher/Cacher mails participant⋅e⋅s</button>
 	<pre id="export-mails" style="display:none">{% spaceless %}
           {% for participant in participants %}{{ participant.email }}, {% endfor %}
 	{% endspaceless %}</pre>
@@ -56,9 +56,7 @@
         <a href="{% url 'bda-rappels' spectacle.id %}">Page d'envoi manuel des mails de rappel</a>
       </div>
 
-    <script type="text/javascript"
+    <script type="text/javascript">
-            src="{% static "js/joequery-Stupid-Table-Plugin/stupidtable.js" %}"></script>
-    <script>
     function toggle(id) {
         var pre = document.getElementById(id) ;
         pre.style.display = pre.style.display == "none" ? "block" : "none" ;

bda/templates/bda/resume_places.html

@@ -10,9 +10,20 @@
                 <td>{{place.spectacle.location}}</td>
                 <td>{{place.spectacle.date}}</td>
                 <td>{% if place.double %}deux places{%else%}une place{% endif %}</td>
+                <td>{% if place.spectacle.listing %}sur listing{% else %}place physique{% endif %}</td>
+                <td>
+                  {% if place.unpaid == 0 %}
+                    Payé
+                  {% elif place.unpaid == 1 %}
+                    Une place à payer ({{place.unpaid_price|floatformat}}€)
+                  {% else %}
+                    Deux places à payer ({{place.unpaid_price|floatformat}}€)
+                  {% endif %}
+                </td>
             </tr>
         {% endfor %}
     </table>
+    <h4 class="bda-prix">Reste à payer : {{ unpaid|floatformat }}€</h4>
     <h4 class="bda-prix">Total à payer : {{ total|floatformat }}€</h4>
     <br/>
     <p>Ne manque pas un spectacle avec le

bda/templates/bda/revente/confirm-shotgun.html

@@ -1,5 +1,5 @@
 {% extends "base_title.html" %}
-{% load staticfiles %}
+{% load static %}
 
 
 {%block realcontent %}

bda/templates/bda/revente/confirmed.html

@@ -1,5 +1,5 @@
 {% extends "base_title.html" %}
-{% load staticfiles %}
+{% load static %}
 
 {% block realcontent %}
 <h2>Inscription à une revente</h2>

bda/templates/bda/revente/mail-success.html

@@ -1,5 +1,5 @@
 {% extends "base_title.html" %}
-{% load staticfiles %}
+{% load static %}
 
 {% block realcontent %}
 

bda/templates/bda/revente/manage.html

@@ -1,12 +1,11 @@
 {% extends "base_title.html" %}
-{% load bootstrap %}
+{% load static %}
 
 {% block realcontent %}
 
 <h2>Gestion des places que je revends</h2>
-{% with resell_attributions=resellform.attributions annul_reventes=annulform.reventes sold_reventes=soldform.reventes %}
 
-{% if resell_attributions %}
+{% if resell_exists %}
 <br />
 
 <h3>Places non revendues</h3>
@@ -17,10 +16,21 @@
     ensuite 1h pour changer d'avis avant que la revente soit confirmée et
     que les notifications soient envoyées aux intéressé·e·s.
   </div>
-    <div class="bootstrap-form-reduce">
   {% csrf_token %}
-        {{ resellform|bootstrap }}
+  <table  class="table table-striped stupidtable">
-    </div>
+    <thead>
+      <tr>
+        <th></th>
+        <th data-sort="string">Titre</th>
+        <th data-sort="string">Lieu</th>
+        <th data-sort="int">Date</th>
+        <th data-sort="int">Prix</th>
+      </tr>
+    </thead>
+    <tbody>
+      {% for checkbox in resellform.attributions %}{{ checkbox }}{% endfor %}
+    </tbody>
+  </table> 
   <div class="form-actions">
     <input type="submit" class="btn btn-primary" name="resell" value="Revendre les places sélectionnées">
   </div>
@@ -29,7 +39,7 @@
 <hr />
 {% endif %}
 
-{% if annul_reventes %}
+{% if annul_exists %}
 <h3>Places en cours de revente</h3>
 <form action="" method="post">
   <div class="bg-info text-info center-block">
@@ -37,22 +47,28 @@
     Vous pouvez annuler les reventes qui n'ont pas encore trouvé preneur·se.
   </div>
   {% csrf_token %}
-  <div class='form-group'>
+  <table  class="table table-striped stupidtable">
-      <div class='multiple-checkbox'>
+    <thead>
-          <ul>
+      <tr>
-              {% for revente in annul_reventes %}
+        <th></th>
-                  <li>{{ revente.tag }} {{ revente.choice_label }}</li>
+        <th data-sort="string">Titre</th>
-              {% endfor %}
+        <th data-sort="int">Date</th>
-          </ul>
+        <th data-sort="string">Lieu</th>
-      </div>
+        <th data-sort="int">Prix</th>
-  </div>
+        <th data-sort="int">Tirage le</th>
+      </tr>
+    </thead>
+    <tbody>
+      {% for checkbox in annulform.reventes %}{{ checkbox }}{% endfor %}
+    </tbody>
+  </table>
   <input type="submit" class="btn btn-primary" name="annul" value="Annuler les reventes sélectionnées">
 </form>
 
 <hr />
 {% endif %}
 
-{% if sold_reventes %}
+{% if sold_exists %}
 <h3>Places revendues</h3>
 <form action="" method="post">
   <div class="bg-info text-info center-block">
@@ -64,17 +80,42 @@
       vous ne parvenez pas à entrer en contact avec la personne tirée au
       sort.
   </div>
-    <div class="bootstrap-form-reduce">
   {% csrf_token %}
-        {{ soldform|bootstrap }}
+  <table  class="table table-striped stupidtable">
-    </div>
+    <thead>
+      <tr>
+        <th></th>
+        <th data-sort="string">Titre</th>
+        <th data-sort="int">Date</th>
+        <th data-sort="string">Lieu</th>
+        <th data-sort="int">Prix</th>
+        <th>Vendue à</th>
+      </tr>
+    </thead>
+    <tbody>
+      {% for checkbox in soldform.reventes %}{{ checkbox }}{% endfor %}
+    </tbody>
+  </table>  
     <button type="submit" class="btn btn-primary" name="transfer">Transférer</button>
     <button type="submit" class="btn btn-primary" name="reinit">Réinitialiser</button>
 </form>
 {% endif %}
-{% if not resell_attributions and not annul_reventes and not sold_reventes %}
+{% if not resell_exists and not annul_exists and not sold_exists %}
 <p>Plus de reventes possibles !</p>
 {% endif %}
 
-{% endwith %}
+<script language="JavaScript">
+  $(function(){
+      $("table.stupidtable").stupidtable();
+  });
+
+  $("tr").click(function() {
+      $(this).find("input[type=checkbox]").click()
+  });
+  
+  $("input[type=checkbox]").click(function(e) {
+    e.stopPropagation();
+  });
+</script>
+
 {% endblock %}

bda/templates/bda/revente/notpaid.html

@@ -1,6 +0,0 @@
-{% extends "base_title.html" %}
-
-{% block realcontent %}
-    <h2><strong>Nope</strong></h2>
-    <p>Avant de revendre des places, il faut aller les payer !</p>
-{% endblock %}

bda/templates/bda/revente/shotgun.html

@@ -2,11 +2,26 @@
 
 {% block realcontent %}
   <h2>Places disponibles immédiatement</h2>
-  {% if shotgun %}
+  {% if spectacles %}
-    <ul class="list-unstyled">
+    <table class="table table-striped stupidtable" id="bda-shotgun">
-    {% for spectacle in shotgun %}
+        <thead>
-    <li><a href="{% url "bda-revente-buy" spectacle.id %}">{{spectacle}}</a></li>
+          <tr>
+            <th data-sort="string">Titre</th>
+            <th data-sort="int">Date</th>
+            <th data-sort="string">Lieu</th>
+            <th data-sort="int">Prix</th>
+            <th></th>
+          </tr>
+        </thead>
+        <tbody>
+          {% for spectacle in spectacles %}
+          <tr>
+            {% include "bda/forms/spectacle_label_table.html" with spectacle=spectacle %}
+            <td class="button"><a role="button" class="btn btn-primary" href="{% url 'bda-revente-buy' spectacle.id %}">Racheter</a>
+          </tr>
           {% endfor %}
+        </tbody>
+      </table>
   {% else %}
     <p> Pas de places disponibles immédiatement, désolé !</p>
   {% endif %}

bda/templates/bda/revente/subscribe.html

@@ -1,12 +1,12 @@
 {% extends "base_title.html" %}
-{% load bootstrap %}
+{% load static %}
 
 {% block realcontent %}
   <h2>Inscriptions pour BdA-Revente</h2>
   <form action="" class="form-horizontal" method="post">
     <div class="bg-info text-info center-block">
         <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
-        Cochez les spectacles pour lesquels vous souhaitez recevoir un
+        Cochez les spectacles pour lesquels vous souhaitez recevoir une
         notification quand une place est disponible en revente. <br />
         Lorsque vous validez vos choix, si un tirage au sort est en cours pour
         un des spectacles que vous avez sélectionné, vous serez automatiquement
@@ -21,14 +21,21 @@
       <button type="button"
               class="btn btn-primary"
               onClick="select(false)">Tout désélectionner</button>
+      <table  class="table table-striped stupidtable">
+        <thead>
+          <tr>
+            <th></th>
+            <th data-sort="string">Titre</th>
+            <th data-sort="int">Date</th>
+            <th data-sort="string">Lieu</th>
+            <th data-sort="int">Prix</th>
+          </tr>
+        </thead>
+        <tbody>
+          {% for checkbox in form.spectacles %}{{ checkbox }}{% endfor %}
+        </tbody>
+      </table>
 
-      <div class="multiple-checkbox">
-        <ul>
-          {% for checkbox in form.spectacles %}
-            <li>{{ checkbox }}</li>
-          {% endfor %}
-        </ul>
-      </div>
     </div>
     <input type="submit"
            class="btn btn-primary"
@@ -36,11 +43,22 @@
   </form>
 
 <script language="JavaScript">
-function select(check) {
+  function select(check) {
       checkboxes = document.getElementsByName("spectacles");
       for(var i=0, n=checkboxes.length; i < n; i++) {
           checkboxes[i].checked = check;
       }
-}
+  }
+  $(function(){
+      $("table.stupidtable").stupidtable();
+  });
+  
+  $("tr").click(function() {
+      $(this).find("input[type=checkbox]").click()
+  });
+
+  $("input[type=checkbox]").click(function(e) {
+    e.stopPropagation();
+  });
 </script>
 {% endblock %}

bda/templates/bda/revente/tirages.html

@@ -1,45 +1,70 @@
 {% extends "base_title.html" %}
-{% load bootstrap %}
+{% load static %}
 
 {% block realcontent %}
 
 <h2>Tirages au sort de reventes</h2>
 
-{% if annulform.reventes %}
+{% if annul_exists %}
 <h3>Les reventes auxquelles vous êtes inscrit·e</h3>
 <form class="form-horizontal" action="" method="post">
     <div class="bg-info text-info center-block">
       <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
-        Vous pouvez vous désinscrire des reventes suivantes tant que le tirage n'a
+      Voici la liste des reventes auxquelles vous êtes inscrit·e ; si vous ne souhaitez plus participer au tirage au sort vous pouvez vous en désister.    
-        pas eu lieu.
     </div>
-    <div class="bootstrap-form-reduce">
     {% csrf_token %}
-        {{ annulform|bootstrap }}
+    <table  class="table table-striped stupidtable">
-    </div>
+        <thead>
+          <tr>
+            <th></th>
+            <th data-sort="string">Titre</th>
+            <th data-sort="int">Date</th>
+            <th data-sort="string">Lieu</th>
+            <th data-sort="int">Prix</th>
+            <th>Vendue par</th>
+            <th data-sort="int">Tirage le</th>
+          </tr>
+        </thead>
+        <tbody>
+          {% for checkbox in annulform.reventes %}{{ checkbox }}{% endfor %}
+        </tbody>
+      </table>
     <div class="form-actions">
         <input type="submit"
                class="btn btn-primary"
                name="annul"
-               value="Se désinscrire des tirages sélectionnés">
+               value="Se désister des tirages sélectionnés">
     </div>
 </form>
-
-<hr />
 {% endif %}
 
-{% if subform.reventes %}
+<hr />
+
+{% if sub_exists %}
 
 <h3>Tirages en cours</h3>
 <form class="form-horizontal" action="" method="post">
     <div class="bg-info text-info center-block">
         <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
-        Vous pouvez vous inscrire aux tirage en cours suivants.
+        Vous pouvez vous inscrire aux tirages en cours suivants.
     </div>
-    <div class="bootstrap-form-reduce">
         {% csrf_token %}
-        {{ subform|bootstrap }}
+        <table  class="table table-striped stupidtable">
-    </div>
+        <thead>
+          <tr>
+            <th></th>
+            <th data-sort="string">Titre</th>
+            <th data-sort="int">Date</th>
+            <th data-sort="string">Lieu</th>
+            <th data-sort="int">Prix</th>
+            <th>Vendue par</th>
+            <th data-sort="int">Tirage le</th>
+          </tr>
+        </thead>
+        <tbody>
+          {% for checkbox in subform.reventes %}{{ checkbox }}{% endfor %}
+        </tbody>
+      </table>
     <div class="form-actions">
         <input type="submit"
                class="btn btn-primary"
@@ -49,4 +74,26 @@
 </form>
 {% endif %}
 
+{% if not annul_exists and not sub_exists %}
+<div class="bg-info text-info center-block">
+  <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
+  Aucune revente n'est active pour le moment !
+</div>
+{% endif %}
+
+<script language="JavaScript">
+  $(function(){
+      $("table.stupidtable").stupidtable();
+  });
+  
+  $("tr").click(function() {
+      $(this).find("input[type=checkbox]").click()
+  });
+
+  $("input[type=checkbox]").click(function(e) {
+    e.stopPropagation();
+  });
+</script>
+
+
 {% endblock %}

bda/templates/descriptions.html

@@ -1,113 +0,0 @@
-{% load staticfiles %}
-
-<!doctype html>
-<html>
-    <head>
-	 	  <base target="_parent"/>
-        <style>
-        @font-face {
-            font-family: josefinsans;
-            src: url({% static "fonts/josefinsans.ttf" %});
-        }
-
-        *::-moz-selection {
-            background: #B0B0B0;
-        }
-
-        *::selection {
-            background: #B0B0B0;
-        }
-
-        .descTable{
-            width: 100%;
-            margin: 0 auto 1em;
-            border-bottom: 2px solid;
-            border-collapse: collapse;
-            border-spacing: 0;
-            font-size: 14px;
-            line-height: 2;
-            max-width: 100%;
-            background-color: transparent;
-            font-family: 'josefinsans', 'Arial';
-            font-weight: 700;
-            color: #5a5a5a;
-        }
-
-		  img{
-		  	   max-width: 100%;
-		  }
-        </style>
-        <meta charset="utf8" />
-    </head>
-	 <script src="https://code.jquery.com/jquery-3.1.0.min.js"></script>
-    <body>
-        {% for show in shows %}
-        <table class="descTable">
-            <thead>
-                <tr>
-                    <th colspan="2"><p style="text-align:center;font-size:22px;">{{ show.title }}</p></th>
-                </tr>
-            </thead>
-            <tbody>
-                <tr>
-                    <td><p style="text-align: left;">{{ show.location }}</p></td><td class="column-2"><p style="text-align: right;">{{ show.category }}</p></td>
-                </tr>
-                <tr>
-                    <td><p style="text-align: left;">{{ show.date|date:"l j F Y - H\hi" }}</p></td><td class="column-2"><p style="text-align: right;">{{ show.slots }} place{{ show.slots|pluralize}} {% if show.slots_description != "" %}({{ show.slots_description }}){% endif %} - {{ show.price }} euro{{ show.price|pluralize}}</p></td>
-                </tr>
-					 {% if show.vips %}
-				 	 <tr>
-					 	  <td colspan="2"><p style="text-align: justify;">{{ show.vips }}</p></td>
-					 </tr>
-					 {% endif %}
-					 <tr>
-                    <td colspan="2">
-                    <p style="text-align: justify;">{{ show.description }}</p>
-                    {% for quote in show.quote_set.all %}
-                    <p style="text-align:center; font-style: italic;">«{{ quote.text }}»{% if quote.author %} - {{ quote.author }}{% endif %}</p>
-                    {% endfor %}
-                    </td>
-                </tr>
-                {% if show.image %}
-                <tr>
-                    <td colspan="2"><p style="text-align:center;"><a href="{{ show.ext_link }}"><img class="imgDesc" style="display: inline;" src="{{ MEDIA_URL }}{{ show.image }}" alt="{{ show.title }}"></a></p></td>
-                </tr>
-                {% endif %}
-            </tbody>
-        </table>
-        {% endfor %}
-		  <script>
-				// Correction de la taille des images
-				
-				/*$(document).ready(function() {
-					$(".descTable").each(function() {
-						$(this).width($("body").width());
-					});
-					
-					$(".imgDesc").on("load", function() {
-				
-						// Dimensions
-						origHeight = 500; // Hauteur souhaitée
-
-						w = $(this).width();
-						h = $(this).height();
-						r = w/h; // Ratio de l'image
-						maxWidth = $("body").width();
-
-						if (r * origHeight > maxWidth)
-						{
-							$(this).width(maxWidth);
-							$(this).height(maxWidth/r);
-						}
-						else
-						{
-							$(this).width(r * origHeight);
-							$(this).height(origHeight);
-						}
-					});
-				});*/
-
-
-		  </script>
-    </body>
-</html>

bda/templates/spectacle_list.html

@@ -1,8 +1,8 @@
 {% extends "base_title.html" %}
-{% load staticfiles %}
+{% load static %}
 
 {% block extra_head %}
-    <link type="text/css" rel="stylesheet" href="{% static "css/bda.css" %}" />
+    <link type="text/css" rel="stylesheet" href="{% static "bda/css/bda.css" %}" />
 {% endblock %}
 
 {% block realcontent %}
@@ -32,9 +32,7 @@
                     {% endfor %}
         </tbody>
         </table>
-	<script type="text/javascript"
+
-	        src="{% static "js/joequery-Stupid-Table-Plugin/stupidtable.js" %}">
-        </script>
     <script type="text/javascript">
         $(function(){
         $("table.etat-bda").stupidtable();
@@ -51,6 +49,5 @@
     <h3> Exports </h3>
     <ul>
         <li><a href="{% url 'bda-unpaid' tirage_id  %}">Mailing list impayés</a>
-        <li><a href="{% url 'bda-descriptions' tirage_id %}">Lien vers les descriptions des spectacles, à utiliser dans une page wordpress</a>
     </ul>
 {% endblock %}

bda/tests/mixins.py

@@ -0,0 +1,65 @@
+from django.utils import timezone
+
+from shared.tests.mixins import ViewTestCaseMixin
+
+from ..models import CategorieSpectacle, Salle, Spectacle, Tirage
+from .utils import create_user
+
+
+class BdAViewTestCaseMixin(ViewTestCaseMixin):
+    def get_users_base(self):
+        return {
+            "bda_other": create_user(username="bda_other"),
+            "bda_member": create_user(username="bda_member", is_cof=True),
+            "bda_staff": create_user(username="bda_staff", is_cof=True, is_buro=True),
+        }
+
+
+class BdATestHelpers:
+    bda_testdata = False
+
+    def setUp(self):
+        super().setUp()
+
+        if self.bda_testdata:
+            self.load_bda_testdata()
+
+    def load_bda_testdata(self):
+        self.tirage = Tirage.objects.create(
+            title="Test tirage",
+            appear_catalogue=True,
+            ouverture=timezone.now(),
+            fermeture=timezone.now(),
+        )
+        self.category = CategorieSpectacle.objects.create(name="Category")
+        self.location = Salle.objects.create(name="here")
+        self.show1 = Spectacle.objects.create(
+            title="foo",
+            date=timezone.now(),
+            location=self.location,
+            price=0,
+            slots=42,
+            tirage=self.tirage,
+            listing=False,
+            category=self.category,
+        )
+        self.show2 = Spectacle.objects.create(
+            title="bar",
+            date=timezone.now(),
+            location=self.location,
+            price=1,
+            slots=142,
+            tirage=self.tirage,
+            listing=False,
+            category=self.category,
+        )
+        self.show3 = Spectacle.objects.create(
+            title="baz",
+            date=timezone.now(),
+            location=self.location,
+            price=2,
+            slots=242,
+            tirage=self.tirage,
+            listing=False,
+            category=self.category,
+        )

bda/tests/test_models.py

@@ -19,8 +19,6 @@ User = get_user_model()
 
 
 class SpectacleReventeTests(TestCase):
-    fixtures = ["gestioncof/management/data/custommail.json"]
-
     def setUp(self):
         now = timezone.now()
 
@@ -69,7 +67,7 @@ class SpectacleReventeTests(TestCase):
         revente = self.rev
 
         wanted_by = [self.p1, self.p2, self.p3]
-        revente.confirmed_entry = wanted_by
+        revente.confirmed_entry.set(wanted_by)
 
         with mock.patch("bda.models.random.choice") as mc:
             # Set winner to self.p1.

bda/tests/test_views.py

@@ -1,221 +1,306 @@
 import json
 from datetime import timedelta
 from unittest import mock
-from urllib.parse import urlencode
 
-from django.contrib.auth.models import User
+from django.contrib.auth import get_user_model
 from django.test import Client, TestCase
-from django.utils import timezone
+from django.urls import reverse
+from django.utils import formats, timezone
 
-from ..models import CategorieSpectacle, Salle, Spectacle, Tirage
+from ..models import Participant, Tirage
+from .mixins import BdATestHelpers, BdAViewTestCaseMixin
+
+User = get_user_model()
 
 
-def create_user(username, is_cof=False, is_buro=False):
+class InscriptionViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
-    user = User.objects.create_user(username=username, password=username)
+    url_name = "bda-tirage-inscription"
-    user.profile.is_cof = is_cof
-    user.profile.is_buro = is_buro
-    user.profile.save()
-    return user
 
+    http_methods = ["GET", "POST"]
 
-def user_is_cof(user):
+    auth_user = "bda_member"
-    return (user is not None) and user.profile.is_cof
+    auth_forbidden = [None, "bda_other"]
 
+    bda_testdata = True
 
-def user_is_staff(user):
+    @property
-    return (user is not None) and user.profile.is_buro
+    def url_kwargs(self):
+        return {"tirage_id": self.tirage.id}
 
+    @property
+    def url_expected(self):
+        return "/gestion/bda/inscription/{}".format(self.tirage.id)
 
-class BdATestHelpers:
+    def test_get_opened(self):
-    def setUp(self):
+        self.tirage.ouverture = timezone.now() - timedelta(days=1)
-        # Some user with different access privileges
+        self.tirage.fermeture = timezone.now() + timedelta(days=1)
-        staff = create_user(username="bda_staff", is_cof=True, is_buro=True)
+        self.tirage.save()
-        staff_c = Client()
-        staff_c.force_login(staff)
 
-        member = create_user(username="bda_member", is_cof=True)
+        resp = self.client.get(self.url)
-        member_c = Client()
-        member_c.force_login(member)
 
-        other = create_user(username="bda_other")
+        self.assertEqual(resp.status_code, 200)
-        other_c = Client()
+        self.assertFalse(resp.context["messages"])
-        other_c.force_login(other)
 
-        self.client_matrix = [
+    def test_get_closed_future(self):
-            (staff, staff_c),
+        self.tirage.ouverture = timezone.now() + timedelta(days=1)
-            (member, member_c),
+        self.tirage.fermeture = timezone.now() + timedelta(days=2)
-            (other, other_c),
+        self.tirage.save()
-            (None, Client()),
-        ]
 
-    def require_custommails(self):
+        resp = self.client.get(self.url)
-        from django.core.management import call_command
 
-        call_command("syncmails", verbosity=0)
+        self.assertEqual(resp.status_code, 200)
-
+        self.assertIn(
-    def check_restricted_access(
+            "Le tirage n'est pas encore ouvert : ouverture le {}".format(
-        self, url, validate_user=user_is_cof, redirect_url=None
+                formats.localize(timezone.template_localtime(self.tirage.ouverture))
-    ):
-        def craft_redirect_url(user):
-            if redirect_url:
-                return redirect_url
-            elif user is None:
-                # client is not logged in
-                login_url = "/login"
-                if url:
-                    login_url += "?{}".format(urlencode({"next": url}, safe="/"))
-                return login_url
-            else:
-                return "/"
-
-        for (user, client) in self.client_matrix:
-            resp = client.get(url, follow=True)
-            if validate_user(user):
-                self.assertEqual(200, resp.status_code)
-            else:
-                self.assertRedirects(resp, craft_redirect_url(user))
-
-
-class TestBdAViews(BdATestHelpers, TestCase):
-    def setUp(self):
-        # Signals handlers on login/logout send messages.
-        # Due to the way the Django' test Client performs login, this raise an
-        # error. As workaround, we mock the Django' messages module.
-        patcher_messages = mock.patch("gestioncof.signals.messages")
-        patcher_messages.start()
-        self.addCleanup(patcher_messages.stop)
-        # Set up the helpers
-        super().setUp()
-        # Some BdA stuff
-        self.tirage = Tirage.objects.create(
-            title="Test tirage",
-            appear_catalogue=True,
-            ouverture=timezone.now(),
-            fermeture=timezone.now(),
-        )
-        self.category = CategorieSpectacle.objects.create(name="Category")
-        self.location = Salle.objects.create(name="here")
-        Spectacle.objects.bulk_create(
-            [
-                Spectacle(
-                    title="foo",
-                    date=timezone.now(),
-                    location=self.location,
-                    price=0,
-                    slots=42,
-                    tirage=self.tirage,
-                    listing=False,
-                    category=self.category,
             ),
-                Spectacle(
+            [str(msg) for msg in resp.context["messages"]],
-                    title="bar",
-                    date=timezone.now(),
-                    location=self.location,
-                    price=1,
-                    slots=142,
-                    tirage=self.tirage,
-                    listing=False,
-                    category=self.category,
-                ),
-                Spectacle(
-                    title="baz",
-                    date=timezone.now(),
-                    location=self.location,
-                    price=2,
-                    slots=242,
-                    tirage=self.tirage,
-                    listing=False,
-                    category=self.category,
-                ),
-            ]
         )
 
-    def test_bda_inscriptions(self):
+    def test_get_closed_past(self):
-        # TODO: test the form
+        self.tirage.ouverture = timezone.now() - timedelta(days=2)
-        url = "/bda/inscription/{}".format(self.tirage.id)
+        self.tirage.fermeture = timezone.now() - timedelta(days=1)
-        self.check_restricted_access(url)
+        self.tirage.save()
 
-    def test_bda_places(self):
+        resp = self.client.get(self.url)
-        url = "/bda/places/{}".format(self.tirage.id)
-        self.check_restricted_access(url)
 
-    def test_etat_places(self):
+        self.assertEqual(resp.status_code, 200)
-        url = "/bda/etat-places/{}".format(self.tirage.id)
+        self.assertIn(
-        self.check_restricted_access(url)
+            " C'est fini : tirage au sort dans la journée !",
+            [str(msg) for msg in resp.context["messages"]],
+        )
 
-    def test_perform_tirage(self):
+    def get_base_post_data(self):
-        # Only staff member can perform a tirage
+        return {
-        url = "/bda/tirage/{}".format(self.tirage.id)
+            "choixspectacle_set-TOTAL_FORMS": "3",
-        self.check_restricted_access(url, validate_user=user_is_staff)
+            "choixspectacle_set-INITIAL_FORMS": "0",
+            "choixspectacle_set-MIN_NUM_FORMS": "0",
+            "choixspectacle_set-MAX_NUM_FORMS": "1000",
+        }
 
-        _, staff_c = self.client_matrix[0]
+    base_post_data = property(get_base_post_data)
+
+    def test_post(self):
+        self.tirage.ouverture = timezone.now() - timedelta(days=1)
+        self.tirage.fermeture = timezone.now() + timedelta(days=1)
+        self.tirage.save()
+
+        data = dict(
+            self.base_post_data,
+            **{
+                "choixspectacle_set-TOTAL_FORMS": "2",
+                "choixspectacle_set-0-id": "",
+                "choixspectacle_set-0-participant": "",
+                "choixspectacle_set-0-spectacle": str(self.show1.pk),
+                "choixspectacle_set-0-double_choice": "1",
+                "choixspectacle_set-0-priority": "2",
+                "choixspectacle_set-1-id": "",
+                "choixspectacle_set-1-participant": "",
+                "choixspectacle_set-1-spectacle": str(self.show2.pk),
+                "choixspectacle_set-1-double_choice": "autoquit",
+                "choixspectacle_set-1-priority": "1",
+            }
+        )
+        resp = self.client.post(self.url, data)
+
+        self.assertEqual(resp.status_code, 200)
+        self.assertIn(
+            "Votre inscription a été mise à jour avec succès !",
+            [str(msg) for msg in resp.context["messages"]],
+        )
+        participant = Participant.objects.get(
+            user=self.users["bda_member"], tirage=self.tirage
+        )
+        self.assertSetEqual(
+            set(
+                participant.choixspectacle_set.values_list(
+                    "priority", "spectacle_id", "double_choice"
+                )
+            ),
+            {(1, self.show2.pk, "autoquit"), (2, self.show1.pk, "1")},
+        )
+
+    def test_post_state_changed(self):
+        self.tirage.ouverture = timezone.now() - timedelta(days=1)
+        self.tirage.fermeture = timezone.now() + timedelta(days=1)
+        self.tirage.save()
+
+        data = {"dbstate": "different"}
+        resp = self.client.post(self.url, data)
+
+        self.assertEqual(resp.status_code, 200)
+        self.assertIn(
+            "Impossible d'enregistrer vos modifications : vous avez apporté d'autres "
+            "modifications entre temps.",
+            [str(msg) for msg in resp.context["messages"]],
+        )
+
+
+class PlacesViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
+    url_name = "bda-places-attribuees"
+
+    auth_user = "bda_member"
+    auth_forbidden = [None, "bda_other"]
+
+    bda_testdata = True
+
+    @property
+    def url_kwargs(self):
+        return {"tirage_id": self.tirage.id}
+
+    @property
+    def url_expected(self):
+        return "/gestion/bda/places/{}".format(self.tirage.id)
+
+
+class EtatPlacesViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
+    url_name = "bda-etat-places"
+
+    auth_user = "bda_member"
+    auth_forbidden = [None, "bda_other"]
+
+    bda_testdata = True
+
+    @property
+    def url_kwargs(self):
+        return {"tirage_id": self.tirage.id}
+
+    @property
+    def url_expected(self):
+        return "/gestion/bda/etat-places/{}".format(self.tirage.id)
+
+
+class TirageViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
+    url_name = "bda-tirage"
+
+    http_methods = ["GET", "POST"]
+
+    auth_user = "bda_staff"
+    auth_forbidden = [None, "bda_other", "bda_member"]
+
+    bda_testdata = True
+
+    @property
+    def url_kwargs(self):
+        return {"tirage_id": self.tirage.id}
+
+    @property
+    def url_expected(self):
+        return "/gestion/bda/tirage/{}".format(self.tirage.id)
+
+    def test_perform_tirage_disabled(self):
         # Cannot be performed if disabled
         self.tirage.enable_do_tirage = False
         self.tirage.save()
-        resp = staff_c.get(url)
+        resp = self.client.get(self.url)
         self.assertTemplateUsed(resp, "tirage-failed.html")
+
+    def test_perform_tirage_opened_registrations(self):
         # Cannot be performed if registrations are still open
         self.tirage.enable_do_tirage = True
         self.tirage.fermeture = timezone.now() + timedelta(seconds=3600)
         self.tirage.save()
-        resp = staff_c.get(url)
+        resp = self.client.get(self.url)
         self.assertTemplateUsed(resp, "tirage-failed.html")
+
+    def test_perform_tirage(self):
         # Otherwise, perform the tirage
+        self.tirage.enable_do_tirage = True
         self.tirage.fermeture = timezone.now()
         self.tirage.save()
-        resp = staff_c.get(url)
+        resp = self.client.get(self.url)
         self.assertTemplateNotUsed(resp, "tirage-failed.html")
 
-    def test_spectacles_list(self):
-        url = "/bda/spectacles/{}".format(self.tirage.id)
-        self.check_restricted_access(url, validate_user=user_is_staff)
 
-    def test_spectacle_detail(self):
+class SpectacleListViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
-        show = self.tirage.spectacle_set.first()
+    url_name = "bda-liste-spectacles"
-        url = "/bda/spectacles/{}/{}".format(self.tirage.id, show.id)
-        self.check_restricted_access(url, validate_user=user_is_staff)
 
-    def test_tirage_unpaid(self):
+    auth_user = "bda_staff"
-        url = "/bda/spectacles/unpaid/{}".format(self.tirage.id)
+    auth_forbidden = [None, "bda_other", "bda_member"]
-        self.check_restricted_access(url, validate_user=user_is_staff)
 
-    def test_send_reminders(self):
+    bda_testdata = True
-        self.require_custommails()
+
-        # Just get the page
+    @property
-        show = self.tirage.spectacle_set.first()
+    def url_kwargs(self):
-        url = "/bda/mails-rappel/{}".format(show.id)
+        return {"tirage_id": self.tirage.id}
-        self.check_restricted_access(url, validate_user=user_is_staff)
+
-        # Actually send the reminder emails
+    @property
-        _, staff_c = self.client_matrix[0]
+    def url_expected(self):
-        resp = staff_c.post(url)
+        return "/gestion/bda/spectacles/{}".format(self.tirage.id)
+
+
+class SpectacleViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
+    url_name = "bda-spectacle"
+
+    auth_user = "bda_staff"
+    auth_forbidden = [None, "bda_other", "bda_member"]
+
+    bda_testdata = True
+
+    @property
+    def url_kwargs(self):
+        return {"tirage_id": self.tirage.id, "spectacle_id": self.show1.id}
+
+    @property
+    def url_expected(self):
+        return "/gestion/bda/spectacles/{}/{}".format(self.tirage.id, self.show1.id)
+
+
+class UnpaidViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
+    url_name = "bda-unpaid"
+
+    auth_user = "bda_staff"
+    auth_forbidden = [None, "bda_other", "bda_member"]
+
+    bda_testdata = True
+
+    @property
+    def url_kwargs(self):
+        return {"tirage_id": self.tirage.id}
+
+    @property
+    def url_expected(self):
+        return "/gestion/bda/spectacles/unpaid/{}".format(self.tirage.id)
+
+
+class SendRemindersViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
+    url_name = "bda-rappels"
+
+    auth_user = "bda_staff"
+    auth_forbidden = [None, "bda_other", "bda_member"]
+
+    bda_testdata = True
+
+    @property
+    def url_kwargs(self):
+        return {"spectacle_id": self.show1.id}
+
+    @property
+    def url_expected(self):
+        return "/gestion/bda/mails-rappel/{}".format(self.show1.id)
+
+    def test_post(self):
+        resp = self.client.post(self.url)
         self.assertEqual(200, resp.status_code)
         # TODO: check that emails are sent
 
-    def test_catalogue_api(self):
-        url_list = "/bda/catalogue/list"
-        url_details = "/bda/catalogue/details?id={}".format(self.tirage.id)
-        url_descriptions = "/bda/catalogue/descriptions?id={}".format(self.tirage.id)
 
-        # Anyone can get
+class CatalogueViewTestCase(BdATestHelpers, BdAViewTestCaseMixin, TestCase):
-        def anyone_can_get(url):
+    auth_user = None
-            self.check_restricted_access(url, validate_user=lambda user: True)
+    auth_forbidden = []
 
-        anyone_can_get(url_list)
+    bda_testdata = True
-        anyone_can_get(url_details)
-        anyone_can_get(url_descriptions)
 
-        # The resulting JSON contains the information
+    def test_api_list(self):
-        _, client = self.client_matrix[0]
+        url_list = "/gestion/bda/catalogue/list"
-
+        resp = self.client.get(url_list)
-        # List
-        resp = client.get(url_list)
         self.assertJSONEqual(
             resp.content.decode("utf-8"),
             [{"id": self.tirage.id, "title": self.tirage.title}],
         )
 
-        # Details
+    def test_api_details(self):
-        resp = client.get(url_details)
+        url_details = "/gestion/bda/catalogue/details?id={}".format(self.tirage.id)
+        resp = self.client.get(url_details)
         self.assertJSONEqual(
             resp.content.decode("utf-8"),
             {
@@ -224,8 +309,11 @@ class TestBdAViews(BdATestHelpers, TestCase):
             },
         )
 
-        # Descriptions
+    def test_api_descriptions(self):
-        resp = client.get(url_descriptions)
+        url_descriptions = "/gestion/bda/catalogue/descriptions?id={}".format(
+            self.tirage.id
+        )
+        resp = self.client.get(url_descriptions)
         raw = resp.content.decode("utf-8")
         try:
             results = json.loads(raw)
@@ -238,6 +326,43 @@ class TestBdAViews(BdATestHelpers, TestCase):
         )
 
 
+# ----- BdA Revente --------------------------------------- #
+
+
+def make_participant(name: str, tirage: Tirage) -> User:
+    user = User.objects.create_user(username=name, password=name)
+    user.profile.is_cof = True
+    user.profile.save()
+    Participant.objects.create(user=user, tirage=tirage)
+    return user
+
+
+class TestReventeManageTest(TestCase):
+    def setUp(self):
+        self.tirage = Tirage.objects.create(
+            title="tirage1",
+            ouverture=timezone.now(),
+            fermeture=timezone.now() + timedelta(days=90),
+        )
+        self.user = make_participant("toto", self.tirage)
+        self.url = reverse("bda-revente-manage", args=[self.tirage.id])
+
+        # Signals handlers on login/logout send messages.
+        # Due to the way the Django' test Client performs login, this raise an
+        # error. As workaround, we mock the Django' messages module.
+        patcher_messages = mock.patch("gestioncof.signals.messages")
+        patcher_messages.start()
+        self.addCleanup(patcher_messages.stop)
+
+    def test_can_get(self):
+        client = Client()
+        client.force_login(
+            self.user, backend="django.contrib.auth.backends.ModelBackend"
+        )
+        r = client.get(self.url)
+        self.assertEqual(r.status_code, 200)
+
+
 class TestBdaRevente:
     pass
     # TODO

bda/tests/utils.py

@@ -0,0 +1,36 @@
+from datetime import timedelta
+
+from django.contrib.auth.models import User
+from django.utils import timezone
+
+from ..models import CategorieSpectacle, Salle, Spectacle, Tirage
+
+
+def create_user(username, is_cof=False, is_buro=False):
+    user = User.objects.create_user(username=username, password=username)
+    user.profile.is_cof = is_cof
+    user.profile.is_buro = is_buro
+    user.profile.save()
+    return user
+
+
+def user_is_cof(user):
+    return (user is not None) and user.profile.is_cof
+
+
+def user_is_staff(user):
+    return (user is not None) and user.profile.is_buro
+
+
+def create_spectacle(**kwargs):
+    defaults = {
+        "title": "Title",
+        "category": CategorieSpectacle.objects.first(),
+        "date": (timezone.now() + timedelta(days=7)).date(),
+        "location": Salle.objects.first(),
+        "price": 10.0,
+        "slots": 20,
+        "tirage": Tirage.objects.first(),
+        "listing": False,
+    }
+    return Spectacle.objects.create(**dict(defaults, **kwargs))

bda/urls.py

@@ -1,75 +1,80 @@
-from django.conf.urls import url
+from django.urls import re_path
 
 from bda import views
 from bda.views import SpectacleListView
 from gestioncof.decorators import buro_required
 
 urlpatterns = [
-    url(
+    re_path(
         r"^inscription/(?P<tirage_id>\d+)$",
         views.inscription,
         name="bda-tirage-inscription",
     ),
-    url(r"^places/(?P<tirage_id>\d+)$", views.places, name="bda-places-attribuees"),
+    re_path(r"^places/(?P<tirage_id>\d+)$", views.places, name="bda-places-attribuees"),
-    url(r"^etat-places/(?P<tirage_id>\d+)$", views.etat_places, name="bda-etat-places"),
+    re_path(
-    url(r"^tirage/(?P<tirage_id>\d+)$", views.tirage),
+        r"^etat-places/(?P<tirage_id>\d+)$", views.etat_places, name="bda-etat-places"
-    url(
+    ),
+    re_path(r"^tirage/(?P<tirage_id>\d+)$", views.tirage, name="bda-tirage"),
+    re_path(
         r"^spectacles/(?P<tirage_id>\d+)$",
         buro_required(SpectacleListView.as_view()),
         name="bda-liste-spectacles",
     ),
-    url(
+    re_path(
         r"^spectacles/(?P<tirage_id>\d+)/(?P<spectacle_id>\d+)$",
         views.spectacle,
         name="bda-spectacle",
     ),
-    url(r"^spectacles/unpaid/(?P<tirage_id>\d+)$", views.unpaid, name="bda-unpaid"),
+    re_path(
-    url(
+        r"^spectacles/unpaid/(?P<tirage_id>\d+)$",
+        views.UnpaidParticipants.as_view(),
+        name="bda-unpaid",
+    ),
+    re_path(
         r"^spectacles/autocomplete$",
         views.spectacle_autocomplete,
         name="bda-spectacle-autocomplete",
     ),
-    url(
+    re_path(
         r"^participants/autocomplete$",
         views.participant_autocomplete,
         name="bda-participant-autocomplete",
     ),
     # Urls BdA-Revente
-    url(
+    re_path(
         r"^revente/(?P<tirage_id>\d+)/manage$",
         views.revente_manage,
         name="bda-revente-manage",
     ),
-    url(
+    re_path(
         r"^revente/(?P<tirage_id>\d+)/subscribe$",
         views.revente_subscribe,
         name="bda-revente-subscribe",
     ),
-    url(
+    re_path(
         r"^revente/(?P<tirage_id>\d+)/tirages$",
         views.revente_tirages,
         name="bda-revente-tirages",
     ),
-    url(
+    re_path(
         r"^revente/(?P<spectacle_id>\d+)/buy$",
         views.revente_buy,
         name="bda-revente-buy",
     ),
-    url(
+    re_path(
         r"^revente/(?P<revente_id>\d+)/confirm$",
         views.revente_confirm,
         name="bda-revente-confirm",
     ),
-    url(
+    re_path(
         r"^revente/(?P<tirage_id>\d+)/shotgun$",
         views.revente_shotgun,
         name="bda-revente-shotgun",
     ),
-    url(r"^mails-rappel/(?P<spectacle_id>\d+)$", views.send_rappel, name="bda-rappels"),
+    re_path(
-    url(
+        r"^mails-rappel/(?P<spectacle_id>\d+)$", views.send_rappel, name="bda-rappels"
-        r"^descriptions/(?P<tirage_id>\d+)$",
+    ),
-        views.descriptions_spectacles,
+    re_path(
-        name="bda-descriptions",
+        r"^catalogue/(?P<request_type>[a-z]+)$", views.catalogue, name="bda-catalogue"
     ),
-    url(r"^catalogue/(?P<request_type>[a-z]+)$", views.catalogue, name="bda-catalogue"),
 ]

bda/views.py

@@ -4,19 +4,19 @@ import random
 import time
 from collections import defaultdict
 
-from custommail.models import CustomMail
-from custommail.shortcuts import send_custom_mail, send_mass_custom_mail
 from django.conf import settings
 from django.contrib import messages
-from django.contrib.auth.decorators import login_required
 from django.core import serializers
-from django.core.urlresolvers import reverse
+from django.core.exceptions import NON_FIELD_ERRORS
+from django.core.mail import send_mail, send_mass_mail
 from django.db import transaction
 from django.db.models import Count, Prefetch
 from django.forms.models import inlineformset_factory
 from django.http import HttpResponseBadRequest, HttpResponseRedirect, JsonResponse
 from django.shortcuts import get_object_or_404, render
+from django.template import loader
 from django.template.defaultfilters import pluralize
+from django.urls import reverse
 from django.utils import formats, timezone
 from django.views.generic.list import ListView
 
@@ -41,8 +41,8 @@ from bda.models import (
     SpectacleRevente,
     Tirage,
 )
-from gestioncof.decorators import buro_required, cof_required
+from gestioncof.decorators import BuroRequiredMixin, buro_required, cof_required
-from utils.views.autocomplete import Select2QuerySetView
+from shared.views import Select2QuerySetView
 
 
 @cof_required
@@ -114,6 +114,7 @@ def places(request, tirage_id):
         "spectacle__date", "spectacle"
     ).select_related("spectacle", "spectacle__location")
     total = sum(place.spectacle.price for place in places)
+    unpaid = 0
     filtered_places = []
     places_dict = {}
     spectacles = []
@@ -124,6 +125,8 @@ def places(request, tirage_id):
             places_dict[place.spectacle].double = True
         else:
             place.double = False
+            place.unpaid = 0
+            place.unpaid_price = 0
             places_dict[place.spectacle] = place
             spectacles.append(place.spectacle)
             filtered_places.append(place)
@@ -132,6 +135,12 @@ def places(request, tirage_id):
                 warning = True
             else:
                 dates.append(date)
+
+        if not place.paid:
+            unpaid += place.spectacle.price
+            places_dict[place.spectacle].unpaid += 1
+            places_dict[place.spectacle].unpaid_price += place.spectacle.price
+
     # On prévient l'utilisateur s'il a deux places à la même date
     if warning:
         messages.warning(
@@ -147,6 +156,7 @@ def places(request, tirage_id):
             "places": filtered_places,
             "tirage": tirage,
             "total": total,
+            "unpaid": unpaid,
         },
     )
 
@@ -188,22 +198,40 @@ def inscription(request, tirage_id):
         ChoixSpectacle,
         fields=("spectacle", "double_choice", "priority"),
         formset=InscriptionInlineFormSet,
+        error_messages={
+            NON_FIELD_ERRORS: {
+                "unique_together": "Vous avez déjà demandé ce voeu plus haut !"
+            }
+        },
     )
 
-    success = False
-    stateerror = False
     if request.method == "POST":
         # use *this* queryset
         dbstate = _hash_queryset(participant.choixspectacle_set.all())
         if "dbstate" in request.POST and dbstate != request.POST["dbstate"]:
-            stateerror = True
             formset = BdaFormSet(instance=participant)
+            messages.error(
+                request,
+                "Impossible d'enregistrer vos modifications "
+                ": vous avez apporté d'autres modifications "
+                "entre temps.",
+            )
         else:
             formset = BdaFormSet(request.POST, instance=participant)
             if formset.is_valid():
                 formset.save()
-                success = True
                 formset = BdaFormSet(instance=participant)
+                participant.accepte_charte = True
+                participant.save()
+                messages.success(
+                    request, "Votre inscription a été mise à jour avec succès !"
+                )
+            else:
+                messages.error(
+                    request,
+                    "Une erreur s'est produite lors de l'enregistrement de vos vœux.  "
+                    "Avez-vous demandé plusieurs fois le même spectacle ?",
+                )
     else:
         formset = BdaFormSet(instance=participant)
     # use *this* queryset
@@ -214,18 +242,6 @@ def inscription(request, tirage_id):
         total_price += choice.spectacle.price
         if choice.double:
             total_price += choice.spectacle.price
-    # Messages
-    if success:
-        messages.success(
-            request, "Votre inscription a été mise à jour avec " "succès !"
-        )
-    if stateerror:
-        messages.error(
-            request,
-            "Impossible d'enregistrer vos modifications "
-            ": vous avez apporté d'autres modifications "
-            "entre temps.",
-        )
     return render(
         request,
         "bda/inscription-tirage.html",
@@ -234,6 +250,7 @@ def inscription(request, tirage_id):
             "total_price": total_price,
             "dbstate": dbstate,
             "tirage": tirage,
+            "charte": participant.accepte_charte,
         },
     )
 
@@ -267,13 +284,13 @@ def do_tirage(tirage_elt, token):
     results = Algorithm(data["shows"], data["members"], choices)(token)
 
     # On compte les places attribuées et les déçus
-    for (_, members, losers) in results:
+    for _, members, losers in results:
         data["total_slots"] += len(members)
         data["total_losers"] += len(losers)
 
     # On calcule le déficit et les bénéfices pour le BdA
     # FIXME: le traitement de l'opéra est sale
-    for (show, members, _) in results:
+    for show, members, _ in results:
         deficit = (show.slots - len(members)) * show.price
         data["total_sold"] += show.slots * show.price
         if deficit >= 0:
@@ -286,8 +303,8 @@ def do_tirage(tirage_elt, token):
     # so assign a single object for each Participant id
     members_uniq = {}
     members2 = {}
-    for (show, members, _) in results:
+    for show, members, _ in results:
-        for (member, _, _, _) in members:
+        for member, _, _, _ in members:
             if member.id not in members_uniq:
                 members_uniq[member.id] = member
                 members2[member] = []
@@ -363,7 +380,7 @@ def tirage(request, tirage_id):
     return render(request, "bda-token.html", {"form": form})
 
 
-@login_required
+@cof_required
 def revente_manage(request, tirage_id):
     """
     Gestion de ses propres reventes :
@@ -374,13 +391,10 @@ def revente_manage(request, tirage_id):
     - Annulation d'une revente après que le tirage a eu lieu
     """
     tirage = get_object_or_404(Tirage, id=tirage_id)
-    participant, created = Participant.objects.get_or_create(
+    participant, created = Participant.objects.annotate_paid().get_or_create(
         user=request.user, tirage=tirage
     )
 
-    if not participant.paid:
-        return render(request, "bda/revente/notpaid.html", {})
-
     resellform = ResellForm(participant, prefix="resell")
     annulform = AnnulForm(participant, prefix="annul")
     soldform = SoldForm(participant, prefix="sold")
@@ -390,7 +404,7 @@ def revente_manage(request, tirage_id):
         if "resell" in request.POST:
             resellform = ResellForm(participant, request.POST, prefix="resell")
             if resellform.is_valid():
-                datatuple = []
+                mails = []
                 attributions = resellform.cleaned_data["attributions"]
                 with transaction.atomic():
                     for attribution in attributions:
@@ -405,16 +419,17 @@ def revente_manage(request, tirage_id):
                             "show": attribution.spectacle,
                             "revente": revente,
                         }
-                        datatuple.append(
+                        mails.append(
                             (
-                                "bda-revente-new",
+                                "BdA-Revente : {}".format(attribution.spectacle),
-                                context,
+                                loader.render_to_string(
+                                    "bda/mails/revente-seller.txt", context=context
+                                ),
                                 settings.MAIL_DATA["revente"]["FROM"],
                                 [participant.user.email],
                             )
                         )
-                        revente.save()
+                    send_mass_mail(mails)
-                    send_mass_custom_mail(datatuple)
         # On annule une revente
         elif "annul" in request.POST:
             annulform = AnnulForm(participant, request.POST, prefix="annul")
@@ -445,6 +460,10 @@ def revente_manage(request, tirage_id):
                         new_date = timezone.now() - SpectacleRevente.remorse_time
                         revente.reset(new_date=new_date)
 
+    sold_exists = soldform.fields["reventes"].queryset.exists()
+    annul_exists = annulform.fields["reventes"].queryset.exists()
+    resell_exists = resellform.fields["attributions"].queryset.exists()
+
     return render(
         request,
         "bda/revente/manage.html",
@@ -453,11 +472,14 @@ def revente_manage(request, tirage_id):
             "soldform": soldform,
             "annulform": annulform,
             "resellform": resellform,
+            "sold_exists": sold_exists,
+            "annul_exists": annul_exists,
+            "resell_exists": resell_exists,
         },
     )
 
 
-@login_required
+@cof_required
 def revente_tirages(request, tirage_id):
     """
     Affiche à un participant la liste de toutes les reventes en cours (pour un
@@ -500,14 +522,22 @@ def revente_tirages(request, tirage_id):
                         ),
                     )
 
+    annul_exists = annulform.fields["reventes"].queryset.exists()
+    sub_exists = subform.fields["reventes"].queryset.exists()
+
     return render(
         request,
         "bda/revente/tirages.html",
-        {"annulform": annulform, "subform": subform},
+        {
+            "annulform": annulform,
+            "subform": subform,
+            "annul_exists": annul_exists,
+            "sub_exists": sub_exists,
+        },
     )
 
 
-@login_required
+@cof_required
 def revente_confirm(request, revente_id):
     revente = get_object_or_404(SpectacleRevente, id=revente_id)
     participant, _ = Participant.objects.get_or_create(
@@ -524,7 +554,7 @@ def revente_confirm(request, revente_id):
     )
 
 
-@login_required
+@cof_required
 def revente_subscribe(request, tirage_id):
     """
     Permet à un participant de sélectionner ses préférences pour les reventes.
@@ -541,7 +571,7 @@ def revente_subscribe(request, tirage_id):
         form = InscriptionReventeForm(tirage, request.POST)
         if form.is_valid():
             choices = form.cleaned_data["spectacles"]
-            participant.choicesrevente = choices
+            participant.choicesrevente.set(choices)
             participant.save()
             for spectacle in choices:
                 qset = SpectacleRevente.objects.filter(attribution__spectacle=spectacle)
@@ -569,18 +599,18 @@ def revente_subscribe(request, tirage_id):
         )
     # Messages
     if success:
-        messages.success(request, "Ton inscription a bien été prise en compte")
+        messages.success(request, "Votre inscription a bien été prise en compte")
     if deja_revente:
         messages.info(
             request,
             "Des reventes existent déjà pour certains de "
-            "ces spectacles, vérifie les places "
+            "ces spectacles, vérifiez les places "
             "disponibles sans tirage !",
         )
     if inscrit_revente:
         shows = map("<li>{!s}</li>".format, inscrit_revente)
         msg = (
-            "Tu as été inscrit à des reventes en cours pour les spectacles "
+            "Vous avez été inscrit·e à des reventes en cours pour les spectacles "
             "<ul>{:s}</ul>".format("\n".join(shows))
         )
         messages.info(request, msg, extra_tags="safe")
@@ -588,7 +618,7 @@ def revente_subscribe(request, tirage_id):
     return render(request, "bda/revente/subscribe.html", {"form": form})
 
 
-@login_required
+@cof_required
 def revente_buy(request, spectacle_id):
     spectacle = get_object_or_404(Spectacle, id=spectacle_id)
     tirage = spectacle.tirage
@@ -618,12 +648,16 @@ def revente_buy(request, spectacle_id):
             "acheteur": request.user,
             "vendeur": revente.seller.user,
         }
-        send_custom_mail(
+
-            "bda-buy-shotgun",
+        send_mail(
-            "bda@ens.fr",
+            "BdA-Revente : {}".format(spectacle.title),
+            loader.render_to_string(
+                "bda/mails/revente-shotgun-seller.txt", context=context
+            ),
+            request.user.email,
             [revente.seller.user.email],
-            context=context,
         )
+
         return render(
             request,
             "bda/revente/mail-success.html",
@@ -637,7 +671,7 @@ def revente_buy(request, spectacle_id):
     )
 
 
-@login_required
+@cof_required
 def revente_shotgun(request, tirage_id):
     tirage = get_object_or_404(Tirage, id=tirage_id)
     spectacles = (
@@ -657,7 +691,7 @@ def revente_shotgun(request, tirage_id):
     )
     shotgun = [sp for sp in spectacles if len(sp.shotguns) > 0]
 
-    return render(request, "bda/revente/shotgun.html", {"shotgun": shotgun})
+    return render(request, "bda/revente/shotgun.html", {"spectacles": shotgun})
 
 
 @buro_required
@@ -676,12 +710,13 @@ def spectacle(request, tirage_id, spectacle_id):
             "username": participant.user.username,
             "email": participant.user.email,
             "given": int(attrib.given),
-            "paid": participant.paid,
+            "paid": attrib.paid,
             "nb_places": 1,
         }
         if participant.id in participants:
             participants[participant.id]["nb_places"] += 1
             participants[participant.id]["given"] += attrib.given
+            participants[participant.id]["paid"] &= attrib.paid
         else:
             participants[participant.id] = participant_info
 
@@ -709,34 +744,37 @@ class SpectacleListView(ListView):
         return context
 
 
-@buro_required
+class UnpaidParticipants(BuroRequiredMixin, ListView):
-def unpaid(request, tirage_id):
+    context_object_name = "unpaid"
-    tirage = get_object_or_404(Tirage, id=tirage_id)
+    template_name = "bda-unpaid.html"
-    unpaid = (
+
-        tirage.participant_set.annotate(nb_attributions=Count("attribution"))
+    def get_queryset(self):
-        .filter(paid=False, nb_attributions__gt=0)
+        return (
+            Participant.objects.annotate_paid()
+            .filter(tirage__id=self.kwargs["tirage_id"], paid=False)
             .select_related("user")
         )
-    return render(request, "bda-unpaid.html", {"unpaid": unpaid})
 
 
 @buro_required
 def send_rappel(request, spectacle_id):
     show = get_object_or_404(Spectacle, id=spectacle_id)
     # Mails d'exemples
-    custommail = CustomMail.objects.get(shortname="bda-rappel")
+    subject = show.title
-    exemple_mail_1place = custommail.render(
+    body_mail_1place = loader.render_to_string(
-        {"member": request.user, "show": show, "nb_attr": 1}
+        "bda/mails/rappel.txt",
+        context={"member": request.user, "show": show, "nb_attr": 1},
     )
-    exemple_mail_2places = custommail.render(
+    body_mail_2places = loader.render_to_string(
-        {"member": request.user, "show": show, "nb_attr": 2}
+        "bda/mails/rappel.txt",
+        context={"member": request.user, "show": show, "nb_attr": 2},
     )
+
     # Contexte
     ctxt = {
         "show": show,
-        "exemple_mail_1place": exemple_mail_1place,
+        "exemple_mail_1place": (subject, body_mail_1place),
-        "exemple_mail_2places": exemple_mail_2places,
+        "exemple_mail_2places": (subject, body_mail_2places),
-        "custommail": custommail,
     }
     # Envoi confirmé
     if request.method == "POST":
@@ -757,25 +795,6 @@ def send_rappel(request, spectacle_id):
     return render(request, "bda/mails-rappel.html", ctxt)
 
 
-def descriptions_spectacles(request, tirage_id):
-    tirage = get_object_or_404(Tirage, id=tirage_id)
-    shows_qs = tirage.spectacle_set.select_related("location").prefetch_related(
-        "quote_set"
-    )
-    category_name = request.GET.get("category", "")
-    location_id = request.GET.get("location", "")
-    if category_name:
-        shows_qs = shows_qs.filter(category__name=category_name)
-    if location_id:
-        try:
-            shows_qs = shows_qs.filter(location__id=int(location_id))
-        except ValueError:
-            return HttpResponseBadRequest(
-                "La variable GET 'location' doit contenir un entier"
-            )
-    return render(request, "descriptions.html", {"shows": shows_qs})
-
-
 def catalogue(request, request_type):
     """
     Vue destinée à communiquer avec un client AJAX, fournissant soit :

bds/admin.py

@@ -0,0 +1,5 @@
+from django.contrib import admin
+
+from bds.models import BDSProfile
+
+admin.site.register(BDSProfile)

bds/apps.py

@@ -0,0 +1,28 @@
+from django.apps import AppConfig, apps as global_apps
+from django.db.models import Q
+from django.db.models.signals import post_migrate
+
+
+def bds_group_perms(app_config, apps=global_apps, **kwargs):
+    try:
+        Permission = apps.get_model("auth", "Permission")
+        Group = apps.get_model("auth", "Group")
+
+        group = Group.objects.get(name="Burô du BDS")
+        perms = Permission.objects.filter(
+            Q(content_type__app_label="bds")
+            | Q(content_type__app_label="auth") & Q(content_type__model="user")
+        )
+        group.permissions.set(perms)
+        group.save()
+
+    except (LookupError, Group.DoesNotExist):
+        return
+
+
+class BdsConfig(AppConfig):
+    name = "bds"
+    verbose_name = "Gestion des adhérent·e·s du BDS"
+
+    def ready(self):
+        post_migrate.connect(bds_group_perms, sender=self)

bds/autocomplete.py

@@ -0,0 +1,63 @@
+from urllib.parse import urlencode
+
+from django.contrib.auth import get_user_model
+from django.db.models import Q
+from django.urls import reverse
+from django.utils.translation import gettext_lazy as _
+
+from shared import autocomplete
+
+User = get_user_model()
+
+
+class BDSMemberSearch(autocomplete.ModelSearch):
+    model = User
+    search_fields = ["username", "first_name", "last_name"]
+    verbose_name = _("Membres du BDS")
+
+    def get_queryset_filter(self, *args, **kwargs):
+        qset_filter = super().get_queryset_filter(*args, **kwargs)
+        qset_filter &= Q(bds__is_member=True)
+        return qset_filter
+
+    def result_uuid(self, user):
+        return user.username
+
+    def result_link(self, user):
+        return reverse("bds:user.update", args=(user.pk,))
+
+
+class BDSOthersSearch(autocomplete.ModelSearch):
+    model = User
+    search_fields = ["username", "first_name", "last_name"]
+    verbose_name = _("Non-membres du BDS")
+
+    def get_queryset_filter(self, *args, **kwargs):
+        qset_filter = super().get_queryset_filter(*args, **kwargs)
+        qset_filter &= Q(bds__isnull=True) | Q(bds__is_member=False)
+        return qset_filter
+
+    def result_uuid(self, user):
+        return user.username
+
+    def result_link(self, user):
+        return reverse("bds:user.update", args=(user.pk,))
+
+
+class BDSLDAPSearch(autocomplete.LDAPSearch):
+    def result_link(self, clipper):
+        url = reverse("bds:user.create.fromclipper", args=(clipper.clipper,))
+        get = {"fullname": clipper.fullname, "mail": clipper.mail}
+
+        return "{}?{}".format(url, urlencode(get))
+
+
+class BDSSearch(autocomplete.Compose):
+    search_units = [
+        ("members", BDSMemberSearch()),
+        ("others", BDSOthersSearch()),
+        ("clippers", BDSLDAPSearch()),
+    ]
+
+
+bds_search = BDSSearch()

bds/forms.py

@@ -0,0 +1,41 @@
+from django import forms
+from django.contrib.auth import get_user_model
+from django.contrib.auth.forms import UserCreationForm
+from django.utils.translation import gettext_lazy as _
+
+from bds.models import BDSProfile
+
+User = get_user_model()
+
+
+class UserForm(forms.ModelForm):
+    is_buro = forms.BooleanField(label=_("Membre du Burô"), required=False)
+
+    class Meta:
+        model = User
+        fields = ["email", "first_name", "last_name"]
+
+
+class UserFromClipperForm(forms.ModelForm):
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.fields["username"].disabled = True
+
+    class Meta:
+        model = User
+        fields = ["username", "email", "first_name", "last_name"]
+
+
+class UserFromScratchForm(UserCreationForm):
+    class Meta:
+        model = User
+        fields = ["username", "email", "first_name", "last_name"]
+
+
+class ProfileForm(forms.ModelForm):
+    class Meta:
+        model = BDSProfile
+        exclude = ["user"]
+        widgets = {
+            "birthdate": forms.DateInput(attrs={"type": "date"}, format="%Y-%m-%d")
+        }

bds/migrations/0001_initial.py

@@ -0,0 +1,141 @@
+# Generated by Django 2.2 on 2019-07-17 12:48
+
+import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
+
+import bds.models
+
+
+class Migration(migrations.Migration):
+    initial = True
+
+    dependencies = [migrations.swappable_dependency(settings.AUTH_USER_MODEL)]
+
+    operations = [
+        migrations.CreateModel(
+            name="BDSProfile",
+            fields=[
+                (
+                    "id",
+                    models.AutoField(
+                        auto_created=True,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                (
+                    "phone",
+                    models.CharField(
+                        blank=True, max_length=20, verbose_name="téléphone"
+                    ),
+                ),
+                (
+                    "occupation",
+                    models.CharField(
+                        choices=[
+                            ("EXT", "Extérieur"),
+                            ("1A", "1A"),
+                            ("2A", "2A"),
+                            ("3A", "3A"),
+                            ("4A", "4A"),
+                            ("MAG", "Magistérien"),
+                            ("ARC", "Archicube"),
+                            ("DOC", "Doctorant"),
+                            ("CST", "CST"),
+                            ("PER", "Personnel ENS"),
+                        ],
+                        default="1A",
+                        max_length=3,
+                        verbose_name="occupation",
+                    ),
+                ),
+                (
+                    "departement",
+                    models.CharField(
+                        blank=True, max_length=50, verbose_name="département"
+                    ),
+                ),
+                (
+                    "birthdate",
+                    models.DateField(
+                        blank=True, null=True, verbose_name="date de naissance"
+                    ),
+                ),
+                (
+                    "mails_bds",
+                    models.BooleanField(
+                        default=False, verbose_name="recevoir les mails du BDS"
+                    ),
+                ),
+                (
+                    "is_buro",
+                    models.BooleanField(
+                        default=False, verbose_name="membre du Burô du BDS"
+                    ),
+                ),
+                (
+                    "has_certificate",
+                    models.BooleanField(
+                        default=False, verbose_name="certificat médical"
+                    ),
+                ),
+                (
+                    "certificate_file",
+                    models.FileField(
+                        blank=True,
+                        upload_to=bds.models.BDSProfile.get_certificate_filename,
+                        verbose_name="fichier de certificat médical",
+                    ),
+                ),
+                (
+                    "ASPSL_number",
+                    models.CharField(
+                        blank=True,
+                        max_length=50,
+                        null=True,
+                        verbose_name="numéro AS PSL",
+                    ),
+                ),
+                (
+                    "FFSU_number",
+                    models.CharField(
+                        blank=True, max_length=50, null=True, verbose_name="numéro FFSU"
+                    ),
+                ),
+                (
+                    "cotisation_period",
+                    models.CharField(
+                        choices=[
+                            ("ANN", "Année"),
+                            ("SE1", "Premier semestre"),
+                            ("SE2", "Deuxième semestre"),
+                            ("NO", "Aucune"),
+                        ],
+                        default="NO",
+                        max_length=3,
+                        verbose_name="inscription",
+                    ),
+                ),
+                (
+                    "registration_date",
+                    models.DateField(
+                        auto_now_add=True, verbose_name="date d'inscription"
+                    ),
+                ),
+                (
+                    "user",
+                    models.OneToOneField(
+                        on_delete=django.db.models.deletion.CASCADE,
+                        related_name="bds",
+                        to=settings.AUTH_USER_MODEL,
+                    ),
+                ),
+            ],
+            options={
+                "verbose_name": "Profil BDS",
+                "verbose_name_plural": "Profils BDS",
+            },
+        )
+    ]

bds/migrations/0002_bds_group.py

@@ -0,0 +1,16 @@
+# Generated by Django 2.2 on 2019-07-17 14:56
+
+from django.db import migrations
+
+
+def create_bds_buro_group(apps, schema_editor):
+    Group = apps.get_model("auth", "Group")
+    Group.objects.get_or_create(name="Burô du BDS")
+
+
+class Migration(migrations.Migration):
+    dependencies = [("bds", "0001_initial")]
+
+    operations = [
+        migrations.RunPython(create_bds_buro_group, migrations.RunPython.noop)
+    ]

bds/migrations/0003_staff_permission.py

@@ -0,0 +1,24 @@
+# Generated by Django 2.2.8 on 2019-12-20 22:48
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("bds", "0002_bds_group"),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name="bdsprofile",
+            options={
+                "permissions": (("is_team", "est membre du burô"),),
+                "verbose_name": "Profil BDS",
+                "verbose_name_plural": "Profils BDS",
+            },
+        ),
+        migrations.RemoveField(
+            model_name="bdsprofile",
+            name="is_buro",
+        ),
+    ]

bds/migrations/0004_is_member_cotiz_type.py

@@ -0,0 +1,33 @@
+# Generated by Django 2.2.8 on 2019-12-22 10:20
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("bds", "0003_staff_permission"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="bdsprofile",
+            name="cotisation_type",
+            field=models.CharField(
+                choices=[
+                    ("ETU", "Étudiant"),
+                    ("NOR", "Normalien"),
+                    ("EXT", "Extérieur"),
+                    ("ARC", "Archicube"),
+                ],
+                default="Normalien",
+                max_length=9,
+                verbose_name="type de cotisation",
+            ),
+            preserve_default=False,
+        ),
+        migrations.AddField(
+            model_name="bdsprofile",
+            name="is_member",
+            field=models.BooleanField(default=False, verbose_name="adhérent⋅e du BDS"),
+        ),
+    ]

bds/migrations/0005_remove_bdsprofile_certificate_file.py

@@ -0,0 +1,16 @@
+# Generated by Django 2.2.14 on 2020-07-27 20:14
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("bds", "0004_is_member_cotiz_type"),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name="bdsprofile",
+            name="certificate_file",
+        ),
+    ]

bds/migrations/0006_bdsprofile_comments.py

@@ -0,0 +1,22 @@
+# Generated by Django 2.2.12 on 2020-08-28 12:14
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("bds", "0005_remove_bdsprofile_certificate_file"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="bdsprofile",
+            name="comments",
+            field=models.TextField(
+                blank=True,
+                help_text="Attention : l'utilisateur·ice dispose d'un droit d'accès"
+                " aux données le/la concernant, dont le contenu de ce champ !",
+                verbose_name="commentaires",
+            ),
+        ),
+    ]

bds/mixins.py

@@ -0,0 +1,122 @@
+from django.contrib.auth.mixins import PermissionRequiredMixin
+from django.core.exceptions import ImproperlyConfigured
+from django.http import HttpResponseRedirect
+from django.views.generic.base import ContextMixin, TemplateResponseMixin, View
+
+
+class StaffRequiredMixin(PermissionRequiredMixin):
+    permission_required = "bds.is_team"
+
+
+class MultipleFormMixin(ContextMixin):
+    """Mixin pour gérer plusieurs formulaires dans la même vue.
+    Le fonctionnement est relativement identique à celui de
+    FormMixin, dont la documentation est disponible ici :
+    https://docs.djangoproject.com/en/3.0/ref/class-based-views/mixins-editing/
+
+    Les principales différences sont :
+    - au lieu de form_class, il faut donner comme attribut un dict de la forme
+    {<form_name>: <form_class>}, avec tous les formulaires à instancier. On
+    peut aussi redéfinir `get_form_classes`
+
+    - les données initiales se récupèrent pour chaque form via l'attribut
+    `<form_name>_initial` ou la fonction `get_<form_name>_initial`. De même,
+    si certaines forms sont des `ModelForm`s, on peut définir la fonction
+    `get_<form_name>_instance`.
+
+    - chaque form a un préfixe rajouté, par défaut <form_name>, mais qui peut
+    être customisé via `prefixes` ou `get_prefixes`.
+    """
+
+    form_classes = {}
+    prefixes = {}
+    initial = {}
+
+    success_url = None
+
+    def get_form_classes(self):
+        return self.form_classes
+
+    def get_initial(self, form_name):
+        initial_attr = "%s_initial" % form_name
+
+        initial_method = "get_%s_initial" % form_name
+        initial_method = getattr(self, initial_method, None)
+
+        if hasattr(self, initial_attr):
+            return getattr(self, initial_attr)
+        elif callable(initial_method):
+            return initial_method()
+        else:
+            return self.initial.copy()
+
+    def get_prefix(self, form_name):
+        return self.prefixes.get(form_name, form_name)
+
+    def get_instance(self, form_name):
+        # Au cas où certaines des forms soient des ModelForms
+        instance_method = "get_%s_instance" % form_name
+        instance_method = getattr(self, instance_method, None)
+
+        if callable(instance_method):
+            return instance_method()
+        else:
+            return None
+
+    def get_form_kwargs(self, form_name):
+        kwargs = {
+            "initial": self.get_initial(form_name),
+            "prefix": self.get_prefix(form_name),
+            "instance": self.get_instance(form_name),
+        }
+
+        if self.request.method in ("POST", "PUT"):
+            kwargs.update({"data": self.request.POST, "files": self.request.FILES})
+
+        return kwargs
+
+    def get_forms(self):
+        form_classes = self.get_form_classes()
+        return {
+            form_name: form_class(**self.get_form_kwargs(form_name))
+            for form_name, form_class in form_classes.items()
+        }
+
+    def get_success_url(self):
+        if not self.success_url:
+            raise ImproperlyConfigured("No URL to redirect to. Provide a success_url.")
+        return str(self.success_url)
+
+    def form_valid(self, forms):
+        # on garde le nom form_valid pour l'interface avec SuccessMessageMixin
+        return HttpResponseRedirect(self.get_success_url())
+
+    def form_invalid(self, forms):
+        """If the form is invalid, render the invalid form."""
+        return self.render_to_response(self.get_context_data(forms=forms))
+
+
+class ProcessMultipleFormView(View):
+    """Équivalent de `ProcessFormView` pour plusieurs forms.
+    Note : il faut que *tous* les formulaires soient valides pour
+    qu'ils soient sauvegardés !
+    """
+
+    def get(self, request, *args, **kwargs):
+        forms = self.get_forms()
+        return self.render_to_response(self.get_context_data(forms=forms))
+
+    def post(self, request, *args, **kwargs):
+        forms = self.get_forms()
+        if all(form.is_valid() for form in forms.values()):
+            return self.form_valid(forms)
+        else:
+            return self.form_invalid(forms)
+
+
+class BaseMultipleFormView(MultipleFormMixin, ProcessMultipleFormView):
+    pass
+
+
+class MultipleFormView(TemplateResponseMixin, BaseMultipleFormView):
+    pass

bds/models.py

@@ -0,0 +1,113 @@
+from datetime import date
+from os.path import splitext
+
+from django.contrib.auth import get_user_model
+from django.db import models
+from django.utils import timezone
+from django.utils.translation import gettext_lazy as _
+
+from shared.utils import choices_length
+
+User = get_user_model()
+
+
+class BDSProfile(models.Model):
+    OCCUPATION_CHOICES = (
+        ("EXT", "Extérieur"),
+        ("1A", "1A"),
+        ("2A", "2A"),
+        ("3A", "3A"),
+        ("4A", "4A"),
+        ("MAG", "Magistérien"),
+        ("ARC", "Archicube"),
+        ("DOC", "Doctorant"),
+        ("CST", "CST"),
+        ("PER", "Personnel ENS"),
+    )
+
+    TYPE_COTIZ_CHOICES = (
+        ("ETU", "Étudiant"),
+        ("NOR", "Normalien"),
+        ("EXT", "Extérieur"),
+        ("ARC", "Archicube"),
+    )
+
+    COTIZ_DURATION_CHOICES = (
+        ("ANN", "Année"),
+        ("SE1", "Premier semestre"),
+        ("SE2", "Deuxième semestre"),
+        ("NO", "Aucune"),
+    )
+
+    def get_certificate_filename(instance, filename):
+        _, ext = splitext(filename)  # récupère l'extension du fichier
+        year = str(date.now().year)
+        return "certifs/{username}-{year}.{ext}".format(
+            username=instance.user.username, year=year, ext=ext
+        )
+
+    user = models.OneToOneField(User, on_delete=models.CASCADE, related_name="bds")
+    phone = models.CharField(_("téléphone"), max_length=20, blank=True)
+    occupation = models.CharField(
+        _("occupation"),
+        default="1A",
+        choices=OCCUPATION_CHOICES,
+        max_length=choices_length(OCCUPATION_CHOICES),
+    )
+    departement = models.CharField(_("département"), max_length=50, blank=True)
+    birthdate = models.DateField(
+        auto_now_add=False,
+        auto_now=False,
+        verbose_name=_("date de naissance"),
+        blank=True,
+        null=True,
+    )
+
+    is_member = models.BooleanField(_("adhérent⋅e du BDS"), default=False)
+
+    mails_bds = models.BooleanField(_("recevoir les mails du BDS"), default=False)
+
+    has_certificate = models.BooleanField(_("certificat médical"), default=False)
+
+    ASPSL_number = models.CharField(
+        _("numéro AS PSL"), max_length=50, blank=True, null=True
+    )
+    FFSU_number = models.CharField(
+        _("numéro FFSU"), max_length=50, blank=True, null=True
+    )
+    cotisation_period = models.CharField(
+        _("inscription"), default="NO", choices=COTIZ_DURATION_CHOICES, max_length=3
+    )
+    registration_date = models.DateField(
+        auto_now_add=True, verbose_name=_("date d'inscription")
+    )
+    cotisation_type = models.CharField(
+        _("type de cotisation"), choices=TYPE_COTIZ_CHOICES, max_length=9
+    )
+
+    comments = models.TextField(
+        _("commentaires"),
+        blank=True,
+        help_text=_(
+            "Attention : l'utilisateur·ice dispose d'un droit d'accès aux données "
+            "le/la concernant, dont le contenu de ce champ !"
+        ),
+    )
+
+    @classmethod
+    def expired_members(cls):
+        now = timezone.now()
+        qs = cls.objects.filter(is_member=True)
+        if now.month > 1 and now.month < 7:
+            return qs.filter(cotisation_period="SE1")
+        elif now.month < 2 or now.month > 8:
+            return qs.none()
+        return qs
+
+    class Meta:
+        verbose_name = _("Profil BDS")
+        verbose_name_plural = _("Profils BDS")
+        permissions = (("is_team", _("est membre du burô")),)
+
+    def __str__(self):
+        return self.user.username

bds/static/bds/images/logo.svg

@@ -0,0 +1,15 @@
+<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 20010904//EN" "http://www.w3.org/TR/2001/REC-SVG-20010904/DTD/svg10.dtd">
+<svg version="1.0" xmlns="http://www.w3.org/2000/svg" width="300px" height="246px" viewBox="0 0 3000 2460" preserveAspectRatio="xMidYMid meet">
+<g id="layer101" fill="#ffffff" stroke="none">
+ <path d="M0 1230 l0 -1230 1500 0 1500 0 0 1230 0 1230 -1500 0 -1500 0 0 -1230z"/>
+ </g>
+<g id="layer102" fill="#3e2263" stroke="none">
+ <path d="M0 1230 l0 -1230 1500 0 1500 0 0 1230 0 1230 -1500 0 -1500 0 0 -1230z m1716 765 c103 -17 204 -46 284 -83 l55 -25 -40 -8 c-80 -16 -169 -10 -245 16 -123 42 -232 59 -385 59 -126 0 -278 -18 -341 -40 -13 -5 -16 -4 -9 4 14 15 166 60 254 76 102 19 315 19 427 1z m-703 -217 c15 -6 27 -13 27 -17 0 -4 -37 -25 -82 -47 -86 -43 -182 -120 -193 -155 -8 -26 32 -113 78 -167 44 -52 46 -62 10 -62 -39 1 -106 33 -165 80 -40 31 -54 38 -64 29 -21 -18 -17 -50 16 -116 57 -113 151 -199 259 -235 49 -16 51 -18 16 -13 -123 19 -260 151 -322 310 -15 39 -14 -81 1 -147 25 -107 55 -176 112 -251 49 -65 52 -73 38 -88 -21 -23 -104 -62 -216 -100 -51 -18 -97 -35 -102 -40 -4 -4 32 -10 80 -14 106 -9 256 13 317 48 l39 22 56 -33 c32 -18 114 -61 184 -95 80 -39 126 -67 123 -74 -6 -19 12 -16 19 3 9 24 28 5 22 -22 -7 -26 8 -34 18 -9 11 29 28 16 21 -18 -6 -30 -5 -31 10 -13 8 11 15 24 15 29 0 4 5 5 10 2 6 -3 8 -18 4 -33 -5 -26 -5 -26 10 -8 15 19 18 19 118 -12 113 -35 284 -72 329 -72 24 0 29 4 29 25 0 34 -36 125 -63 158 l-22 28 -59 -30 c-74 -39 -165 -43 -233 -10 -24 11 -43 25 -43 30 0 4 28 6 63 2 49 -4 71 -2 98 12 46 23 79 55 79 75 0 16 -48 66 -135 139 l-40 33 43 -23 c55 -30 166 -121 188 -154 9 -13 13 -29 9 -36 -15 -23 -100 -69 -129 -69 -17 0 -38 -5 -46 -10 -12 -8 -10 -10 10 -10 49 1 104 19 149 51 l45 31 24 -28 c33 -40 72 -128 79 -180 10 -79 -25 -96 -233 -114 -84 -7 -148 -7 -218 0 -106 12 -269 51 -317 76 -28 14 -31 13 -62 -10 -56 -43 -167 -89 -225 -94 l-55 -5 6 29 c2 16 14 55 26 86 21 57 27 112 15 142 -3 9 -34 38 -68 64 l-62 49 -104 7 c-166 12 -192 34 -83 70 34 12 65 24 68 28 4 4 -7 27 -24 52 -118 171 -124 402 -16 571 18 29 50 88 70 131 42 93 99 154 168 184 82 35 212 44 280 18z m540 -39 c60 -16 70 -41 75 -196 l5 -134 -39 3 -39 3 -5 114 c-4 93 -8 116 -22 125 -31 19 -41 -7 -47 -126 l-6 -113 -32 -3 c-41 -4 -44 6 -34 151 9 143 25 172 101 186 3 0 22 -4 43 -10z m322 -34 c0 -30 0 -30 -64 -33 l-64 -3 7 -130 7 -129 -35 0 -36 0 0 165 0 166 93 -3 92 -3 0 -30z m115 -55 l5 -84 18 27 c21 33 39 34 60 5 15 -22 16 -18 16 60 l1 82 35 0 35 0 -2 -162 -3 -163 -40 0 c-38 0 -41 2 -53 43 -16 49 -24 48 -42 -7 -12 -38 -16 -41 -50 -41 -34 0 -38 3 -44 31 -8 43 -8 284 1 293 4 4 19 6 33 4 24 -3 25 -6 30 -88z m370 -42 c193 -243 228 -505 100 -759 -61 -120 -195 -260 -328 -343 -66 -41 -60 -26 13 33 36 29 92 85 125 126 182 230 209 500 75 753 -42 79 -45 89 -45 163 0 44 4 79 9 79 5 0 28 -24 51 -52z m-1111 -294 c76 -30 94 -102 41 -156 -27 -27 -29 -32 -17 -47 34 -40 25 -100 -19 -120 -30 -13 -143 -15 -164 -1 -11 7 -15 42 -18 164 l-4 156 23 9 c37 15 115 13 158 -5z m571 -9 c84 -43 61 -135 -43 -173 -46 -18 -52 -23 -52 -48 0 -39 30 -42 79 -9 l37 25 -3 -41 c-4 -49 -32 -76 -90 -85 -32 -6 -41 -2 -68 24 -24 25 -30 39 -30 74 0 57 21 85 87 114 40 18 54 29 51 42 -5 29 -71 28 -111 -2 -19 -14 -36 -26 -39 -26 -11 0 -16 55 -7 72 24 45 131 63 189 33z m-303 -12 c56 -26 75 -65 71 -150 -3 -67 -6 -75 -38 -108 -43 -44 -94 -61 -146 -48 l-39 9 -3 144 c-1 79 0 150 2 157 8 19 110 17 153 -4z"/>
+ <path d="M1503 633 c4 -3 10 -3 14 0 3 4 0 7 -7 7 -7 0 -10 -3 -7 -7z"/>
+ <path d="M1532 448 c3 -7 15 -14 29 -16 23 -2 23 -2 5 13 -24 18 -39 20 -34 3z"/>
+ <path d="M1140 1231 c0 -45 10 -61 36 -61 26 0 64 32 64 53 0 23 -23 37 -62 37 -35 0 -38 -2 -38 -29z"/>
+ <path d="M1140 1070 c0 -35 17 -45 56 -36 20 5 25 12 22 34 -3 23 -8 27 -40 30 -36 3 -38 2 -38 -28z"/>
+ <path d="M1430 1151 c0 -72 3 -91 14 -91 28 0 47 13 61 41 31 60 10 121 -47 135 l-28 6 0 -91z"/>
+ </g>
+
+</svg>