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/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --

Reply via email to