Discussion :

[Lafite84]

Bonjour à tous,

Je suis un problème de syntaxe dans mon trigger. 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
26
27
28
29
30
31
BEGIN
 
declare @PINCODE as varchar(10);
declare @PCVISVALIDE as varchar(10);
declare @PCVNUM as varchar(50);
declare @shell as varchar(8000);
declare ins_cur cursor FAST_FORWARD for (select PCVNUM from inserted);
 
declare @pcvnumcde as varchar(50);
set @pcvnumcde = (SELECT PCVNUM from inserted);
 
 
declare @numdevis as varchar(50);
set @numdevis = (SELECT TOP (1) Entete_Devis.PCVNUM AS Numdevis FROM PIECEVENTELIGNES AS Ligne_Devis INNER JOIN
                         PIECEVENTES AS Entete_Devis ON Ligne_Devis.PCVID = Entete_Devis.PCVID INNER JOIN
                         PIECEVENTELIGNES AS Ligne_Cde ON Ligne_Devis.PLVID = Ligne_Cde.PLVIDORG INNER JOIN
                         PIECEVENTES AS Entete_CDE ON Ligne_Cde.PCVID = Entete_CDE.PCVID
						 where Entete_CDE.PCVNUM=''+@pcvnumcde+'');
 
  IF UPDATE (PCVNUM)
  BEGIN  
SELECT @PINCODE=I.PINCODE, @PCVNUM=I.PCVNUM FROM INSERTED I;
 
IF @PINCODE = 'CDECLI' 
 
 
                  set @shell = 'echo ' + @numdevis + ';1''>> "H:\Site Web\Out\orders_in\"accepted' + @numdevis + '.csv';
				  exec master..xp_cmdshell @shell;
 
                  				  end
								  end;

Si j'exécute ce trigger, je n'obtiens pas de fichier dans mon dossier. Par contre, si je mets le code suivant, j'obtiens un fichier mais avec comme nom ma requête.

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
BEGIN
 
declare @PINCODE as varchar(10);
declare @PCVISVALIDE as varchar(10);
declare @PCVNUM as varchar(50);
declare @shell as varchar(8000);
declare ins_cur cursor FAST_FORWARD for (select PCVNUM from inserted);
 
declare @pcvnumcde as varchar(50);
set @pcvnumcde = (SELECT PCVNUM from inserted);
 
 
declare @numdevis as varchar(50);
set @numdevis ='(SELECT TOP (1) Entete_Devis.PCVNUM AS Numdevis FROM PIECEVENTELIGNES AS Ligne_Devis INNER JOIN
                         PIECEVENTES AS Entete_Devis ON Ligne_Devis.PCVID = Entete_Devis.PCVID INNER JOIN
                         PIECEVENTELIGNES AS Ligne_Cde ON Ligne_Devis.PLVID = Ligne_Cde.PLVIDORG INNER JOIN
                         PIECEVENTES AS Entete_CDE ON Ligne_Cde.PCVID = Entete_CDE.PCVID
						 where Entete_CDE.PCVNUM='''+@pcvnumcde+')';
 
 
  IF UPDATE (PCVNUM)
  BEGIN  
SELECT @PINCODE=I.PINCODE, @PCVNUM=I.PCVNUM FROM INSERTED I;
 
IF @PINCODE = 'CDECLI' 
 
 
                  set @shell = 'echo ' + @numdevis + ';1''>> "H:\Site Web\Out\orders_in\"accepted' + @numdevis + '.csv';
				  exec master..xp_cmdshell @shell;
 
                  				  end
								  end;

Pourriez vous m'aider s'il vous plait?

D'avance, merci.

Julien

[JeanYvette]

Bonjour bonjour,

Tout d'abord, si dans votre deuxième proposition vous avez la requête, c'est normal, vous traitez votre variable comme un string.

Pour la première proposition, il vous faut adapter votre code. En effet, là vous ne remplacez pas votre variable par votre valeur. Il vous faut passer par du sql dynamique avec un bel exemple sur le lien suivant :
https://blog.sqlauthority.com/2018/1...to-a-variable/

Bisous bisous

[Lafite84]

Bonjour et merci pour votre aide,

Tout d'abord, merci pour l'information concernant la seconde requête. Je n'avais pas vu que je traité le champ comme string.

Concernant le sql dynamique, je ne savais. J'ai essayé d'utiliser l'exemple donnée mais ça ne marche toujours pas.

Voici le code rectifié. Peux être aurez vous une idée.

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
BEGIN
 
declare @PINCODE as varchar(10);
declare @PCVISVALIDE as varchar(10);
declare @PCVNUM as varchar(50);
declare @shell as varchar(8000);
declare ins_cur cursor FAST_FORWARD for (select PCVNUM from inserted);
 
 
declare @sqlCommand as Nvarchar(4000);
declare @numdevis as VARCHAR(50);
declare @pcvnumcde as varchar(50);
set @pcvnumcde = (SELECT PCVNUM from inserted);
 
 
set @sqlCommand ='SELECT TOP (1) @numdevis = Entete_Devis.PCVNUM FROM PIECEVENTELIGNES AS Ligne_Devis INNER JOIN
                         PIECEVENTES AS Entete_Devis ON Ligne_Devis.PCVID = Entete_Devis.PCVID INNER JOIN
                         PIECEVENTELIGNES AS Ligne_Cde ON Ligne_Devis.PLVID = Ligne_Cde.PLVIDORG INNER JOIN
                         PIECEVENTES AS Entete_CDE ON Ligne_Cde.PCVID = Entete_CDE.PCVID
						 where Entete_CDE.PCVNUM=@pcvnumcde';
 
EXEC sp_executesql @sqlCommand, N'@pcvnumcde VARCHAR(50), @numdevis VARCHAR(50) OUTPUT',
@pcvnumcde=@pcvnumcde, @numdevis=@numdevis OUTPUT
SELECT @numdevis ReturnedName
  IF UPDATE (PCVNUM)
  BEGIN  
SELECT @PINCODE=I.PINCODE, @PCVNUM=I.PCVNUM FROM INSERTED I;
 
IF @PINCODE = 'CDECLI' 
 
                 /* set @shell = 'echo EP;' + @PCVNUM + '>> "\\192.168.1.9\data srvlx\Factures\"PDF' + @PCVNUM + '.txt';*/
                  set @shell = 'echo ' + @numdevis + ';1''>> "H:\Site Web\Out\orders_in\"accepted' + @pcvnumcde + '.csv';
				  exec master..xp_cmdshell @shell;
 
                  				  end
								  end;

D'avance, merci.
Cordialement
Julien

[SQLpro]

Pouvez vous expliquer fonctionnellement ce que vous cherchez à faire ?
Parce que :

• Les déclencheurs sont par nature ensembliste dans SQL Server alors que votre logique semble unitaire
• il 'est pas d'usage de faire appel à un curseur dans un déclencheur, sauf à vouloir pourrir les performances
• il 'est pas d'usage de faire appel à un processus externe (xp_cmdshell) dans un déclencheur, sauf à vouloir pourrir les performances et la sécurité

A +

[Lafite84]

Bonsoir SQLpro,

Tout d'abord merci pour votre réponse. Je vous explique donc en détail le besoin de ce déclencheur.

La table sur laquelle agit ce déclencheur est une table de pièces de vente. ce déclencheur doit me générer un fichier (ou deux fichiers) de façon à mettre à jour mon site internet.

Ce déclencheur doit au moment ou je transforme ma pièce de vente créer un (ou des) fichiers pour que mes clients puissent suivre l'avancement de leur devis/commande. Voici un exemple qui sera plus parlant :
un client me passe une commande sur mon site internet. Mon site me donne un fichier qui est intégré via un automate de mon erp.
J'intègre donc ce fichier en tant que devis dans mon logiciel. après accord par mon client, je transforme mon devis en commande. A ce moment là, j'ai besoin de générer un fichier qui, déposé sur le ftp de mon site, donne l'information que la commande est validée. (le déclencheur que j'essaye de faire)
Lors de la transformation de ma commande en bon de livraison, j'ai besoin de deux fichiers : un qui, via mon automate, me permet de générer le bon de livraison en pdf et un autre qui , déposé sur le ftp de mon site, donne l'information et le fichier de la livraison de la commande.
enfin, lors de la création de la facture, j'ai besoin de générer, de nouveau, deux fichiers comme pour la livraison pour la facture.

Je suis conscient que la création ou l'utilisation de déclencheur sur sql serveur peut ralentir la base et les temps d'accès certaines transformation ne sont pas quotidiennes.

Du coup, auriez vous une idée pour réaliser ces éléments sans que j'ai besoin d'intervenir (en dehors des actions dans l'ERP) ?

D'avance, merci.

Cordialement

Julien

[jowo]

Bonjour,

1) Votre ERP ne génère-t-il pas des événements lors d'un changement d'état ?

2 )Serait-il envisageable de lors du trigger créer une entrée dans une tables et qu'un processus par polling (exemple ~10secondes) lise cette entrée et qu'il génère les fichiers adéquats ?

3) Même que 2 mais l'information est mise dans une queue et le processus au lieu du polling lise cette dernière queue pour générer les fichiers ?

4) Au lieu de générer les fichiers au changement d'état qu'ils soient générés au moment de la demande ?

Je pense qu'il existe d'autres solutions alternatives... Ce ne sont que des pistes plus ou moins compliquées à mettre en œuvre.