Discussion :

[ninsekh]

Bonjour amis!

je viens de créer le trigger TRG_CONSO à partir de la syntaxe suivante:

Code :

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

1
2
3
4
5
6
7
8
 
CREATE OR REPLACE trg_conso BEFORE INSERT ON dwh_conso_str
REFERENCING NEW AS N 
FOR EACH ROW
BEGIN
      IF (N.ND IN (SELECT ND FROM dwh_conso_str)) THEN
      ABORT TRANSACTION
END

.

Par la suite j'ai voulu l'executer en effectuant une insertion dans la table dwh_conso_str. Voici le script d'insertion:

Code :

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

1
2
 
INSERT INTO dwh_conso_str VALUES ('24494761','12','15','01/08/2008','2','31/07/2008')

Lorsque je lance la requete, l'erreur suivante apparait:
<< ORA-04098 trigger DWHDEV.TRG_CONSO is invalid and failed revalidation>>. je ne comprends pas pourquoi le trigger refuse de s'executer. Pouvez vous m'aidez à executer un trigger sous TOAD,SVP?

PRECISIONS: TOAD V.8,systeme client Win XP, Server unix,Base ORACLE 10g.

[jmguiche]

Un bout de PL sans ";" , c'est louche.
Un bout de PL avec une commande "abort transaction", c'est nouveau et je ne connais pas, mais je soupçonne que cela n'existe pas.

Comme ton PL n'en est probablement pas, ton trigger se crée avec une erreur. Quand on crée une procédure, une fonction, un package ou un trigger, il faut jeter un oeil aux messages d'erreur transmis par oracle.

Comme il se crée avec une erreur, quand tu essaye de l'executer, les programmeurs d'oracle on la grande gentillesse de t'envoyer un message disant :
"votre trigger est invalide et n'a pas pu etre revalidé" (sous entendu, quand vous avez demander d'executer le pl invalide, j'ai essayé de le revalider et cela n'a pas marché).
Evidement c'est en anglais.

Dans la doc, on lit :

ORA-04098: trigger 'string.string' is invalid and failed re-validation
Cause: A trigger was attempted to be retrieved for execution and was found to be invalid. This also means that compilation/authorization failed for the trigger.
Action: Options are to resolve the compilation/authorization errors, disable the trigger, or drop the trigger.

En plus, si j'ai bien compris ce que vous souhaitez faire, il me semble plus judicieux d'envisager la création d'un index unique.

En Oracle, on ne met pas "abort transaction" (sauf nouveauté ), on met rollback.
Placer un rollback dans un trigger me semble bien aventureux, je ne sais même pas si c'est possible, je n'ai jamais essayé de le faire. Par contre, il est possible de lever une exception.

Et dans tout ça, je ne vois pas le rapport avec toad... A moins que toad ne montre pas les erreurs quand il crée des triggers (ce dont je doute) et dans ce cas il faut vite le desinstaler !

[McM]

Toad met bien les erreurs lors d'une compilation. Faut être dans le bon ecran pour compiler les procédures, trigger etc...(Procedure Editor)

Je vois plein de bugs dans ce code :
1/ Pas de ; après le END (facile jmguiche l'a dit)

2/ L'appel à new et old se fait avec :new.colonne ou :old.colonne
(Sauf le cas particulier de la condition when du trigger)

3/ Du IF avec du SELECT ne fonctionne pas

4/ Pas de lecture possible de la table sur laquelle le trigger repose (ORA-04091 MUTATING TABLE)

5/ Jmguiche, je confirme : pas de commit/Rollback dans les triggers.
Il faut utiliser un Raise

Code :

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

1
2
3
4
5
6
7
CREATE OR REPLACE trg_conso BEFORE INSERT ON dwh_conso_str
REFERENCING NEW AS N 
FOR EACH ROW
BEGIN
      IF (N.ND IN (SELECT ND FROM dwh_conso_str)) THEN
      ABORT TRANSACTION
END

En plus, si j'ai bien compris ce que vous souhaitez faire, il me semble plus judicieux d'envisager la création d'un index unique.

Tout à fait. Sauf si les données actuelles ne le permettent pas et qu'on veut bloquer les insertions suivantes.

[djanolivier]

Il suffit de mettre un point virgule à la fin du code. C est tout.

Enjoy toi!!!