added Mongo JSON serializer to correctly serialize ObjectId + datetime.datetime serialization

[erans]

No description provided.

[arikfr]

Why is this needed?

>>> isinstance(datetime.datetime.now(), datetime.date)
True

[erans]

Mongo (and other runners) sometimes return a datetime.datetime object so I thought instead of having it in the mongodb json serializer I'll add it here.

datetime.date != datetime.datetime :-)

[arikfr]

From what I understand datetime.datetime inherits datetime.date, and hence why isinstance(datetime.datetime.now(), datetime.date) == True. You see different behavior?

[erans]

Yet without this line the deserializer yells it cant deserialize
datetime.datetime.

That's why I added it.

On Tue, Jun 16, 2015, 21:40 Arik Fraimovich notifications@github.com
wrote:

In redash/utils.py
#455 (comment):

@@ -95,9 +95,9 @@ def default(self, o):
if isinstance(o, decimal.Decimal):
return float(o)

•    if isinstance(o, datetime.date):
  

•    if isinstance(o, datetime.date) or isinstance(o, datetime.datetime):
  

From what I understand datetime.datetime inherits datetime.date, and
hence why isinstance(datetime.datetime.now(), datetime.date) == True. You
see different behavior?

—
Reply to this email directly or view it on GitHub
https://github.com/EverythingMe/redash/pull/455/files#r32555487.

[erans]

OK. My bad. It accidentally called the original class. I'll remove it.