From 979e02ec73e0e81dba715a71370ba34a0ad4d122 Mon Sep 17 00:00:00 2001
From: Richard Schreiber <wiffbi@gmail.com>
Date: Wed, 15 Jun 2022 13:25:31 +0200
Subject: [PATCH] Improve free price input auto-checking/selecting items and
 addons

---
 .../event/fragment_addon_choice.html            | 14 ++++++++++++++
 .../event/fragment_product_list.html            |  6 ++++++
 src/pretix/static/pretixpresale/js/ui/main.js   | 17 ++++++++++++++---
 3 files changed, 34 insertions(+), 3 deletions(-)

diff --git a/src/pretix/presale/templates/pretixpresale/event/fragment_addon_choice.html b/src/pretix/presale/templates/pretixpresale/event/fragment_addon_choice.html
index 9ed81b38a..74a901d7c 100644
--- a/src/pretix/presale/templates/pretixpresale/event/fragment_addon_choice.html
+++ b/src/pretix/presale/templates/pretixpresale/event/fragment_addon_choice.html
@@ -131,6 +131,7 @@
                                         <div class="input-group input-group-price">
                                             <span class="input-group-addon">{{ event.currency }}</span>
                                             <input type="number" class="form-control input-item-price"
+                                                    id="price-variation-{{form.pos.pk}}-{{ item.pk }}-{{ var.pk }}"
                                                     placeholder="0"
                                                     min="{% if event.settings.display_net_prices %}{{ var.display_price.net|money_numberfield:event.currency }}{% else %}{{ var.display_price.gross|money_numberfield:event.currency }}{% endif %}"
                                                     name="cp_{{ form.pos.pk }}_variation_{{ item.id }}_{{ var.id }}_price"
@@ -173,6 +174,9 @@
                                             <label class="item-checkbox-label">
                                                 <input type="checkbox" value="1"
                                                         {% if var.initial %}checked="checked"{% endif %}
+                                                        {% if item.free_price %}
+                                                           data-checked-onchange="price-variation-{{form.pos.pk}}-{{ item.pk }}-{{ var.pk }}"
+                                                        {% endif %}
                                                         id="cp_{{ form.pos.pk }}_variation_{{ item.id }}_{{ var.id }}"
                                                         name="cp_{{ form.pos.pk }}_variation_{{ item.id }}_{{ var.id }}"
                                                         data-exclusive-prefix="cp_{{ form.pos.pk }}_variation_{{ item.id }}_"
@@ -181,6 +185,9 @@
                                         {% else %}
                                             <input type="number" class="form-control input-item-count" placeholder="0" min="0"
                                                     {% if var.initial %}value="{{ var.initial }}"{% endif %}
+                                                    {% if item.free_price %}
+                                                       data-checked-onchange="price-variation-{{form.pos.pk}}-{{ item.pk }}-{{ var.pk }}"
+                                                    {% endif %}
                                                     max="{{ c.max_count }}"
                                                     id="cp_{{ form.pos.pk }}_variation_{{ item.id }}_{{ var.id }}"
                                                     name="cp_{{ form.pos.pk }}_variation_{{ item.id }}_{{ var.id }}"
@@ -245,6 +252,7 @@
                                 <div class="input-group input-group-price">
                                     <span class="input-group-addon">{{ event.currency }}</span>
                                     <input type="number" class="form-control input-item-price" placeholder="0"
+                                            id="price-item-{{ form.pos.pk }}-{{ item.pk }}"
                                             min="{% if event.settings.display_net_prices %}{{ item.display_price.net|money_numberfield:event.currency }}{% else %}{{ item.display_price.gross|money_numberfield:event.currency }}{% endif %}"
                                             name="cp_{{ form.pos.pk }}_item_{{ item.id }}_price"
                                             title="{% blocktrans trimmed with item=item.name %}Modify price for {{ item }}{% endblocktrans %}"
@@ -285,6 +293,9 @@
                             {% if c.max_count == 1 or not c.multi_allowed %}
                                 <label class="item-checkbox-label">
                                     <input type="checkbox" value="1"
+                                            {% if item.free_price %}
+                                               data-checked-onchange="price-item-{{ form.pos.pk }}-{{ item.pk }}"
+                                            {% endif %}
                                             {% if c.max_count == 1 and c.min_count == 1 and c.items|length == 1 %}
                                                 checked="checked"
                                                 required="required"
@@ -298,6 +309,9 @@
                                 </label>
                             {% else %}
                                 <input type="number" class="form-control input-item-count" placeholder="0" min="0"
+                                        {% if item.free_price %}
+                                           data-checked-onchange="price-item-{{ form.pos.pk }}-{{ item.pk }}"
+                                        {% endif %}
                                         max="{{ c.max_count }}"
                                         {% if item.initial %}value="{{ item.initial }}"{% endif %}
                                         name="cp_{{ form.pos.pk }}_item_{{ item.id }}"
diff --git a/src/pretix/presale/templates/pretixpresale/event/fragment_product_list.html b/src/pretix/presale/templates/pretixpresale/event/fragment_product_list.html
index f95611f81..fe3dd501d 100644
--- a/src/pretix/presale/templates/pretixpresale/event/fragment_product_list.html
+++ b/src/pretix/presale/templates/pretixpresale/event/fragment_product_list.html
@@ -192,6 +192,9 @@
                                         {% else %}
                                             <input type="number" class="form-control input-item-count" placeholder="0" min="0"
                                                    {% if not ev.presale_is_running %}disabled{% endif %}
+                                                    {% if item.free_price %}
+                                                       data-checked-onchange="price-variation-{{ item.pk }}-{{ var.pk }}"
+                                                    {% endif %}
                                                    max="{{ var.order_max }}"
                                                    id="variation_{{ item.id }}_{{ var.id }}"
                                                    name="variation_{{ item.id }}_{{ var.id }}"
@@ -321,6 +324,9 @@
                                 <input type="number" class="form-control input-item-count" placeholder="0" min="0"
                                        {% if not ev.presale_is_running %}disabled{% endif %}
                                        {% if itemnum == 1 %}value="1"{% endif %}
+                                        {% if item.free_price %}
+                                           data-checked-onchange="price-item-{{ item.pk }}"
+                                        {% endif %}
                                        max="{{ item.order_max }}"
                                        name="item_{{ item.id }}"
                                        id="item_{{ item.id }}"
diff --git a/src/pretix/static/pretixpresale/js/ui/main.js b/src/pretix/static/pretixpresale/js/ui/main.js
index 72204ba62..d30f68f89 100644
--- a/src/pretix/static/pretixpresale/js/ui/main.js
+++ b/src/pretix/static/pretixpresale/js/ui/main.js
@@ -650,11 +650,22 @@ $(function () {
     // Lightbox
     lightbox.init();
 
-    // free-range price input auto-check checkbox
+    // free-range price input auto-check checkbox/set count-input to 1 if 0
     $("[data-checked-onchange]").each(function() {
-        var checkbox = this;
+        var countInput = this;
         $("#" + this.getAttribute("data-checked-onchange")).on("change", function() {
-            checkbox.checked = true;
+            if (countInput.type === "checkbox") {
+                if (countInput.checked) return;
+                countInput.checked = true;
+            }
+            else if (countInput.type === "number" && !countInput.valueAsNumber) {
+                countInput.value = "1";
+            }
+            else {
+                return;
+            }
+            // in case of a change, trigger event
+            $(countInput).trigger("change");
         });
     });
 });