diff --git a/djangorestframework/mixins.py b/djangorestframework/mixins.py index 99f6719b3..b80f5894c 100644 --- a/djangorestframework/mixins.py +++ b/djangorestframework/mixins.py @@ -150,7 +150,9 @@ class RequestMixin(object): return self._content - + # TODO: Modify this so that it happens implictly, rather than being called explicitly + # ie accessing any of .DATA, .FILES, .content_type, .stream or .method will force + # form overloading. def perform_form_overloading(self): """ Check the request to see if it is using form POST '_method'/'_content'/'_content_type' overrides. @@ -199,6 +201,7 @@ class RequestMixin(object): content_type}) + # TODO: Acutally this needs to go into Resource def validate(self, content): """ Validate, cleanup, and type-ify the request content. @@ -209,6 +212,7 @@ class RequestMixin(object): return content + # TODO: Acutally this needs to go into Resource def get_bound_form(self, content=None): """ Return a bound form instance for the given content, @@ -229,7 +233,7 @@ class RequestMixin(object): @property def default_parser(self): - """Return the view's most preferred renderer. + """Return the view's most preferred parser. (This has no behavioral effect, but is may be used by documenting renderers)""" return self.parsers[0] @@ -256,8 +260,8 @@ class ResponseMixin(object): renderers = () - # TODO: wrap this behavior around dispatch(), ensuring it works well with - # existing Django classes that use render_to_response. + # TODO: wrap this behavior around dispatch(), ensuring it works + # out of the box with existing Django classes that use render_to_response. def render(self, response): """ Takes a ``Response`` object and returns an ``HttpResponse``. diff --git a/djangorestframework/resource.py b/djangorestframework/resource.py index 044424987..6b039059f 100644 --- a/djangorestframework/resource.py +++ b/djangorestframework/resource.py @@ -6,6 +6,7 @@ import decimal import inspect import re +# TODO: Replace this with new Serializer code based on Forms API. class Resource(object): """A Resource determines how an object maps to a serializable entity. @@ -61,7 +62,7 @@ class Resource(object): if inspect.ismethod(f) and len(inspect.getargspec(f)[0]) == 1: ret = _any(f()) else: - ret = unicode(thing) # TRC TODO: Change this back! + ret = unicode(thing) # TRC return ret