Re: [pgAdmin4][Patch]: RM #3066 - Create script doesn't add quotation marks to column names for the "add index" functions

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: RM #3066 - Create script doesn't add quotation marks to column names for the "add index" functions
Date: 2018-02-20 16:47:51
Message-ID: CA+OCxoyE_Xn=+vSCBuyctLOEKyf1if8LeBPPJ5uW0ffzyuOobQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks, applied.

On Mon, Feb 19, 2018 at 1:27 PM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

>
>
> On Mon, Feb 19, 2018 at 5:19 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Mon, Feb 19, 2018 at 7:56 AM, Khushboo Vashi <
>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> Please find the attached patch to fix RM #3066 - Create script doesn't
>>> add quotation marks to column names for the "add index" functions
>>>
>>
>> This causes additional quoting on regular tables. Please see the
>> generated SQL for the index below:
>>
>> Please find the attached updated patch.
>
>
>> -- Table: public."CamelCase"
>>
>> -- DROP TABLE public."CamelCase";
>>
>> CREATE TABLE public."CamelCase"
>> (
>> "ID" bigint NOT NULL DEFAULT nextval('"CamelCase_ID_seq"'::regclass),
>> "SomeColumn" text COLLATE pg_catalog."default",
>> b boolean,
>> CONSTRAINT "CamelCase_pkey" PRIMARY KEY ("ID")
>> )
>> WITH (
>> OIDS = FALSE
>> )
>> TABLESPACE pg_default;
>>
>> ALTER TABLE public."CamelCase"
>> OWNER to postgres;
>>
>> -- Index: MixedCase_IDX
>>
>> -- DROP INDEX public."MixedCase_IDX";
>>
>> CREATE INDEX "MixedCase_IDX"
>> ON public."CamelCase" USING btree
>> ("""ID""")
>> TABLESPACE pg_default;
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-02-20 16:50:06 pgAdmin 4 commit: Fix alignment issues in keyboard shortcut options. Fi
Previous Message Dave Page 2018-02-20 16:47:38 pgAdmin 4 commit: Ensure column names on indexes on views are properly