From 68de1aefeadc6abf1ef7c0ffc518c846a7dbfb6f Mon Sep 17 00:00:00 2001
From: Gabriel Augendre <gabriel@augendre.info>
Date: Sat, 2 Jun 2018 17:51:53 +0200
Subject: [PATCH] Implement teacher lists confirmation. #19

---
 .../0022_teacher_has_confirmed_list.py        | 18 +++++
 manuels/models.py                             |  5 ++
 .../templates/manuels/confirm_teacher.html    | 29 +++++++
 .../manuels/list_books_supplies.html          | 75 ++++++++++++-------
 manuels/urls.py                               |  3 +-
 manuels/views.py                              | 21 +++++-
 manuels_collection/settings.py                |  5 ++
 7 files changed, 128 insertions(+), 28 deletions(-)
 create mode 100644 manuels/migrations/0022_teacher_has_confirmed_list.py
 create mode 100644 manuels/templates/manuels/confirm_teacher.html

diff --git a/manuels/migrations/0022_teacher_has_confirmed_list.py b/manuels/migrations/0022_teacher_has_confirmed_list.py
new file mode 100644
index 0000000..592c7c8
--- /dev/null
+++ b/manuels/migrations/0022_teacher_has_confirmed_list.py
@@ -0,0 +1,18 @@
+# Generated by Django 2.0.5 on 2018-06-02 15:36
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('manuels', '0021_auto_20180602_1638'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='teacher',
+            name='has_confirmed_list',
+            field=models.BooleanField(default=False, verbose_name='a confirmé les listes'),
+        ),
+    ]
diff --git a/manuels/models.py b/manuels/models.py
index ffbd7b4..6722e1e 100644
--- a/manuels/models.py
+++ b/manuels/models.py
@@ -48,6 +48,11 @@ class Teacher(BaseModel):
         help_text='Utilisée pour vous transmettre votre lien personnel',
         unique=True
     )
+    has_confirmed_list = models.BooleanField(
+        'a confirmé les listes',
+        default=False,
+        blank=True
+    )
 
     def get_absolute_url(self):
         from django.urls import reverse
diff --git a/manuels/templates/manuels/confirm_teacher.html b/manuels/templates/manuels/confirm_teacher.html
new file mode 100644
index 0000000..3ac300b
--- /dev/null
+++ b/manuels/templates/manuels/confirm_teacher.html
@@ -0,0 +1,29 @@
+{% extends 'manuels/base.html' %}
+
+{% load bootstrap4 %}
+{% load static %}
+
+{% block content %}
+    <div class="row">
+        <div class="col-8 offset-2">
+            <h1>Bienvenue {{ teacher.full_name }}</h1>
+            <h2>
+                {% block title %}Confirmer les listes{% endblock %}
+                <a href="{% url 'list_books' teacher.pk %}" class="btn btn-secondary">Retour à vos listes</a>
+            </h2>
+            <div class="alert alert-danger">
+                <h4 class="alert-heading"><i class="fas fa-info-circle"></i> Danger</h4>
+                <p class="mb-0">
+                    <strong>Êtes-vous certain&centerdot;e de vouloir confirmer vos listes ?
+                        Cette action est définitive : vous ne pourrez plus les modifier ensuite.</strong>
+                </p>
+            </div>
+            <form action="" method="post" class="form">
+                {% csrf_token %}
+                {% buttons %}
+                    <button type="submit" class="btn btn-danger">Confirmer</button>
+                {% endbuttons %}
+            </form>
+        </div>
+    </div>
+{% endblock %}
diff --git a/manuels/templates/manuels/list_books_supplies.html b/manuels/templates/manuels/list_books_supplies.html
index d847c75..fe9f859 100644
--- a/manuels/templates/manuels/list_books_supplies.html
+++ b/manuels/templates/manuels/list_books_supplies.html
@@ -7,8 +7,15 @@
 {% block content %}
     <div class="row">
         <div class="col-12">
-            <h1>Bienvenue {{ teacher.full_name }} <a href="{% url 'clear_teacher' %}" class="btn btn-warning">Se
-                déconnecter</a></h1>
+            <h1>Bienvenue {{ teacher.full_name }}
+                <a href="{% url 'clear_teacher' %}" class="btn btn-warning">
+                    <i class="fas fa-sign-out-alt"></i> Se déconnecter</a>
+
+                {% if not teacher.has_confirmed_list %}
+                    <a href="{% url 'confirm_teacher' pk=teacher.pk %}" class="btn btn-danger">
+                        <i class="fas fa-check-circle"></i> Confirmer ma liste</a>
+                {% endif %}
+            </h1>
         </div>
     </div>
     <div class="alert alert-warning" role="alert">
