Discussion :

[arwen]

Bonjour,

je souhaite créer un déclencheur et envoyer un mail d'avertissement aux utilisateurs enregistrés.
La création du déclencheur ne pose pas de problèmes.
J'ai une table qui contient tous les utilisateurs avec leur adresse mail.
Dans le déclencheur, il y a une requête qui sélectionne les utlisateurs suivant un critère de région. La sélection s'effectue sans difficultés.

Par contre, je souhaite faire une concaténation de tous les adresses mails pour éviter d'envoyer plusieurs le même message. C'est dans la concaténation que ça plante et je vois pas comment faire.

Voici 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
CREATE TRIGGER nvperso
ON PERSONNE
FOR UPDATE
AS
	Declare @Nom nvarchar(50), @Prenom nvarchar(50), @msg nvarchar(50), @EmailT varchar(50), @EmailReg varchar(300)
   Select @Nom = NomPersonne from inserted
   Select @Prenom = PrenomPersonne from inserted
   Set @msg = @Nom +' ' +@Prenom + ' a été modifié'
 
   DECLARE Email_cursor CURSOR FOR
   Select Email from Utilisateurs inner join Communes on Utilisateurs.Region = Communes.Region
	inner join inserted on Communes.IdCommune = inserted.IdCommuPersonne
   OPEN Email_cursor
   FETCH NEXT FROM Email_cursor INTO @EmailT
   WHILE @@FETCH_STATUS = 0
        BEGIN
	set @EmailReg = @EmailReg + @EmailT + ';'  --cette ligne ne marche pas
   	FETCH NEXT FROM Email_cursor INTO @EmailT
       END
   CLOSE Email_cursor
   DEALLOCATE Email_cursor
 
   EXEC master..xp_sendmail @recipients = 'toto@toto.fr',
      @message = @EmailT,
      @subject = @msg

Dans le message, j'ai rien du tout. En testant dans l'analyseur de requêtes, c'est au niveau du set @EmailReg = ... que ça marche pas. Toutes les autres instructions sont executées.

Est ce quelqu'un a une idée ?

Merci !

[soazig]

Bonjour,
Est-ce que tu n'aurais pas plus de 6 destinataires (emails)?
Car pour 6 emails cela te fait au maximum, 6*(50+1)=306 caractères ce qui dépasse la capacité de ta variable @EmailReg qui est de 300.
le +1 vient du ';'

Par ailleurs, je crois que ton trigger ne marcherait pas si tu mettais à jour plusieurs enregistrements de la table Personne en même temps.
A+
Soazig

[arwen]

Merci !

Le trigger ne marche pas s'il y a plusieurs mises à jour dans la base de données en même temps ? A quoi c'est dû ?

Comment je pourrais faire alors pour envoyer un message d'alerte à certaines personnes lors de mises à jour dans la base de données ?

[soazig]

Bonjour,

Je n'ai jamais créé de trigger, mais si j'ai compris la théorie lorsque tu mets à jour plusieurs enregistrements, tu obtiens plusieurs enregistrements dans la table inserted. comme ton

Code :

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

Select @Nom = NomPersonne from inserted

présuppose qu'il n'y a qu'un enregistrement dans inserted, tu auras peut-être un problème.

Est-ce que ton problème venait ou non de la taille de la variable @EmailReg ?

A+
Soazig

[arwen]

Ouf !

en fait, le problème c'est que j'avais des champs nuls de temps en temps dans la variable @EmailT (Les champs ne sont pas toujours remplis).
Et j'ai découvert que le null est absorbant !! donc voilà pourquoi @EmailReg était vide.

En faisant un test sur @EmailT pour vérifier s'il y a une valeur, ça fonctionne !

Merci pour les suggestions !

[mr_qno]

Ouf !

en fait, le problème c'est que j'avais des champs nuls de temps en temps dans la variable @EmailT (Les champs ne sont pas toujours remplis).
Et j'ai découvert que le null est absorbant !! donc voilà pourquoi @EmailReg était vide.

En faisant un test sur @EmailT pour vérifier s'il y a une valeur, ça fonctionne !

Merci pour les suggestions !

Mais ça je te l'avais signalé dans un autre post de ta part il me semble, ça t'aurait permis de gagner du temps.