Making JSONParser python 3 compatible

Author: Hamed Ahmadi

rest_framework_json_api/parsers.py

@@ -1,13 +1,16 @@
 """
 Parsers
 """
+import json
+
+from django.conf import settings
 from rest_framework import parsers
 from rest_framework.exceptions import ParseError
 
 from . import utils, renderers, exceptions
 
 
-class JSONParser(parsers.JSONParser):
+class JSONParser(parsers.BaseParser):
     """
     A JSON API client will send a payload that looks like this:
 
@@ -58,7 +61,9 @@ def parse(self, stream, media_type=None, parser_context=None):
         """
         Parses the incoming bytestream as JSON and returns the resulting data
         """
-        result = super(JSONParser, self).parse(stream, media_type=media_type, parser_context=parser_context)
+
+        result = self._convert_context_to_json(stream, media_type, parser_context)
+
         data = result.get('data')
 
         if data:
@@ -100,3 +105,18 @@ def parse(self, stream, media_type=None, parser_context=None):
 
         else:
             raise ParseError('Received document does not contain primary data')
+
+    def _convert_context_to_json(self, stream, media_type=None, parser_context=None):
+        parser_context = parser_context or {}
+
+        try:
+            string = stream.read()
+            if hasattr(string, 'decode'):
+                encoding = parser_context.get('encoding', settings.DEFAULT_CHARSET)
+                data = string.decode(encoding)
+            else:
+                data = string
+
+            return json.loads(data)
+        except ValueError as exc:
+            raise ParseError('JSON parse error - %s' % six.text_type(exc))