Merge branch 'master' into Kerl/fix_32_do_tirage

.gitignore

@@ -7,3 +7,4 @@ settings.py
 venv/
 .vagrant
 /src
+media/

apache/wsgi.py

@@ -6,7 +6,7 @@ https://docs.djangoproject.com/en/1.7/howto/deployment/wsgi/
 """
 
 import os
-os.environ.setdefault("DJANGO_SETTINGS_MODULE", "cof.settings")
-
 from django.core.wsgi import get_wsgi_application
+
+os.environ.setdefault("DJANGO_SETTINGS_MODULE", "cof.settings")
 application = get_wsgi_application()

bda/admin.py

@@ -4,31 +4,42 @@ from django.core.mail import send_mail
 
 from django.contrib import admin
 from django.db.models import Sum, Count
-from bda.models import Spectacle, Salle, Participant, ChoixSpectacle, Attribution, Tirage
+from bda.models import Spectacle, Salle, Participant, ChoixSpectacle,\
+    Attribution, Tirage
 from django import forms
 
 from datetime import timedelta
 
+import autocomplete_light
+
+
 class ChoixSpectacleInline(admin.TabularInline):
     model = ChoixSpectacle
     sortable_field_name = "priority"
 
+
 class AttributionInline(admin.TabularInline):
     model = Attribution
 
+
 class ParticipantAdmin(admin.ModelAdmin):
     inlines = [AttributionInline]
+
     def get_queryset(self, request):
         return Participant.objects.annotate(nb_places=Count('attributions'),
                                             total=Sum('attributions__price'))
+
     def nb_places(self, obj):
         return obj.nb_places
     nb_places.admin_order_field = "nb_places"
     nb_places.short_description = "Nombre de places"
+
     def total(self, obj):
         tot = obj.total
-        if tot: return u"%.02f €" % tot
-        else: return u"0 €"
+        if tot:
+            return u"%.02f €" % tot
+        else:
+            return u"0 €"
     total.admin_order_field = "total"
     total.short_description = "Total à payer"
     list_display = ("user", "nb_places", "total", "paid", "paymenttype",
@@ -50,7 +61,8 @@ 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 !
+l'année, et nous t'invitons à nous contacter si l'une d'entre elles
+t'intéresse !
 --
 Le Bureau des Arts
 
@@ -69,15 +81,16 @@ pour les spectacles suivants :
 L'intégralité de ces places de spectacles est à régler dès maintenant et AVANT
 le %s, au bureau du COF pendant les heures de permanences (du lundi au vendredi
 entre 12h et 14h, et entre 18h et 20h). 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.
+é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 theâtres correspondants. Pour chaque spectacle, vous recevrez un mail 
-quelques jours avant la représentation vous indiquant le mode de retrait.
+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 theâ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 :
@@ -94,7 +107,8 @@ Le Bureau des Arts
                 for attrib in attribs:
                     attribs_text += u"- 1 place pour %s\n" % attrib
                 deadline = member.tirage.fermeture + timedelta(days=7)
-                mail = mail % (name, attribs_text, deadline.strftime('%d %b %Y'))
+                mail = mail % (name, attribs_text,
+                               deadline.strftime('%d %b %Y'))
                 send_mail("Résultats du tirage au sort", mail,
                           "bda@ens.fr", [member.user.email],
                           fail_silently=True)
@@ -105,9 +119,11 @@ Le Bureau des Arts
         else:
             message_bit = u"%d membres ont" % count
             plural = "s"
-        self.message_user(request, u"%s été informé%s avec succès." % (message_bit, plural))
+        self.message_user(request, u"%s été informé%s avec succès."
+                          % (message_bit, plural))
     send_attribs.short_description = u"Envoyer les résultats par mail"
 
+
 class AttributionAdminForm(forms.ModelForm):
     def clean(self):
         cleaned_data = super(AttributionAdminForm, self).clean()
@@ -115,26 +131,36 @@ class AttributionAdminForm(forms.ModelForm):
         spectacle = cleaned_data.get("spectacle")
         if participant and spectacle:
             if participant.tirage != spectacle.tirage:
-                raise forms.ValidationError(u"Erreur : le participant et le spectacle n'appartiennent pas au même tirage")
+                raise forms.ValidationError(
+                    u"Erreur : le participant et le spectacle n'appartiennent"
+                    u"pas au même tirage")
         return cleaned_data
 
+
 class AttributionAdmin(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', 'participant__user__username', 'participant__user__first_name', 'participant__user__last_name')
+    search_fields = ('spectacle__title', 'participant__user__username',
+                     'participant__user__first_name',
+                     'participant__user__last_name')
     form = AttributionAdminForm
 
-import autocomplete_light
+
 class ChoixSpectacleAdmin(admin.ModelAdmin):
     form = autocomplete_light.modelform_factory(ChoixSpectacle, exclude=[])
+
     def tirage(self, obj):
         return obj.participant.tirage
-    list_display = ("participant", "tirage", "spectacle", "priority", "double_choice")
+    list_display = ("participant", "tirage", "spectacle", "priority",
+                    "double_choice")
     list_filter = ("double_choice", "participant__tirage")
-    search_fields = ('participant__user__username', 'participant__user__first_name', 'participant__user__last_name')
+    search_fields = ('participant__user__username',
+                     'participant__user__first_name',
+                     'participant__user__last_name')
+
 
 class SpectacleAdmin(admin.ModelAdmin):
     model = Spectacle
@@ -142,6 +168,7 @@ class SpectacleAdmin(admin.ModelAdmin):
     list_filter = ("location", "tirage",)
     search_fields = ("title", "location__name")
 
+
 class TirageAdmin(admin.ModelAdmin):
     model = Tirage
     list_display = ("title", "ouverture", "fermeture", "active",

bda/algorithm.py

@@ -6,6 +6,7 @@ from django.db.models import Max
 
 import random
 
+
 class Algorithm(object):
 
     shows = None
@@ -19,7 +20,8 @@ class Algorithm(object):
               show.requests
             - on crée des tables de demandes pour chaque personne, afin de
               pouvoir modifier les rankings"""
-        self.max_group = 2 * choices.aggregate(Max('priority'))['priority__max']
+        self.max_group = \
+            2 * choices.aggregate(Max('priority'))['priority__max']
         self.shows = []
         showdict = {}
         for show in shows:
@@ -39,8 +41,10 @@ class Algorithm(object):
             member_shows[member] = {}
         for choice in choices:
             member = choice.participant
-            if choice.spectacle in member_shows[member]: continue
-            else: member_shows[member][choice.spectacle] = True
+            if choice.spectacle in member_shows[member]:
+                continue
+            else:
+                member_shows[member][choice.spectacle] = True
             showdict[choice.spectacle].requests.append(member)
             showdict[choice.spectacle].nrequests += 2 if choice.double else 1
             self.ranks[member][choice.spectacle] = next_rank[member]
@@ -63,13 +67,14 @@ class Algorithm(object):
     def __call__(self, seed):
         random.seed(seed)
         results = []
-        shows = sorted(self.shows, key = lambda x: x.nrequests / x.slots, reverse = True)
+        shows = sorted(self.shows, key=lambda x: x.nrequests / x.slots,
+                       reverse=True)
         for show in shows:
             # On regroupe tous les gens ayant le même rang
             groups = dict([(i, []) for i in range(1, self.max_group + 1)])
             for member in show.requests:
                 if self.ranks[member][show] == 0:
-                    raise RuntimeError, (member, show.title)
+                    raise RuntimeError(member, show.title)
                 groups[self.ranks[member][show]].append(member)
             # On passe à l'attribution
             winners = []
@@ -82,7 +87,8 @@ class Algorithm(object):
                         if len(winners) + 1 < show.slots:
                             self.appendResult(winners, member, show)
                             self.appendResult(winners, member, show)
-                        elif not self.choices[member][show].autoquit and len(winners) < show.slots:
+                        elif not self.choices[member][show].autoquit \
+                                and len(winners) < show.slots:
                             self.appendResult(winners, member, show)
                             self.appendResult(losers, member, show)
                         else:
@@ -97,4 +103,3 @@ class Algorithm(object):
                             self.IncrementRanks(member, i)
             results.append((show, winners, losers))
         return results
-

bda/autocomplete_light_registry.py

@@ -2,8 +2,11 @@ import autocomplete_light
 
 from bda.models import Participant, Spectacle
 
-autocomplete_light.register(Participant, search_fields=('user__username','user__first_name','user__last_name'),
+autocomplete_light.register(
+    Participant, search_fields=('user__username', 'user__first_name',
+                                'user__last_name'),
     autocomplete_js_attributes={'placeholder': 'participant...'})
 
-autocomplete_light.register(Spectacle, search_fields=('title',),
+autocomplete_light.register(
+    Spectacle, search_fields=('title', ),
     autocomplete_js_attributes={'placeholder': 'spectacle...'})

bda/forms.py

@@ -4,12 +4,14 @@ from django import forms
 from django.forms.models import BaseInlineFormSet
 from bda.models import Spectacle
 
+
 class BaseBdaFormSet(BaseInlineFormSet):
     def clean(self):
         """Checks that no two articles have the same title."""
         super(BaseBdaFormSet, self).clean()
         if any(self.errors):
-            # Don't bother validating the formset unless each form is valid on its own
+            # Don't bother validating the formset unless each form is valid on
+            # its own
             return
         spectacles = []
         for i in range(0, self.total_form_count()):
@@ -19,23 +21,27 @@ class BaseBdaFormSet(BaseInlineFormSet):
             spectacle = form.cleaned_data['spectacle']
             delete = form.cleaned_data['DELETE']
             if not delete and spectacle in spectacles:
-                raise forms.ValidationError("Vous ne pouvez pas vous " + \
-                        "inscrire deux fois pour le même spectacle.")
+                raise forms.ValidationError(
+                    "Vous ne pouvez pas vous inscrire deux fois pour le "
+                    "même spectacle.")
             spectacles.append(spectacle)
 
+
 class TokenForm(forms.Form):
     token = forms.CharField(widget=forms.widgets.Textarea())
 
+
 class SpectacleModelChoiceField(forms.ModelChoiceField):
     def label_from_instance(self, obj):
         return u"%s le %s (%s) à %.02f€" % (obj.title, obj.date_no_seconds(),
                                             obj.location, obj.price)
 
+
 class ResellForm(forms.Form):
     count = forms.ChoiceField(choices=(("1", "1"), ("2", "2"),))
     spectacle = SpectacleModelChoiceField(queryset=Spectacle.objects.none())
 
     def __init__(self, participant, *args, **kwargs):
         super(ResellForm, self).__init__(*args, **kwargs)
-        self.fields['spectacle'].queryset = participant.attributions.all().distinct()
-
+        self.fields['spectacle'].queryset = participant.attributions.all() \
+            .distinct()

bda/models.py

@@ -6,6 +6,7 @@ from django.db import models
 from django.contrib.auth.models import User
 from django.utils.translation import ugettext_lazy as _
 
+
 class Tirage(models.Model):
     title = models.CharField("Titre", max_length=300)
     ouverture = models.DateTimeField("Date et heure d'ouverture du tirage")
@@ -21,6 +22,7 @@ class Tirage(models.Model):
     def __unicode__(self):
         return u"%s - %s" % (self.title, self.date_no_seconds())
 
+
 class Salle(models.Model):
     name = models.CharField("Nom", max_length=300)
     address = models.TextField("Adresse")
@@ -28,6 +30,7 @@ class Salle(models.Model):
     def __unicode__(self):
         return self.name
 
+
 class Spectacle(models.Model):
     title = models.CharField("Titre", max_length=300)
     date = models.DateTimeField("Date & heure")
@@ -63,6 +66,7 @@ PAYMENT_TYPES = (
     ("autre", u"Autre"),
 )
 
+
 class Participant(models.Model):
     user = models.ForeignKey(User)
     choices = models.ManyToManyField(Spectacle,
@@ -73,7 +77,8 @@ class Participant(models.Model):
                                           related_name="attributed_to")
     paid = models.BooleanField(u"A payé", default=False)
     paymenttype = models.CharField(u"Moyen de paiement",
-            max_length=6, choices=PAYMENT_TYPES, blank=True)
+                                   max_length=6, choices=PAYMENT_TYPES,
+                                   blank=True)
     tirage = models.ForeignKey(Tirage)
 
     def __unicode__(self):
@@ -85,12 +90,14 @@ DOUBLE_CHOICES = (
     ("double", "2 places sinon rien"),
 )
 
+
 class ChoixSpectacle(models.Model):
     participant = models.ForeignKey(Participant)
     spectacle = models.ForeignKey(Spectacle, related_name="participants")
     priority = models.PositiveIntegerField("Priorité")
     double_choice = models.CharField("Nombre de places",
-            default="1", choices=DOUBLE_CHOICES, max_length=10)
+                                     default="1", choices=DOUBLE_CHOICES,
+                                     max_length=10)
 
     def get_double(self):
         return self.double_choice != "1"
@@ -106,6 +113,7 @@ class ChoixSpectacle(models.Model):
         verbose_name = "voeu"
         verbose_name_plural = "voeux"
 
+
 class Attribution(models.Model):
     participant = models.ForeignKey(Participant)
     spectacle = models.ForeignKey(Spectacle, related_name="attribues")
@@ -113,4 +121,3 @@ class Attribution(models.Model):
 
     def __unicode__(self):
         return u"%s -- %s" % (self.participant, self.spectacle)
-

bda/urls.py

@@ -3,7 +3,8 @@
 from django.conf.urls import url, patterns
 from bda.views import SpectacleListView
 
-urlpatterns = patterns('',
+urlpatterns = patterns(
+    '',
     url(r'inscription/(?P<tirage_id>\d+)$',
         'bda.views.inscription',
         name='bda-tirage-inscription'),

bda/views.py

@@ -18,11 +18,13 @@ from datetime import timedelta
 import time
 
 from gestioncof.decorators import cof_required, buro_required
-from bda.models import Spectacle, Participant, ChoixSpectacle, Attribution, Tirage
+from bda.models import Spectacle, Participant, ChoixSpectacle, Attribution,\
+    Tirage
 from bda.algorithm import Algorithm
 
 from bda.forms import BaseBdaFormSet, TokenForm, ResellForm
 
+
 @cof_required
 def etat_places(request, tirage_id):
     tirage = get_object_or_404(Tirage, id=tirage_id)
@@ -60,12 +62,14 @@ def etat_places(request, tirage_id):
     return render(request, "etat-places.html",
                   {"spectacles": spectacles, "total": total, 'tirage': tirage})
 
+
 def _hash_queryset(queryset):
     data = serializers.serialize("json", queryset)
     hasher = hashlib.sha256()
     hasher.update(data)
     return hasher.hexdigest()
 
+
 @cof_required
 def places(request, tirage_id):
     tirage = get_object_or_404(Tirage, id=tirage_id)
@@ -99,6 +103,7 @@ def places(request, tirage_id):
                    "total": total,
                    "warning": warning})
 
+
 @cof_required
 def places_ics(request, tirage_id):
     tirage = get_object_or_404(Tirage, id=tirage_id)
@@ -114,7 +119,8 @@ def places_ics(request, tirage_id):
             places_dict[place.spectacle].double = True
         else:
             place.double = False
-            place.spectacle.dtend = place.spectacle.date + timedelta(seconds=7200)
+            place.spectacle.dtend = place.spectacle.date \
+                + timedelta(seconds=7200)
             places_dict[place.spectacle] = place
             spectacles.append(place.spectacle)
             filtered_places.append(place)
@@ -122,6 +128,7 @@ def places_ics(request, tirage_id):
                   {"participant": participant,
                    "places": filtered_places}, content_type="text/calendar")
 
+
 @cof_required
 def inscription(request, tirage_id):
     tirage = get_object_or_404(Tirage, id=tirage_id)
@@ -137,8 +144,10 @@ def inscription(request, tirage_id):
         choices = participant.choixspectacle_set.order_by("priority").all()
         return render(request, "resume_inscription.html",
                       {"error_title": "C'est fini !",
-                    "error_description": u"Tirage au sort dans la journée !",
+                       "error_description":
+                           u"Tirage au sort dans la journée !",
                        "choices": choices})
+
     def formfield_callback(f, **kwargs):
         if f.name == "spectacle":
             kwargs['queryset'] = tirage.spectacle_set
@@ -170,7 +179,8 @@ def inscription(request, tirage_id):
     total_price = 0
     for choice in participant.choixspectacle_set.all():
         total_price += choice.spectacle.price
-        if choice.double: total_price += choice.spectacle.price
+        if choice.double:
+            total_price += choice.spectacle.price
     return render(request, "inscription-bda.html",
                   {"formset": formset,
                    "success": success,
@@ -179,6 +189,7 @@ def inscription(request, tirage_id):
                    'tirage': tirage,
                    "stateerror": stateerror})
 
+
 def do_tirage(request, tirage_id):
     tirage_elt = get_object_or_404(Tirage, id=tirage_id)
     form = TokenForm(request.POST)
@@ -188,8 +199,8 @@ def do_tirage(request, tirage_id):
     data = {}
     shows = tirage_elt.spectacle_set.select_related().all()
     members = tirage_elt.participant_set.all()
-    choices = ChoixSpectacle.objects.filter(spectacle__tirage=tirage_elt).order_by(
-            'participant', 'priority').select_related().all()
+    choices = ChoixSpectacle.objects.filter(spectacle__tirage=tirage_elt) \
+        .order_by('participant', 'priority').select_related().all()
     algo = Algorithm(shows, members, choices)
     results = algo(form.cleaned_data["token"])
     total_slots = 0
@@ -219,8 +230,9 @@ def do_tirage(request, tirage_id):
     data["duration"] = time.time() - start
     if request.user.is_authenticated():
         members2 = {}
-        members_uniq = {} # Participant objects are not shared accross spectacle results,
-                          # So assign a single object for each Participant id
+        # Participant objects are not shared accross spectacle results,
+        # so assign a single object for each Participant id
+        members_uniq = {}
         for (show, members, _) in results:
             for (member, _, _, _) in members:
                 if member.id not in members_uniq:
@@ -247,6 +259,7 @@ def do_tirage(request, tirage_id):
     else:
         return render(request, "bda-attrib.html", data)
 
+
 @buro_required
 def tirage(request, tirage_id):
     tirage_elt = get_object_or_404(Tirage, id=tirage_id)
@@ -260,6 +273,7 @@ def tirage(request, tirage_id):
         form = TokenForm()
     return render(request, "bda-token.html", {"form": form})
 
+
 def do_resell(request, form):
     spectacle = form.cleaned_data["spectacle"]
     count = form.cleaned_data["count"]
