Thanks for pointing this out, Joe. I did run into one problem that this
solution creates: the appendNode method of the order object does not use the
fieldName key; rather, it uses the field key. Changing "fieldAlias" to
"fieldName" in the abstractGateway causes getFieldExpression to break when
it is called for an order field. I had to change the appendNode method of
the order object, line 38, from:
<cfset node.field =
getQuery().findObject(node.objectAlias).getObjectMetadata().getField(node.fieldAlias).name
/>
to:
<cfset node.fieldName =
getQuery().findObject(node.objectAlias).getObjectMetadata().getField(node.fieldAlias).name
/>
It seems to be working, but since I'm not intimately familiar with the code,
this could have unintended consequences.
Christopher Bradford
Alive! LLP
----- Original Message -----
From: "Joe Rinehart" <[EMAIL PROTECTED]>
To: <[email protected]>
Sent: Friday, June 09, 2006 6:29 AM
Subject: SPAM-LOW: Re: [Reactor for CF] Sanity check please...
Using latest Reactor from SVN, I think there's a bug in when it
translates field aliases to column names: it's using the alias
instead of the name attrib in the where clause.
Doug'll actually be down here at my house later today, so I'll see if
I can get him to commit the fix. I got it working by opening
reactor.base.AbstractGateway and
changing line 632 from:
<cfreturn arguments.Convention.formatFieldName(arguments.node.fieldAlias,
arguments.node.objectAlias) />
to:
<cfreturn arguments.Convention.formatFieldName(arguments.node.fieldName,
arguments.node.objectAlias) />
Doug'll have final call as to whether this was the right place to do
it, but it looks right to me.
-Joe
--
Get Glued!
The Model-Glue ColdFusion Framework
http://www.model-glue.com
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
-- --
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/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --