<cfcatch type="Any"><!--- do nothing ---></cfcatch>
Never _ever_ do that! If you're going to catch an exception, do something with it. Don't just bury it. I can't tell you how many "that's not possible" moments I've had while working with legacy code that contains buried exceptions. Also, I think a method named "authenticate" should return a boolean. If you want to get the user's information, I would do that in a separate method, named something like "retrieveUser." And I would have that method return an object rather than a query. HTH, Patrick -- Patrick McElhaney 704.560.9117 http://pmcelhaney.weblogs.us ---------------------------------------------------------- You are subscribed to cfcdev. To unsubscribe, send an email to [email protected] with the words 'unsubscribe cfcdev' as the subject of the email. CFCDev is run by CFCZone (www.cfczone.org) and supported by CFXHosting (www.cfxhosting.com). An archive of the CFCDev list is available at www.mail-archive.com/[email protected]
