From f18959c0a1d643fab3b08921a004f158d4ba4720 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20P=C3=A9pin?= <martin.pepin@ens.fr>
Date: Wed, 1 Nov 2017 17:26:40 +0100
Subject: [PATCH] BdA-Revente: meaningful names, some help tests

---
 bda/templates/bda/revente/manage.html    | 58 +++++++++++++++++++-----
 bda/templates/bda/revente/subscribe.html | 39 ++++++++++------
 bda/templates/bda/revente/tirages.html   | 40 ++++++++++++----
 bda/views.py                             | 18 ++++++++
 gestioncof/static/css/cof.css            | 11 +++++
 gestioncof/templates/home.html           |  6 +--
 6 files changed, 136 insertions(+), 36 deletions(-)

diff --git a/bda/templates/bda/revente/manage.html b/bda/templates/bda/revente/manage.html
index 0912babb..8162d55d 100644
--- a/bda/templates/bda/revente/manage.html
+++ b/bda/templates/bda/revente/manage.html
@@ -3,50 +3,84 @@
 
 {% block realcontent %}
 
-<h2>Revente de place</h2>
+<h2>Gestion des places que je revends</h2>
 {% with resell_attributions=resellform.attributions annul_attributions=annulform.attributions sold_attributions=soldform.attributions %}
 
 {% if resellform.attributions %}
+<br />
+
 <h3>Places non revendues</h3>
 <form class="form-horizontal" action="" method="post">
-    {% csrf_token %}
-    {{resellform|bootstrap}}
+    <div class="bg-info text-info center-block">
+        <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
+        Cochez les places que vous souhaitez revendre, et validez. Vous aurez
+        ensuite 1h pour changer d'avis avant que la revente soit confirmée et
+        que les notifications soient envoyées aux intéressé·e·s.
+    </div>
+    <div class="bootstrap-form-reduce">
+        {% csrf_token %}
+        {{ resellform|bootstrap }}
+    </div>
     <div class="form-actions">
         <input type="submit" class="btn btn-primary" name="resell" value="Revendre les places sélectionnées">
     </div>
 </form>
+
+<hr />
 {% endif %}
-<br>
+
 {% if annul_attributions or overdue %}
 <h3>Places en cours de revente</h3>
 <form action="" method="post">
+    {% if annul_attributions %}
+        <div class="bg-info text-info center-block">
+            <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
+            Vous pouvez annuler les places mises en vente il y a moins d'une heure.
+        </div>
+    {% endif %}
     {% csrf_token %}
     <div class='form-group'>
         <div class='multiple-checkbox'>
             <ul>
                 {% for attrib in annul_attributions %}
-                    <li>{{attrib.tag}} {{attrib.choice_label}}</li>
+                    <li>{{ attrib.tag }} {{ attrib.choice_label }}</li>
                 {% endfor %}
                 {% for attrib in overdue %}
                     <li>
                         <input type="checkbox" style="visibility:hidden">
-                        {{attrib.spectacle}}
+                        {{ attrib.spectacle }}
                     </li>
                 {% endfor %}
+            </ul>
+        </div>
+    </div>
     {% if annul_attributions %}
     <input type="submit" class="btn btn-primary" name="annul" value="Annuler les reventes sélectionnées">
     {% endif %}
 </form>
+
+<hr />
 {% endif %}
-<br>
+
 {% if sold_attributions %}
 <h3>Places revendues</h3>
-    <form action="" method="post">
+<form action="" method="post">
+    <div class="bg-info text-info center-block">
+        <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
+        Pour chaque revente, vous devez soit l'annuler soit la confirmer pour
+        transférer la place la place à la personne tirée au sort.
+
+        L'annulation sert par exemple à pouvoir remettre la place en jeu si
+        vous ne parvenez pas à entrer en contact avec la personne tirée au
+        sort.
+    </div>
+    <div class="bootstrap-form-reduce">
         {% csrf_token %}
-        {{soldform|bootstrap}}
-        <button type="submit" class="btn btn-primary" name="transfer">Transférer</button>
-        <button type="submit" class="btn btn-primary" name="reinit">Réinitialiser</button>
-    </form>
+        {{ soldform|bootstrap }}
+    </div>
+    <button type="submit" class="btn btn-primary" name="transfer">Transférer</button>
+    <button type="submit" class="btn btn-primary" name="reinit">Réinitialiser</button>
+</form>
 {% endif %}
 {% if not resell_attributions and not annul_attributions and not overdue and not sold_attributions %}
 <p>Plus de reventes possibles !</p>
