Oui le code aurait du être

sparky/ = 12 Septembre, 2007 - 14:18

Oui le code aurait du être
CREATE OR REPLACE FUNCTION majdate()
RETURNS "trigger" AS
$BODY$
DECLARE

BEGIN
NEW.lastmodified := now();
RETURN NEW;
END;
$BODY$
LANGUAGE 'plpgsql' VOLATILE;

Je vois deux problèmes :

Guillaume Lelarge/ = 12 Septembre, 2007 - 14:21

Je vois deux problèmes :

1. trigger sur instruction... si l'UPDATE met à jour plusieurs lignes, comment le moteur peut-il savoir la ligne que vous souhaitez modifier ? il faut donc passer par un trigger en mode ligne (FOR EACH ROW) ;

2. vous exécutez un UPDATE sur un trigger INSERT/UPDATE --> récursivité assurée :) il faut donc modifier la ligne en cours de modification, d'où changer le trigger AFTER en trigger BEFORE et apporter la modification de la colonne avec l'opérateur d'affectation « := »

Cela devrait donner quelque chose comme cela (attention, non testé) :

CREATE OR REPLACE FUNCTION majdate()
RETURNS "trigger" AS
$BODY$
BEGIN
NEW.lastmodified := now();
RETURN NEW;
END;
$BODY$
LANGUAGE 'plpgsql' VOLATILE;

CREATE TRIGGER triggermajdate
BEFORE INSERT OR UPDATE
ON utilisateur
FOR EACH ROW
EXECUTE PROCEDURE majdate();

--
Guillaume Lelarge.
Administrateur de bases de données
http://dalibo.com - http://dalibo.org

Merci

cioran/ = 12 Septembre, 2007 - 15:44

Super !
Merci beaucoup à tous les deux, ça marche maintenant. En effet, j'oubliais l'affectation de type "Pascal" ... :) Et j'ai une idée plus claire maintenant sur les mots clés NEW, OLD, BEFORE, AFTER.
Encore merci pour votre réactivité.

A+
Claudiu