Merge branch 'feature/166/rendre-telephone-non-obligatoire' into 'develop'

[166] Rendre le telephone non obligatoire

Closes #166

See merge request la-chariotte/la-chariotte!108

la_chariotte/order/forms.py

@@ -18,6 +18,10 @@ class GroupedOrderForm(forms.ModelForm):
         widget=forms.TimeInput(attrs={"type": "time"}),
         initial=datetime.time(hour=23, minute=59, second=59),
     )
+    is_phone_mandatory = forms.BooleanField(
+        label="Numéro de téléphone obligatoire pour les participants",
+        required=False,
+    )
 
     class Meta:
         model = GroupedOrder
@@ -29,6 +33,7 @@ class GroupedOrderForm(forms.ModelForm):
             "delivery_slot",
             "place",
             "description",
+            "is_phone_mandatory",
         ]
         widgets = {
             "name": forms.TextInput(

la_chariotte/order/migrations/0027_groupedorder_phone_mandatory.py

@@ -0,0 +1,20 @@
+# Generated by Django 4.2.11 on 2024-04-14 08:58
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("order", "0026_groupedorder_delivery_slot"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="groupedorder",
+            name="is_phone_mandatory",
+            field=models.BooleanField(
+                default=False, verbose_name="Numéro de téléphone obligatoire"
+            ),
+        ),
+    ]

la_chariotte/order/migrations/0028_alter_order_created_date.py

@@ -0,0 +1,19 @@
+# Generated by Django 4.2 on 2024-10-20 13:02
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("order", "0027_groupedorder_phone_mandatory"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="order",
+            name="created_date",
+            field=models.DateTimeField(
+                auto_now_add=True, verbose_name="Date et heure de commande"
+            ),
+        ),
+    ]

la_chariotte/order/models.py

@@ -27,6 +27,9 @@ class GroupedOrder(models.Model):
     description = models.TextField("Description", null=True, blank=True)
     total_price = models.DecimalField(max_digits=10, decimal_places=2, default=0)
     code = models.CharField(auto_created=True)
