Discussion :

[claralavraie]

Bonjour,

j'ai créé le trigger suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
CREATE OR REPLACE TRIGGER NOM_TRIG AFTER UPDATE OF STATUS ON TABLE1
FOR EACH ROW
WHEN (NEW.STATUS=11)
BEGIN
  UPDATE TABLE1
  SET NUM = ltrim(substr(lpad(id, 5), -5) ),
      DATE_DELIVERY = SYSDATE
  WHERE ID = :NEW.ID;
END;

Je pense qu'il doit y avoir un problème avec mon trigger mais ou?
J'ai l'erreur suivante:

Code :

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

1
2
3
4
 
ORA-04091: table table1 is mutating, trigger/function may not see it
ORA-06512: at "NOM_TRIG", line 2
ORA-04088: error during execution of trigger 'NOM_TRIG'

Pouvez vous m'aider?

[plaineR]

Il n'est pas possible de faire un update dans un trigger sur la table de ce trigger. Fais une recherche sur table mutante sur le forum, tu trouveras des solutions de contournement

[claralavraie]

il faudrait que je n'interroge pas la table mutante.
Mais cela n'est pas possible à faire.
Donc il faudrait par exemple que j'insère les données pour lesquelles le status est à 11 dans une table temporaire puis ensuite que je mette à jour les données dans ma table avec cette table temporaire.
Est ce exact?

[plaineR]

C'est une solution parmis d'autres :
http://sgbd.developpez.com/oracle/ora-04091/#LE

[claralavraie]

Quelle est la meilleure solution?
Table temporaire, exception table mutante?....
ou toutes sont bonnes?

[orafrance]

et pourquoi pas :

Code :

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

1
2
3
 
  :new.NUM := ltrim(substr(lpad(id, 5), -5) );
  :new.DATE_DELIVERY = SYSDATE;

[plaineR]

Toutes sont bonnes, tout dépend du contexte et de ce que tu veux (peux) faire.

Personnellement, j'évite l'exception, car si oracle me signale une erreur, je ne passe pas outre. J'ai une préférence pour la table temporaire.

[plaineR]

et pourquoi pas :

Code :

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

1
2
3
 
  :new.NUM := ltrim(substr(lpad(id, 5), -5) );
  :new.DATE_DELIVERY = SYSDATE;

Je suppose qu'il y a plus d'une ligne à mettre à jour :

il faudrait que je n'interroge pas la table mutante.
Mais cela n'est pas possible à faire.

Sinon, c'est LA solution, en effet.

[claralavraie]

et pourquoi pas :

Code :

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

1
2
3
 
  :new.NUM := ltrim(substr(lpad(id, 5), -5) );
  :new.DATE_DELIVERY = SYSDATE;

cela ne fonctionne pas

[orafrance]

cela ne fonctionne pas

mais encore ?

Essaye :

Code :

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

1
2
3
SELECT ltrim(substr(lpad(:old.id, 5), -5) ), SYSDATE
INTO :new.NUM,:new.DATE_DELIVERY
FROM DUAL;

[plaineR]

cela ne fonctionne pas

Nous voilà bien avancé

[claralavraie]

Code :

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

1
2
3
SELECT ltrim(substr(lpad(:old.id, 5), -5) ), SYSDATE
INTO :new.NUM,:new.DATE_DELIVERY
FROM DUAL;

J'ai essayé et j'ai l'erreur suivante:

Code :

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

ORA-04084: cannot change NEW values for this trigger type

[orafrance]

1°) ce serait pas mal d'essayer de chercher plutôt qu'appliquer les propositions à l'aveuglette... sinon, tu ne progresseras jamais

2°) est-il possible de faire un BEFORE UPDATE plutôt que AFTER ?

[claralavraie]

1°) ce serait pas mal d'essayer de chercher plutôt qu'appliquer les propositions à l'aveuglette... sinon, tu ne progresseras jamais

Ne t'en fais pas avant de poster cette discussion j'ai essayé et tu ne sais pas ce que je fais de mon coté. Alors abstiens toi de dire pareil chose!Merci!

Et mettre before au leiu de after ne change rien.

[orafrance]

non mais attend, c'était évident après relecture que le code que je proposais était faux... pour le débuggage t'es sensé pouvoir te débrouiller il me semble

Alors maintenant regardons l'erreur : http://ora-04084.ora-code.com/

Je lis : Cause: New trigger variables can only be changed in before row insert or update triggers.

ce qui est bien ton cas... tu ne peux donc pas avoir la même erreur

Merci de faire un minimum d'effort, et si tu travailles par ailleurs, merci de nous dire les essais infructeux que tu as fait avec le message que cela générait.

[claralavraie]

non mais attend, c'était évident après relecture que le code que je proposais était faux... pour le débuggage t'es sensé pouvoir te débrouiller il me semble

Oui je sais me debrouiller comme une grande :-(

Je n'ai jamais dit que j'avais la même erreur.
ça me remet mon problème de table mutante!

[orafrance]

c'est pas possible ???

Tu peux coller le code STP ?

[claralavraie]

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
16
17
 
CREATE OR REPLACE TRIGGER NOM_TRIG BEFORE UPDATE OF STATUS ON TABLE1
FOR EACH ROW
WHEN (NEW.STATUS=11)
  DECLARE
  NUM TABLE1.num_ticket%type; 
  DATE_D DATE;
BEGIN
    SELECT ltrim(substr(lpad(:old.id, 5), -5) ), SYSDATE INTO NUM, DATE_D
  FROM DUAL;	
 
  UPDATE TABLE1
  SET NUM_TICKET = NUM,
      DATE_DELIVERY = DATE_D
  WHERE ORDERS_ID= :NEW.ORDERS_ID;
 
END;

Voici le code.

[orafrance]

l'update n'est pas utile pour mettre à jour les valeurs de la ligne, il faut modifier les valeurs :NEW.<nom_colonne> comme je l'ai montré.

[claralavraie]

Ah en effet cela fonctionne très bien.
je ne savais pas qu'il était possible de faire comme cela.
Cela m'écite ainsi la table temporaire.
Merci beaucoup!