Re: basic trigger using OLD not working?

The RAISE statement requires a % marker for each variable you want to
include in the output text.

--
Guy Rouillier

-----Original Message-----
From: pgsql-general-owner(at)postgresql(dot)org
[mailto:pgsql-general-owner(at)postgresql(dot)org] On Behalf Of Rick Casey
Sent: Thursday, February 24, 2005 5:51 PM
To: pgsql-general(at)postgresql(dot)org
Subject: Re: [GENERAL] basic trigger using OLD not working?


Thanks much! That met with partial success; but getting closer.

The error message about OLD went away (thankfully!), but there
is still no data from the OLD variable. Here is the code again:

create trigger PEDIGREES_hist_del_trig
AFTER DELETE
on PEDIGREES
FOR EACH ROW
EXECUTE PROCEDURE logPedigreesDel();


CREATE OR REPLACE FUNCTION logPedigreesDel() RETURNS TRIGGER AS
'
begin
RAISE EXCEPTION ''OLD.famindid = '', OLD.famindid;
return OLD;
end;
' LANGUAGE plpgsql;

Which when it fires results in:
cgb_lab_data=# \i testphdtrig.sql
psql:testphdtrig.sql:1: ERROR: OLD.famindid =

Regards,
rick



Tom Lane wrote:

Rick Casey <rick(dot)casey(at)colorado(dot)edu>
<mailto:rick(dot)casey(at)colorado(dot)edu> writes:

Here is the code that creates the delete
trigger:
create trigger PEDIGREES_hist_del_trig
AFTER DELETE
on PEDIGREES
EXECUTE PROCEDURE logPedigreesDel();

I think you forgot FOR EACH ROW. By default, the above
creates a
STATEMENT trigger, in which you don't have access to
individual rows.

regards, tom lane