Skip to content
GitLab
Menu
Projects
Groups
Snippets
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
Menu
Open sidebar
thalia
concrexit
Commits
9035c25c
Commit
9035c25c
authored
Dec 06, 2017
by
Wietse Kuipers
Browse files
Merge branch 'fix/pizzas-api-orders-dict-get' into 'master'
Use get for dict access in order serializer See merge request
!629
parents
505813cb
95e40fc5
Changes
1
Hide whitespace changes
Inline
Side-by-side
website/pizzas/api/serializers.py
View file @
9035c25c
...
...
@@ -33,11 +33,11 @@ class AdminOrderSerializer(serializers.ModelSerializer):
fields
=
(
'pk'
,
'paid'
,
'product'
,
'name'
,
'member'
)
def
validate
(
self
,
attrs
):
if
attrs
[
'member'
]
and
attrs
[
'name'
]
:
if
attrs
.
get
(
'member'
)
and
attrs
.
get
(
'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'
]
):
if
not
(
attrs
.
get
(
'member'
)
or
attrs
.
get
(
'name'
)
):
attrs
[
'member'
]
=
self
.
context
[
'request'
].
member
return
super
().
validate
(
attrs
)
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment