From 574aaad745915c1830f9909b97a2991e3bf82b0a Mon Sep 17 00:00:00 2001
From: ludo <lstephan@clipper.ens.fr>
Date: Thu, 16 Jun 2016 15:09:14 +0200
Subject: [PATCH] =?UTF-8?q?gestion=20des=20erreurs=20et=20des=20messages?=
 =?UTF-8?q?=20associ=C3=A9s?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 bda/templates/bda-participants.html |  5 ++++-
 bda/views.py                        | 27 +++++++++++++++++++++++----
 2 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/bda/templates/bda-participants.html b/bda/templates/bda-participants.html
index 5cc678d1..ed3767a3 100644
--- a/bda/templates/bda-participants.html
+++ b/bda/templates/bda-participants.html
@@ -2,6 +2,9 @@
 
 {% block realcontent %}
     <h2>{{ spectacle }}</h2>
+    {% for message in messages %}
+      <h3>{{ message }}</h3>
+    {% endfor %}
       <table class='etat-bda' align="center">
         <thead>
           <tr>
@@ -36,7 +39,7 @@
         {% csrf_token %}
         <tr>
           <td></td>
-          <td><input type="text" name="clipper" size=8></td> 
+          <td><input type="text" name="username" size=8></td> 
           <td><select name="nb_places">
                 <option value=1>1 place</option>
                 <option value=2>2 places</option>
diff --git a/bda/views.py b/bda/views.py
index e004854f..c8e250f1 100644
--- a/bda/views.py
+++ b/bda/views.py
@@ -14,6 +14,7 @@ from django.utils import timezone
 from django.views.generic.list import ListView
 from django.http import HttpResponseRedirect
 from django.core.urlresolvers import reverse
+from django.contrib import messages
 
 from datetime import timedelta
 import time
@@ -319,13 +320,31 @@ def spectacle(request, tirage_id, spectacle_id):
 def add_attrib(request, tirage_id, spectacle_id):
     tirage = get_object_or_404(Tirage, id=tirage_id)
     spectacle = get_object_or_404(Spectacle, id = spectacle_id, tirage=tirage)
-    # TODO : erreur
-    part= tirage.participant_set.get(user__username=request.POST['clipper'])
-    attrib = Attribution(participant=part, spectacle=spectacle, given=request.POST['given'])
+    try:
+        part=tirage.participant_set.get(user__username=request.POST['username'])
+    except Participant.DoesNotExist:
+        messages.add_message(request, messages.ERROR, 
+            u"Erreur : utilisateur %s non trouvé" % request.POST['username'])
+        return HttpResponseRedirect(reverse('bda-spectacle', 
+                                    args=(tirage_id, spectacle_id,)))
+
+    places_owned = len(spectacle.attribues.filter(participant=part))
+
+    if int(request.POST['nb_places'])+places_owned > 2:
+        messages.add_message(request, messages.ERROR,
+                        "Erreur: on ne peut pas avoir plus de deux places")
+        return HttpResponseRedirect(reverse('bda-spectacle', 
+                                    args=(tirage_id, spectacle_id,)))
+
+    attrib = Attribution(participant=part, spectacle=spectacle,
+                        given=('given' in request.POST))
     attrib.save()
     if request.POST['nb_places']==2:
         attrib.save()
-    return HttpResponseRedirect(reverse('bda-spectacle', args=(tirage_id, spectacle_id,)))
+    messages.add_message(request, messages.SUCCESS, 
+                        "Attribution réussie !")
+    return HttpResponseRedirect(reverse('bda-spectacle',
+                                        args=(tirage_id, spectacle_id,)))
     
 class SpectacleListView(ListView):
     model = Spectacle