Re: PATCH: Support smallserial columns in Edit Data window - Mailing list pgadmin-hackers

From J.F. Oster
Subject Re: PATCH: Support smallserial columns in Edit Data window
Date
Msg-id 46937069.20141028194612@mail.ru
Whole thread Raw
In response to Re: PATCH: Support smallserial columns in Edit Data window  (Sanket Mehta <sanket.mehta@enterprisedb.com>)
Responses Re: PATCH: Support smallserial columns in Edit Data window
List pgadmin-hackers
Hello Sanket,

Steps to reveal issues:

1. Prepare test tables:
        CREATE TABLE tserial2 (
          a smallserial NOT NULL,
          dat text,
          CONSTRAINT tserial2_pkey PRIMARY KEY (a)
        );

        -- Create these for reference (to compare):
        CREATE TABLE tserial(
          a serial NOT NULL,
          dat text,
          CONSTRAINT tserial_pkey PRIMARY KEY (a)
        );
        CREATE TABLE tserial8(
          a bigserial NOT NULL,
          dat text,
          CONSTRAINT tserial8_pkey PRIMARY KEY (a)
        );

2. Open Edit data window for tserial2. Bug#1: the header for column "a" shows
datatype "smallint" instead of "smallserial".

3. Insert one row into tserial2.

4. Select whole new row by pressing on row header and copy it to
clipboard (Ctrl-C or with context menu on row header).

5. Move to last row (*) and Paste into it (Ctrl-V or context menu).
Bug#2: A confirmation dialog should appear ("This table contains
serial ..."), but it doesn't.


Tuesday, October 28, 2014, 1:27:29 PM, you wrote:


SM> Can you please explain in which case it will create issue if bug is not applied?


SM> On Mon, Oct 27, 2014 at 4:14 PM, Dave Page <dpage@pgadmin.org> wrote:
SM> Ashesh, can you review/commit this please?

SM> On Fri, Oct 24, 2014 at 8:12 PM, J.F. Oster <jinfroster@mail.ru> wrote:
>> Hello!
>>
>> This patch adds a few missing lines of code to handle smallserial
>> columns the same way as serial and bigserial.
>>

--
Best regards,
 J.F.



pgadmin-hackers by date:

Previous
From: Sanket Mehta
Date:
Subject: Re: PATCH: Clear undo/redo history when a file is opened in Query editor
Next
From: Ashesh Vashi
Date:
Subject: Re: pgAdmin III commit: Beautify the query shown in the SQL pane for the fu