From: | "A(dot) Kretschmer" <andreas(dot)kretschmer(at)schollglas(dot)com> |
---|---|
To: | pgsql-general(at)postgresql(dot)org |
Subject: | Re: Simple Trigger Error |
Date: | 2006-12-20 06:47:53 |
Message-ID: | 20061220064752.GA7143@a-kretschmer.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
am Wed, dem 20.12.2006, um 9:31:54 +0530 mailte Parthan folgendes:
> Hi,
> I am trying to create a simple trigger which inserts a couple of fields
> into a table, when data in inserted into another table.
>
> I have two tables, 'addressbook' and 'phonebook'. When I insert data
> into the addressbook (whose fields are name, address1, address2,
> address3, phonenum), I want it to add 'name' and 'phonenum' field values
> into 'phonebook' table.
>
> The following is the trigger i wrote for the above..
>
> ---- Begin of Code ---
>
> DROP TRIGGER phonebook on addressbook;
>
> CREATE OR REPLACE FUNCTION add_to_phonebook() RETURNS TRIGGER AS $phonebook$
> DECLARE
> new_name varchar;
> new_phone varchar;
>
> BEGIN
> IF(TG_OP='INSERT') THEN
> INSERT INTO phonebook(name,phonenum)
> VALUES(NEW.name,NEW.phonenum);
> END IF;
> RETURN NEW;
> END;
>
> $phonebook$ LANGUAGE plpgsql;
>
> CREATE TRIGGER phonebook AFTER INSERT ON addressbook FOR EACH ROW
> EXECUTE PROCEDURE add_to_phonebook();
>
> ----- End of code ---
Sorry, i can't reproduce your problem:
test=# create table tele (name text, phone text);
CREATE TABLE
test=*# create table adr (id serial, name text, phone text, city text);
NOTICE: CREATE TABLE will create implicit sequence "adr_id_seq" for serial column "adr.id"
CREATE TABLE
test=*# commit;
COMMIT
test=# create or replace function add_phone() returns trigger as $$ begin IF(TG_OP='INSERT') THEN insert into tele (name, phone) values (new.name,new.phone);end if;return new; end; $$ language plpgsql;
CREATE FUNCTION
test=*# commit;
COMMIT
test=# CREATE TRIGGER addphone after insert on adr for each row execute procedure add_phone();
CREATE TRIGGER
test=*# commit;
COMMIT
test=# insert into adr (name, phone,city) values ('ich', '123', 'Dresden');
INSERT 0 1
test=*# select * from tele;
name | phone
------+-------
ich | 123
(1 row)
This is very similar to your example and it works.
But i have questions/suggestions:
- you have never-used variables in your function. Perhaps you have an
older version from the function with an error and the wrong version
runs?
- perhaps, you have an other trigger on phonebook that calls recursive
the trigger on addressbook?
- you can use a RULE instead a Trigger, an example:
test=# create rule r_adr as on insert to adr do also insert into tele(name) values (new.name);
CREATE RULE
test=*# commit;
COMMIT
test=# insert into adr (name, phone,city) values ('du', '456', 'wilsdruff');
INSERT 0 1
test=*# select * from tele;
name | phone
------+-------
ich | 123
du | 456
du |
(3 rows)
Now i have a TRIGGER and a RULE, and both works ;-)
I think, you should check your tables with \d addressbook and \d
phonebook to ensure that everything is okay.
Andreas
--
Andreas Kretschmer
Kontakt: Heynitz: 035242/47215, D1: 0160/7141639 (mehr: -> Header)
GnuPG-ID: 0x3FFF606C, privat 0x7F4584DA http://wwwkeys.de.pgp.net
From | Date | Subject | |
---|---|---|---|
Next Message | Richard Huxton | 2006-12-20 07:11:58 | Re: Unable to start server - winxp |
Previous Message | Parthan | 2006-12-20 04:01:54 | Simple Trigger Error |