Commit 4140eac1 authored by Tom van Bussel's avatar Tom van Bussel
Browse files

Merge branch 'feature/pizzas-order-create-admins' into 'master'

Fix creating orders for other users using the pizza API

See merge request thalia/concrexit!605
parents 9771a7c0 f1e48bba
from django.utils.translation import ugettext_lazy as _
from rest_framework import serializers
from rest_framework.exceptions import ValidationError
from pizzas.models import Product, PizzaEvent, Order
......@@ -29,3 +31,13 @@ class AdminOrderSerializer(serializers.ModelSerializer):
class Meta:
model = Order
fields = ('pk', 'paid', 'product', 'name', 'member')
def validate(self, attrs):
if attrs['member'] and attrs['name']:
raise ValidationError({
'member': _('Either specify a member or a name'),
'name': _('Either specify a member or a name'),
if not (attrs['member'] or attrs['name']):
attrs['member'] = self.context['request'].member
return super().validate(attrs)
......@@ -72,7 +72,10 @@ class OrderViewset(ModelViewSet):
if serializer.validated_data.get('name'):
if self.request.user.has_perm('pizzas.change_order'):
except IntegrityError:
raise ValidationError('Something went wrong when saving the order')
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment