Fixed PPAS trigger when condition - Mailing list pgadmin-hackers

From Sanket Mehta
Subject Fixed PPAS trigger when condition
Date
Msg-id CA+yw=mPrafcx7xdC-08Pjf5e6SoP4AxX76UJpnEs84ubndduOw@mail.gmail.com
Whole thread Raw
Responses Re: Fixed PPAS trigger when condition
List pgadmin-hackers
Hi Dave 

There is one issue with PPAS while trigger creation as mentioned below:

create table a (a int not null primary key, b int);
create table b (a int not null primary key, b int);
--make trigger with when condition
create trigger tr_a after insert on a for each row
when (:new.a > 0)
begin
insert into b values (:new.*);
end;

After creating the trigger using above query, When condition is not visible in property window of that trigger. As per current implementation, it fetches the when part of trigger using below logic:

substring(pg_get_triggerdef(t.oid), 'WHEN (.*) EXECUTE PROCEDURE') 

this will not work with PPAS as we have not provided any procedure to be executed so it will return null. To fix this issue I have changed the above logic as below:

COALESCE(substring(pg_get_triggerdef(t.oid), 'WHEN (.*) EXECUTE PROCEDURE'), substring(pg_get_triggerdef(t.oid), 'WHEN (.*)  \\$trigger')) 

Above logic has worked with PPAS and PG both. Attached is the patch file.

Please review it and if it looks good to you then please do commit the code


Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb
Attachment

pgadmin-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: Re: restore postgres 8.4, help me
Next
From: Dave Page
Date:
Subject: pgAdmin III commit: Fix support for triggers with inline code on PPAS