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:348995 Subscription: http://www.houseoffusion.com/groups/cf-talk/subscribe.cfm Unsubscribe: http://www.houseoffusion.com/groups/cf-talk/unsubscribe.cfm