Re: [pgadmin-hackers] PATCH: Fix the issue in JSON decoding (pgAdmin4)

2016-06-03 Thread Ashesh Vashi
At the moment - I have committed the code, but - I think, we should rely on some external utility to make it work better with date, timestamp, etc. Please take a look at this mail-chain. https://www.postgresql.org/message-id/87mvnauugi.fsf%40metapensiero.it -- Thanks & Regards, Ashesh Vashi

Re: [pgadmin-hackers] PATCH: Fix the issue in JSON decoding (pgAdmin4)

2016-06-02 Thread Murtuza Zabuawala
Hi, I was not sure about all the internal representation of psycopg date/time format against python’s datetime for JSON decoding format, so I used logical OR. Thanks, Murtuza > On 02-Jun-2016, at 3:10 pm, Ashesh Vashi > wrote: > > On Thu, Jun 2, 2016 at 12:04

Re: [pgadmin-hackers] PATCH: Fix the issue in JSON decoding (pgAdmin4)

2016-06-02 Thread Ashesh Vashi
On Thu, Jun 2, 2016 at 12:04 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi, > > PFA patch to fix the issue in json decoding while dealing with date > format, causing QueryTool to fail to render data. > RM#1280 > Shouldn't the check have 'and' instead of 'or' bitwise

[pgadmin-hackers] PATCH: Fix the issue in JSON decoding (pgAdmin4)

2016-06-02 Thread Murtuza Zabuawala
Hi, PFA patch to fix the issue in json decoding while dealing with date format, causing QueryTool to fail to render data. RM#1280 -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company RM_1280.patch Description: Binary data -- Sent via