@@ -25,13 +32,19 @@
         <div class="col-12">
             <h2>
                 Liste des livres demandés
-                <a href="{% url 'add_book' pk=teacher.pk %}" class="btn btn-primary">Ajouter un livre</a>
+                {% if not teacher.has_confirmed_list %}
+                    <a href="{% url 'add_book' pk=teacher.pk %}" class="btn btn-primary">
+                        <i class="fas fa-plus-circle"></i> Ajouter un livre
+                    </a>
+                {% endif %}
             </h2>
             <div class="table-responsive">
                 <table class="table table-hover table-sm">
                     <thead>
                     <tr>
-                        <th scope="col">Modifier</th>
+                        {% if not teacher.has_confirmed_list %}
+                            <th scope="col">Modifier</th>
+                        {% endif %}
                         <th scope="col">Classe</th>
                         <th scope="col">Discipline</th>
                         <th scope="col">Titre</th>
@@ -47,16 +60,18 @@
                     <tbody>
                     {% for book in books %}
                         <tr>
-                            <th scope="row">
-                                <div class="btn-group">
-                                    <a title="Modifier"
-                                       href="{% url 'edit_book' teacher_pk=book.teacher.pk pk=book.pk %}"
-                                       class="btn btn-sm btn-secondary"><i class="fas fa-edit"></i></a>
-                                    <a title="Supprimer"
-                                       href="{% url 'delete_book' teacher_pk=book.teacher.pk pk=book.pk %}"
-                                       class="btn btn-sm btn-danger"><i class="fas fa-trash"></i></a>
-                                </div>
-                            </th>
+                            {% if not teacher.has_confirmed_list %}
+                                <th scope="row">
+                                    <div class="btn-group">
+                                        <a title="Modifier"
+                                           href="{% url 'edit_book' teacher_pk=book.teacher.pk pk=book.pk %}"
+                                           class="btn btn-sm btn-secondary"><i class="fas fa-edit"></i></a>
+                                        <a title="Supprimer"
+                                           href="{% url 'delete_book' teacher_pk=book.teacher.pk pk=book.pk %}"
+                                           class="btn btn-sm btn-danger"><i class="fas fa-trash"></i></a>
+                                    </div>
+                                </th>
+                            {% endif %}
                             <td>{{ book.level }}</td>
                             <td>{{ book.field }}</td>
                             <td>{{ book.title }}</td>
@@ -81,13 +96,19 @@
         <div class="col-12">
             <h2>
                 Liste des fournitures demandées
-                <a href="{% url 'add_supplies' pk=teacher.pk %}" class="btn btn-primary">Ajouter des fournitures</a>
+                {% if not teacher.has_confirmed_list %}
+                    <a href="{% url 'add_supplies' pk=teacher.pk %}" class="btn btn-primary">
+                        <i class="fas fa-plus-circle"></i> Ajouter des fournitures
+                    </a>
+                {% endif %}
             </h2>
             <div class="table-responsive">
                 <table class="table table-hover table-sm">
                     <thead>
                     <tr>
-                        <th scope="col">Modifier</th>
+                        {% if not teacher.has_confirmed_list %}
+                            <th scope="col">Modifier</th>
+                        {% endif %}
                         <th scope="col">Classe</th>
                         <th scope="col">Disciplines</th>
                         <th scope="col">Liste de fournitures</th>
@@ -96,16 +117,18 @@
                     <tbody>
                     {% for supply in supplies %}
                         <tr>
-                            <th scope="row">
-                                <div class="btn-group">
-                                    <a title="Modifier"
-                                       href="{% url 'edit_supplies' teacher_pk=supply.teacher.pk pk=supply.pk %}"
-                                       class="btn btn-sm btn-secondary"><i class="fas fa-edit"></i></a>
-                                    <a title="Supprimer"
-                                       href="{% url 'delete_supplies' teacher_pk=supply.teacher.pk pk=supply.pk %}"
-                                       class="btn btn-sm btn-danger"><i class="fas fa-trash"></i></a>
-                                </div>
-                            </th>
+                            {% if not teacher.has_confirmed_list %}
+                                <th scope="row">
+                                    <div class="btn-group">
+                                        <a title="Modifier"
+                                           href="{% url 'edit_supplies' teacher_pk=supply.teacher.pk pk=supply.pk %}"
+                                           class="btn btn-sm btn-secondary"><i class="fas fa-edit"></i></a>
+                                        <a title="Supprimer"
+                                           href="{% url 'delete_supplies' teacher_pk=supply.teacher.pk pk=supply.pk %}"
+                                           class="btn btn-sm btn-danger"><i class="fas fa-trash"></i></a>
+                                    </div>
+                                </th>
+                            {% endif %}
                             <td>{{ supply.level }}</td>
                             <td>{{ supply.fields }}</td>
                             <td>{{ supply.supplies|linebreaksbr }}</td>
