Re: Next patch for 8.5 support: WHEN clause in CREATE TRIGGER

Le mercredi 25 novembre 2009 à 09:48:31, Dave Page a écrit :
> On Tue, Nov 24, 2009 at 6:48 PM, Guillaume Lelarge
>
> <guillaume(at)lelarge(dot)info> wrote:
> > Hi,
> >
> > Here is a patch to add the support for this new 8.5 functionality. Ticket
> > is http://code.pgadmin.org/trac/ticket/99, commitfest item is
> > https://commitfest.postgresql.org/action/patch_view?id=198.
>
> Browsing the patch, I don't see anything too nasty :-)
>

Héhé.

> > I don't really like the way I get the old WHEN clause, but I don't find
> > another way.
>
> I assume it's stored as a parsetree in the catalogs? It's not ideal to
> extract it using a regexp, but it's not the end of the world.
>

AFAIK, there's no other way right now.

> > Another thing that bothers me is the text field: it's one line
> > only. Perhaps a bit small?
>
> Maybe - is there enough slack in the comment textbox to steal a line
> or two, and add a vertical scrollbar?
>

Yeah. I didn't remember at that time how to do it with the new dialog layout.
But I found and the new patch fixes this.

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com