Hey Doug, I can't commit this fix as-is. It looks like (read messages below) the fix breaks something else.
-Joe On 6/12/06, Doug Hughes <[EMAIL PROTECTED]> wrote:
Joe will be committing the fix to this later today. That, and a nifty new feature for using CS-created objects from within reactor. Doug -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Christopher Bradford Sent: Monday, June 12, 2006 3:54 PM To: [email protected] Subject: Re: SPAM-LOW: RE: [Reactor for CF] Sanity check please... 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/ -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- Reactor for ColdFusion Mailing List [email protected] Archives at: http://www.mail-archive.com/reactor%40doughughes.net/ -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
-- 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/ -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
