Re: Patch to enable/disable rule on a table - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Patch to enable/disable rule on a table
Date
Msg-id 47470965.8080307@postgresql.org
Whole thread Raw
In response to Patch to enable/disable rule on a table  (Guillaume Lelarge <guillaume@lelarge.info>)
Responses Re: Patch to enable/disable rule on a table
List pgadmin-hackers
Guillaume Lelarge wrote:
> Hi all,
>
> Here is a patch to support this new PostgreSQL 8.3 feature : the
> possibility to enable and disable a rule on a table.

Thanks, applied with minor changes:

- Correct the naming of pgRule::iSetEnabled to pgRule::SetEnabled, and
added iSetEnabled with the correct functionality. In a nutshell, the
functions prefixed with 'i' are used to setup the internal state of the
object (usually called by CreateObjects in the appropriate factory), but
not change it, whilst those without the 'i' actually change the object
and it's internal state. This was also broken in the Trigger
enable/disable code which I guess you probably copied - fixed there too.

- Added code to prevent the option being offered for rules under views
as the syntax isn't there to allow rules on views (or rules that make
views) to be enabled or disabled.

- Re-jigged the menus a touch as they were starting to look a little
messy with various new options being added.

Cheers, Dave.

pgadmin-hackers by date:

Previous
From: svn@pgadmin.org
Date:
Subject: SVN Commit by dpage: r6827 - in trunk/pgadmin3: . pgadmin/frm pgadmin/include/schema pgadmin/schema
Next
From: "Joshua D. Drake"
Date:
Subject: Re: pgadmin3 1.8.0 for ubuntu/dapper and