Discussion :

[scary]

Bonjour,

Voila mon problème est le suivant, je veux créer un trigger qui me permette d'empêcher la mise à jour ou la suppression du compte root de mon site.

Voici donc celle que j'ai fait:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
CREATE OR REPLACE FUNCTION trig_root_member() RETURNS TRIGGER AS
$trig_root_member$
	BEGIN
		IF OLD.pseudo = 'root' THEN
                    RETURN NULL;
		ELSE
		    RETURN OLD;
                END IF;
	END;
$trig_root_member$ LANGUAGE plpgsql;
 
DROP TRIGGER IF EXISTS trig_root_member ON membre;
CREATE TRIGGER trig_root_member BEFORE DELETE OR UPDATE ON membre
FOR EACH STATEMENT
EXECUTE PROCEDURE trig_root_member();

Or lorsque j'essaie de modifier ou supprimer mon compte root postgresql me met cette erreur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
ERREUR: l'enregistrement « old » n'est pas encore affectée
État SQL :55000
Détail :La structure de ligne d'un enregistrement pas encore affecté est indéterminée.
Contexte : fonction PL/pgsql « trig_root_member », ligne 2 à IF

Et je ne comprend pas cette erreur Quelqu'un aurait une idée à me proposer ? Merci d'avance

[ok.Idriss]

Salut.

Personnellement, j'utiliserai un RAISE EXCEPTION pour empêcher la mise à jour ou la suppression...

Voici comment j'aurai fait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
CREATE OR REPLACE FUNCTION trig_root_member() RETURNS TRIGGER AS $$
	BEGIN
		IF OLD.pseudo = 'root' THEN
                    RAISE EXCEPTION 'message d\'erreur ...'; -- stop les traitements
                END IF;
                RETURN NEW;
	END; 
$$ LANGUAGE plpgsql;
 
CREATE TRIGGER trig_root_member BEFORE INSERT
	ON auteur FOR EACH ROW
	EXECUTE PROCEDURE trig_root_member();

Cordialement,
Idriss

[estofilo]

Il faudrait un trigger FOR EACH STATEMENT au lieu de FOR EACH ROW dans la définition du trigger, sans quoi NEW et OLD ne sont pas accessibles.
C'est la raison du message d'erreur.

EDIT: lire Il faudrait un trigger FOR EACH ROW au lieu de FOR EACH STATEMENT.

[ok.Idriss]

Il faudrait un trigger FOR EACH STATEMENT au lieu de FOR EACH ROW dans la définition du trigger, sans quoi NEW et OLD ne sont pas accessibles.
C'est la raison du message d'erreur.

Ou peut être le contraire, non ? Il utilise FOR EACH STATMENT, il a une erreur. J'utilise FOR EACH ROW et j'ai bien accès aux objets NEW et OLD, non ?

Cordialement,
Idriss

[estofilo]

Ou peut être le contraire, non ? Il utilise FOR EACH STATMENT, il a une erreur. J'utilise FOR EACH ROW et j'ai bien accès aux objets NEW et OLD, non ?

Euh oui désolé j'ai écrit l'inverse de ce que je voulais.
C'est bien FOR EACH ROW qu'il faut utiliser.

[scary]

Merci c'était exactement ce qu'il me fallait