diff options
| author | Tom Christie | 2011-05-10 12:59:13 +0100 | 
|---|---|---|
| committer | Tom Christie | 2011-05-10 12:59:13 +0100 | 
| commit | a2575c1191104df024b41c58825b9a24d4c4ae2d (patch) | |
| tree | 738b0911d77d4344b61b2bfb525ab1ce7a615a4d /djangorestframework | |
| parent | cc4c9a9518bda227df78e511242172773358c95c (diff) | |
| download | django-rest-framework-a2575c1191104df024b41c58825b9a24d4c4ae2d.tar.bz2 | |
Bits of cleanup
Diffstat (limited to 'djangorestframework')
| -rw-r--r-- | djangorestframework/mixins.py | 12 | ||||
| -rw-r--r-- | djangorestframework/resource.py | 3 | 
2 files changed, 10 insertions, 5 deletions
diff --git a/djangorestframework/mixins.py b/djangorestframework/mixins.py index 99f6719b..b80f5894 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 04442498..6b039059 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  | 
