January 28, 2005 at 3:39 am
morning ![]()
I have a trigger and I need to call a sproc within it.
I am getting an error.Yet when copy the sproc's T-SQL into the trigger, then no problem.
Is this by design?I'd have thought that it'd be better practice to include a simple sproc call within the trigger, rather than littering triggers with extra sql statements.
cheers,
yogi
January 28, 2005 at 3:44 am
It's hard to say, you don't say what error is, or what the trigger and sproc are supposed to do. ![]()
In general, there is nothing that says that you can't or shouldn't call a proc from a trigger - it depends. If the statements are required for the trigger to function, I wouldn't consider the code placed there then as 'litter'. Though, for maintenance and readability's sake it might be a good idea to encapsulate 'functions' in separate procs, but then again - It depends. ![]()
just my .02 though
/Kenneth
January 28, 2005 at 4:16 am
cheers Kenneth ![]()
I definitely believe that it's better to encapsulate whenever possible. I'll include the trigger code, to see if that helps to clarify
<sql>
CREATE TRIGGER [trigUserProductPermissionDelete] ON dbo.tblUserProductPermission
FOR DELETE
AS
spReadFromTblUserProductPermission
</sql>
<error>
error 170: Incorrect syntax near 'spReadFromTblUserProductPermission'
</error>
yet, if I paste the code in-line, then no problem.
I also tried to prefix the name of the sproc with "dbo." but no joy.
I can confirm that:
1) the sproc exists within this database
2) I can run the sproc no problem thru QA
3) the sproc requires no parameters
I think that I am not CALLING the sproc correctly.
I tried to put a "RUN" a "DO" and a "CALL" before the sproc, as a guess, but, hee haw.
Any ideas?
cheers,
yogi
January 28, 2005 at 5:39 am
Did you try putting EXEC in front of proc name?
Good Hunting!
AJ Ahrens
webmaster@kritter.net
January 28, 2005 at 5:52 am
he he
, I did now
..everything but the kitchen sink...
I'm a bit rusty.
thanks bud,
yogi
Viewing 5 posts - 1 through 5 (of 5 total)
You must be logged in to reply to this topic. Login to reply