aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTom Christie2013-08-27 11:22:19 +0100
committerTom Christie2013-08-27 11:22:19 +0100
commit8d590ebfded0968e458f8e3a87efabec8384586e (patch)
tree4e5f591042ce90873fb21948960639d0d5ad5f31
parentc3e273a90e08cc5215f9e9ea0508b62809b181a4 (diff)
downloaddjango-rest-framework-8d590ebfded0968e458f8e3a87efabec8384586e.tar.bz2
First hacky pass at displaying raw data
-rw-r--r--rest_framework/parsers.py9
-rw-r--r--rest_framework/renderers.py27
-rw-r--r--rest_framework/serializers.py2
-rw-r--r--rest_framework/tests/test_serializer.py2
4 files changed, 35 insertions, 5 deletions
diff --git a/rest_framework/parsers.py b/rest_framework/parsers.py
index 96bfac84..c635505a 100644
--- a/rest_framework/parsers.py
+++ b/rest_framework/parsers.py
@@ -10,9 +10,9 @@ from django.core.files.uploadhandler import StopFutureHandlers
from django.http import QueryDict
from django.http.multipartparser import MultiPartParser as DjangoMultiPartParser
from django.http.multipartparser import MultiPartParserError, parse_header, ChunkIter
-from rest_framework.compat import yaml, etree
+from rest_framework.compat import etree, six, yaml
from rest_framework.exceptions import ParseError
-from rest_framework.compat import six
+from rest_framework.renderers import UnicodeJSONRenderer
import json
import datetime
import decimal
@@ -32,6 +32,8 @@ class BaseParser(object):
media_type = None
+ supports_html_forms = False
+
def parse(self, stream, media_type=None, parser_context=None):
"""
Given a stream to read from, return the parsed representation.
@@ -47,6 +49,7 @@ class JSONParser(BaseParser):
"""
media_type = 'application/json'
+ renderer_class = UnicodeJSONRenderer
def parse(self, stream, media_type=None, parser_context=None):
"""
@@ -91,6 +94,7 @@ class FormParser(BaseParser):
"""
media_type = 'application/x-www-form-urlencoded'
+ supports_html_forms = True
def parse(self, stream, media_type=None, parser_context=None):
"""
@@ -109,6 +113,7 @@ class MultiPartParser(BaseParser):
"""
media_type = 'multipart/form-data'
+ supports_html_forms = True
def parse(self, stream, media_type=None, parser_context=None):
"""
diff --git a/rest_framework/renderers.py b/rest_framework/renderers.py
index b30f2ea9..cc8de959 100644
--- a/rest_framework/renderers.py
+++ b/rest_framework/renderers.py
@@ -24,7 +24,7 @@ from rest_framework.settings import api_settings
from rest_framework.request import clone_request
from rest_framework.utils import encoders
from rest_framework.utils.breadcrumbs import get_breadcrumbs
-from rest_framework import exceptions, parsers, status, VERSION
+from rest_framework import exceptions, status, VERSION
class BaseRenderer(object):
@@ -482,7 +482,7 @@ class BrowsableAPIRenderer(BaseRenderer):
if method in ('DELETE', 'OPTIONS'):
return True # Don't actually need to return a form
- if not getattr(view, 'get_serializer', None) or not parsers.FormParser in view.parser_classes:
+ if not getattr(view, 'get_serializer', None) or not any(parser.supports_html_forms for parser in view.parser_classes):
return
serializer = view.get_serializer(instance=obj)
@@ -561,6 +561,29 @@ class BrowsableAPIRenderer(BaseRenderer):
view = renderer_context['view']
request = renderer_context['request']
response = renderer_context['response']
+
+ obj = getattr(view, 'object', None)
+ if getattr(view, 'get_serializer', None):
+ serializer = view.get_serializer(instance=obj)
+ else:
+ serializer = None
+
+ parsers = []
+ for parser_class in view.parser_classes:
+ content = None
+ renderer_class = getattr(parser_class, 'renderer_class', None)
+ if renderer_class and serializer:
+ renderer = renderer_class()
+ context = renderer_context.copy()
+ context['indent'] = 4
+ content = renderer.render(serializer.data, accepted_media_type, context)
+ print content
+ parsers.append({
+ 'media_type': parser_class.media_type,
+ 'content': content
+ })
+
+
media_types = [parser.media_type for parser in view.parser_classes]
renderer = self.get_default_renderer(view)
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py
index abff6898..202d3a09 100644
--- a/rest_framework/serializers.py
+++ b/rest_framework/serializers.py
@@ -304,6 +304,8 @@ class BaseSerializer(WritableField):
ret.empty = obj is None
for field_name, field in self.fields.items():
+ if obj is None and field.read_only:
+ continue
field.initialize(parent=self, field_name=field_name)
key = self.get_field_key(field_name)
value = field.field_to_native(obj, field_name)
diff --git a/rest_framework/tests/test_serializer.py b/rest_framework/tests/test_serializer.py
index c2497660..7c2a276e 100644
--- a/rest_framework/tests/test_serializer.py
+++ b/rest_framework/tests/test_serializer.py
@@ -158,7 +158,7 @@ class BasicTests(TestCase):
'email': '',
'content': '',
'created': None,
- 'sub_comment': ''
+ #'sub_comment': ''
}
self.assertEqual(serializer.data, expected)