CGM_Public/pretix_cgo
3
0
Fork 0
forked from CGM_Public/pretix_original
Code Pull Requests Actions Activity

Fix N+1 query found by sentry (PRETIXEU-AC2)

Browse Source
This commit is contained in:
Raphael Michel
2024-07-16 11:25:48 +02:00
parent fb7f4d1160
commit 2a0c3da8c4
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/pretix/api/serializers/__init__.py
+3
View File
@@ -21,6 +21,7 @@
# #
import json import json
from django.db.models import prefetch_related_objects
from rest_framework import serializers from rest_framework import serializers
from rest_framework.exceptions import ValidationError from rest_framework.exceptions import ValidationError
@@ -80,6 +81,7 @@ class SalesChannelMigrationMixin:
def to_internal_value(self, data): def to_internal_value(self, data):
if "sales_channels" in data: if "sales_channels" in data:
prefetch_related_objects([self.organizer], "sales_channels")
all_channels = { all_channels = {
s.identifier for s in s.identifier for s in
self.organizer.sales_channels.all() self.organizer.sales_channels.all()
@@ -109,6 +111,7 @@ class SalesChannelMigrationMixin:
def to_representation(self, value): def to_representation(self, value):
value = super().to_representation(value) value = super().to_representation(value)
if value.get("all_sales_channels"): if value.get("all_sales_channels"):
prefetch_related_objects([self.organizer], "sales_channels")
value["sales_channels"] = sorted([ value["sales_channels"] = sorted([
s.identifier for s in s.identifier for s in
self.organizer.sales_channels.all() self.organizer.sales_channels.all()