Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Relationship fail to deserialize missing value #130

Closed
kumy opened this issue Jun 20, 2018 · 1 comment
Closed

Relationship fail to deserialize missing value #130

kumy opened this issue Jun 20, 2018 · 1 comment

Comments

@kumy
Copy link
Contributor

kumy commented Jun 20, 2018

While upgrading our dependency from 0.16.0 to 0.20.0, we got a new stack trace

[…]
  File "/home/kumy/GIT/geokrety-api/venv/local/lib/python2.7/site-packages/marshmallow/schema.py", line 660, in _do_load
    index_errors=self.opts.index_errors,
  File "/home/kumy/GIT/geokrety-api/venv/local/lib/python2.7/site-packages/marshmallow/marshalling.py", line 295, in deserialize
    index=(index if index_errors else None)
  File "/home/kumy/GIT/geokrety-api/venv/local/lib/python2.7/site-packages/marshmallow/marshalling.py", line 68, in call_and_store
    value = getter_func(data)
  File "/home/kumy/GIT/geokrety-api/venv/local/lib/python2.7/site-packages/marshmallow/marshalling.py", line 288, in <lambda>
    data
  File "/home/kumy/GIT/geokrety-api/venv/local/lib/python2.7/site-packages/marshmallow_jsonapi/fields.py", line 208, in deserialize
    if 'links' in value:
TypeError: argument of type '_Missing' is not iterable

Posted data are:

{'data': {'relationships': {}, 'attributes': {'comment': 'My news comment'}, 'type': 'news-comment'}}
kumy added a commit to kumy/marshmallow-jsonapi that referenced this issue Jun 20, 2018
@kumy
Copy link
Contributor Author

kumy commented Jun 20, 2018

Please see PR #131

@sloria sloria closed this as completed in d9e284e Jul 15, 2018
sloria added a commit that referenced this issue Jul 15, 2018
Fix #130: Relationship fail to deserialize missing value
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant