Solved

Sql trigger error - SQL 2000

Posted on 2014-09-10
10
207 Views
Last Modified: 2014-09-16
Not that familiar with triggers...... we already have one set up for this table but here is the error we get:       updated is not an object      - when an update to the table occurs.
Here  is the trigger info:
~~~~~~~~~~~~~~~~~~~~~~
CREATE TRIGGER  Davis_Upd_TblNotify  ON [dbo].[dtMavis]
     FOR UPDATE

AS
BEGIN
   declare @COID varchar(5);
   declare @Site varchar(2);
   declare @Tmp varchar(8);
   declare @dteupd  datetime;
   Set @dteupd = getdate();

   Select @COID=Inserted.COID  From  updated i;
  Set @Site =(Select SubString(WhoUpd,6,2) From Updated)

       Update DbTblNotify  Set DteUpd=getDate() Where COID=@COID AND TranID=3 And Site=@Site

       If @@ROWCOUNT=0
          Insert Into DbTblNotify(COID,TranID,Site,DteUpd) Values(@COID,3,@Site,@dteupd)
      End
~~~~~~~~~~~

The error we get says Updates is not an object...... I have tried numerous variations of the above... but still get same error. {have tried it with the "i", without the "i", ...)

What am I doing wrong (this is for Sql 2000
0
Comment
Question by:datafarmer
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 5
  • 4
10 Comments
 
LVL 29

Expert Comment

by:sammySeltzer
ID: 40315735
I don't think trigger has UPDATED.

It is either selected from inserted or deleted but not updated

I believe you want to select from Inserted.

Or if you wish to update based on updated record, then use After Update trigger.
0
 

Author Comment

by:datafarmer
ID: 40315753
I tried that as well....
~~~~~~~~~~~~
CREATE TRIGGER Davis_Upd_TblNotify  ON [dbo].[dtMavis]      AFTER UPDATE
AS
BEGIN
   declare @COID varchar(5);
   declare @Site varchar(2);
   declare @Tmp varchar(8);
   declare @dteupd  datetime;
   Set @dteupd = getdate();

   Select @COID=Inserted.COID  From  updated i;
  Set @Site =(Select SubString(WhoUpd,6,2) From Updated )

       Update DbTblNotify  Set DteUpd=getDate() Where COID=@COID AND TranID=3 And Site=@Site

       If @@ROWCOUNT=0
          Insert Into DbTblNotify(COID,TranID,Site,DteUpd) Values(@COID,3,@Site,@dteupd)
      End
~~~~~~~~~~~
So to be clear, when I try to update dtMavis.. exact error message is "ODBC Sql Server Driver] Invalid object name 'updated'
0
 

Author Comment

by:datafarmer
ID: 40315766
Let me add to the comment...
So if  I use:  
 CREATE TRIGGER Davis_Upd_TblNotify  ON [dbo].[dtMavis]      AFTER UPDATE
AS
BEGIN
   declare @COID varchar(5);
   declare @Site varchar(2);
   declare @Tmp varchar(8);
   declare @dteupd  datetime;
   Set @dteupd = getdate();

   Select @COID=(Select COID  From  updated)
  Set @Site =(Select SubString(WhoUpd,6,2) From Updated )

** That works as far as the trigger actually firing with out an error... but then I get a blank COID, which let me to believe that I was still using the incorrect syntax - because I am not getting a COID.
True, the COID itself, is not being updated.. but I would expect to get the COID back.
0
What is SQL Server and how does it work?

The purpose of this paper is to provide you background on SQL Server. It’s your self-study guide for learning fundamentals. It includes both the history of SQL and its technical basics. Concepts and definitions will form the solid foundation of your future DBA expertise.

 
LVL 29

Expert Comment

by:sammySeltzer
ID: 40315768
You can't use:

 Select @COID=Inserted.COID  From  updated i;

You have to change it to:

 Select @COID=Inserted.COID  From  inserted i;

However, depending on what you are trying to do, you can try to find out if a table is updated with a value and then update it using this link:

http://msdn.microsoft.com/en-us/library/ms187326.aspx
0
 

Author Comment

by:datafarmer
ID: 40315790
Let me be show the updated trigger:
CREATE TRIGGER Davis_Upd_TblNotify  ON [dbo].[dtMavis]      AFTER UPDATE
AS
BEGIN

    declare @COID varchar(5);
   declare @Site varchar(2);
   declare @Tmp varchar(8);
   declare @dteupd  datetime;
   Set @dteupd = getdate();

   
  Set @COID =(Select COID From inserted i)
  Set @Site =(Select SubString(WhoUpd,6,2) From inserted i)

      
       Update DbTblNotify  Set DteUpd=getDate(), EdtSq=EdtSq+1 Where COID=@COID AND TranID=3 And Site=@Site

       If @@ROWCOUNT=0
          Insert Into DbTblNotify(COID,TranID,Site,DteUpd) Values(@COID,3,@Site,@dteupd)
      End
~~~
The above does work.... but I am not getting the COID back, even though there is a COID in the record.
What happens then, is I try to add the COID, TraniD, Site... and get a duplicate key error because a blank @COID has already been added (along with a blank @site field

All that I am trying to do is update the DbTblNotify with the COID (from the Record being updated) and the SiteID.   Both seem to values seem to be blank, even though there are valid values in record.
0
 
LVL 29

Expert Comment

by:sammySeltzer
ID: 40315821
Ok let's start with this:

 Set @COID =(Select COID From inserted i)
   Set @Site =(Select SubString(WhoUpd,6,2) From inserted i)

what are you trying to accomplish here?

The code tells me that when a record is inserted grad the COID and whoUpd of that inserted record.

Is that what you really want to do?

I don't think your code reflects that.

Also, you can streamline code above into just one line:

 SELECT @COID =COID,  @Site = SubString(WhoUpd,6,2) From inserted i)

Finally, you will need to test your insert and update by hardcoding the values.

If they work, then you know where the problem lies.

Take a quick look at the link I provided to see how to check for an updated record before performing any action.
0
 

Author Comment

by:datafarmer
ID: 40315851
This exactly what I am trying to accomplish:
If this query runs:
Update dtMavis  Set Cat='AC',Dteupd=getdate() where COID='FSL' AND Recnum='Aaa000003'

I want the DbTblNotify  (table) to be updated with the COID (in this case FSL), the current date/time and
the site ID that would get from the WhoUpd field.    

This is what I coded...
CREATE TRIGGER  Mavis_Upd_TblNotify  ON dtMavis     AFTER UPDATE
AS
BEGIN

    declare @COID varchar(5);
   declare @Site varchar(2);
   declare @dteupd  datetime;
   Set @dteupd = getdate();

   
       Select @COID = COID, @Site = SubString(WhoUpd,6,2) From updated
---  What I still don't understand is...  I would think that since I relying on a row to be updated,
---  and not inserted.... I would use updated.  But when I do, I get invalid object  = updated.  
---  When I use inserted... I get null values on COID and site ID... both of which have values.
---- I did read the article, thank you... but I guess I since I don't care which columns are updated,
 --- as long as the record is being updated... what I am in missing.

      
       Update DbTblNotify  Set DteUpd=getDate(), EdtSq=EdtSq+1  Where COID=@COID AND TranID=3 And Site=@Site

       If @@ROWCOUNT=0
          Insert Into DbTblNotify(COID,TranID,Site,DteUpd) Values(@COID,3,@Site,@dteupd)
      End
~~~~~
Apologize for being dense... but I guess I need a slap to see what I am missing.
0
 
LVL 69

Accepted Solution

by:
Scott Pletcher earned 500 total points
ID: 40315899
You need to do this set-based rather than assuming only one row is being UPDATEd.  SQL Server only calls a trigger once per UPDATE/statement, no matter how many rows are affected.


CREATE TRIGGER Davis_Upd_TblNotify
  ON [dbo].[dtMavis]
  AFTER UPDATE
AS
SET NOCOUNT ON;

UPDATE dtn
SET
    DteUpd = GETDATE(),
    EdtSq = EdtSq+1
FROM inserted i
INNER JOIN DbTblNotify dtn ON
    dtn.COID = i.COID AND
    dtn.Site = SUBSTRING(i.WhoUpd, 6, 2) AND
    dtn.TranID = 3

INSERT INTO DbTblNotify ( COID, TranID, Site, EdtSq, DteUpd )
SELECT
    i.COID, 3 AS TranID,
    SUBSTRING(i.WhoUpd, 6, 2) AS Site,
    1 AS EdtSq, GETDATE() AS DteUpd
FROM inserted i
LEFT OUTER JOIN DbTblNotify dtn ON
    dtn.COID = i.COID AND
    dtn.Site = SUBSTRING(i.WhoUpd, 6, 2) AND
    dtn.TranID = 3
WHERE
    dtn.COID IS NULL

GO
0
 
LVL 29

Expert Comment

by:sammySeltzer
ID: 40315906
You use a trigger because you want certain operation to occur when triggered by something.

In your case it seems, when a table is updated with WhoUpd field, you want to perform an update or insert.

Something like this:

CREATE TRIGGER  Mavis_Upd_TblNotify  ON 
dtMavis    
 AFTER UPDATE
AS
IF (UPDATE WhoUpd )
BEGIN
Update DbTblNotify  d 
Set DteUpd=getDate(), EdtSq=EdtSq+1 
FROM theOtherTable t
INNER JOIN t ON d.TranID= t.TranID
Where d.TranID=3 
END
GO

Open in new window


Try this and see what happens.

You may have to make changes based on your understanding of your domain and data structure.

If this works, then you can work in the INSERT bit.
0
 

Author Closing Comment

by:datafarmer
ID: 40326891
Flipping awesome... yes this works.....   Thank you so much!
0

Featured Post

Enroll in June's Course of the Month

June’s Course of the Month is now available! Experts Exchange’s Premium Members, Team Accounts, and Qualified Experts have access to a complimentary course each month as part of their membership—an extra way to sharpen your skills and increase training.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

I have a large data set and a SSIS package. How can I load this file in multi threading?
In the first part of this tutorial we will cover the prerequisites for installing SQL Server vNext on Linux.
This video shows, step by step, how to configure Oracle Heterogeneous Services via the Generic Gateway Agent in order to make a connection from an Oracle session and access a remote SQL Server database table.
Using examples as well as descriptions, and references to Books Online, show the documentation available for datatypes, explain the available data types and show how data can be passed into and out of variables.

728 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question