MetricsMiddleware: Do not record pretix_view_duration_seconds for urls with no url.url_name

src/pretix/base/forms/questions.py

@@ -1025,10 +1025,9 @@ class BaseInvoiceAddressForm(forms.ModelForm):
                 'autocomplete': 'address-level2',
             }),
             'company': forms.TextInput(attrs={
-                'data-display-dependency': '#id_is_business_1',
                 'autocomplete': 'organization',
             }),
-            'vat_id': forms.TextInput(attrs={'data-display-dependency': '#id_is_business_1'}),
+            'vat_id': forms.TextInput(),
             'internal_reference': forms.TextInput,
         }
         labels = {
@@ -1059,10 +1058,8 @@ class BaseInvoiceAddressForm(forms.ModelForm):
 
         super().__init__(*args, **kwargs)
 
-        # If an individual or company address is acceptable, #id_is_business_0 == individual, _1 == company.
-        # However, if only company addresses are acceptable, #id_is_business_0 == company and is the only choice
-        self.fields["company"].widget.attrs["data-display-dependency"] = f'#id_{self.add_prefix("is_business")}_{int(not self.company_required)}'
-        self.fields["vat_id"].widget.attrs["data-display-dependency"] = f'#id_{self.add_prefix("is_business")}_{int(not self.company_required)}'
+        self.fields["company"].widget.attrs["data-display-dependency"] = f'input[name="{self.add_prefix("is_business")}"][value="business"]'
+        self.fields["vat_id"].widget.attrs["data-display-dependency"] = f'input[name="{self.add_prefix("is_business")}"][value="business"]'
 
         if not self.ask_vat_id:
             del self.fields['vat_id']
@@ -1143,9 +1140,9 @@ class BaseInvoiceAddressForm(forms.ModelForm):
         )
         if self.address_required and not self.company_required and not self.all_optional:
             if not event.settings.invoice_name_required:
-                self.fields['name_parts'].widget.attrs['data-required-if'] = f'#id_{self.add_prefix("is_business")}_0'
+                self.fields['name_parts'].widget.attrs['data-required-if'] = f'input[name="{self.add_prefix("is_business")}"][value="individual"]'
             self.fields['name_parts'].widget.attrs['data-no-required-attr'] = '1'
-            self.fields['company'].widget.attrs['data-required-if'] = f'#id_{self.add_prefix("is_business")}_1'
+            self.fields['company'].widget.attrs['data-required-if'] = f'input[name="{self.add_prefix("is_business")}"][value="business"]'
 
         if not event.settings.invoice_address_beneficiary:
             del self.fields['beneficiary']

src/pretix/control/logdisplay.py

@@ -351,7 +351,6 @@ class CheckinErrorLogEntryType(OrderLogEntryType):
                 return str(plain_without_dt).format_map(data)
 
 
-@log_entry_types.new('pretix.event.checkin')
 class CheckinLogEntryType(CheckinErrorLogEntryType):
     def display(self, logentry: LogEntry, data):
         if data.get('type') == Checkin.TYPE_EXIT:
@@ -378,7 +377,6 @@ class CheckinLogEntryType(CheckinErrorLogEntryType):
             )
 
 
-@log_entry_types.new()
 class OrderConsentLogEntryType(OrderLogEntryType):
     action_type = 'pretix.event.order.consent'
 
@@ -388,7 +386,6 @@ class OrderConsentLogEntryType(OrderLogEntryType):
         )
 
 
-@log_entry_types.new()
 class OrderCanceledLogEntryType(OrderLogEntryType):
     action_type = 'pretix.event.order.canceled'
 
@@ -400,7 +397,6 @@ class OrderCanceledLogEntryType(OrderLogEntryType):
             return _('The order has been canceled.')
 
 
-@log_entry_types.new()
 class OrderPrintLogEntryType(OrderLogEntryType):
     action_type = 'pretix.event.order.print'
 

src/pretix/helpers/metrics/middleware.py

@@ -62,7 +62,8 @@ class MetricsMiddleware(object):
         t0 = time.perf_counter()
         resp = self.get_response(request)
         tdiff = time.perf_counter() - t0
-        pretix_view_duration_seconds.observe(tdiff, status_code=resp.status_code, method=request.method,
-                                             url_name=url.namespace + ':' + url.url_name)
+        if url.url_name:
+            pretix_view_duration_seconds.observe(tdiff, status_code=resp.status_code, method=request.method,
+                                                 url_name=url.namespace + ':' + url.url_name)
 
         return resp