It may be more than a style preference. I'm not sure that "No" as a string value is always equivalent to a true/false boolean value. My suggestion was intended to help identify if that might actually be your issue.
I've experienced intermittent issues with FileExists and trying to delete files in similar logic as yours. Carl On 12/7/2011 9:14 AM, Shannon Rhodes wrote: > I understand and agree with your style preferences, but neither complaint > with legacy code would cause issues. > >> Since DirectoryExists returns a true/false, you don't need the IS "No" >> part of the statement. Also, DirectoryExists expects you to pass it a >> variable, so you don't need the quotes and pound signs around >> variables.dirpath: >> >> <cfif NOT DirectoryExists(variables.dirpath)> >> <cfdirectory action="CREATE" directory="#variables.dirpath#"> >> </cfif> >> >> HTH, >> Carl >> >> On 12/6/2011 1:56 PM, Shann > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| Order the Adobe Coldfusion Anthology now! http://www.amazon.com/Adobe-Coldfusion-Anthology/dp/1430272155/?tag=houseoffusion Archive: http://www.houseoffusion.com/groups/cf-talk/message.cfm/messageid:348997 Subscription: http://www.houseoffusion.com/groups/cf-talk/subscribe.cfm Unsubscribe: http://www.houseoffusion.com/groups/cf-talk/unsubscribe.cfm
