reactor.query.order

Christopher Bradford
Alive! LLP
----- Original Message ----- From: "Joshua Scott" <[EMAIL PROTECTED]>
To: <[email protected]>
Sent: Monday, June 12, 2006 1:16 PM
Subject: SPAM-LOW: RE: [Reactor for CF] Sanity check please...


What cfc is this in? I am having the same problem.

- JS


------------------------------------------------
Joshua Scott
Resonant Media Technologies, LLC.
http://www.resonantmedia.com | http://ponderings.wordpress.com


"It is impossible to get out of a problem by using the same type of thinking
that it took to get into the problem." -- Albert Einstein




-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On
Behalf Of Christopher Bradford
Sent: Monday, June 12, 2006 12:36 PM
To: [email protected]
Subject: Re: [Reactor for CF] Sanity check please...

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.fi
eldAlias).name
/>

to:

<cfset node.fieldName =
getQuery().findObject(node.objectAlias).getObjectMetadata().getField(node.fi
eldAlias).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/
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
-- --



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