SQL Trigger Error - - sql-server

Question

I am trying to develop one trigger, but I seem to be getting the following error:
A cursor with the name 'CONTACT_CURSOR' does not exist. 5339B6CC-5480-4B24-AC5D-3A4C0EAF64EC 3F0F445B-7BA3-4311-944A-E372D41FD307 The statement has been terminated.
Here is the trigger I am trying to develop:
USE [124959test]
GO
/****** Object: Trigger [dbo].[TRG_AUTOCAMPAIGN_Based_on_IDStatus_Value_Latest] Script Date: 05/28/2015 10:48:50 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
ALTER TRIGGER [dbo].[TRG_AUTOCAMPAIGN_Based_on_IDStatus_Value_Latest]
ON [dbo].[SN_Contact2]
FOR INSERT, UPDATE
AS
BEGIN
SET NOCOUNT ON;
IF (UPDATE(contact_category)) -- selector field is the one that determines if an action shou dbe taken
BEGIN
DECLARe #CAMPAIGNSTARTTIME datetime
DECLARE #DELETEOLD INT
DECLARE #TRIGGERFIELD VARCHAR(100)
DECLARE #USERID VARCHAR(36)
DECLARE #CONTACTID VARCHAR(36)
DECLARE #CAMPAIGNID VARCHAR(36)
DECLARE #SCHEDULEFOR VARCHAR(36)
DECLARE #COMMINGLE VARCHAR(100)
DECLARE #DROPSHIP VARCHAR(100)
DECLARE #EDDM VARCHAR(100)
DECLARE #EXPEDITEDDROPSHIP VARCHAR(100)
DECLARE #EXPEDITEDSERVICES VARCHAR(100)
DECLARE #FIRSTCLASS VARCHAR (100)
DECLARE #FLATS VARCHAR(100)
DECLARE #FREIGHTROVER VARCHAR(100)
DECLARE #INTERNATIONAL VARCHAR(100)
DECLARE #MAILROVER VARCHAR(100)
DECLARE #OBBASHIP VARCHAR(100)
DECLARE #OBBATRACK VARCHAR(100)
DECLARE #PARCEL VARCHAR(100)
DECLARE #PRODUCTION VARCHAR(100)
SELECT #DELETEOLD = 0
DECLARE CONTACT_CURSOR CURSOR FOR
SELECT Contact_ContactID, SNL_RecordManagerID,contact_category,cust_commingle,CUST_Drop_Ship,
cust_eddm,CUST_Expedited_Drop_Ship,CUST_Expedited_Services,CUST_First_Class,CUST_Flats,
CUST_Freight_Rover,CUST_International,CUST_MailRover,CUST_OBBAShip,CUST_OBBATrack,
CUST_Parcel,CUST_Production FROM inserted
--where snl_recordmanagerid = '1C42B435-4A81-4109-A008-34AE91C2D48F'
OPEN CONTACT_CURSOR
FETCH NEXT FROM CONTACT_CURSOR
INTO #CONTACTID, #USERID, #TRIGGERFIELD, #COMMINGLE, #DROPSHIP, #EDDM, #EXPEDITEDDROPSHIP, #EXPEDITEDSERVICES,
#FIRSTCLASS, #FLATS, #FREIGHTROVER, #INTERNATIONAL, #MAILROVER, #OBBASHIP, #OBBATRACK, #PARCEL, #PRODUCTION
WHILE ##FETCH_STATUS = 0
BEGIN
IF #TRIGGERFIELD = 'Client' AND #USERID = '5339B6CC-5480-4B24-AC5D-3A4C0EAF64EC'
--or #USERID = '277D668E-F18F-48E7-B7C1-2FDCF50F1B36'
--or #USERID = '7FF6E867-8DDA-47FE-9A2F-E7E042FFAC56'
/* This is to launch the New Client Workflow*/
BEGIN
SET #CAMPAIGNID = 'D25053D9-EF07-4184-98F5-EC6DB1814B7E' -- CampaignID of the New client Campaign
SET #Campaignstarttime = getdate()
EXEC usp_AssignCampaign #CONTACTID, #USERID, #CAMPAIGNID, #DELETEOLD, #CAMPAIGNSTARTTIME
/* This is to launch the Send Welcome Package Workflow*/
SET #CAMPAIGNID = '08118AC6-456B-4BEC-BD09-F09A0FDC893D' -- CampaignID of the Welcome Package Campaign.
SET #Campaignstarttime = getdate()
SET #USERID = '9E283EF8-208F-4A60-BACA-555F3BECEBAB' -- The UserID of to whom the to-do task will be scheduled to.
EXEC usp_AssignCampaign #CONTACTID, #USERID, #CAMPAIGNID, #DELETEOLD, #CAMPAIGNSTARTTIME
END
ELSE
BEGIN
IF #TRIGGERFIELD = 'Prospect'
BEGIN
--Update sn_contact2 set cust_newsletter = Yes' -- When ID/Status = Prospect/Hot Prospect update Newsletter field to Yes.
SET #CAMPAIGNID = 'B1649041-FB74-40D1-807C-F969D8AE4D40' -- CampaignID of the Opt-out Email with LinkedIn Follow Up Campaign
SET #Campaignstarttime = getdate()
EXEC usp_AssignCampaign #CONTACTID, #USERID, #CAMPAIGNID, #DELETEOLD, #CAMPAIGNSTARTTIME
END
ELSE
BEGIN
IF #TRIGGERFIELD = 'Hot Prospect'
BEGIN
--Update sn_contact2 set cust_newsletter = Yes' -- When ID/Status = Prospect/Hot Prospect update Newsletter field to Yes.
SET #CAMPAIGNID = 'B1649041-FB74-40D1-807C-F969D8AE4D40' -- CampaignID of the Opt-out Email with LinkedIn Follow Up Campaign
SET #Campaignstarttime = getdate()
EXEC usp_AssignCampaign #CONTACTID, #USERID, #CAMPAIGNID, #DELETEOLD, #CAMPAIGNSTARTTIME
end
end
FETCH NEXT FROM CONTACT_CURSOR
INTO #CONTACTID, #USERID, #TRIGGERFIELD, #COMMINGLE, #DROPSHIP, #EDDM, #EXPEDITEDDROPSHIP, #EXPEDITEDSERVICES,
#FIRSTCLASS, #FLATS, #FREIGHTROVER, #INTERNATIONAL, #MAILROVER, #OBBASHIP, #OBBATRACK, #PARCEL, #PRODUCTION
END
CLOSE CONTACT_CURSOR
DEALLOCATE CONTACT_CURSOR
END
end
end
Any thoughts?

