Re: [pgAdmin][Patch] RM 5463 - CSV download quotes numeric columns - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: [pgAdmin][Patch] RM 5463 - CSV download quotes numeric columns
Date
Msg-id CAM9w-_mpZAMqZU99ewg4aeSoakeRSLZW=bOghoFmNLO47QRJ9w@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][Patch] RM 5463 - CSV download quotes numeric columns  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Responses Re: [pgAdmin][Patch] RM 5463 - CSV download quotes numeric columns  (Yogesh Mahajan <yogesh.mahajan@enterprisedb.com>)
List pgadmin-hackers
Hi Yogesh,

The patch looks good to me except that the newly added import can be clubbed with the existing one.

+from psycopg2.extensions import DECIMAL as _DECIMAL

 import psycopg2

 from psycopg2.extensions import encodings


On Thu, Jul 9, 2020 at 1:24 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Aditya

Can you please review it.

On Wed, Jul 8, 2020 at 4:17 PM Yogesh Mahajan <yogesh.mahajan@enterprisedb.com> wrote:
Hi,

Please find a patch which fixes the issue of quoting values of numeric data types like 'bigint, numeric, double precision & real'.

The issue was caused because global type caster are registered for connection in order to avoid javascript compatibility issue for large size data types like real, numeric, double precision, bigint.
Typcaster used to cast these data type values into String, causing quotes to numeric data in downloaded CSV files in case of  "CSV quoting = Strings".

As part of fix, type caster is temporarily unregistered for these numeric data types while creating CSV file & registered back after csv file creation.



Thanks,
Yogesh Mahajan
QA - Team
EnterpriseDB Corporation

Phone: +91-9741705709


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246


--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin][RM5569] [SQL] Bad display of tables storage parameters in the "SQL" tab (correction of bug #5398 not complete)
Next
From: Yogesh Mahajan
Date:
Subject: Re: [pgAdmin][Patch] RM 5463 - CSV download quotes numeric columns