As stated, you are missing a comma after one of the params. That may be causing the error you are receiving. Also, there are some fundamental issues with the code you posted beyond the error you are receiving. I only point these out to help so please don't take as criticism. First, you get an orderid at the top of the code. The lock is pointless in that scenario and will just add unnecessary overhead. How do you know that is the correct id? Just grabbing the max can result in getting the wrong id. It is highly probable that the id returned could be the wrong id. There should be something in place to make sure you are getting the id for the order for this user/request. Next, this code: <cfoutput><cfset getoid = #qGetorderID.oid#></cfoutput> You don't need to put cfoutput around a set statement. Also, unless you are concatenating in quotes or outputting you don't need to put pound signs around vars. You should change it to... <cfset getoid = qGetorderID.oid> Try to avoid using evaluate. It is slow and generally not necessary. You could rewrite those set statements as... <cfset getqty = form[qty & idx]> Thanks, --Dave
... View more