The simplest would be to remove all the dynamic SQL and just use a simple insert possibly with NULLIFs in ths values clause if the app passes blanks that you want to store as nulls or similar. Is there some reaon youi NEED to manually string the SQL together like this?

Mike