diff --git a/bda/templates/bda/revente/subscribe.html b/bda/templates/bda/revente/subscribe.html
index fcf57345..9a193908 100644
--- a/bda/templates/bda/revente/subscribe.html
+++ b/bda/templates/bda/revente/subscribe.html
@@ -4,28 +4,41 @@
 {% block realcontent %}
   <h2>Inscriptions pour BdA-Revente</h2>
   <form action="" class="form-horizontal" method="post">
+    <div class="bg-info text-info center-block">
+        <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
+        Cochez les spectacles pour lesquels vous souhaitez recevoir un
+        notification quand une place est disponible en revente. <br />
+        Lorsque vous validez vos choix, si un tirage au sort est en cours pour
+        un des spectacles que vous avez sélectionné, vous serez automatiquement
+        inscrit à ce tirage.
+    </div>
+    <br />
     {% csrf_token %}
     <div class="form-group">
-        <h3>Spectacles</h3>
-        <br/>
-      <button type="button" class="btn btn-primary" onClick="select(true)">Tout sélectionner</button>
-      <button type="button" class="btn btn-primary" onClick="select(false)">Tout désélectionner</button>
+      <button type="button"
+              class="btn btn-primary"
+              onClick="select(true)">Tout sélectionner</button>
+      <button type="button"
+              class="btn btn-primary"
+              onClick="select(false)">Tout désélectionner</button>
 
-        <div class="multiple-checkbox">
-            <ul>
-                {% for checkbox in form.spectacles %}
-                <li>{{checkbox}}</li>
-                {%endfor%}
-            </ul>
-        </div>
+      <div class="multiple-checkbox">
+        <ul>
+          {% for checkbox in form.spectacles %}
+            <li>{{ checkbox }}</li>
+          {% endfor %}
+        </ul>
+      </div>
     </div>
