Problem with code like this is the max variable length will be hit you when you least expect it. There is also the bigger problem of a delimiter or...

As Sushila said it may be obvious to some that it is sensible to move column names into variables of the same name prefixed by @ but I have seen...

Prefixing stored procedure names adds clutter and makes finding thim in Query analiser difficult. After all SELECT * FROM sysobjects WHERE xtype='P' get them all!

Try this:

CREATE FUNCTION RemoveTimeFromDate (@DateToModify datetime)

--input like: 01/01/2003 12:34:56

--output: 01/01/2003 00:00:00

--Ian Stone December 2003, issue1.0

RETURNS datetime

AS

BEGIN

DECLARE @ReturnDate datetime

SET @ReturnDate=DATEADD(hh,-DATEPART(hh,@DateToModify),@DateToModify)

...