> I would argue that that is not only a horrible abuse (LOCKING > on SEVER to set something in SESSION), but that this > situation doesn't even need a lock at all. The race condition > here seems unimportant. In fact, it is locking for one CFSET > but not the other. Get rid of this lock.
Not to mention the use of the very much deprecated ParameterExists()!!! Wow... I haven't used that function since CF 4.01 when we migrated everything to IsDefined() and now with MX, StructKeyExists()... What version of CF is that code running on? Paul ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| Introducing the Fusion Authority Quarterly Update. 80 pages of hard-hitting, up-to-date ColdFusion information by your peers, delivered to your door four times a year. http://www.fusionauthority.com/quarterly Archive: http://www.houseoffusion.com/groups/CF-Talk/message.cfm/messageid:251702 Subscription: http://www.houseoffusion.com/groups/CF-Talk/subscribe.cfm Unsubscribe: http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=89.70.4