@@ -270,12 +284,14 @@ Je souhaite revendre %s pour %s le %s (%s) à %.02f€.
 Contactez moi par email si vous êtes intéressé·e·s !
 
 %s (%s)""" % (places, spectacle.title, spectacle.date_no_seconds(),
-        spectacle.location, spectacle.price, request.user.get_full_name(),
-        request.user.email)
+              spectacle.location, spectacle.price,
+              request.user.get_full_name(), request.user.email)
     send_mail("%s" % spectacle, mail,
               request.user.email, ["bda-revente@lists.ens.fr"],
               fail_silently=False)
-    return render(request, "bda-success.html", {"show": spectacle, "places": places})
+    return render(request, "bda-success.html",
+                  {"show": spectacle, "places": places})
+
 
 @login_required
 def revente(request, tirage_id):
@@ -290,7 +306,9 @@ def revente(request, tirage_id):
             return do_resell(request, form)
     else:
         form = ResellForm(participant)
-    return render(request, "bda-revente.html", {"form": form, 'tirage': tirage})
+    return render(request, "bda-revente.html",
+                  {"form": form, 'tirage': tirage})
+
 
 @buro_required
 def spectacle(request, tirage_id, spectacle_id):
@@ -318,19 +336,23 @@ def spectacle(request, tirage_id, spectacle_id):
     return render(request, "bda-participants.html",
                   {"spectacle": spectacle, "participants": participants_info})
 
+
 class SpectacleListView(ListView):
     model = Spectacle
     template_name = 'spectacle_list.html'
+
     def get_queryset(self):
         self.tirage = get_object_or_404(Tirage, id=self.kwargs['tirage_id'])
         categories = self.tirage.spectacle_set.all()
         return categories
+
     def get_context_data(self, **kwargs):
         context = super(SpectacleListView, self).get_context_data(**kwargs)
         context['tirage_id'] = self.tirage.id
         context['tirage_name'] = self.tirage.title
         return context
 
+
 @buro_required
 def unpaid(request, tirage_id):
     tirage = get_object_or_404(Tirage, id=tirage_id)
@@ -339,6 +361,7 @@ def unpaid(request, tirage_id):
         .filter(paid=False, nb_attributions__gt=0).all()
     return render(request, "bda-unpaid.html", {"unpaid": unpaid})
 
+
 @buro_required
 def liste_spectacles_ics(request, tirage_id):
     tirage = get_object_or_404(Tirage, id=tirage_id)
@@ -348,4 +371,3 @@ def liste_spectacles_ics(request, tirage_id):
     return render(request, "liste_spectacles.ics",
                   {"spectacles": spectacles, "tirage": tirage},
                   content_type="text/calendar")
-

cof/settings_dev.py

@@ -116,6 +116,12 @@ USE_TZ = True
 
 STATIC_URL = '/static/'
 
+# Media upload (through ImageField, SiteField)
+# https://docs.djangoproject.com/en/1.9/ref/models/fields/
+
+MEDIA_ROOT = os.path.join(BASE_DIR, 'media/')
+MEDIA_URL = '/media/'
+
 # Various additional settings
 SITE_ID = 1
 
@@ -148,11 +154,14 @@ RECAPTCHA_PUBLIC_KEY = "DUMMY"
 RECAPTCHA_PRIVATE_KEY = "DUMMY"
 RECAPTCHA_USE_SSL = True
 
-# On ne veut pas la vérification de INTERNAL_IPS faite par la debug-toolbar car
-# cela interfère avec l'utilisation de Vagrant. En effet, l'adresse de la
-# machine physique n'est pas forcément connue, et peut difficilement être mise
-# dans les INTERNAL_IPS.
+
 def show_toolbar(request):
+    """
+    On ne veut pas la vérification de INTERNAL_IPS faite par la debug-toolbar
+    car cela interfère avec l'utilisation de Vagrant. En effet, l'adresse de la
+    machine physique n'est pas forcément connue, et peut difficilement être
+    mise dans les INTERNAL_IPS.
+    """
     if not DEBUG:
         return False
     if request.is_ajax():

cof/urls.py

@@ -1,17 +1,23 @@
+# -*-coding:utf-8 -*
+
+from django.conf import settings
 from django.conf.urls import patterns, include, url
+from django.conf.urls.static import static
 
 import autocomplete_light
-autocomplete_light.autodiscover()
 
 from django.contrib import admin
-admin.autodiscover()
 
 from django.views.generic.base import TemplateView
 
 from gestioncof.urls import export_patterns, petitcours_patterns, \
     surveys_patterns, events_patterns
 
-urlpatterns = patterns('',
+autocomplete_light.autodiscover()
+admin.autodiscover()
+
+urlpatterns = patterns(
+    '',
     # Page d'accueil
     url(r'^$', 'gestioncof.views.home', name='home'),
     # Le BdA
@@ -50,7 +56,8 @@ urlpatterns = patterns('',
     url(r'^registration/empty$', 'gestioncof.views.registration_form2',
         name="empty-registration"),
     # Autocompletion
-    url(r'^autocomplete/registration$', 'gestioncof.autocomplete.autocomplete'),
+    url(r'^autocomplete/registration$',
+        'gestioncof.autocomplete.autocomplete'),
     url(r'^autocomplete/', include('autocomplete_light.urls')),
     # Interface admin
     url(r'^admin/logout/', 'gestioncof.views.logout'),
@@ -66,5 +73,9 @@ urlpatterns = patterns('',
     url(r'^utile_bda/bda_diff$', 'gestioncof.views.liste_bdadiff'),
     url(r'^utile_cof/diff_cof$', 'gestioncof.views.liste_diffcof'),
     url(r'^utile_bda/bda_revente$', 'gestioncof.views.liste_bdarevente'),
-)
-
+) + \
+    (static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)
+        if settings.DEBUG
+        else [])
+# Si on est en production, MEDIA_ROOT est servi par Apache.
+# Il faut dire à Django de servir MEDIA_ROOT lui-même en développement.

gestioncof/admin.py

@@ -9,9 +9,14 @@ from django.core.urlresolvers import reverse
 from django.utils.safestring import mark_safe
 from django.utils.translation import ugettext_lazy as _
 
-def add_link_field(target_model='', field='', link_text=unicode, desc_text=unicode):
+import autocomplete_light
+
+
+def add_link_field(target_model='', field='', link_text=unicode,
+                   desc_text=unicode):
     def add_link(cls):
         reverse_name = target_model or cls.model.__name__.lower()
+
         def link(self, instance):
             app_name = instance._meta.app_label
             reverse_path = "admin:%s_%s_change" % (app_name, reverse_name)
@@ -19,60 +24,74 @@ def add_link_field(target_model='', field='', link_text=unicode, desc_text=unico
             if not link_obj.id:
                 return ""
             url = reverse(reverse_path, args=(link_obj.id,))
-            return mark_safe("<a href='%s'>%s</a>" % (url, link_text(link_obj)))
+            return mark_safe("<a href='%s'>%s</a>"
+                             % (url, link_text(link_obj)))
         link.allow_tags = True
         link.short_description = desc_text(reverse_name + ' link')
         cls.link = link
-        cls.readonly_fields = list(getattr(cls, 'readonly_fields', [])) + ['link']
+        cls.readonly_fields =\
+            list(getattr(cls, 'readonly_fields', [])) + ['link']
         return cls
     return add_link
 
+
 class SurveyQuestionAnswerInline(admin.TabularInline):
     model = SurveyQuestionAnswer
 
+
 @add_link_field(desc_text=lambda x: "Réponses",
                 link_text=lambda x: "Éditer les réponses")
 class SurveyQuestionInline(admin.TabularInline):
     model = SurveyQuestion
 
+
 class SurveyQuestionAdmin(admin.ModelAdmin):
     inlines = [
         SurveyQuestionAnswerInline,
     ]
 
+
 class SurveyAdmin(admin.ModelAdmin):
     inlines = [
         SurveyQuestionInline,
     ]
 
+
 class EventOptionChoiceInline(admin.TabularInline):
     model = EventOptionChoice
 
+
 @add_link_field(desc_text=lambda x: "Choix",
                 link_text=lambda x: "Éditer les choix")
 class EventOptionInline(admin.TabularInline):
     model = EventOption
 
+
 class EventCommentFieldInline(admin.TabularInline):
     model = EventCommentField
 
+
 class EventOptionAdmin(admin.ModelAdmin):
     inlines = [
         EventOptionChoiceInline,
     ]
 
+
 class EventAdmin(admin.ModelAdmin):
     inlines = [
         EventOptionInline,
         EventCommentFieldInline,
     ]
 
+
 class CofProfileInline(admin.StackedInline):
     model = CofProfile
     inline_classes = ("collapse open",)
 
+
 class FkeyLookup(object):
-    def __init__(self, fkeydecl, short_description=None, admin_order_field=None):
+    def __init__(self, fkeydecl, short_description=None,
+                 admin_order_field=None):
         self.fk, fkattrs = fkeydecl.split('__', 1)
         self.fkattrs = fkattrs.split('__')
 
@@ -81,14 +100,18 @@ class FkeyLookup(object):
 
     def __get__(self, obj, klass):
         if obj is None:
-            return self  # hack required to make Django validate (if obj is
-                         # None, then we're a class, and classes are callable
-                         # <wink>)
+            """
+            hack required to make Django validate (if obj is
+            None, then we're a class, and classes are callable
+            <wink>)
+            """
+            return self
         item = getattr(obj, self.fk)
         for attr in self.fkattrs:
             item = getattr(item, attr)
         return item
 
+
 def ProfileInfo(field, short_description, boolean=False):
     def getter(self):
         try:
@@ -99,14 +122,17 @@ def ProfileInfo(field, short_description, boolean=False):
     getter.boolean = boolean
     return getter
 
-User.profile_login_clipper = FkeyLookup("profile__login_clipper", "Login clipper")
+User.profile_login_clipper = FkeyLookup("profile__login_clipper",
+                                        "Login clipper")
 User.profile_num = FkeyLookup("profile__num", "Numéro")
 User.profile_phone = ProfileInfo("phone", "Téléphone")
 User.profile_occupation = ProfileInfo("occupation", "Occupation")
 User.profile_departement = ProfileInfo("departement", "Departement")
 User.profile_mailing_cof = ProfileInfo("mailing_cof", "ML COF", True)
 User.profile_mailing_bda = ProfileInfo("mailing_bda", "ML BDA", True)
-User.profile_mailing_bda_revente = ProfileInfo("mailing_bda_revente", "ML BDA-R", True)
+User.profile_mailing_bda_revente = ProfileInfo("mailing_bda_revente",
+                                               "ML BDA-R", True)
+
 
 class UserProfileAdmin(UserAdmin):
     def is_buro(self, obj):
@@ -116,6 +142,7 @@ class UserProfileAdmin(UserAdmin):
             return False
     is_buro.short_description = 'Membre du Buro'
     is_buro.boolean = True
+
     def is_cof(self, obj):
         try:
             return obj.profile.is_cof
@@ -136,13 +163,15 @@ class UserProfileAdmin(UserAdmin):
         CofProfileInline,
     ]
 
-import autocomplete_light
+
 def user_unicode(self):
     if self.first_name and self.last_name:
         return u"%s %s (%s)" % (self.first_name, self.last_name, self.username)
     else:
         return self.username
 User.__unicode__ = user_unicode
+
+
 class EventRegistrationAdmin(admin.ModelAdmin):
     form = autocomplete_light.modelform_factory(EventRegistration, exclude=[])
     list_display = ('__unicode__', 'event', 'user', 'paid')
@@ -150,15 +179,18 @@ class EventRegistrationAdmin(admin.ModelAdmin):
     search_fields = ('user__username', 'user__first_name', 'user__last_name',
                      'user__email', 'event__title')
 
+
 class PetitCoursAbilityAdmin(admin.ModelAdmin):
     list_display = ('user', 'matiere', 'niveau', 'agrege')
     search_fields = ('user__username', 'user__first_name', 'user__last_name',
                      'user__email', 'matiere__name', 'niveau')
     list_filter = ('matiere', 'niveau', 'agrege')
 
+
 class PetitCoursAttributionAdmin(admin.ModelAdmin):
     list_display = ('user', 'demande', 'matiere', 'rank', )
 
+
 class PetitCoursAttributionCounterAdmin(admin.ModelAdmin):
     list_display = ('user', 'matiere', 'count', )
     list_filter = ('matiere',)
@@ -171,6 +203,7 @@ class PetitCoursAttributionCounterAdmin(admin.ModelAdmin):
         queryset.update(count=0)
     reset.short_description = u"Remise à zéro du compteur"
 
+
 class PetitCoursDemandeAdmin(admin.ModelAdmin):
     list_display = ('name', 'email', 'agrege_requis', 'niveau', 'created',
                     'traitee', 'processed')
@@ -188,6 +221,7 @@ admin.site.register(CustomMail)
 admin.site.register(PetitCoursSubject)
 admin.site.register(PetitCoursAbility, PetitCoursAbilityAdmin)
 admin.site.register(PetitCoursAttribution, PetitCoursAttributionAdmin)
-admin.site.register(PetitCoursAttributionCounter, PetitCoursAttributionCounterAdmin)
+admin.site.register(PetitCoursAttributionCounter,
+                    PetitCoursAttributionCounterAdmin)
 admin.site.register(PetitCoursDemande, PetitCoursDemandeAdmin)
 admin.site.register(EventRegistration, EventRegistrationAdmin)

gestioncof/autocomplete.py

@@ -5,6 +5,7 @@ from django.db.models import Q
 from django.contrib.auth.models import User
 from gestioncof.models import CofProfile, Clipper
 
+
 def autocomplete(request):
     if "q" not in request.GET:
         raise Http404
@@ -34,9 +35,12 @@ def autocomplete(request):
                  | Q(username__icontains=bit))
     queries['members'] = queries['members'].distinct()
     queries['users'] = queries['users'].distinct()
-    usernames = list(queries['members'].values_list('login_clipper', flat = 'True')) \
-              + list(queries['users'].values_list('profile__login_clipper', flat = 'True'))
-    queries['clippers'] = queries['clippers'].exclude(username__in = usernames).distinct()
+    usernames = list(queries['members'].values_list('login_clipper',
+                                                    flat='True')) \
+        + list(queries['users'].values_list('profile__login_clipper',
+                                            flat='True'))
+    queries['clippers'] = queries['clippers'] \
+        .exclude(username__in=usernames).distinct()
     # add clippers
 
     data.update(queries)

gestioncof/autocomplete_light_registry.py

@@ -2,5 +2,6 @@ import autocomplete_light
 
 from django.contrib.auth.models import User
 
-autocomplete_light.register(User, search_fields=('username','first_name','last_name'),
+autocomplete_light.register(
+    User, search_fields=('username', 'first_name', 'last_name'),
     autocomplete_js_attributes={'placeholder': 'membre...'})

gestioncof/csv_views.py

@@ -3,10 +3,12 @@ from django.http import HttpResponse, HttpResponseForbidden
 from django.template.defaultfilters import slugify
 from django.apps import apps
 
+
 def export(qs, fields=None):
     model = qs.model
     response = HttpResponse(content_type='text/csv')
-    response['Content-Disposition'] = 'attachment; filename=%s.csv' % slugify(model.__name__)
+    response['Content-Disposition'] = 'attachment; filename=%s.csv' \
+        % slugify(model.__name__)
     writer = csv.writer(response)
     # Write headers to CSV file
     if fields:
@@ -29,10 +31,13 @@ def export(qs, fields=None):
     # Return CSV file to browser as download
     return response
 
-def admin_list_export(request, model_name, app_label, queryset=None, fields=None, list_display=True):
+
+def admin_list_export(request, model_name, app_label, queryset=None,
+                      fields=None, list_display=True):
     """
     Put the following line in your urls.py BEFORE your admin include
-    (r'^admin/(?P<app_label>[\d\w]+)/(?P<model_name>[\d\w]+)/csv/', 'util.csv_view.admin_list_export'),
+    (r'^admin/(?P<app_label>[\d\w]+)/(?P<model_name>[\d\w]+)/csv/',
+    'util.csv_view.admin_list_export'),
     """
     if not request.user.is_staff:
         return HttpResponseForbidden()
@@ -47,12 +52,17 @@ def admin_list_export(request, model_name, app_label, queryset=None, fields=None
             fields = None
     return export(queryset, fields)
     """
-    Create your own change_list.html for your admin view and put something like this in it:
+    Create your own change_list.html for your admin view and put something
+    like this in it:
     {% block object-tools %}
     <ul class="object-tools">
-        <li><a href="csv/{%if request.GET%}?{{request.GET.urlencode}}{%endif%}" class="addlink">Export to CSV</a></li>
+        <li><a href="csv/{%if request.GET%}?{{request.GET.urlencode}}
+        {%endif%}" class="addlink">Export to CSV</a></li>
     {% if has_add_permission %}
-        <li><a href="add/{% if is_popup %}?_popup=1{% endif %}" class="addlink">{% blocktrans with cl.opts.verbose_name|escape as name %}Add {{ name }}{% endblocktrans %}</a></li>
+        <li><a href="add/{% if is_popup %}?_popup=1{% endif %}"
+                class="addlink">
+            {% blocktrans with cl.opts.verbose_name|escape as name %}
+            Add {{ name }}{% endblocktrans %}</a></li>
     {% endif %}
     </ul>
     {% endblock %}

gestioncof/decorators.py

@@ -1,5 +1,6 @@
 from django_cas_ng.decorators import user_passes_test
 
+
 def is_cof(user):
     try:
         profile = user.profile
@@ -8,7 +9,9 @@ def is_cof(user):
         return False
 
 cof_required = user_passes_test(lambda u: is_cof(u))
-cof_required_customdenied = user_passes_test(lambda u: is_cof(u), login_url = "cof-denied")
+cof_required_customdenied = user_passes_test(lambda u: is_cof(u),
+                                             login_url="cof-denied")
+
 
 def is_buro(user):
     try:

gestioncof/forms.py

@@ -10,6 +10,7 @@ from gestioncof.models import CofProfile, EventCommentValue
 from gestioncof.widgets import TriStateCheckbox
 from gestioncof.shared import lock_table, unlock_table
 
+
 class EventForm(forms.Form):
     def __init__(self, *args, **kwargs):
         event = kwargs.pop("event")
@@ -25,16 +26,20 @@ class EventForm(forms.Form):
                     choices[choice.event_option.id].append(choice.id)
         all_choices = choices
         for option in event.options.all():
-            choices = [(choice.id, choice.value) for choice in option.choices.all()]
+            choices = [(choice.id, choice.value)
+                       for choice in option.choices.all()]
             if option.multi_choices:
