From 21b4dacee9efc4747ae04a2b39543d54f5dc632b Mon Sep 17 00:00:00 2001
From: Mira Weller <weller@rami.io>
Date: Mon, 25 Nov 2024 13:37:54 +0100
Subject: [PATCH] Correctly apply "required" attribute to address state field

(we try to use it from the country field, which is a <select> tag with default value and therefore never has a "required" attribute - we need to take the information from the .required class of the container)
---
 src/pretix/static/pretixbase/js/addressform.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/pretix/static/pretixbase/js/addressform.js b/src/pretix/static/pretixbase/js/addressform.js
index 5468ed9c0..2e6ea0fde 100644
--- a/src/pretix/static/pretixbase/js/addressform.js
+++ b/src/pretix/static/pretixbase/js/addressform.js
@@ -5,6 +5,7 @@ $(function () {
         var dependent = $(this),
             counter = 0,
             dependency = $(this).closest(".panel-body, form").find('select[name$=country]'),
+            depRequired = dependency.closest(".form-group").is(".required"),
             update = function (ev) {
                 counter++;
                 var curCounter = counter;
@@ -24,8 +25,8 @@ $(function () {
                             }
                             dependent.append(o);
                         });
-                        dependent.closest(".form-group").show();
-                        dependent.prop('required', dependency.prop("required"));
+                        dependent.closest(".form-group").show().toggleClass('required', depRequired);
+                        dependent.prop('required', depRequired);
                     } else {
                         dependent.closest(".form-group").hide();
                         dependent.prop("required", false);
@@ -37,7 +38,8 @@ $(function () {
         if (dependent.find("option").length === 1) {
             dependent.closest(".form-group").hide();
         } else {
-            dependent.prop('required', dependency.prop("required"));
+            dependent.closest(".form-group").toggleClass('required', depRequired);
+            dependent.prop('required', depRequired);
         }
         dependency.on("change", update);
     });