Voucher import: Fix validation quirks

This commit is contained in:
Raphael Michel
2024-05-03 16:06:40 +02:00
parent 806124304a
commit 9da65f60d7
2 changed files with 6 additions and 5 deletions

View File

@@ -75,7 +75,7 @@ class MaxUsagesColumn(IntegerColumnMixin, ImportColumn):
] ]
def clean(self, value, previous_values): def clean(self, value, previous_values):
if value is None: if value is None and previous_values.get("code"):
raise ValidationError(_('The maximum number of usages must be set.')) raise ValidationError(_('The maximum number of usages must be set.'))
return super().clean(value, previous_values) return super().clean(value, previous_values)

View File

@@ -370,10 +370,11 @@ class Voucher(LoggedModel):
'redeemed': redeemed 'redeemed': redeemed
} }
) )
if data.get('max_usages', 1) < data.get('min_usages', 1): if data.get('min_usages') is not None:
raise ValidationError( if data.get('max_usages', 1) < data.get('min_usages', 1):
_('The maximum number of usages may not be lower than the minimum number of usages.'), raise ValidationError(
) _('The maximum number of usages may not be lower than the minimum number of usages.'),
)
@staticmethod @staticmethod
def clean_subevent(data, event): def clean_subevent(data, event):