-  <input type="submit" class="btn btn-primary" value="S'inscrire pour les places sélectionnées">
+    <input type="submit"
+           class="btn btn-primary"
+           value="S'inscrire pour les places sélectionnées">
   </form>
 
 <script language="JavaScript">
 function select(check) {
     checkboxes = document.getElementsByName("spectacles");
-    for(var i=0, n=checkboxes.length;i<n;i++) {
+    for(var i=0, n=checkboxes.length; i < n; i++) {
         checkboxes[i].checked = check;
     }
 }
diff --git a/bda/templates/bda/revente/tirages.html b/bda/templates/bda/revente/tirages.html
index bd738673..91d39c90 100644
--- a/bda/templates/bda/revente/tirages.html
+++ b/bda/templates/bda/revente/tirages.html
@@ -4,25 +4,49 @@
 {% block realcontent %}
 
 <h2>Tirages au sort de reventes</h2>
+
 {% if annulform.reventes %}
-<h3>Mes inscriptions</h3>
+<h3>Les reventes auxquelles vous êtes inscrit·e</h3>
 <form class="form-horizontal" action="" method="post">
-    {% csrf_token %}
-    {{annulform|bootstrap}}
+    <div class="bg-info text-info center-block">
+        <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
+        Vous pouvez vous désinscrire des reventes suivantes tant que le tirage n'a
+        pas eu lieu
+    </div>
+    <div class="bootstrap-form-reduce">
+        {% csrf_token %}
+        {{ annulform|bootstrap }}
+    </div>
     <div class="form-actions">
-        <input type="submit" class="btn btn-primary" name="annul" value="Se désinscrire des tirages sélectionnés">
+        <input type="submit"
+               class="btn btn-primary"
+               name="annul"
+               value="Se désinscrire des tirages sélectionnés">
     </div>
 </form>
+
+<hr />
 {% endif %}
-<br>
+
 {% if subform.reventes %}
+
 <h3>Tirages en cours</h3>
 <form class="form-horizontal" action="" method="post">
-    {% csrf_token %}
-    {{subform|bootstrap}}
+    <div class="bg-info text-info center-block">
+        <span class="glyphicon glyphicon-info-sign" aria-hidden="true"></span>
+        Vous pouvez vous inscrire aux tirage en cours suivants
+    </div>
+    <div class="bootstrap-form-reduce">
+        {% csrf_token %}
+        {{ subform|bootstrap }}
+    </div>
     <div class="form-actions">
-        <input type="submit" class="btn btn-primary" name="subscribe" value="S'inscrire aux tirages sélectionnés">
+        <input type="submit"
+               class="btn btn-primary"
+               name="subscribe"
+               value="S'inscrire aux tirages sélectionnés">
     </div>
 </form>
 {% endif %}
+
 {% endblock %}
diff --git a/bda/views.py b/bda/views.py
index fb1a2e82..67fa2486 100644
--- a/bda/views.py
+++ b/bda/views.py
@@ -350,6 +350,14 @@ def tirage(request, tirage_id):
 
 @login_required
 def revente_manage(request, tirage_id):
+    """
+    Gestion de ses propres reventes :
+    - Création d'une revente
+    - Annulation d'une revente
+    - Confirmation d'une revente = transfert de la place à la personne qui
+      rachète
+    - Annulation d'une revente après que le tirage a eu lieu
+    """
     tirage = get_object_or_404(Tirage, id=tirage_id)
     participant, created = Participant.objects.get_or_create(
         user=request.user, tirage=tirage)
@@ -435,6 +443,10 @@ def revente_manage(request, tirage_id):
 
 @login_required
 def revente_tirages(request, tirage_id):
+    """
+    Affiche à un participant la liste de toutes les reventes en cours (pour un
+    tirage donné) et lui permet de s'inscrire et se désinscrire à ces reventes.
+    """
     tirage = get_object_or_404(Tirage, id=tirage_id)
     participant, _ = Participant.objects.get_or_create(
             user=request.user, tirage=tirage)
@@ -492,6 +504,12 @@ def revente_confirm(request, revente_id):
 
 @login_required
 def revente_subscribe(request, tirage_id):
+    """
+    Permet à un participant de sélectionner ses préférences pour les reventes.
+    Il recevra des notifications pour les spectacles qui l'intéressent et il
+    est automatiquement inscrit aux reventes en cours au moment où il ajoute un
+    spectacle à la liste des spectacles qui l'intéressent.
+    """
     tirage = get_object_or_404(Tirage, id=tirage_id)
     participant, _ = Participant.objects.get_or_create(
         user=request.user, tirage=tirage)
diff --git a/gestioncof/static/css/cof.css b/gestioncof/static/css/cof.css
index 91cf726a..84c81d60 100644
--- a/gestioncof/static/css/cof.css
+++ b/gestioncof/static/css/cof.css
@@ -1140,3 +1140,14 @@ p.help-block {
   margin: 5px auto;
   width: 90%;
 }
+
+div.bg-info {
+  border-radius: 3px;
+  padding: 0.3em 1em;
+  margin-left: 1em;
+  margin-right: 1em;
+}
+
+.bootstrap-form-reduce > .form-group {
+  margin-top: -16px;
+}
diff --git a/gestioncof/templates/home.html b/gestioncof/templates/home.html
index 943ef780..0de68c8d 100644
--- a/gestioncof/templates/home.html
+++ b/gestioncof/templates/home.html
@@ -43,9 +43,9 @@
                             <li><a href="{% url "bda-etat-places" tirage.id %}">État des demandes</a></li>
                         {% else %}
                             <li><a href="{% url "bda-places-attribuees" tirage.id %}">Mes places</a></li> 
-                            <li><a href="{% url "bda-revente-manage" tirage.id %}">Gestion de mes reventes</a></li> 
-                            <li><a href="{% url "bda-revente-tirages" tirage.id %}">Reventes en cours</a></li> 
-                            <li><a href="{% url "bda-revente-subscribe" tirage.id %}">S'inscrire à BdA-Revente</a></li>
+                            <li><a href="{% url "bda-revente-manage" tirage.id %}">Gérer les places que je revends</a></li> 
+                            <li><a href="{% url "bda-revente-tirages" tirage.id %}">Voir les reventes en cours</a></li> 
+                            <li><a href="{% url "bda-revente-subscribe" tirage.id %}">Indiquer les spectacles qui m'intéressent</a></li>
                             <li><a href="{% url "bda-revente-shotgun" tirage.id %}">Places disponibles immédiatement</a></li>
                         {% endif %}
                     </ul>