diff options
| author | Tom Christie | 2015-01-23 14:28:59 +0000 | 
|---|---|---|
| committer | Tom Christie | 2015-01-23 14:28:59 +0000 | 
| commit | 39f26c9eca6fa8b749f9197ad78e5cba69870e50 (patch) | |
| tree | 86ebd311194a185217e6fefb31d16316bf80779f /rest_framework | |
| parent | 37dc2520f9adbaf54de759a1fdc41985ebd38a0e (diff) | |
| parent | 4201c9fb01beae84fc34a5b74e138e721de42de1 (diff) | |
| download | django-rest-framework-39f26c9eca6fa8b749f9197ad78e5cba69870e50.tar.bz2 | |
Merge master
Diffstat (limited to 'rest_framework')
| -rw-r--r-- | rest_framework/parsers.py | 2 | ||||
| -rw-r--r-- | rest_framework/renderers.py | 3 | ||||
| -rw-r--r-- | rest_framework/response.py | 7 | ||||
| -rw-r--r-- | rest_framework/utils/model_meta.py | 10 | ||||
| -rw-r--r-- | rest_framework/utils/serializer_helpers.py | 10 | 
5 files changed, 27 insertions, 5 deletions
| diff --git a/rest_framework/parsers.py b/rest_framework/parsers.py index dd069840..7a51bb05 100644 --- a/rest_framework/parsers.py +++ b/rest_framework/parsers.py @@ -200,7 +200,7 @@ class FileUploadParser(BaseParser):              if 'filename*' in filename_parm:                  return self.get_encoded_filename(filename_parm)              return force_text(filename_parm['filename']) -        except (AttributeError, KeyError): +        except (AttributeError, KeyError, ValueError):              pass      def get_encoded_filename(self, filename_parm): diff --git a/rest_framework/renderers.py b/rest_framework/renderers.py index f970a363..6256acdd 100644 --- a/rest_framework/renderers.py +++ b/rest_framework/renderers.py @@ -305,6 +305,9 @@ class HTMLFormRenderer(BaseRenderer):      })      def render_field(self, field, parent_style): +        if isinstance(field, serializers.HiddenField): +            return '' +          style = dict(self.default_style[field])          style.update(field.style)          if 'template_pack' not in style: diff --git a/rest_framework/response.py b/rest_framework/response.py index d6ca1aad..7f90bae1 100644 --- a/rest_framework/response.py +++ b/rest_framework/response.py @@ -81,10 +81,13 @@ class Response(SimpleTemplateResponse):      def __getstate__(self):          """ -        Remove attributes from the response that shouldn't be cached +        Remove attributes from the response that shouldn't be cached.          """          state = super(Response, self).__getstate__() -        for key in ('accepted_renderer', 'renderer_context', 'data'): +        for key in ( +            'accepted_renderer', 'renderer_context', 'resolver_match', +            'client', 'request', 'wsgi_request', '_closable_objects' +        ):              if key in state:                  del state[key]          return state diff --git a/rest_framework/utils/model_meta.py b/rest_framework/utils/model_meta.py index b9fb6f67..dd92f8b6 100644 --- a/rest_framework/utils/model_meta.py +++ b/rest_framework/utils/model_meta.py @@ -121,12 +121,17 @@ def _get_reverse_relationships(opts):      """      Returns an `OrderedDict` of field names to `RelationInfo`.      """ +    # Note that we have a hack here to handle internal API differences for +    # this internal API across Django 1.7 -> Django 1.8. +    # See: https://code.djangoproject.com/ticket/24208 +      reverse_relations = OrderedDict()      for relation in opts.get_all_related_objects():          accessor_name = relation.get_accessor_name() +        related = getattr(relation, 'related_model', relation.model)          reverse_relations[accessor_name] = RelationInfo(              model_field=None, -            related_model=relation.model, +            related_model=related,              to_many=relation.field.rel.multiple,              has_through_model=False          ) @@ -134,9 +139,10 @@ def _get_reverse_relationships(opts):      # Deal with reverse many-to-many relationships.      for relation in opts.get_all_related_many_to_many_objects():          accessor_name = relation.get_accessor_name() +        related = getattr(relation, 'related_model', relation.model)          reverse_relations[accessor_name] = RelationInfo(              model_field=None, -            related_model=relation.model, +            related_model=related,              to_many=True,              has_through_model=(                  (getattr(relation.field.rel, 'through', None) is not None) diff --git a/rest_framework/utils/serializer_helpers.py b/rest_framework/utils/serializer_helpers.py index ab057862..87bb3ac0 100644 --- a/rest_framework/utils/serializer_helpers.py +++ b/rest_framework/utils/serializer_helpers.py @@ -19,6 +19,11 @@ class ReturnDict(OrderedDict):      def __repr__(self):          return dict.__repr__(self) +    def __reduce__(self): +        # Pickling these objects will drop the .serializer backlink, +        # but preserve the raw data. +        return (dict, (dict(self),)) +  class ReturnList(list):      """ @@ -33,6 +38,11 @@ class ReturnList(list):      def __repr__(self):          return list.__repr__(self) +    def __reduce__(self): +        # Pickling these objects will drop the .serializer backlink, +        # but preserve the raw data. +        return (list, (list(self),)) +  class BoundField(object):      """ | 
