Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit 73130e3

Browse files
committed
Fixed case where a missing meta_field value would error
1 parent 3b93077 commit 73130e3

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

‎rest_framework_json_api/renderers.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -325,7 +325,7 @@ def extract_meta(serializer, resource):
325325
meta = getattr(serializer.child, 'Meta', None)
326326
else:
327327
meta = getattr(serializer, 'Meta', None)
328-
meta_fields = getattr(meta, 'meta_fields', {})
328+
meta_fields = getattr(meta, 'meta_fields', [])
329329
data = OrderedDict()
330330
for field_name in meta_fields:
331331
data.update({

‎rest_framework_json_api/serializers.py‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -148,10 +148,10 @@ def get_field_names(self, declared_fields, info):
148148
We override the parent to omit explicity defined meta fields (such
149149
as SerializerMethodFields) from the list of declared fields
150150
"""
151-
meta_fields = getattr(self.Meta, 'meta_fields', None)
151+
meta_fields = getattr(self.Meta, 'meta_fields', [])
152152

153153
declared = OrderedDict()
154-
for field_name in declared_fields.keys():
154+
for field_name in set(declared_fields.keys()):
155155
field = declared_fields[field_name]
156156
if field_name not in meta_fields:
157157
declared[field_name] = field

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /