Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fixed #16850 -- Made sure we always represent Decimal instances with …
…JS strings when serializing.

We force this when calling simplejson dump() to isolate us from changes of
default behavior to encode them with JavaScript numbers instead that could
introduce loss of precision when decoding back from JSON to Python.

Thanks Raphael Hertzog for the report and Bob Ippolito for his help.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@17228 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information
ramiro committed Dec 17, 2011
1 parent e31c43a commit b5d9ad9
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions django/core/serializers/json.py
Expand Up @@ -18,6 +18,7 @@ class Serializer(PythonSerializer):
internal_use_only = False

def end_serialization(self):
self.options.update({'use_decimal': False}) # Use JS strings to represent Python Decimal instances (ticket #16850)
simplejson.dump(self.objects, self.stream, cls=DjangoJSONEncoder, **self.options)

def getvalue(self):
Expand Down

0 comments on commit b5d9ad9

Please sign in to comment.