Ticket #11970: 11970-with-test2.diff

django/core/serializers/json.py

@@ -8 +8 @@
 
 from django.core.serializers.python import Serializer as PythonSerializer
 from django.core.serializers.python import Deserializer as PythonDeserializer
+from django.core.serializers import base
 from django.utils import datetime_safe
 from django.utils import simplejson
 
@@ -24 +25 @@
         if callable(getattr(self.stream, 'getvalue', None)):
             return self.stream.getvalue()
 
+
 def Deserializer(stream_or_string, **options):
     """
     Deserialize a stream or string of JSON data.
@@ -32 +34 @@
         stream = StringIO(stream_or_string)
     else:
         stream = stream_or_string
-    for obj in PythonDeserializer(simplejson.load(stream), **options):
-        yield obj
+    try:
+        for obj in PythonDeserializer(simplejson.load(stream), **options):
+            yield obj
+    except ValueError, e:
+        # Map to deserializer error
+        raise base.DeserializationError(e.args)
+
 
 class DjangoJSONEncoder(simplejson.JSONEncoder):
     """

tests/regressiontests/serializers_regress/tests.py

@@ -374 +374 @@
         with self.assertRaises(SerializerDoesNotExist):
             serializers.get_deserializer("nonsense")
 
+    def test_json_deserializer_exception(self):
+        from django.core.serializers import base
+        with self.assertRaises(base.DeserializationError):
+            data = serializers.deserialize("json", """[{"pk":1}""")
+            for obj in data:
+                pass
+
 def serializerTest(format, self):
 
     # Create all the objects defined in the test data