Answer 1

I think you have an END in the wrong place. It's pushing your Fetch Next out of scope of your Cursor declaration. Try amending the end of your code to the following...
BEGIN
--Update sn_contact2 set cust_newsletter = Yes' -- When ID/Status = Prospect/Hot Prospect update Newsletter field to Yes.
SET #CAMPAIGNID = 'B1649041-FB74-40D1-807C-F969D8AE4D40' -- CampaignID of the Opt-out Email with LinkedIn Follow Up Campaign
SET #Campaignstarttime = getdate()
EXEC usp_AssignCampaign #CONTACTID, #USERID, #CAMPAIGNID, #DELETEOLD, #CAMPAIGNSTARTTIME
end
end
END
FETCH NEXT FROM CONTACT_CURSOR
INTO #CONTACTID, #USERID, #TRIGGERFIELD, #COMMINGLE, #DROPSHIP, #EDDM, #EXPEDITEDDROPSHIP, #EXPEDITEDSERVICES,
#FIRSTCLASS, #FLATS, #FREIGHTROVER, #INTERNATIONAL, #MAILROVER, #OBBASHIP, #OBBATRACK, #PARCEL, #PRODUCTION
END
CLOSE CONTACT_CURSOR
DEALLOCATE CONTACT_CURSOR
END
end

Answer 2

There are couple problems with the code. First you need to move FETCH right before END that matches BEGIN after WHILE. Next thing you need to move CLOSE and DEALLOCATE to outside.
FETCH NEXT FROM CONTACT_CURSOR
INTO #CONTACTID, #USERID, #TRIGGERFIELD, #COMMINGLE, #DROPSHIP, #EDDM, #EXPEDITEDDROPSHIP,
#EXPEDITEDSERVICES, #FIRSTCLASS, #FLATS, #FREIGHTROVER, #INTERNATIONAL, #MAILROVER,
#OBBASHIP, #OBBATRACK, #PARCEL, #PRODUCTION
END
CLOSE CONTACT_CURSOR
DEALLOCATE CONTACT_CURSOR
END
END
This is basic outline of the cursor
DECLARE Some_Cursor CURSOR
OPEN Some_Cursor
FETCH NEXT FROM Some_Cursor
WHILE ##FETCH_STATUS = 0
BEGIN
FETCH NEXT FROM Some_Cursor
END
CLOSE Some_Cursor
DEALLOCATE Some_Cursor

Answer 3

Appreciate all the help guys! I have made this trigger running, after I verify it works correctly, I will then refine or restructure this so it would no longer be using contact cursors.
Appreciate the help!
Thanks again!
-Mike