Discussion :

[BTB36]

Bonjour
J'essaie de créer un trigger qui envoie un email lorsque le compteur atteint la valeur souhaiter
Ci joint le code

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
create or replace TRIGGER email
   AFTER UPDATE
   ON T_METER
   FOR EACH ROW
 
   DECLARE
   CURSOR NS
   IS
      SELECT EREQ_SERIAL_NUMBER, MDMT_DESCRIPTION
        FROM T_METER , T_EQUIPMENT
       WHERE PK_METER = MDMT_CUMULATIVE_UNITS
       AND ereq_code = MDMT_CODE;
 
   objet       VARCHAR2 (100 BYTE);
   MESSAGE     VARCHAR2 (500 BYTE);
   recipient   vest_tools.tablevarchar100;
BEGIN
 
   IF MDMT_CUMULATIVE_UNITS >= MDMT_NUMBER3
   THEN
      objet := 'Fin de vie de l'' Adapter'|| MDMT_CODE;
 
      FOR vie IN NS
      LOOP
         MESSAGE :=
               'L'' Adapter '
            || MDMT_DESCRIPTION
            || ' avec le numero de série '
            || EREQ_SERIAL_NUMBER
            || ' à attend son cycle de fin de vie ';
         MESSAGE :=
               MESSAGE
            || CHR (10)
            || ' ---- NE PAS REPONDRE ---'
            || CHR (10)
            || '---- Mail  envoyé automatiquement par Coswin  ----';

lors de l’exécution du tregger j'obtiens le message suivants:

Erreur(22,4): PL/SQL: Statement ignored
Erreur(22,7): PLS-00201: l'identificateur 'MDMT_CUMULATIVE_UNITS' doit être déclaré

[mnitu]

Le message semble assez clair: il connait pas que-est ce MDMT_CUMULATIVE_UNITS.

[BTB36]

Comment peux-je le déclarer ?

sachons que le champ MDMT_CUMULATIVE_UNITS est utilisable dans la table T_METER

[mnitu]

Il y a un tutoriel PL/SQL sur ce site.

[Ikebukuro]

Tu déclares un curseur mais tu ne l'utilises pas... où est le OPEN, le FETCH, le CLOSE?

Il faudrait dans ton SELECT mettre les colonnes MDMT_CUMULATIVE_UNITS et MDMT_NUMBER3 et les stocker dans deux variables.

En outre je te suggère fortement de préfixer tes variables par v_ pour une meilleure lisibilité, et, même mieux, v_var_ pour VARCHAR2.

Je crois aussi que le code
IF MDMT_CUMULATIVE_UNITS >= MDMT_NUMBER3
devrait être remplacé par
IF v_MDMT_CUMULATIVE_UNITS >= v_MDMT_NUMBER3

[mnitu]

...
En outre je te suggère fortement de préfixer tes variables par v_ pour une meilleure lisibilité, et, même mieux, v_var_ pour VARCHAR2.
...

C'est loin d'être un bon conseil.