-                initial = [] if option.id not in all_choices else all_choices[option.id]
-                field = forms.MultipleChoiceField(label = option.name,
+                initial = [] if option.id not in all_choices \
+                    else all_choices[option.id]
+                field = forms.MultipleChoiceField(
+                    label=option.name,
                     choices=choices,
                     widget=CheckboxSelectMultiple,
                     required=False,
                     initial=initial)
             else:
-                initial = None if option.id not in all_choices else all_choices[option.id][0]
+                initial = None if option.id not in all_choices \
+                    else all_choices[option.id][0]
                 field = forms.ChoiceField(label=option.name,
                                           choices=choices,
                                           widget=RadioSelect,
@@ -48,6 +53,7 @@ class EventForm(forms.Form):
             if name.startswith('option_'):
                 yield (self.fields[name].option_id, value)
 
+
 class SurveyForm(forms.Form):
     def __init__(self, *args, **kwargs):
         survey = kwargs.pop("survey")
@@ -61,16 +67,20 @@ class SurveyForm(forms.Form):
                 else:
                     answers[answer.survey_question.id].append(answer.id)
         for question in survey.questions.all():
-            choices = [(answer.id, answer.answer) for answer in question.answers.all()]
+            choices = [(answer.id, answer.answer)
+                       for answer in question.answers.all()]
             if question.multi_answers:
-                initial = [] if question.id not in answers else answers[question.id]
-                field = forms.MultipleChoiceField(label = question.question,
+                initial = [] if question.id not in answers\
+                    else answers[question.id]
+                field = forms.MultipleChoiceField(
+                    label=question.question,
                     choices=choices,
                     widget=CheckboxSelectMultiple,
                     required=False,
                     initial=initial)
             else:
-                initial = None if question.id not in answers else answers[question.id][0]
+                initial = None if question.id not in answers\
+                    else answers[question.id][0]
                 field = forms.ChoiceField(label=question.question,
                                           choices=choices,
                                           widget=RadioSelect,
@@ -84,6 +94,7 @@ class SurveyForm(forms.Form):
             if name.startswith('question_'):
                 yield (self.fields[name].question_id, value)
 
+
 class SurveyStatusFilterForm(forms.Form):
     def __init__(self, *args, **kwargs):
         survey = kwargs.pop("survey")
@@ -91,11 +102,13 @@ class SurveyStatusFilterForm(forms.Form):
         for question in survey.questions.all():
             for answer in question.answers.all():
                 name = "question_%d_answer_%d" % (question.id, answer.id)
-                if self.is_bound and self.data.get(self.add_prefix(name), None):
+                if self.is_bound \
+                        and self.data.get(self.add_prefix(name), None):
                     initial = self.data.get(self.add_prefix(name), None)
                 else:
                     initial = "none"
-                field = forms.ChoiceField(label = "%s : %s" % (question.question, answer.answer),
+                field = forms.ChoiceField(
+                    label="%s : %s" % (question.question, answer.answer),
                     choices=[("yes", "yes"), ("no", "no"), ("none", "none")],
                     widget=TriStateCheckbox,
                     required=False,
@@ -107,7 +120,9 @@ class SurveyStatusFilterForm(forms.Form):
     def filters(self):
         for name, value in self.cleaned_data.items():
             if name.startswith('question_'):
-                yield (self.fields[name].question_id, self.fields[name].answer_id, value)
+                yield (self.fields[name].question_id,
+                       self.fields[name].answer_id, value)
+
 
 class EventStatusFilterForm(forms.Form):
     def __init__(self, *args, **kwargs):
@@ -116,11 +131,13 @@ class EventStatusFilterForm(forms.Form):
         for option in event.options.all():
             for choice in option.choices.all():
                 name = "option_%d_choice_%d" % (option.id, choice.id)
-                if self.is_bound and self.data.get(self.add_prefix(name), None):
+                if self.is_bound \
+                        and self.data.get(self.add_prefix(name), None):
                     initial = self.data.get(self.add_prefix(name), None)
                 else:
                     initial = "none"
-                field = forms.ChoiceField(label = "%s : %s" % (option.name, choice.value),
+                field = forms.ChoiceField(
+                    label="%s : %s" % (option.name, choice.value),
                     choices=[("yes", "yes"), ("no", "no"), ("none", "none")],
                     widget=TriStateCheckbox,
                     required=False,
@@ -135,7 +152,8 @@ class EventStatusFilterForm(forms.Form):
         else:
             initial = "none"
         field = forms.ChoiceField(label="Événement payé",
-                                  choices = [("yes", "yes"),("no","no"),("none","none")],
+                                  choices=[("yes", "yes"), ("no", "no"),
+                                           ("none", "none")],
                                   widget=TriStateCheckbox,
                                   required=False,
                                   initial=initial)
@@ -144,10 +162,12 @@ class EventStatusFilterForm(forms.Form):
     def filters(self):
         for name, value in self.cleaned_data.items():
             if name.startswith('option_'):
-                yield (self.fields[name].option_id, self.fields[name].choice_id, value)
+                yield (self.fields[name].option_id,
+                       self.fields[name].choice_id, value)
             elif name == "event_has_paid":
                 yield ("has_paid", None, value)
 
+
 class UserProfileForm(forms.ModelForm):
     first_name = forms.CharField(label=_(u'Prénom'), max_length=30)
     last_name = forms.CharField(label=_(u'Nom'), max_length=30)
@@ -174,7 +194,9 @@ class UserProfileForm(forms.ModelForm):
 
     class Meta:
         model = CofProfile
-        fields = ("phone", "mailing_cof", "mailing_bda", "mailing_bda_revente",)
+        fields = ("phone", "mailing_cof", "mailing_bda",
+                  "mailing_bda_revente", )
+
 
 class RegistrationUserForm(forms.ModelForm):
     def __init__(self, *args, **kw):
@@ -185,6 +207,7 @@ class RegistrationUserForm(forms.ModelForm):
         model = User
         fields = ("username", "first_name", "last_name", "email")
 
+
 class RegistrationProfileForm(forms.ModelForm):
     def __init__(self, *args, **kw):
         super(RegistrationProfileForm, self).__init__(*args, **kw)
@@ -224,24 +247,30 @@ class RegistrationProfileForm(forms.ModelForm):
 
     class Meta:
         model = CofProfile
-        fields = ("login_clipper", "num", "phone", "occupation", "departement", "is_cof", "type_cotiz", "mailing_cof", "mailing_bda", "mailing_bda_revente", "comments")
+        fields = ("login_clipper", "num", "phone", "occupation",
+                  "departement", "is_cof", "type_cotiz", "mailing_cof",
+                  "mailing_bda", "mailing_bda_revente", "comments")
 
 STATUS_CHOICES = (('no', 'Non'),
                   ('wait', 'Oui mais attente paiement'),
                   ('paid', 'Oui payé'),)
 
+
 class AdminEventForm(forms.Form):
-    status = forms.ChoiceField(label = "Inscription", choices = STATUS_CHOICES, widget = RadioSelect)
+    status = forms.ChoiceField(label="Inscription",
+                               choices=STATUS_CHOICES, widget=RadioSelect)
 
     def __init__(self, *args, **kwargs):
         event = kwargs.pop("event")
         self.event = event
         registration = kwargs.pop("current_registration", None)
-        current_choices = registration.options.all() if registration is not None else []
+        current_choices = \
+            registration.options.all() if registration is not None\
+            else []
         paid = kwargs.pop("paid", None)
-        if paid == True:
+        if paid is True:
             kwargs["initial"] = {"status": "paid"}
-        elif paid == False:
+        elif paid is False:
             kwargs["initial"] = {"status": "wait"}
         else:
             kwargs["initial"] = {"status": "no"}
@@ -254,16 +283,20 @@ class AdminEventForm(forms.Form):
                 choices[choice.event_option.id].append(choice.id)
         all_choices = choices
         for option in event.options.all():
-            choices = [(choice.id, choice.value) for choice in option.choices.all()]
+            choices = [(choice.id, choice.value)
+                       for choice in option.choices.all()]
             if option.multi_choices:
-                initial = [] if option.id not in all_choices else all_choices[option.id]
-                field = forms.MultipleChoiceField(label = option.name,
+                initial = [] if option.id not in all_choices\
+                    else all_choices[option.id]
+                field = forms.MultipleChoiceField(
+                    label=option.name,
                     choices=choices,
                     widget=CheckboxSelectMultiple,
                     required=False,
                     initial=initial)
             else:
-                initial = None if option.id not in all_choices else all_choices[option.id][0]
+                initial = None if option.id not in all_choices\
+                    else all_choices[option.id][0]
                 field = forms.ChoiceField(label=option.name,
                                           choices=choices,
                                           widget=RadioSelect,
@@ -275,10 +308,12 @@ class AdminEventForm(forms.Form):
             initial = commentfield.default
             if registration is not None:
                 try:
-                    initial = registration.comments.get(commentfield = commentfield).content
+                    initial = registration.comments \
+                        .get(commentfield=commentfield).content
                 except EventCommentValue.DoesNotExist:
                     pass
-            widget = forms.Textarea if commentfield.fieldtype == "text" else forms.TextInput
+            widget = forms.Textarea if commentfield.fieldtype == "text" \
+                else forms.TextInput
             field = forms.CharField(label=commentfield.name,
                                     widget=widget,
                                     required=False,
@@ -295,4 +330,3 @@ class AdminEventForm(forms.Form):
         for name, value in self.cleaned_data.items():
             if name.startswith('comment_'):
                 yield (self.fields[name].comment_id, value)
-

gestioncof/migrations/0003_event_image.py

@@ -0,0 +1,19 @@
+# -*- coding: utf-8 -*-
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('gestioncof', '0002_enable_unprocessed_demandes'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='event',
+            name='image',
+            field=models.ImageField(upload_to=b'imgs/events/', null=True, verbose_name=b'Image', blank=True),
+        ),
+    ]

gestioncof/models.py

@@ -29,9 +29,11 @@ TYPE_COMMENT_FIELD = (
     ('char', _(u"Texte court")),
 )
 
+
 def choices_length(choices):
     return reduce(lambda m, choice: max(m, len(choice[0])), choices, 0)
 
+
 class CofProfile(models.Model):
     user = models.OneToOneField(User, related_name="profile")
     login_clipper = models.CharField("Login clipper", max_length=8, blank=True)
@@ -41,19 +43,26 @@ class CofProfile(models.Model):
     occupation = models.CharField(_(u"Occupation"),
                                   default="1A",
                                   choices=OCCUPATION_CHOICES,
-                             max_length = choices_length (OCCUPATION_CHOICES))
-    departement = models.CharField (_(u"Département"), max_length = 50, blank = True)
+                                  max_length=choices_length(
+                                      OCCUPATION_CHOICES))
+    departement = models.CharField(_(u"Département"), max_length=50,
+                                   blank=True)
     type_cotiz = models.CharField(_(u"Type de cotisation"),
                                   default="normalien",
                                   choices=TYPE_COTIZ_CHOICES,
-                             max_length = choices_length (TYPE_COTIZ_CHOICES))
+                                  max_length=choices_length(
+                                      TYPE_COTIZ_CHOICES))
     mailing_cof = models.BooleanField("Recevoir les mails COF", default=False)
     mailing_bda = models.BooleanField("Recevoir les mails BdA", default=False)
-    mailing_bda_revente = models.BooleanField("Recevoir les mails de revente de places BdA", default = False)
-    comments = models.TextField("Commentaires visibles uniquement par le Buro", blank = True)
+    mailing_bda_revente = models.BooleanField(
+        "Recevoir les mails de revente de places BdA", default=False)
+    comments = models.TextField(
+        "Commentaires visibles uniquement par le Buro", blank=True)
     is_buro = models.BooleanField("Membre du Burô", default=False)
-    petits_cours_accept = models.BooleanField("Recevoir des petits cours", default = False)
-    petits_cours_remarques = models.TextField(_(u"Remarques et précisions pour les petits cours"),
+    petits_cours_accept = models.BooleanField(
+        "Recevoir des petits cours", default=False)
+    petits_cours_remarques = models.TextField(
+        _(u"Remarques et précisions pour les petits cours"),
         blank=True, default="")
 
     class Meta:
@@ -63,22 +72,26 @@ class CofProfile(models.Model):
     def __unicode__(self):
         return unicode(self.user.username)
 
+
 def create_user_profile(sender, instance, created, **kwargs):
     if created:
         CofProfile.objects.get_or_create(user=instance)
 post_save.connect(create_user_profile, sender=User)
 
+
 class Club(models.Model):
     name = models.CharField("Nom", max_length=200)
     description = models.TextField("Description")
     respos = models.ManyToManyField(User, related_name="clubs_geres")
     membres = models.ManyToManyField(User, related_name="clubs")
 
+
 class CustomMail(models.Model):
     shortname = models.SlugField(max_length=50, blank=False)
     title = models.CharField("Titre", max_length=200, blank=False)
     content = models.TextField("Contenu", blank=False)
-    comments = models.TextField("Informations contextuelles sur le mail", blank = True)
+    comments = models.TextField("Informations contextuelles sur le mail",
+                                blank=True)
 
     class Meta:
         verbose_name = "Mails personnalisables"
@@ -86,13 +99,17 @@ class CustomMail(models.Model):
     def __unicode__(self):
         return u"%s: %s" % (self.shortname, self.title)
 
+
 class Event(models.Model):
     title = models.CharField("Titre", max_length=200)
     location = models.CharField("Lieu", max_length=200)
     start_date = models.DateField("Date de début", blank=True, null=True)
     end_date = models.DateField("Date de fin", blank=True, null=True)
     description = models.TextField("Description", blank=True)
-    registration_open = models.BooleanField("Inscriptions ouvertes", default = True)
+    image = models.ImageField("Image", blank=True, null=True,
+                              upload_to="imgs/events/")
+    registration_open = models.BooleanField("Inscriptions ouvertes",
+                                            default=True)
     old = models.BooleanField("Archiver (événement fini)", default=False)
 
     class Meta:
@@ -101,10 +118,12 @@ class Event(models.Model):
     def __unicode__(self):
         return unicode(self.title)
 
+
 class EventCommentField(models.Model):
     event = models.ForeignKey(Event, related_name="commentfields")
     name = models.CharField("Champ", max_length=200)
-    fieldtype = models.CharField("Type", max_length = 10, choices = TYPE_COMMENT_FIELD, default = "text")
+    fieldtype = models.CharField("Type", max_length=10,
+                                 choices=TYPE_COMMENT_FIELD, default="text")
     default = models.TextField("Valeur par défaut", blank=True)
 
     class Meta:
@@ -113,11 +132,14 @@ class EventCommentField(models.Model):
     def __unicode__(self):
         return unicode(self.name)
 
+
 class EventCommentValue(models.Model):
     commentfield = models.ForeignKey(EventCommentField, related_name="values")
-    registration = models.ForeignKey("EventRegistration", related_name = "comments")
+    registration = models.ForeignKey("EventRegistration",
+                                     related_name="comments")
     content = models.TextField("Contenu", blank=True, null=True)
 
+
 class EventOption(models.Model):
     event = models.ForeignKey(Event, related_name="options")
     name = models.CharField("Option", max_length=200)
@@ -129,6 +151,7 @@ class EventOption(models.Model):
     def __unicode__(self):
         return unicode(self.name)
 
+
 class EventOptionChoice(models.Model):
     event_option = models.ForeignKey(EventOption, related_name="choices")
     value = models.CharField("Valeur", max_length=200)
@@ -139,11 +162,13 @@ class EventOptionChoice(models.Model):
     def __unicode__(self):
         return unicode(self.value)
 
+
 class EventRegistration(models.Model):
     user = models.ForeignKey(User)
     event = models.ForeignKey(Event)
     options = models.ManyToManyField(EventOptionChoice)
-    filledcomments = models.ManyToManyField(EventCommentField, through = EventCommentValue)
+    filledcomments = models.ManyToManyField(EventCommentField,
+                                            through=EventCommentValue)
     paid = models.BooleanField("A payé", default=False)
 
     class Meta:
@@ -151,7 +176,9 @@ class EventRegistration(models.Model):
         unique_together = ("user", "event")
 
     def __unicode__(self):
-        return u"Inscription de %s à %s" % (unicode(self.user), unicode(self.event.title))
+        return u"Inscription de %s à %s" % (unicode(self.user),
+                                            unicode(self.event.title))
+
 
 class Survey(models.Model):
     title = models.CharField("Titre", max_length=200)
@@ -165,6 +192,7 @@ class Survey(models.Model):
     def __unicode__(self):
         return unicode(self.title)
 
+
 class SurveyQuestion(models.Model):
     survey = models.ForeignKey(Survey, related_name="questions")
     question = models.CharField("Question", max_length=200)
@@ -176,6 +204,7 @@ class SurveyQuestion(models.Model):
     def __unicode__(self):
         return unicode(self.question)
 
+
 class SurveyQuestionAnswer(models.Model):
     survey_question = models.ForeignKey(SurveyQuestion, related_name="answers")
     answer = models.CharField("Réponse", max_length=200)
@@ -186,15 +215,18 @@ class SurveyQuestionAnswer(models.Model):
     def __unicode__(self):
         return unicode(self.answer)
 
+
 class SurveyAnswer(models.Model):
     user = models.ForeignKey(User)
     survey = models.ForeignKey(Survey)
-    answers = models.ManyToManyField(SurveyQuestionAnswer, related_name = "selected_by")
+    answers = models.ManyToManyField(SurveyQuestionAnswer,
+                                     related_name="selected_by")
 
     class Meta:
         verbose_name = "Réponses"
         unique_together = ("user", "survey")
 
+
 class Clipper(models.Model):
     username = models.CharField("Identifiant", max_length=20)
     fullname = models.CharField("Nom complet", max_length=200)

gestioncof/petits_cours_models.py

@@ -4,6 +4,7 @@ from django.db import models
 from django.contrib.auth.models import User
 from django.utils.translation import ugettext_lazy as _
 
+
 def choices_length(choices):
     return reduce(lambda m, choice: max(m, len(choice[0])), choices, 0)
 
@@ -16,6 +17,7 @@ LEVELS_CHOICES = (
     ('other', _(u"Autre (préciser dans les commentaires)")),
 )
 
+
 class PetitCoursSubject(models.Model):
     name = models.CharField(_(u"Matière"), max_length=30)
     users = models.ManyToManyField(User, related_name="petits_cours_matieres",
@@ -28,6 +30,7 @@ class PetitCoursSubject(models.Model):
     def __unicode__(self):
         return self.name
 
+
 class PetitCoursAbility(models.Model):
     user = models.ForeignKey(User)
     matiere = models.ForeignKey(PetitCoursSubject, verbose_name=_(u"Matière"))
@@ -41,7 +44,9 @@ class PetitCoursAbility(models.Model):
         verbose_name_plural = "Compétences des petits cours"
 
     def __unicode__(self):
-        return u"%s - %s - %s" % (self.user.username, self.matiere, self.niveau)
+        return u"%s - %s - %s" % (self.user.username,
+                                  self.matiere, self.niveau)
+
 
 class PetitCoursDemande(models.Model):
     name = models.CharField(_(u"Nom/prénom"), max_length=200)
@@ -50,12 +55,12 @@ class PetitCoursDemande(models.Model):
                              max_length=20, blank=True)
     quand = models.CharField(
         _(u"Quand ?"),
-            help_text=_(u"Indiquez ici la période désirée pour les petits" \
-                    + " cours (vacances scolaires, semaine, week-end)."),
+        help_text=_(u"Indiquez ici la période désirée pour les petits"
+                    " cours (vacances scolaires, semaine, week-end)."),
         max_length=300, blank=True)
     freq = models.CharField(
         _(u"Fréquence"),
-            help_text=_(u"Indiquez ici la fréquence envisagée " \
+        help_text=_(u"Indiquez ici la fréquence envisagée "
                     + "(hebdomadaire, 2 fois par semaine, ...)"),
         max_length=300, blank=True)
     lieu = models.CharField(
@@ -85,7 +90,9 @@ class PetitCoursDemande(models.Model):
         verbose_name_plural = "Demandes de petits cours"
 
     def __unicode__(self):
-        return u"Demande %d du %s" % (self.id, self.created.strftime("%d %b %Y"))
+        return u"Demande %d du %s" % (self.id,
+                                      self.created.strftime("%d %b %Y"))
+
 
 class PetitCoursAttribution(models.Model):
     user = models.ForeignKey(User)
@@ -104,6 +111,7 @@ class PetitCoursAttribution(models.Model):
         return u"Attribution de la demande %d à %s pour %s" \
             % (self.demande.id, self.user.username, self.matiere)
 
+
 class PetitCoursAttributionCounter(models.Model):
     user = models.ForeignKey(User)
     matiere = models.ForeignKey(PetitCoursSubject, verbose_name=_("Matiere"))
@@ -116,4 +124,3 @@ class PetitCoursAttributionCounter(models.Model):
     def __unicode__(self):
         return u"%d demandes envoyées à %s pour %s" \
             % (self.count, self.user.username, self.matiere)
-

gestioncof/petits_cours_views.py

@@ -26,11 +26,13 @@ from datetime import datetime
 import base64
 import simplejson
 
+
 def render_template(template_path, data):
     tmpl = loader.get_template(template_path)
     context = Context(data)
     return tmpl.render(context)
 
+
 class DemandeListView(ListView):
     model = PetitCoursDemande
     template_name = "petits_cours_demandes_list.html"
@@ -43,6 +45,7 @@ class DemandeListView(ListView):
     def dispatch(self, *args, **kwargs):
         return super(DemandeListView, self).dispatch(*args, **kwargs)
 
+
 @buro_required
 def details(request, demande_id):
     demande = get_object_or_404(PetitCoursDemande, id=demande_id)
@@ -51,30 +54,36 @@ def details(request, demande_id):
                   {"demande": demande,
                    "attributions": attributions})
 
+
 def _get_attrib_counter(user, matiere):
-    counter, created = PetitCoursAttributionCounter.objects.get_or_create(user = user,
-                                                                          matiere = matiere)
+    counter, created = PetitCoursAttributionCounter \
+        .objects.get_or_create(user=user, matiere=matiere)
     if created:
-        mincount = PetitCoursAttributionCounter.objects.filter(matiere = matiere).exclude(user = user).all().aggregate(Min('count'))
+        mincount = PetitCoursAttributionCounter.objects \
+            .filter(matiere=matiere).exclude(user=user).all() \
+            .aggregate(Min('count'))
         counter.count = mincount['count__min']
         counter.save()
     return counter
 
+
 def _get_demande_candidates(demande, redo=False):
     for matiere in demande.matieres.all():
-        candidates = PetitCoursAbility.objects.filter(matiere = matiere, niveau = demande.niveau)
+        candidates = PetitCoursAbility.objects.filter(matiere=matiere,
+                                                      niveau=demande.niveau)
         candidates = candidates.filter(user__profile__is_cof=True,
                                        user__profile__petits_cours_accept=True)
         if demande.agrege_requis:
             candidates = candidates.filter(agrege=True)
         if redo:
-            attributions = PetitCoursAttribution.objects.filter(demande = demande,
-                                                                matiere = matiere).all()
+            attributions = PetitCoursAttribution.objects \
+                .filter(demande=demande, matiere=matiere).all()
             for attrib in attributions:
                 candidates = candidates.exclude(user=attrib.user)
         candidates = candidates.order_by('?').select_related().all()
         yield (matiere, candidates)
 
+
 @buro_required
 def traitement(request, demande_id, redo=False):
     demande = get_object_or_404(PetitCoursDemande, id=demande_id)
@@ -110,10 +119,12 @@ def traitement(request, demande_id, redo = False):
     return _finalize_traitement(request, demande, proposals,
                                 proposed_for, unsatisfied, attribdata, redo)
 
+
 @buro_required
 def retraitement(request, demande_id):
     return traitement(request, demande_id, redo=True)
 
+
 def _finalize_traitement(request, demande, proposals, proposed_for,
                          unsatisfied, attribdata, redo=False, errors=None):
     proposals = proposals.items()
@@ -123,7 +134,11 @@ def _finalize_traitement(request, demande, proposals, proposed_for,
     mainmail = render_template("petits-cours-mail-demandeur.txt",
                                {"proposals": proposals,
                                 "unsatisfied": unsatisfied,
-                                "extra": "<textarea name=\"extra\" style=\"width:99%; height: 90px;\"></textarea>"})
+                                "extra":
+                                    '<textarea name="extra" '
+                                    'style="width:99%; height: 90px;">'
+                                    '</textarea>'
+                                })
     return render(request, "traitement_demande_petit_cours.html",
                   {"demande": demande,
                    "unsatisfied": unsatisfied,
@@ -131,18 +146,22 @@ def _finalize_traitement(request, demande, proposals, proposed_for,
                    "proposed_for": proposed_for,
                    "proposed_mails": proposed_mails,
                    "mainmail": mainmail,
-                   "attribdata": base64.b64encode(simplejson.dumps(attribdata)),
+                   "attribdata":
+                   base64.b64encode(simplejson.dumps(attribdata)),
                    "redo": redo,
                    "errors": errors,
                    })
 
+
 def _generate_eleve_email(demande, proposed_for):
     proposed_mails = []
     for user, matieres in proposed_for:
-        msg = render_template("petits-cours-mail-eleve.txt", {"demande": demande, "matieres": matieres})
+        msg = render_template("petits-cours-mail-eleve.txt",
+                              {"demande": demande, "matieres": matieres})
         proposed_mails.append((user, msg))
     return proposed_mails
 
+
 def _traitement_other_preparing(request, demande):
     redo = "redo" in request.POST
     unsatisfied = []
@@ -157,15 +176,18 @@ def _traitement_other_preparing(request, demande):
             attribdata[matiere.id] = []
             proposals[matiere] = []
             for choice_id in range(min(3, len(candidates))):
-                choice = int(request.POST["proposal-%d-%d" % (matiere.id, choice_id)])
+                choice = int(
+                    request.POST["proposal-%d-%d" % (matiere.id, choice_id)])
                 if choice == -1:
                     continue
                 if choice not in candidates:
-                    errors.append(u"Choix invalide pour la proposition %d en %s" % (choice_id + 1, matiere))
+                    errors.append(u"Choix invalide pour la proposition %d"
+                                  "en %s" % (choice_id + 1, matiere))
                     continue
                 user = candidates[choice]
                 if user in proposals[matiere]:
-                    errors.append(u"La proposition %d en %s est un doublon" % (choice_id + 1, matiere))
+                    errors.append(u"La proposition %d en %s est un doublon"
+                                  % (choice_id + 1, matiere))
                     continue
                 proposals[matiere].append(user)
                 attribdata[matiere.id].append(user.id)
@@ -176,10 +198,15 @@ def _traitement_other_preparing(request, demande):
             if not proposals[matiere]:
                 errors.append(u"Aucune proposition pour %s" % (matiere,))
             elif len(proposals[matiere]) < 3:
-                errors.append(u"Seulement %d proposition%s pour %s" % (len(proposals[matiere]), "s" if len(proposals[matiere]) > 1 else "", matiere))
+                errors.append(u"Seulement %d proposition%s pour %s"
+                              % (len(proposals[matiere]),
+                                 "s" if len(proposals[matiere]) > 1 else "",
+                                 matiere))
         else:
             unsatisfied.append(matiere)
-    return _finalize_traitement(request, demande, proposals, proposed_for, unsatisfied, attribdata, errors = errors)
+    return _finalize_traitement(request, demande, proposals, proposed_for,
+                                unsatisfied, attribdata, errors=errors)
+
 
 def _traitement_other(request, demande, redo):
     if request.method == "POST":
@@ -220,6 +247,7 @@ def _traitement_other(request, demande, redo):
                    "proposed_for": proposed_for,
                    })
 
+
 def _traitement_post(request, demande):
     proposals = {}
     proposed_for = {}
@@ -258,13 +286,15 @@ def _traitement_post(request, demande):
         mails_to_send.append(msg)
     mails_to_send.append(EmailMessage("Cours particuliers ENS", mainmail,
                                       frommail, [demande.email],
-                                      [bccaddress], headers = {'Reply-To': replyto}))
+                                      [bccaddress],
+                                      headers={'Reply-To': replyto}))
     connection = mail.get_connection(fail_silently=True)
     connection.send_messages(mails_to_send)
     lock_table(PetitCoursAttributionCounter, PetitCoursAttribution, User)
     for matiere in proposals:
         for rank, user in enumerate(proposals[matiere]):
-            counter = PetitCoursAttributionCounter.objects.get(user = user, matiere = matiere)
+            counter = PetitCoursAttributionCounter.objects.get(user=user,
+                                                               matiere=matiere)
             counter.count += 1
             counter.save()
             attrib = PetitCoursAttribution(user=user, matiere=matiere,
@@ -280,11 +310,13 @@ def _traitement_post(request, demande):
                    "redo": redo,
                    })
 
+
 class BaseMatieresFormSet(BaseInlineFormSet):
     def clean(self):
         super(BaseMatieresFormSet, self).clean()
         if any(self.errors):
-            # Don't bother validating the formset unless each form is valid on its own
+            # Don't bother validating the formset unless each form is
+            # valid on its own
             return
         matieres = []
         for i in range(0, self.total_form_count()):
@@ -295,16 +327,20 @@ class BaseMatieresFormSet(BaseInlineFormSet):
             niveau = form.cleaned_data['niveau']
             delete = form.cleaned_data['DELETE']
             if not delete and (matiere, niveau) in matieres:
-                raise forms.ValidationError("Vous ne pouvez pas vous inscrire deux fois pour la même matiere avec le même niveau.")
+                raise forms.ValidationError(
+                    "Vous ne pouvez pas vous inscrire deux fois pour la "
+                    "même matiere avec le même niveau.")
             matieres.append((matiere, niveau))
 
+
 @login_required
 def inscription(request):
     profile, created = CofProfile.objects.get_or_create(user=request.user)
     if not profile.is_cof:
         return redirect("cof-denied")
     MatieresFormSet = inlineformset_factory(User, PetitCoursAbility,
-                                            fields = ("matiere", "niveau", "agrege",),
+                                            fields=("matiere", "niveau",
+                                                    "agrege",),
                                             formset=BaseMatieresFormSet)
     success = False
     if request.method == "POST":
@@ -314,8 +350,10 @@ def inscription(request):
             profile.petits_cours_accept = "receive_proposals" in request.POST
             profile.petits_cours_remarques = request.POST["remarques"]
             profile.save()
-            lock_table(PetitCoursAttributionCounter, PetitCoursAbility, User, PetitCoursSubject)
-            abilities = PetitCoursAbility.objects.filter(user = request.user).all()
+            lock_table(PetitCoursAttributionCounter, PetitCoursAbility, User,
+                       PetitCoursSubject)
+            abilities = PetitCoursAbility.objects \
+                .filter(user=request.user).all()
             for ability in abilities:
                 counter = _get_attrib_counter(ability.user, ability.matiere)
             unlock_tables()
@@ -323,7 +361,11 @@ def inscription(request):
             formset = MatieresFormSet(instance=request.user)
     else:
         formset = MatieresFormSet(instance=request.user)
-    return render(request, "inscription-petit-cours.html", {"formset": formset, "success": success, "receive_proposals": profile.petits_cours_accept, "remarques": profile.petits_cours_remarques})
+    return render(request, "inscription-petit-cours.html",
+                  {"formset": formset, "success": success,
+                   "receive_proposals": profile.petits_cours_accept,
+                   "remarques": profile.petits_cours_remarques})
+
 
 class DemandeForm(ModelForm):
     captcha = ReCaptchaField(attrs={'theme': 'clean', 'lang': 'fr'})
@@ -334,9 +376,11 @@ class DemandeForm(ModelForm):
 
     class Meta:
         model = PetitCoursDemande
-        fields = ('name', 'email', 'phone', 'quand', 'freq', 'lieu', 'matieres', 'agrege_requis', 'niveau', 'remarques')
+        fields = ('name', 'email', 'phone', 'quand', 'freq', 'lieu',
+                  'matieres', 'agrege_requis', 'niveau', 'remarques')
         widgets = {'matieres': forms.CheckboxSelectMultiple}
 
+
 @csrf_exempt
 def demande(request):
     success = False
@@ -347,7 +391,9 @@ def demande(request):
             success = True
     else:
         form = DemandeForm()
-    return render(request, "demande-petit-cours.html", {"form": form, "success": success})
+    return render(request, "demande-petit-cours.html", {"form": form,
+                                                        "success": success})
+
 
 @csrf_exempt
 def demande_raw(request):
@@ -359,4 +405,5 @@ def demande_raw(request):
             success = True
     else:
         form = DemandeForm()
-    return render(request, "demande-petit-cours-raw.html", {"form": form, "success": success})
+    return render(request, "demande-petit-cours-raw.html",
+                  {"form": form, "success": success})

gestioncof/shared.py

@@ -12,6 +12,7 @@ from gestioncof.models import CofProfile, CustomMail
 
 User = get_user_model()
 
+
 class COFCASBackend(CASBackend):
     def authenticate_cas(self, ticket, service, request):
         """Verifies CAS ticket and gets or creates User object"""
@@ -57,6 +58,7 @@ class COFCASBackend(CASBackend):
             user.save()
         return user
 
+
 def context_processor(request):
     '''Append extra data to the context of the given request'''
     data = {
@@ -65,17 +67,20 @@ def context_processor (request):
            }
     return data
 
+
 def lock_table(*models):
     query = "LOCK TABLES "
     for i, model in enumerate(models):
         table = model._meta.db_table
-        if i > 0: query += ", "
+        if i > 0:
+            query += ", "
         query += "%s WRITE" % table
     cursor = connection.cursor()
     cursor.execute(query)
     row = cursor.fetchone()
     return row
 
+
 def unlock_tables(*models):
     cursor = connection.cursor()
     cursor.execute("UNLOCK TABLES")
@@ -84,8 +89,10 @@ def unlock_tables(*models):
 
 unlock_table = unlock_tables
 
+
 def send_custom_mail(to, shortname, context=None, from_email="cof@ens.fr"):
-    if context is None: context = {}
+    if context is None:
+        context = {}
     if isinstance(to, DjangoUser):
         context["nom"] = to.get_full_name()
         context["prenom"] = to.first_name

gestioncof/templates/registration.html

@@ -10,6 +10,7 @@
                 <h2>Inscription d'un nouveau membre</h2>
                 <input type="text" name="q" id="search_autocomplete" spellcheck="false" />
                 <div id="form-placeholder"></div>
+                <span class="yourlabs-autocomplete"></span>
                 <script type="text/javascript">
                 $(document).ready(function() {
                     $('input#search_autocomplete').yourlabsAutocomplete({
@@ -18,6 +19,9 @@
                         id: 'search_autocomplete',
                         choiceSelector: 'li:has(a)',
                         placeholder: "Chercher un utilisateur par nom, prénom ou identifiant clipper",
+                        container: $("#main-content"),
+                        box: $(".yourlabs-autocomplete"),
+                        fixPosition: function() {},
                     });
                     $('input#search_autocomplete').bind(
                         'selectChoice',

gestioncof/urls.py

@@ -19,7 +19,8 @@ petitcours_patterns = [
         name='petits-cours-demande-raw'),
     url(r'^demandes$', DemandeListView.as_view(),
         name='petits-cours-demandes-list'),
-    url(r'^demandes/(?P<demande_id>\d+)$', 'gestioncof.petits_cours_views.details',
+    url(r'^demandes/(?P<demande_id>\d+)$',
+        'gestioncof.petits_cours_views.details',
         name='petits-cours-demande-details'),
     url(r'^demandes/(?P<demande_id>\d+)/traitement$',
         'gestioncof.petits_cours_views.traitement',
@@ -38,5 +39,3 @@ events_patterns = [
     url(r'^(?P<event_id>\d+)$', 'gestioncof.views.event'),
     url(r'^(?P<event_id>\d+)/status$', 'gestioncof.views.event_status'),
 ]
-
-

gestioncof/views.py

@@ -8,8 +8,10 @@ from django.contrib.auth.decorators import login_required
 from django.contrib.auth.views import login as django_login_view
 from django.contrib.auth.models import User
 
-from gestioncof.models import Survey, SurveyAnswer, SurveyQuestion, SurveyQuestionAnswer
-from gestioncof.models import Event, EventRegistration, EventOption, EventOptionChoice
+from gestioncof.models import Survey, SurveyAnswer, SurveyQuestion, \
+    SurveyQuestionAnswer
+from gestioncof.models import Event, EventRegistration, EventOption, \
+    EventOptionChoice
 from gestioncof.models import EventCommentField, EventCommentValue
 from gestioncof.shared import send_custom_mail
 from gestioncof.models import CofProfile, Clipper
@@ -20,20 +22,25 @@ from gestioncof.forms import UserProfileForm, EventStatusFilterForm, \
 
 from bda.models import Tirage
 
+
 @login_required
 def home(request):
     data = {"surveys": Survey.objects.filter(old=False).all(),
             "events": Event.objects.filter(old=False).all(),
-            "open_surveys": Survey.objects.filter(survey_open=True, old=False).all(),
-            "open_events": Event.objects.filter(registration_open=True, old=False).all(),
+            "open_surveys":
+                Survey.objects.filter(survey_open=True, old=False).all(),
+            "open_events":
+                Event.objects.filter(registration_open=True, old=False).all(),
             "open_tirages": Tirage.objects.filter(active=True).all()}
     return render(request, "home.html", data)
 
+
 def login(request):
     if request.user.is_authenticated():
         return redirect("gestioncof.views.home")
     return render(request, "login_switch.html", {})
 
+
 def login_ext(request):
     if request.method == "POST" and "username" in request.POST:
         try:
@@ -41,13 +48,16 @@ def login_ext(request):
             if not user.has_usable_password() or user.password in ("", "!"):
                 profile, created = CofProfile.objects.get_or_create(user=user)
                 if profile.login_clipper:
-                    return render(request, "error.html", {"error_type": "use_clipper_login"})
+                    return render(request, "error.html",
+                                  {"error_type": "use_clipper_login"})
                 else:
-                    return render(request, "error.html", {"error_type": "no_password"})
+                    return render(request, "error.html",
+                                  {"error_type": "no_password"})
         except User.DoesNotExist:
             pass
     return django_login_view(request, template_name='login.html')
 
+
 @login_required
 def logout(request):
     try:
@@ -59,6 +69,7 @@ def logout(request):
     else:
         return redirect("django.contrib.auth.views.logout")
 
+
 @login_required
 def survey(request, survey_id):
     survey = get_object_or_404(Survey, id=survey_id)
@@ -70,7 +81,8 @@ def survey(request, survey_id):
         form = SurveyForm(request.POST, survey=survey)
         if request.POST.get('delete'):
             try:
-                current_answer = SurveyAnswer.objects.get(user = request.user, survey = survey)
+                current_answer = SurveyAnswer.objects.get(user=request.user,
+                                                          survey=survey)
                 current_answer.delete()
                 current_answer = None
             except SurveyAnswer.DoesNotExist:
@@ -82,7 +94,8 @@ def survey(request, survey_id):
             if form.is_valid():
                 all_answers = []
                 for question_id, answers_ids in form.answers():
-                    question = get_object_or_404(SurveyQuestion, id = question_id,
+                    question = get_object_or_404(SurveyQuestion,
+                                                 id=question_id,
                                                  survey=survey)
                     if type(answers_ids) != list:
                         answers_ids = [answers_ids]
@@ -97,21 +110,29 @@ def survey(request, survey_id):
                                                    survey_question=question)
                         all_answers.append(answer)
                 try:
-                    current_answer = SurveyAnswer.objects.get(user = request.user, survey = survey)
+                    current_answer = SurveyAnswer.objects.get(
+                        user=request.user, survey=survey)
                 except SurveyAnswer.DoesNotExist:
-                    current_answer = SurveyAnswer(user = request.user, survey = survey)
+                    current_answer = SurveyAnswer(user=request.user,
+                                                  survey=survey)
                     current_answer.save()
                 current_answer.answers = all_answers
                 current_answer.save()
                 success = True
     else:
         try:
-            current_answer = SurveyAnswer.objects.get(user = request.user, survey = survey)
-            form = SurveyForm(survey = survey, current_answers = current_answer.answers)
+            current_answer = SurveyAnswer.objects.get(user=request.user,
+                                                      survey=survey)
+            form = SurveyForm(survey=survey,
+                              current_answers=current_answer.answers)
         except SurveyAnswer.DoesNotExist:
             current_answer = None
             form = SurveyForm(survey=survey)
-    return render(request, "survey.html", {"survey": survey, "form": form, "success": success, "deleted": deleted, "current_answer": current_answer})
+    return render(request, "survey.html", {"survey": survey, "form": form,
+                                           "success": success,
+                                           "deleted": deleted,
+                                           "current_answer": current_answer})
+
 
 def get_event_form_choices(event, form):
     all_choices = []
@@ -132,17 +153,20 @@ def get_event_form_choices(event, form):
             all_choices.append(choice)
     return all_choices
 
+
 def update_event_form_comments(event, form, registration):
     for commentfield_id, value in form.comments():
         field = get_object_or_404(EventCommentField, id=commentfield_id,
                                   event=event)
         if value == field.default:
             continue
-        (storage, _) = EventCommentValue.objects.get_or_create(commentfield = field,
+        (storage, _) = EventCommentValue.objects.get_or_create(
+                            commentfield=field,
                             registration=registration)
         storage.content = value
         storage.save()
 
+
 @login_required
 def event(request, event_id):
     event = get_object_or_404(Event, id=event_id)
@@ -153,17 +177,23 @@ def event(request, event_id):
         form = EventForm(request.POST, event=event)
         if form.is_valid():
             all_choices = get_event_form_choices(event, form)
-            (current_registration, _) = EventRegistration.objects.get_or_create(user = request.user, event = event)
+            (current_registration, _) = \
+                EventRegistration.objects.get_or_create(user=request.user,
+                                                        event=event)
             current_registration.options = all_choices
             current_registration.save()
             success = True
     else:
         try:
-            current_registration = EventRegistration.objects.get(user = request.user, event = event)
-            form = EventForm(event = event, current_choices = current_registration.options)
+            current_registration = \
+                EventRegistration.objects.get(user=request.user, event=event)
+            form = EventForm(event=event,
+                             current_choices=current_registration.options)
         except EventRegistration.DoesNotExist:
             form = EventForm(event=event)
-    return render(request, "event.html", {"event": event, "form": form, "success": success})
+    return render(request, "event.html",
+                  {"event": event, "form": form, "success": success})
+
 
 def clean_post_for_status(initial):
     d = initial.copy()
@@ -173,6 +203,7 @@ def clean_post_for_status(initial):
             d[k[3:]] = v
     return d
 
+
 @buro_required
 def event_status(request, event_id):
     event = get_object_or_404(Event, id=event_id)
@@ -185,15 +216,19 @@ def event_status(request, event_id):
                 if value == "yes":
                     registrations_query = registrations_query.filter(paid=True)
                 elif value == "no":
-                    registrations_query = registrations_query.filter(paid = False)
+                    registrations_query = registrations_query.filter(
+                                                paid=False)
                 continue
-            choice = get_object_or_404(EventOptionChoice, id = choice_id, event_option__id = option_id)
+            choice = get_object_or_404(EventOptionChoice, id=choice_id,
+                                       event_option__id=option_id)
             if value == "none":
                 continue
             if value == "yes":
-                registrations_query = registrations_query.filter(options__id__exact = choice.id)
+                registrations_query = registrations_query.filter(
+                                            options__id__exact=choice.id)
             elif value == "no":
-                registrations_query = registrations_query.exclude(options__id__exact = choice.id)
+                registrations_query = registrations_query.exclude(
+                                            options__id__exact=choice.id)
     user_choices = registrations_query.prefetch_related("user").all()
     options = EventOption.objects.filter(event=event).all()
     choices_count = {}
@@ -203,7 +238,11 @@ def event_status(request, event_id):
     for user_choice in user_choices:
         for choice in user_choice.options.all():
             choices_count[choice.id] += 1
-    return render(request, "event_status.html", {"event": event, "user_choices": user_choices, "options": options, "choices_count": choices_count, "form": form})
+    return render(request, "event_status.html",
+                  {"event": event, "user_choices": user_choices,
+                   "options": options, "choices_count": choices_count,
+                   "form": form})
+
 
 @buro_required
 def survey_status(request, survey_id):
@@ -213,13 +252,16 @@ def survey_status(request, survey_id):
     form = SurveyStatusFilterForm(post_data or None, survey=survey)
     if form.is_valid():
         for question_id, answer_id, value in form.filters():
-            answer = get_object_or_404(SurveyQuestionAnswer, id = answer_id, survey_question__id = question_id)
+            answer = get_object_or_404(SurveyQuestionAnswer, id=answer_id,
+                                       survey_question__id=question_id)
             if value == "none":
                 continue
             if value == "yes":
-                answers_query = answers_query.filter(answers__id__exact = answer.id)
+                answers_query = answers_query.filter(
+                                    answers__id__exact=answer.id)
             elif value == "no":
-                answers_query = answers_query.exclude(answers__id__exact = answer.id)
+                answers_query = answers_query.exclude(
+                                    answers__id__exact=answer.id)
     user_answers = answers_query.prefetch_related("user").all()
     questions = SurveyQuestion.objects.filter(survey=survey).all()
     answers_count = {}
@@ -229,7 +271,11 @@ def survey_status(request, survey_id):
     for user_answer in user_answers:
         for answer in user_answer.answers.all():
             answers_count[answer.id] += 1
-    return render(request, "survey_status.html", {"survey": survey, "user_answers": user_answers, "questions": questions, "answers_count": answers_count, "form": form})
+    return render(request, "survey_status.html",
+                  {"survey": survey, "user_answers": user_answers,
+                   "questions": questions, "answers_count": answers_count,
+                   "form": form})
+
 
 @login_required
 def profile(request):
@@ -243,10 +289,12 @@ def profile(request):
         form = UserProfileForm(instance=request.user.profile)
     return render(request, "profile.html", {"form": form, "success": success})
 
+
 def registration_set_ro_fields(user_form, profile_form):
     user_form.fields['username'].widget.attrs['readonly'] = True
     profile_form.fields['login_clipper'].widget.attrs['readonly'] = True
 
+
 @buro_required
 def registration_form(request, login_clipper=None, username=None):
     member = None
@@ -261,7 +309,8 @@ def registration_form(request, login_clipper = None, username = None):
             user_form = RegistrationUserForm()
             profile_form = RegistrationProfileForm()
             user_form.fields['username'].initial = login_clipper
-            user_form.fields['email'].initial = login_clipper + "@clipper.ens.fr"
+            user_form.fields['email'].initial = \
+                login_clipper + "@clipper.ens.fr"
             profile_form.fields['login_clipper'].initial = login_clipper
             if clipper.fullname:
                 bits = clipper.fullname.split(" ")
@@ -280,7 +329,10 @@ def registration_form(request, login_clipper = None, username = None):
         # new user
         user_form = RegistrationUserForm()
         profile_form = RegistrationProfileForm()
-    return render(request, "registration_form.html", {"user_form": user_form, "profile_form": profile_form, "member": member, "login_clipper": login_clipper})
+    return render(request, "registration_form.html",
+                  {"user_form": user_form, "profile_form": profile_form,
+                   "member": member, "login_clipper": login_clipper})
+
 
 @buro_required
 def registration_form2(request, login_clipper=None, username=None):
@@ -298,7 +350,8 @@ def registration_form2(request, login_clipper = None, username = None):
             profile_form = RegistrationProfileForm()
             event_forms = [AdminEventForm(event=event) for event in events]
             user_form.fields['username'].initial = login_clipper
-            user_form.fields['email'].initial = login_clipper + "@clipper.ens.fr"
+            user_form.fields['email'].initial = \
+                login_clipper + "@clipper.ens.fr"
             profile_form.fields['login_clipper'].initial = login_clipper
             if clipper.fullname:
                 bits = clipper.fullname.split(" ")
@@ -316,8 +369,12 @@ def registration_form2(request, login_clipper = None, username = None):
         event_forms = []
         for event in events:
             try:
-                current_registration = EventRegistration.objects.get(user = member, event = event)
-                form = AdminEventForm(event = event, current_registration = current_registration, paid = current_registration.paid)
+                current_registration = EventRegistration.objects.get(
+                                            user=member, event=event)
+                form = AdminEventForm(
+                            event=event,
+                            current_registration=current_registration,
+                            paid=current_registration.paid)
             except EventRegistration.DoesNotExist:
                 form = AdminEventForm(event=event)
             event_forms.append(form)
@@ -326,7 +383,11 @@ def registration_form2(request, login_clipper = None, username = None):
         user_form = RegistrationUserForm()
         profile_form = RegistrationProfileForm()
         event_forms = [AdminEventForm(event=event) for event in events]
-    return render(request, "registration_form.html", {"user_form": user_form, "profile_form": profile_form, "member": member, "login_clipper": login_clipper, "event_forms": event_forms})
+    return render(request, "registration_form.html",
+                  {"user_form": user_form, "profile_form": profile_form,
+                   "member": member, "login_clipper": login_clipper,
+                   "event_forms": event_forms})
+
 
 @buro_required
 def registration(request):
@@ -338,10 +399,12 @@ def registration(request):
         user_form = RegistrationUserForm(request_dict)
         profile_form = RegistrationProfileForm(request_dict)
         events = Event.objects.filter(old=False).all()
-        event_forms = [AdminEventForm(request_dict, event = event) for event in events]
+        event_forms = \
+            [AdminEventForm(request_dict, event=event) for event in events]
         user_form.is_valid()
         profile_form.is_valid()
-        for event_form in event_forms: event_form.is_valid()
+        for event_form in event_forms:
+            event_form.is_valid()
         member = None
         login_clipper = None
         if "user_exists" in request_dict and request_dict["user_exists"]:
@@ -350,7 +413,8 @@ def registration(request):
                 member = User.objects.filter(username=username).get()
                 (profile, _) = CofProfile.objects.get_or_create(user=member)
                 user_form = RegistrationUserForm(request_dict, instance=member)
-                profile_form = RegistrationProfileForm(request_dict, instance = profile)
+                profile_form = RegistrationProfileForm(request_dict,
+                                                       instance=profile)
             except User.DoesNotExist:
                 try:
                     clipper = Clipper.objects.filter(username=username).get()
@@ -358,15 +422,19 @@ def registration(request):
                 except Clipper.DoesNotExist:
                     pass
         for form in event_forms:
-            if not form.is_valid(): break
-            if form.cleaned_data['status'] == 'no': continue
+            if not form.is_valid():
+                break
+            if form.cleaned_data['status'] == 'no':
+                continue
             all_choices = get_event_form_choices(form.event, form)
-        if user_form.is_valid() and profile_form.is_valid() and not any([not form.is_valid() for form in event_forms]):
+        if user_form.is_valid() and profile_form.is_valid() \
+                and not any([not form.is_valid() for form in event_forms]):
             member = user_form.save()
             (profile, _) = CofProfile.objects.get_or_create(user=member)
             was_cof = profile.is_cof
             request_dict["num"] = profile.num
-            profile_form = RegistrationProfileForm(request_dict, instance = profile)
+            profile_form = RegistrationProfileForm(request_dict,
+                                                   instance=profile)
             profile_form.is_valid()
             profile_form.save()
             (profile, _) = CofProfile.objects.get_or_create(user=member)
@@ -375,29 +443,43 @@ def registration(request):
             for form in event_forms:
                 if form.cleaned_data['status'] == 'no':
                     try:
-                        current_registration = EventRegistration.objects.get(user = member, event = form.event)
+                        current_registration = EventRegistration.objects.get(
+                            user=member, event=form.event)
                         current_registration.delete()
                     except EventRegistration.DoesNotExist:
                         pass
                     continue
                 all_choices = get_event_form_choices(form.event, form)
-                (current_registration, created_reg) = EventRegistration.objects.get_or_create(user = member, event = form.event)
+                (current_registration, created_reg) = \
+                    EventRegistration.objects.get_or_create(user=member,
+                                                            event=form.event)
                 update_event_form_comments(event, form, current_registration)
                 current_registration.options = all_choices
-                current_registration.paid = (form.cleaned_data['status'] == 'paid')
+                current_registration.paid = \
+                    (form.cleaned_data['status'] == 'paid')
                 current_registration.save()
                 if event.title == "Mega 15" and created_reg:
-                    field = EventCommentField.objects.get(event = event, name = "Commentaires")
+                    field = EventCommentField.objects.get(event=event,
+                                                          name="Commentaires")
                     try:
-                        comments = EventCommentValue.objects.get(commentfield = field, registration = current_registration).content
+                        comments = EventCommentValue.objects.get(
+                            commentfield=field,
+                            registration=current_registration).content
                     except EventCommentValue.DoesNotExist:
                         comments = field.default
                     send_custom_mail(member, "mega", {"remarques": comments})
             success = True
-        return render(request, "registration_post.html", {"success": success, "user_form": user_form, "profile_form": profile_form, "member": member, "login_clipper": login_clipper, "event_forms": event_forms})
+        return render(request, "registration_post.html",
+                      {"success": success,
+                       "user_form": user_form,
+                       "profile_form": profile_form,
+                       "member": member,
+                       "login_clipper": login_clipper,
+                       "event_forms": event_forms})
     else:
         return render(request, "registration.html")
 
+
 @buro_required
 def export_members(request):
     response = HttpResponse(content_type='text/csv')
@@ -406,11 +488,14 @@ def export_members(request):
     writer = unicodecsv.writer(response)
     for profile in CofProfile.objects.filter(is_cof=True).all():
         user = profile.user
-        bits = [profile.num, user.username, user.first_name, user.last_name, user.email, profile.phone, profile.occupation, profile.departement, profile.type_cotiz]
+        bits = [profile.num, user.username, user.first_name, user.last_name,
+                user.email, profile.phone, profile.occupation,
+                profile.departement, profile.type_cotiz]
         writer.writerow([unicode(bit) for bit in bits])
 
     return response
 
+
 @buro_required
 def csv_export_mega(filename, qs):
     response = HttpResponse(content_type='text/csv')
@@ -420,12 +505,17 @@ def csv_export_mega(filename, qs):
     for reg in qs.all():
         user = reg.user
         profile = user.profile
-        comments = "---".join([comment.content for comment in reg.comments.all()])
-        bits = [user.username, user.first_name, user.last_name, user.email, profile.phone, profile.num, profile.comments if profile.comments else "", comments]
+        comments = "---".join(
+            [comment.content for comment in reg.comments.all()])
+        bits = [user.username, user.first_name, user.last_name, user.email,
+                profile.phone, profile.num,
+                profile.comments if profile.comments else "", comments]
+
         writer.writerow([unicode(bit) for bit in bits])
 
     return response
 
+
 @buro_required
 def export_mega_remarksonly(request):
     filename = 'remarques_mega_2015.csv'
@@ -439,11 +529,13 @@ def export_mega_remarksonly(request):
         reg = val.registration
         user = reg.user
         profile = user.profile
-        bits = [user.username, user.first_name, user.last_name, user.email, profile.phone, profile.num, profile.comments, val.content]
+        bits = [user.username, user.first_name, user.last_name, user.email,
+                profile.phone, profile.num, profile.comments, val.content]
         writer.writerow([unicode(bit) for bit in bits])
 
     return response
 
+
 @buro_required
 def export_mega_bytype(request, type):
     types = {"orga-actif": "Orga élève",
@@ -457,57 +549,72 @@ def export_mega_bytype(request, type):
     event = Event.objects.get(title="Mega 15")
     type_option = event.options.get(name="Type")
     participant_type = type_option.choices.get(value=types[type]).id
-    qs = EventRegistration.objects.filter(event = event).filter(options__id__exact = participant_type)
+    qs = EventRegistration.objects.filter(event=event).filter(
+        options__id__exact=participant_type)
     return csv_export_mega(type + '_mega_2015.csv', qs)
 
+
 @buro_required
 def export_mega_orgas(request):
     event = Event.objects.get(title="Mega 15")
     type_option = event.options.get(name="Type")
     participant_type_a = type_option.choices.get(value="Conscrit étudiant").id
     participant_type_b = type_option.choices.get(value="Conscrit élève").id
-    qs = EventRegistration.objects.filter(event = event).exclude(options__id__in = (participant_type_a, participant_type_b))
+    qs = EventRegistration.objects.filter(event=event).exclude(
+        options__id__in=(participant_type_a, participant_type_b))
     return csv_export_mega('orgas_mega_15.csv', qs)
 
+
 @buro_required
 def export_mega_participants(request):
     event = Event.objects.get(title="Mega 15")
     type_option = event.options.get(name="Type")
     participant_type_a = type_option.choices.get(value="Conscrit étudiant").id
     participant_type_b = type_option.choices.get(value="Conscrit élève").id
-    qs = EventRegistration.objects.filter(event = event).filter(options__id__in = (participant_type_a, participant_type_b))
+    qs = EventRegistration.objects.filter(event=event).filter(
+        options__id__in=(participant_type_a, participant_type_b))
     return csv_export_mega('participants_mega_15.csv', qs)
 
+
 @buro_required
 def export_mega(request):
     event = Event.objects.filter(title="Mega 15")
-    qs = EventRegistration.objects.filter(event = event).order_by("user__username")
+    qs = EventRegistration.objects.filter(event=event) \
+        .order_by("user__username")
     return csv_export_mega('all_mega_2015.csv', qs)
 
+
 @buro_required
 def utile_cof(request):
     return render(request, "utile_cof.html", {})
 
+
 @buro_required
 def utile_bda(request):
     tirages = Tirage.objects.all()
     return render(request, "utile_bda.html", {'tirages': tirages})
 
+
 @buro_required
 def liste_bdadiff(request):
     titre = "BdA diffusion"
     personnes = CofProfile.objects.filter(mailing_bda=True, is_cof=True).all()
-    return render(request, "liste_mails.html", {"titre": titre, "personnes": personnes})
+    return render(request, "liste_mails.html",
+                  {"titre": titre, "personnes": personnes})
+
 
 @buro_required
 def liste_bdarevente(request):
     titre = "BdA revente"
-    personnes = CofProfile.objects.filter(mailing_bda_revente = True, is_cof = True).all()
-    return render(request, "liste_mails.html", {"titre": titre, "personnes": personnes})
+    personnes = CofProfile.objects.filter(mailing_bda_revente=True,
+                                          is_cof=True).all()
+    return render(request, "liste_mails.html", {"titre": titre,
+                                                "personnes": personnes})
+
 
 @buro_required
 def liste_diffcof(request):
     titre = "Diffusion COF"
     personnes = CofProfile.objects.filter(mailing_cof=True, is_cof=True).all()
-    return render(request, "liste_mails.html", {"titre": titre, "personnes": personnes})
-
+    return render(request, "liste_mails.html", {"titre": titre,
+                                                "personnes": personnes})

gestioncof/widgets.py

@@ -2,8 +2,8 @@ from django.forms.widgets import Widget
 from django.forms.utils import flatatt
 from django.utils.safestring import mark_safe
 
-class TriStateCheckbox(Widget):
 
+class TriStateCheckbox(Widget):
     def __init__(self, attrs=None, choices=()):
         super(TriStateCheckbox, self).__init__(attrs)
         # choices can be any iterable, but we may need to render this widget
@@ -12,7 +12,8 @@ class TriStateCheckbox(Widget):
         self.choices = list(choices)
 
     def render(self, name, value, attrs=None, choices=()):
-        if value is None: value = 'none'
+        if value is None:
+            value = 'none'
         final_attrs = self.build_attrs(attrs, value=value)
         output = [u"<span class=\"tristate\"%s></span>" % flatatt(final_attrs)]
         return mark_safe('\n'.join(output))

tirage_bda.py

@@ -15,16 +15,19 @@ if __name__ == "__main__":
     start = time.time()
     shows = Spectacle.objects.all()
     members = Participant.objects.all()
-    choices = ChoixSpectacle.objects.order_by('participant', 'priority').select_related().all()
+    choices = ChoixSpectacle.objects.order_by('participant', 'priority') \
+        .select_related().all()
     available_slots = Spectacle.objects.aggregate(Sum('slots'))['slots__sum']
     cursor = connection.cursor()
-    cursor.execute("SELECT SUM(`slots` * `price`) AS `total` FROM `bda_spectacle`;")
+    cursor.execute(
+        "SELECT SUM(`slots` * `price`) AS `total` FROM `bda_spectacle`;")
     total_price = cursor.fetchone()[0]
     print "%d spectacles" % len(shows)
     print "%d places" % available_slots
     print "%d participants" % len(members)
     print "%d demandes" % len(choices)
-    print "%d places demandées" % (len(choices) + len(choices.filter(double = True).all()))
+    print "%d places demandées" % (len(choices)
+                                   + len(choices.filter(double=True).all()))
     print "%.02f€ à brasser" % total_price
     print "Récupération: %.2fs" % (time.time() - start)
     start_init = time.time()
@@ -54,7 +57,8 @@ if __name__ == "__main__":
             members2[member].append(show)
             member.total += show.price
         if len(members) < show.slots:
-            print "%d place(s) invendue(s) pour %s" % (show.slots - len(members), show)
+            print "%d place(s) invendue(s) pour %s" \
+                % (show.slots - len(members), show)
     members2 = members2.items()
     print "Temps total: %.2fs" % (time.time() - start)
     print "Requêtes SQL:"