+    is_phone_mandatory = models.BooleanField(
+        default=False, verbose_name="Numéro de téléphone obligatoire"
+    )
 
     def create_code_from_pk(self):
         """When a grouped order is created, a unique code is generated, to be used to

la_chariotte/order/templates/order/grouped_order_create.html

@@ -1,5 +1,7 @@
 {% extends 'base.html' %}
 
+{% load crispy_forms_tags %}
+
 {% block title %}Nouvelle commande groupée{% endblock %}
 
 {% block content %}
@@ -10,8 +12,9 @@
         <p class="title">Nouvelle commande groupée</p>
         <div class="columns">
             <div class="column is-8">
-                <form method="post" onsubmit="deadlinePassedCheck(event)">{% csrf_token %}
-                    {{ form.as_p }}
+                <form method="post" onsubmit="deadlinePassedCheck(event)">
+                    {% csrf_token %}
+                    {{ form | crispy }}
                     <div class="buttons">
                         <a class="button is-light" href="{% url 'order:index' %}">Annuler</a>
                         <input class="button is-primary" type="submit" value="Suivant">

la_chariotte/order/templates/order/grouped_order_detail.html

@@ -158,16 +158,18 @@
                                        value="{{ order_author.last_name }}" required></p>
                         </div>
                         <div class="column">
-                            <p><label for="phone">Numéro de téléphone :</label>
-                                <input id="phone" type="tel" pattern="[0-9]{10}" placeholder="0601020304" name="phone"
-                                       value="{{ order_author.phone }}" required></p>
+                            <p><label for="phone">Numéro de téléphone {% if not is_phone_mandatory %}<em>(facultatif)</em> {% endif %}:</label>
+                                <input id="phone" type="tel" pattern="[0-9]{10}" 
+                                placeholder="0601020304" name="phone"
+                                       value="{{ order_author.phone }}"
+                                       {% if is_phone_mandatory %}required{% endif %}></p>
                             <p><label for="email">Adresse mail : </label>
                                 <input id="email" type="email" placeholder="exemple@mail.fr" name="email"
                                        value="{{ order_author.email }}" required></p>
                         </div>
                     </div>
-                    <p><label for="note">Note à l'organisateur·ice</label>
-                        <textarea id="note" rows=3 placeholder="(facultatif)" name="note">{{ note }}</textarea></p>
+                    <p><label for="note">Note à l'organisateur·ice<em> (facultatif)</em> :</label>
+                        <textarea id="note" rows=3 name="note">{{ note }}</textarea></p>
 
                     <div class="buttons">
                         <button id="submit" type="submit" value="Order" class="button is-primary">
@@ -186,7 +188,6 @@
         // Compute total price whenever a value in input is modified
         document.getElementById("inputs-parent").addEventListener("change", function () {
             inputs = [...document.getElementsByTagName("input")].filter(input => input.getAttribute("name").indexOf("quantity_") === 0);  //filter the inputs to get the quantity inputs only
-
             prices = {{ prices_dict | safe }};
             let total_price = 0;
 

la_chariotte/order/templates/order/grouped_order_overview.html

@@ -161,7 +161,7 @@
         <tr>
           <td>{{ order.author }}</td>
           <td>{{ order.price }} €</td>
-          <td><a href="mailto:{{ order.author.email }}">{{ order.author.email }}</a></td>
+          <td><a href="mailto:{{ order.author.email }}">{{ order.author.email }}</a>{% if order.author.phone %} / {{ order.author.phone }}{% endif %}</td>
           <td>
             <button class="button is-info is-small js-modal-trigger" data-target="order-detail-{{ order.id }}">
             Voir

la_chariotte/order/templates/order/grouped_order_update.html

@@ -1,5 +1,7 @@
 {% extends 'base.html' %}
 
+{% load crispy_forms_tags %}
+
 {% block title %}Modifier la commande groupée{% endblock %}
 
 {% block content %}
@@ -9,13 +11,14 @@
 <div class="box">
     <p class="title">{{ grouped_order.name }} - modifier</p>
     <form method="post" onsubmit="deadlinePassedCheck(event)">{% csrf_token %}
-        {{ form.as_p }}
+        {{ form | crispy }}
         <div class="buttons">
             <a class="button is-light" href="{% url 'order:index' %}">Annuler</a>
             <input class="button is-primary" type="submit" value="Suivant">
         </div>
     </form>
 </div>
+
 {% endblock %}
 
 {% block extra_js %}

la_chariotte/order/views/grouped_order.py

@@ -112,7 +112,8 @@ class GroupedOrderDetailView(generic.DetailView):
         return get_object_or_404(GroupedOrder, code=self.kwargs.get("code"))
 
     def get_context_data(self, **kwargs):
-        items = self.get_object().item_set.all()
+        grouped_order = self.get_object()
+        items = grouped_order.item_set.all()
 
         remaining_qty = {item.id: item.get_remaining_nb() for item in items}
         prices_dict = {item.id: item.price for item in items}
@@ -133,6 +134,8 @@ class GroupedOrderDetailView(generic.DetailView):
                 "prices_dict": json.dumps(prices_dict, cls=DjangoJSONEncoder),
                 "remaining_qty": remaining_qty,
                 "order_author": order_author,
+                # Used to set if the phone is required in the form
+                "is_phone_mandatory": grouped_order.is_phone_mandatory,
             }
         )
         return context
@@ -361,7 +364,7 @@ class ExportGroupedOrderToCSVView(GroupedOrderExportView):
         response = http.HttpResponse(
             content_type="text/csv",
             headers={
-                "Content-Disposition": f'attachment; filename="{ context["object"].name }-commandes"'
+                "Content-Disposition": f'attachment; filename="{context["object"].name}-commandes"'
             },
         )
         writer = csv.writer(response)

la_chariotte/tests/test_order_views_grouped_order.py

@@ -662,6 +662,36 @@ class TestGroupedOrderDetailView:
         assert order.price == 4
         assert item.get_remaining_nb() == 16
 
+    def test_phone_not_required_display(self, client, other_user):
+        """a user orders something without entering phone when it is required"""
+        grouped_order = create_grouped_order(
+            days_before_delivery_date=5,
+            days_before_deadline=2,
+            name="gr order test",
+            orga_user=other_user,
+        )
+        assert grouped_order.is_phone_mandatory == True
+        item = models.Item.objects.create(
+            name="test item 1", grouped_order=grouped_order, price=1, max_limit=2
+        )
+        detail_url = reverse(
+            "order:grouped_order_detail",
+            kwargs={
+                "code": grouped_order.code,
+            },
+        )
+        response = client.get(detail_url)
+        assert response.status_code == 200
+        assert "gr order test" in response.content.decode()
+        assert (
+            "Numéro de téléphone <em>(facultatif)</em>" not in response.content.decode()
+        )
+        grouped_order.is_phone_mandatory = False
+        grouped_order.save()
+        response = client.get(detail_url)
+        assert "gr order test" in response.content.decode()
+        assert "Numéro de téléphone <em>(facultatif)</em>" in response.content.decode()
+
 
 class TestGroupedOrderOverview:
     def test_get_overview(self, client_log):

la_chariotte/tests/utils.py

@@ -9,12 +9,20 @@ pytestmark = pytest.mark.django_db
 
 
 def create_grouped_order(
-    days_before_delivery_date, days_before_deadline, name, orga_user
+    days_before_delivery_date,
+    days_before_deadline,
+    name,
+    orga_user,
+    is_phone_mandatory=True,
 ):
     date = timezone.now().date() + datetime.timedelta(days=days_before_delivery_date)
     deadline = timezone.now() + datetime.timedelta(days=days_before_deadline)
     grouped_order = models.GroupedOrder.objects.create(
-        name=name, orga=orga_user, delivery_date=date, deadline=deadline
+        name=name,
+        orga=orga_user,
+        delivery_date=date,
+        deadline=deadline,
+        is_phone_mandatory=is_phone_mandatory,
     )
     grouped_order.create_code_from_pk()
     grouped_order.save()