From e5abba89bd3d46bfe2cf65f3412cd012d202ba1a Mon Sep 17 00:00:00 2001
From: Thomas Girod <thgirod@hotmail.com>
Date: Tue, 10 Jan 2023 16:20:06 +0100
Subject: [PATCH] Les tests fonctionnent

---
 counter/models.py                             |  8 ++--
 counter/static/counter/js/counter_click.js    |  9 ++++
 counter/templates/counter/counter_click.jinja |  7 ++-
 counter/tests.py                              | 28 ++++++------
 counter/views.py                              | 45 +++++++++++--------
 5 files changed, 59 insertions(+), 38 deletions(-)

diff --git a/counter/models.py b/counter/models.py
index 564d6a3b..af83d492 100644
--- a/counter/models.py
+++ b/counter/models.py
@@ -451,11 +451,11 @@ class Counter(models.Model):
         Show if the counter authorize the refilling with physic money
         """
 
-        if (
-            self.id in SITH_COUNTER_OFFICES
-        ):  # If the counter is the counters 'AE' or 'BdF', the refiling are authorized
+        if self.type != "BAR":
+            return False
+        if self.id in SITH_COUNTER_OFFICES:
+            # If the counter is either 'AE' or 'BdF', refills are authorized
             return True
-
         is_ae_member = False
         ae = Club.objects.get(unix_name=SITH_MAIN_CLUB["unix_name"])
         for barman in self.get_barmen_list():
diff --git a/counter/static/counter/js/counter_click.js b/counter/static/counter/js/counter_click.js
index e7e05a83..46f22e93 100644
--- a/counter/static/counter/js/counter_click.js
+++ b/counter/static/counter/js/counter_click.js
@@ -12,6 +12,15 @@ document.addEventListener('alpine:init', () => {
             return total / 100;
         },
 
+        async handle_code(event) {
+            const code = $(event.target).find("#code_field").val().toUpperCase();
+            if(["FIN", "ANN"].includes(code)) {
+                $(event.target).submit();
+            } else {
+                await this.handle_action(event);
+            }
+        },
+
         async handle_action(event) {
             const payload = $(event.target).serialize();
             let request = new Request(click_api_url, {
diff --git a/counter/templates/counter/counter_click.jinja b/counter/templates/counter/counter_click.jinja
index 8abe2b10..1a44c7f3 100644
--- a/counter/templates/counter/counter_click.jinja
+++ b/counter/templates/counter/counter_click.jinja
@@ -56,14 +56,17 @@
             <h5>{% trans %}Selling{% endtrans %}</h5>
             <div>
                 {% set counter_click_url = url('counter:click', counter_id=counter.id, user_id=customer.user.id) %}
+
+                {# Formulaire pour rechercher un produit en tapant son code dans une barre de recherche #}
                 <form method="post" action=""
-                      class="code_form" @submit.prevent="handle_action">
+                      class="code_form" @submit.prevent="handle_code">
                     {% csrf_token %}
                     <input type="hidden" name="action" value="code">
                     <label for="code_field"></label>
                     <input type="text" name="code" value="" class="focus" id="code_field"/>
                     <input type="submit" value="{% trans %}Go{% endtrans %}"/>
                 </form>
+
                 <template x-for="error in errors">
                     <div class="alert alert-red" x-text="error">
                     </div>
@@ -71,7 +74,7 @@
                 <p>{% trans %}Basket: {% endtrans %}</p>
 
                 <ul>
-                    <template x-for="[id, item] in Object.entries(basket)">
+                    <template x-for="[id, item] in Object.entries(basket)" :key="id">
                         <div>
                             <form method="post" action="" class="inline del_product_form"
                                   @submit.prevent="handle_action">
diff --git a/counter/tests.py b/counter/tests.py
index 846d93e9..8e0c7750 100644
--- a/counter/tests.py
+++ b/counter/tests.py
@@ -62,12 +62,12 @@ class CounterTest(TestCase):
             reverse("counter:details", kwargs={"counter_id": self.mde.id}),
             {"code": "4000k", "counter_token": counter_token},
         )
-        location = response.get("location")
+        counter_url = response.get("location")
         response = self.client.get(response.get("location"))
         self.assertTrue(">Richard Batsbak</" in str(response.content))
 
         self.client.post(
-            location,
+            counter_url,
             {
                 "action": "refill",
                 "amount": "5",
@@ -75,14 +75,14 @@ class CounterTest(TestCase):
                 "bank": "OTHER",
             },
         )
-        self.client.post(location, {"action": "code", "code": "BARB"})
-        self.client.post(location, {"action": "add_product", "product_id": "4"})
-        self.client.post(location, {"action": "del_product", "product_id": "4"})
-        self.client.post(location, {"action": "code", "code": "2xdeco"})
-        self.client.post(location, {"action": "code", "code": "1xbarb"})
-        response = self.client.post(location, {"action": "finish"})
+        self.client.post(counter_url, "action=code&code=BARB", content_type="text/xml")
+        self.client.post(counter_url, "action=add_product&product_id=4", content_type="text/xml")
+        self.client.post(counter_url, "action=del_product&product_id=4", content_type="text/xml")
+        self.client.post(counter_url, "action=code&code=2xdeco", content_type="text/xml")
+        self.client.post(counter_url, "action=code&code=1xbarb", content_type="text/xml")
+        response = self.client.post(counter_url, "action=code&code=fin", content_type="text/xml")
 
-        response_get = self.client.get(location)
+        response_get = self.client.get(response.get("location"))
         response_content = response_get.content.decode("utf-8")
         self.assertTrue("2 x Barbar" in str(response_content))
         self.assertTrue("2 x Déconsigne Eco-cup" in str(response_content))
@@ -97,7 +97,7 @@ class CounterTest(TestCase):
         )
 
         response = self.client.post(
-            location,
+            counter_url,
             {
                 "action": "refill",
                 "amount": "5",
@@ -107,7 +107,7 @@ class CounterTest(TestCase):
         )
         self.assertTrue(response.status_code == 200)
 
-        response = self.client.post(
+        self.client.post(
             reverse("counter:login", kwargs={"counter_id": self.foyer.id}),
             {"username": self.krophil.username, "password": "plop"},
         )
@@ -124,10 +124,10 @@ class CounterTest(TestCase):
             reverse("counter:details", kwargs={"counter_id": self.foyer.id}),
             {"code": "4000k", "counter_token": counter_token},
         )
-        location = response.get("location")
+        counter_url = response.get("location")
 
         response = self.client.post(
-            location,
+            counter_url,
             {
                 "action": "refill",
                 "amount": "5",
@@ -143,7 +143,7 @@ class CounterStatsTest(TestCase):
         call_command("populate")
         self.counter = Counter.objects.filter(id=2).first()
 
-    def test_unothorized_user_fail(self):
+    def test_unauthorised_user_fail(self):
         # Test with not login user
         response = self.client.get(reverse("counter:stats", args=[self.counter.id]))
         self.assertTrue(response.status_code == 403)
diff --git a/counter/views.py b/counter/views.py
index bafa7197..2416ce78 100644
--- a/counter/views.py
+++ b/counter/views.py
@@ -302,10 +302,7 @@ class CounterClick(CounterTabsMixin, CanViewMixin, DetailView):
     current_tab = "counter"
 
     def render_to_response(self, *args, **kwargs):
-        if len(self.request.POST) == 0 and len(self.request.body) != 0:
-            # when using the fetch API, the django request.POST dict is empty
-            # this is but a wretched contrivance which must be replaced as soon as possible
-            # by a proper separation between the api endpoints of the counter
+        if self.is_ajax(self.request):
             response = {"errors": []}
             status = HTTPStatus.OK
 
@@ -404,8 +401,9 @@ class CounterClick(CounterTabsMixin, CanViewMixin, DetailView):
             self.operator = self.customer.user
         else:
             self.operator = self.object.get_random_barman()
-        action = parse_qs(request.body.decode()).get("action", [""])[0]
-
+        action = self.request.POST.get("action", None)
+        if action is None:
+            action = parse_qs(request.body.decode()).get("action", [""])[0]
         if action == "add_product":
             self.add_product(request)
         elif action == "add_student_card":
@@ -477,6 +475,16 @@ class CounterClick(CounterTabsMixin, CanViewMixin, DetailView):
             self.compute_record_product(request, product)
         )
 
+    @staticmethod
+    def is_ajax(request):
+        # when using the fetch API, the django request.POST dict is empty
+        # this is but a wretched contrivance which strive to replace
+        # the deprecated django is_ajax() method
+        # and which must be replaced as soon as possible
+        # by a proper separation between the api endpoints of the counter
+        return len(request.POST) == 0 and len(request.body) != 0
+
+
     def add_product(self, request, q=1, p=None):
         """
         Add a product to the basket
