> you can't explicitly specify "field asc" Sean, you *can* explicitly assign asc. For example, this works fine:
sortByFieldList(UserID asc, DateCreated, Username desc) Baz You can -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Sean Corfield Sent: Friday, February 10, 2006 10:17 PM To: [email protected] Subject: Re: Reactor For CF sortByFieldList now accepts asc/desc On 2/10/06, Mark Mazelin <[EMAIL PROTECTED]> wrote: > I believe your code will break if you have a field that contains "desc" > in it. It might perhaps be hard to see in Baz's code but he has a space before desc so it will not break in that situation. You'd specify "field desc". I don't like Baz's fix because it isn't symmetric: you can't explicitly specify "field asc". I don't like the "fieldname sortorder" string approach at all. It's a hack, IMO. Unfortunately, right now, I don't have a better suggestion. So I'd say: don't change the framework until a cleaner way to do this can be agreed on (and remember that Doug has the final say). -- Sean A Corfield -- http://corfield.org/ Got frameworks? "If you're not annoying somebody, you're not really alive." -- Margaret Atwood -- Reactor for ColdFusion Mailing List -- [email protected] -- Archives at http://www.mail-archive.com/reactor%40doughughes.net/ -- Reactor for ColdFusion Mailing List -- [email protected] -- Archives at http://www.mail-archive.com/reactor%40doughughes.net/
