forked from CGM_Public/pretix_original
Assert that the item count is a positive integer (#221)
Thanks to David Gullasch for pointing this one out.
This commit is contained in:
committed by
Raphael Michel
parent
5346473f75
commit
4fa631ab97
@@ -56,13 +56,23 @@ class CartActionMixin:
|
||||
parts = parts[:-1]
|
||||
else:
|
||||
voucher = None
|
||||
|
||||
try:
|
||||
amount = int(value)
|
||||
except ValueError:
|
||||
messages.error(self.request, _('Please enter numbers only.'))
|
||||
return []
|
||||
if amount <= 0:
|
||||
messages.error(self.request, _('Please enter positive numbers only.'))
|
||||
return []
|
||||
|
||||
price = self.request.POST.get('price_' + "_".join(parts[1:]), "")
|
||||
if key.startswith('item_'):
|
||||
try:
|
||||
items.append({
|
||||
'item': int(parts[1]),
|
||||
'variation': None,
|
||||
'count': int(value),
|
||||
'count': amount,
|
||||
'price': price,
|
||||
'voucher': voucher
|
||||
})
|
||||
@@ -74,7 +84,7 @@ class CartActionMixin:
|
||||
items.append({
|
||||
'item': int(parts[1]),
|
||||
'variation': int(parts[2]),
|
||||
'count': int(value),
|
||||
'count': amount,
|
||||
'price': price,
|
||||
'voucher': voucher
|
||||
})
|
||||
|
||||
Reference in New Issue
Block a user