Re: [Reactor for CF] Dynamically returning table names from aliases in custom gateway queries

[[EMAIL PROTECTED]]

Brian, I think we're saying the same thing: I just meant that the method to return the name should be on the level of the Gateway (next to the getAlias() method).  W. On Sep 2, 2006, at 01:12, Brian Kotek wrote: I'm not sure I would agree. I think the Gateway should know the alias of the table, but there is no reason for it to know what table name the alias maps to. If I've defined an alias for a table in my XML, and for some reason the table name changes or is moved to a different schema, I would have to go update all the custom Gateway methods to use the new table name. If we had a getTableName() as I am suggesting, you wouldn't have to change anything except the Reactor XML file. Basically, this seems so simple to implement and such low-hanging fruit that I can't see any possible reason why it shouldn't be included. On 9/1/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: Logically speaking, the gateway for a specific table should know the name (and the alias) of the table... Is it possible to add a getName() method next to the getAlias() ? (I don't have the code here, and besides I'm not very familiar with the core code, so I  can't figure it out myself right now...) Wouter On Sep 1, 2006, at 21:00, Mr. Darby wrote: Brian, That's a great idea.  Currently, I have found the method getAlias, that returns either the tables' name if an alias is not defined or the alias for the table object as defined in the reactor.xml.  Having another method that ensures you are returned the table's name regardless of an alias defined would be better. On 9/1/06, Brian Kotek < [EMAIL PROTECTED]> wrote: Hi folks, I had posted about this a long time back but I don't think anything came of it so I wanted to bring it up again. In a custom gateway query, currently you have to explictly specify the name(s) of the tables you want to use in the query, such as: select userID from mydb.myschema.user Now since we've gone through the trouble of defining aliases for our tables in the Reactor XML, I find this undesirable. I'd rather do something like this: select userID from #getTableName('user')# The function to do this is very simple (though if we go the route of adding a separate "schema" attribute to the object tag it would have to take that into consideration):     <cffunction name="getTableName" access="private" returntype="string" output="false" hint="I return the table name for the passed table alias.">         <cfargument name="tableAlias" type="string" required="true" hint="Alias of target table." />         <cfreturn _getConfig().getObjectConfig(arguments.tableAlias).object.xmlAttributes.name />     </cffunction> Now I ask, wouldn't it be quite easy to add this to a base class within Reactor somewhere? I can only imagine that many, many people have hardcoded table names in their custom queries and it seems that this could be avoided very easily. Thoughts? -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- Reactor for ColdFusion Mailing List reactor@doughughes.net Archives at: http://www.mail-archive.com/reactor%40doughughes.net/ -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- Reactor for ColdFusion Mailing List reactor@doughughes.net Archives at: http://www.mail-archive.com/reactor%40doughughes.net/ -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- Reactor for ColdFusion Mailing List reactor@doughughes.net Archives at: http://www.mail-archive.com/reactor%40doughughes.net/ -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- Reactor for ColdFusion Mailing List reactor@doughughes.net Archives at: http://www.mail-archive.com/reactor%40doughughes.net/ -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- Reactor for ColdFusion Mailing List reactor@doughughes.net Archives at: http://www.mail-archive.com/reactor%40doughughes.net/ -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --