Discussion :

[mad_martigan]

Bonjour à tous,

Je souhaiterais créer un trigger dynamiquement dans une procédure stockée.
Le nom de la base ainsi que le nom de la table sur laquelle le trigger sera créé sont passés en paramètres.

J'obtiens malheureusement le message d'erreur suivant lorsque j'exécute la procédure stockée:

'CREATE TRIGGER' doit être la première instruction d'un lot de requêtes.

La procédure stockée vérifie l'existance du trigger, le supprime le cas échéant puis le crée.

Quelle peut être la solution?

Merci

[Christianbt]

bonjour,

il nous faudrait malheureusement plus de détails sur votre script, avez-vous identifiez à quel niveau se situe l'erreur dans votre code ?

Christian

[iberserk]

'CREATE TRIGGER' doit être la première instruction d'un lot de requêtes.

C'est parce que 'CREATE TRIGGER' doit être la première instruction d'un lot de requêtes

Blague à part postez votre code complet...
Placez un GO juste avant votre CREATE TRIGGER cela devrait solutionner votre problème...

[mad_martigan]

Ok, je poste 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
 
SET @SQL_Create_Trigger = '	
 
		USE ['+@DB_Name+']
 
		IF  EXISTS (SELECT * FROM sys.triggers WHERE object_id = OBJECT_ID(N''[dbo].[TRG_01_Log_Modif_'+@TableName+']''))
		DROP TRIGGER [dbo].[TRG_01_Log_Modif_T_'+@TableName+']
 
		CREATE TRIGGER TRG_01_Log_Modif_'+@TableName+' ON dbo.'+@TableName+' FOR INSERT, UPDATE, DELETE
		AS
 
		DECLARE 
			@DBName VARCHAR(500),
			@Bit INT ,
			@Field INT ,
			@MaxField INT ,
			@Char INT ,
			@FieldName VARCHAR(128) ,
			@TableName VARCHAR(128) ,
			@PKCols VARCHAR(1000) ,
			@SQL VARCHAR(2000), 
			@UpdateDate VARCHAR(21) ,
			@UserName VARCHAR(128) ,
			@Type CHAR(1) ,
			@PKSelect VARCHAR(1000),
			@ColumnType VARCHAR(500)
 
			...
 
 
'
 
 
 
EXEC (@SQL_Create_Trigger)

Je ne poste que l'essentiel.

Si je mets un 'GO' avant le 'CREATE TRIGGER' je pense qu'il y aura un problème...

[aieeeuuuuu]

Si je mets un 'GO' avant le 'CREATE TRIGGER' je pense qu'il y aura un problème...

pourquoi donc ?

prenez juste soin de mettre GO sur une ligne à part, sans autre instruction TSQL

[iberserk]

Si je mets un 'GO' avant le 'CREATE TRIGGER' je pense qu'il y aura un problème...

A part si vous estimez que le fait que çà marche soit un problème je ne vois pas lequel

[mad_martigan]

Effectivement pas de problème, ça fonctionne

[mad_martigan]

Bonjour,

Je rouvre le ticket car finalement cela ne fonctionne pas..

J'ai pourtant bien mis un GO une ligne avant le CREATE TRIGGER:

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
 
SET @SQL_Create_Trigger = '	
 
		USE ['+@DB_Name+']
		GO
 
		IF  EXISTS (SELECT * FROM sys.triggers WHERE object_id = OBJECT_ID(N''[dbo].[TRG_01_Log_Modif_'+@TableName+']''))
		DROP TRIGGER [dbo].[TRG_01_Log_Modif_'+@TableName+']
 
		GO
		CREATE TRIGGER TRG_01_Log_Modif_'+@TableName+' ON dbo.'+@TableName+' FOR INSERT, UPDATE, DELETE
		AS
 
		DECLARE 
			@DBName VARCHAR(500),
			@Bit INT ,
			@Field INT ,
			@MaxField INT ,
.
.
.

Merci,

[aieeeuuuuu]

est-ce toujours la même erreur ????

[mad_martigan]

oui toujours la même...

[iberserk]

Bonjour, je n'avais pas vu que la requête était en SQL dynamique...

Celà ne peut pas marcher...

Vous devez la découper en plusieurs requêtes en pensant à préfixeer vos noms de tables avec le nom de votre base en lieu et place du USE...

[SQLpro]

GO n'est pas une instruction SQL. C'est une instruction qui s'adresse à l'interpréteur de requête. Vous ne pouvez donc pas mettre GO dasn un script SQL Dynamique.
La solution est de couper votre script SQL en autant d'exécution que d'instruction.

A +

[iberserk]

GO n'est pas une instruction SQL. C'est une instruction qui s'adresse à l'interpréteur de requête. Vous ne pouvez donc pas mettre GO dasn un script SQL Dynamique.
La solution est de couper votre script SQL en autant d'exécution que d'instruction.

A +

C'est dailleurs ce que fais smo...

[mad_martigan]

Ok, merci pour vos réponses.
J'ai découpé le script comme prévu et il semble que cela fonctionne bien