Prevent requiring a membership without selecting any types

This commit is contained in:
Raphael Michel
2023-01-06 23:16:46 +01:00
parent b3be64b9f3
commit ff09ed422c

View File

@@ -595,6 +595,13 @@ class ItemUpdateForm(I18nModelForm):
"Gift card products should not be admission products at the same time."
)
)
if d.get('require_membership') and not d.get('require_membership_types'):
self.add_error(
'require_membership_types',
_(
"If a valid membership is required, at least one valid membership type needs to be selected."
)
)
return d
def clean_picture(self):
@@ -792,6 +799,17 @@ class ItemVariationForm(I18nModelForm):
}),
}
def clean(self):
d = super().clean()
if d.get('require_membership') and not d.get('require_membership_types'):
self.add_error(
'require_membership_types',
_(
"If a valid membership is required, at least one valid membership type needs to be selected."
)
)
return d
def save(self, commit=True):
instance = super().save(commit)
self.meta_fields = []