Order change: Fix list of unchangeable add-ons not filtered to category (Z#23223330)

* Fix mail headers being None

* update tests

src/pretix/base/validators.py

@@ -95,6 +95,7 @@ class OrganizerSlugBanlistValidator(BanlistValidator):
         'csp_report',
         'widget',
         'lead',
+        'scheduling',
     ]
 
 

src/pretix/presale/views/order.py

@@ -1510,7 +1510,10 @@ class OrderChangeMixin:
                             'max_count': iao.max_count,
                             'iao': iao,
                             'items': [i for i in items if not i.require_voucher],
-                            'items_missing': {k: v for k, v in current_addon_products_missing.items() if v},
+                            'items_missing': {
+                                k: v for k, v in current_addon_products_missing.items()
+                                if v and k[0].category_id == iao.addon_category_id
+                            },
                         })
 
         return positions

src/pretix/static/pretixcontrol/js/ui/editor.js

@@ -674,11 +674,10 @@ var editor = {
             $("#toolbox").find("button[data-action=middle]").toggleClass('active', o.verticalAlign === 'middle');
             $("#toolbox").find("button[data-action=bottom]").toggleClass('active', o.verticalAlign === 'bottom');
 
-            console.log("_update_toolbox_values", o.scaleY, o.scaleX)
             if (o.scaleY !== 1 || o.scaleX !== 1) {
                 o.set({
-                    height: Math.max(o.height * o.scaleY, editor._mm2px(10.01)),
-                    width: Math.max(o.width * o.scaleX, editor._mm2px(10.01)),
+                    height: o.height * o.scaleY,
+                    width: o.width * o.scaleX,
                     scaleX: 1,
                     scaleY: 1
                 });

src/tests/base/test_mail.py

@@ -170,7 +170,6 @@ def test_queue_state_sent(env):
         subject='Test',
         body_plain='Test',
         sender='sender@example.com',
-        headers={},
     )
     assert m.status == OutgoingMail.STATUS_QUEUED
     mail_send_task.apply(kwargs={
@@ -188,7 +187,6 @@ def test_queue_state_permanent_failure(env):
         subject='Test',
         body_plain='Test',
         sender='sender@example.com',
-        headers={},
     )
     assert m.status == OutgoingMail.STATUS_QUEUED
     mail_send_task.apply(kwargs={
@@ -210,7 +208,6 @@ def test_queue_state_retry_failure(env, monkeypatch):
         subject='Test',
         body_plain='Test',
         sender='sender@example.com',
-        headers={},
     )
     assert m.status == OutgoingMail.STATUS_QUEUED
     mail_send_task.apply(kwargs={
@@ -236,7 +233,6 @@ def test_queue_state_foreign_key_handling():
         subject='Test',
         body_plain='Test',
         sender='sender@example.com',
-        headers={},
     )
     mail_sent = OutgoingMail.objects.create(
         organizer=o,
@@ -245,7 +241,6 @@ def test_queue_state_foreign_key_handling():
         subject='Test',
         body_plain='Test',
         sender='sender@example.com',
-        headers={},
         status=OutgoingMail.STATUS_SENT,
     )