Re: Control for displaying "auto vacuum" fields into grid

From: Surinder Kumar <surinder(dot)kumar(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Control for displaying "auto vacuum" fields into grid
Date: 2016-03-23 16:04:31
Message-ID: CAM5-9D8=DMS75mhNCpPP3jn6=_JkT=drNkUr9gPRXBs5xLLDBg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

On Fri, Mar 18, 2016 at 8:58 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> On Fri, Mar 18, 2016 at 10:37 AM, Surinder Kumar
> <surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
> > Hi,
> >
> > PFA control for displaying auto vacuum fields into grid. This control is
> > common for
> > Materialized View Node and Table Node.
> >
> > Usage:
> >
> > {
> > id: 'vacuum_table', label: '{{ _("Vacuum Table") }}',
> > model: VacuumTableModel, editable: false, type: 'collection',
> > canEdit: true, group: '{{ _("Table") }}',
> > mode: ['edit', 'create'], url: 'get_vacuum_defaults',
> > control: Backform.VacuumCollectionControl.extend({
> > grid_columns :[
> > {
> > name: 'label', label: '{{ _("Label") }}',
> > cell: 'string', editable: false
> > },
> > {
> > name: 'value', label: '{{ _("Value") }}',
> > cellFunction: cellFunction, editable: function(m) {
> > if(m.handler.has('autovacuum_enabled')) {
> > return m.handler.get('autovacuum_enabled');
> > }
> > return !m.handler.isNew();
> > }
> > },
> > {
> > name: 'setting', label: '{{ _("Default value") }}',
> > cellFunction: cellFunction, editable: false
> > }
> > ]
> > }),
> >
> >
> > When using this control, provide following parameters in schema:
> > 1. model
> > 2. url - to fetch default values for auto vacuum fields.
> > 3. grid columns - Name of the columns to display in the grid.
> >
> >
> > Please review the patch.
>
> It's hard to review this without being able to test. Do you have a
> simple test case?
>

I have submitted patch for materialized view which is using this control.
so you can test it in materialized view.

>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Ashesh Vashi 2016-03-24 07:30:42 pgAdmin 4 commit: The changed value of a multiline/text control was not
Previous Message Dave Page 2016-03-23 16:03:25 pgAdmin 4 commit: String improvements.