Re: PATCH: Added Node Type & Catalog objects [pgAdmin4]

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PATCH: Added Node Type & Catalog objects [pgAdmin4]
Date: 2016-03-18 07:15:47
Message-ID: CAKKotZSrb=YrxbaasHQF9DXA-roFmzmA7cVC3tVPMdhYwL58cw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Dave,

Even after installing PosGIS I’m not able to create new custom external
type because the SQL does not return any value (Same behaviour as is in
pgAdmin3) , But I’m able to list all of it’s properties of external types
same as pgAdmin3 (PFA screenshots).

I have also attached patch for plain fields control (without lable &
border) for backform. (Please apply it before testing new type patch)

Thanks,

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On Thu, Mar 17, 2016 at 4:13 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> On Thu, Mar 17, 2016 at 10:08 AM, Murtuza Zabuawala
> <murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
> >
> > - When editing a type, I cannot select a Grantee on the Privileges tab.
> >
> > This is general issue with privilege control.
> > [We will add it in TODO list]
>
> OK.
>
> > - When adding a composite type, I cannot type/select directly in the
> > grid, I have to expand the row.
> >
> > Because currently in Backgrid we do not have functionality like if I edit
> > one cell second cell re-renders according the value of first cell,
> > If we expand and select the type the other two fields (length &
> > precision)are dependent on the value of type for composite types they
> become
> > enable/disable accordingly.
>
> That's must-have functionality. Please add a TODO for it.
>
> > - If I click ADD to add a new member to a composite type whilst the
> > previous row is expanded, the new row is added, but the expanded panel
> > remains linked to the first row.
> >
> > - If I then click on the Edit button on the new row, I see two
> > expanded panels, neither of which is attached to their row (and
> > they're in the wrong order). I would only expect to see one panel at a
> > time.
> >
> > This is general with sub-node collection control.
> > [We will add it in TODO list]
>
> OK.
>
> > - When creating an External type, the Input, Output, Send, Receive and
> > Analyze function lists are all blank. Is this expected? I would have
> > thought there would be system functions listed as there are for Typmod
> > in/out.
> >
> > Dave, I was not able to test External type functionality completely
> because
> > it requires external type which is not inbuilt in postgres.
> > Once you create that custom type using C/C++ code, it will be listed in
> > those combobox by sql we are using to fetch external function types
> added by
> > user in postgres.
>
> I wonder if installing PostGIS would allow that testing to be done.
>
> > - Why does "Range" type have a frame around it?
> >
> > So that we can make visible/hide all the control/elements related to
> Range
> > type as whole instead having separate code for each control/element.
> > That frame is because we used ‘FieldsetControl’ of backform to group all
> the
> > elements of Range type.
>
> Can you make it a plain div with no border/title? That would fit in
> better with the existing styling, assuming the columns line up
> properly.
>
> Thanks.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Attachment Content-Type Size
Screen Shot 2016-03-18 at 11.44.35 am.png image/png 287.1 KB
types_node_v4.patch application/octet-stream 106.9 KB
plain_fieldset_control_v1.patch application/octet-stream 2.3 KB
image/png 66.3 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2016-03-18 07:26:41 Re: PATCH: Added Node Type & Catalog objects [pgAdmin4]
Previous Message Harshal Dhumal 2016-03-18 06:32:22 Re: Control for selecting multiple columns [pgadmin4]