@@ -578,8 +586,6 @@ class CounterClick(CounterTabsMixin, CanViewMixin, DetailView):
                 request.session["basket"][pid]["qty"] -= 1
             if request.session["basket"][pid]["qty"] <= 0:
                 del request.session["basket"][pid]
-        else:
-            request.session["basket"][pid] = None
         request.session.modified = True
 
     def parse_code(self, request):
@@ -590,6 +596,10 @@ class CounterClick(CounterTabsMixin, CanViewMixin, DetailView):
             - <int>X<str>, where the integer is the quantity and str the code
         """
         string = parse_qs(request.body.decode())["code"][0].upper()
+        if string == "FIN":
+            return self.finish(request)
+        elif string == "ANN":
+            return self.cancel(request)
         regex = re.compile(r"^((?P<nb>[0-9]+)X)?(?P<code>[A-Z0-9]+)$")
         m = regex.match(string)
         if m is not None:
@@ -664,17 +674,16 @@ class CounterClick(CounterTabsMixin, CanViewMixin, DetailView):
 
     def refill(self, request):
         """Refill the customer's account"""
-        if self.get_object().type == "BAR" and self.object.can_refill():
-            form = RefillForm(request.POST)
-            if form.is_valid():
-                form.instance.counter = self.object
-                form.instance.operator = self.operator
-                form.instance.customer = self.customer
-                form.instance.save()
-            else:
-                self.refill_form = form
-        else:
+        if not self.object.can_refill():
             raise PermissionDenied
+        form = RefillForm(request.POST)
+        if form.is_valid():
+            form.instance.counter = self.object
+            form.instance.operator = self.operator
+            form.instance.customer = self.customer
+            form.instance.save()
+        else:
+            self.refill_form = form
 
     def get_context_data(self, **kwargs):
         """Add customer to the context"""