diff --git a/manuels/urls.py b/manuels/urls.py
index 1cbc72b..1c8eac0 100644
--- a/manuels/urls.py
+++ b/manuels/urls.py
@@ -1,7 +1,7 @@
 from django.urls import path
 
 from manuels.views import AddBookView, ListBooksView, clear_teacher_view, AddSuppliesView, EditBookView, \
-    EditSuppliesView, DeleteBookView, DeleteSuppliesView
+    EditSuppliesView, DeleteBookView, DeleteSuppliesView, ConfirmTeacherView
 
 urlpatterns = [
     path('teacher/<uuid:pk>/add_book', AddBookView.as_view(), name='add_book'),
@@ -11,5 +11,6 @@ urlpatterns = [
     path('teacher/<uuid:teacher_pk>/book/<int:pk>/delete', DeleteBookView.as_view(), name='delete_book'),
     path('teacher/<uuid:teacher_pk>/supplies/<int:pk>', EditSuppliesView.as_view(), name='edit_supplies'),
     path('teacher/<uuid:teacher_pk>/supplies/<int:pk>/delete', DeleteSuppliesView.as_view(), name='delete_supplies'),
+    path('teacher/<uuid:pk>/confirm', ConfirmTeacherView.as_view(), name='confirm_teacher'),
     path('clear', clear_teacher_view, name='clear_teacher'),
 ]
diff --git a/manuels/views.py b/manuels/views.py
index dce0ba3..ff5fd13 100644
--- a/manuels/views.py
+++ b/manuels/views.py
@@ -2,7 +2,7 @@ from django.contrib import messages
 from django.contrib.auth.mixins import PermissionRequiredMixin
 from django.shortcuts import get_object_or_404, redirect
 from django.urls import reverse
-from django.views.generic import CreateView, ListView, UpdateView, DeleteView
+from django.views.generic import CreateView, ListView, UpdateView, DeleteView, FormView
 
 from manuels.forms import AddBookForm, AddSuppliesForm, EditBookForm, EditSuppliesForm
 from manuels.models import Teacher, Book, SuppliesRequirement
@@ -70,6 +70,13 @@ class ItemView(BaseTeacherView):
     verb = None
     button_class = 'primary'
 
+    def dispatch(self, request, *args, **kwargs):
+        response = super().dispatch(request, *args, **kwargs)
+        if self.teacher.has_confirmed_list:
+            messages.error(request, "Vous avez déjà confirmé vos listes. Il n'est plus possible de les modifier.")
+            return redirect('list_books', pk=self.teacher.pk)
+        return response
+
     def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)
         context['item'] = self.item_text
@@ -180,3 +187,15 @@ def clear_teacher_view(request):
     if request.session['teacher_pk']:
         del request.session['teacher_pk']
     return redirect('home_page')
+
+
+class ConfirmTeacherView(BaseTeacherView, UpdateView):
+    model = Teacher
+    fields = []
+    template_name = 'manuels/confirm_teacher.html'
+
+    def form_valid(self, form):
+        response = super().form_valid(form)
+        self.object.has_confirmed_list = True
+        self.object.save()
+        return response
diff --git a/manuels_collection/settings.py b/manuels_collection/settings.py
index 030d3f1..0cc1427 100644
--- a/manuels_collection/settings.py
+++ b/manuels_collection/settings.py
@@ -14,6 +14,7 @@ import os
 
 # Build paths inside the project like this: os.path.join(BASE_DIR, ...)
 import dj_database_url
+from django.contrib.messages import constants as messages
 
 BASE_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
 
@@ -165,3 +166,7 @@ ANYMAIL = {
 }
 
 EMAIL_BACKEND = 'anymail.backends.mailgun.EmailBackend'
+
+MESSAGE_TAGS = {
+    messages.ERROR: 'danger',
+}