On 2/10/09 21:43, Avi Miller wrote:
> Hey,
>
> Luke Kanies wrote:
>>> --- a/lib/puppet/rails/param_value.rb
>>> +++ b/lib/puppet/rails/param_value.rb
>>> @@ -48,7 +48,7 @@ class Puppet::Rails::ParamValue< ActiveRecord::Base
>>>
>>> # returns an array of hash containing all the parameters of a
>>> given resource
>>> def self.find_all_params_from_resource(db_resource)
>>> - params = db_resource.connection.select_all("SELECT v.id,
>>> v.value, v.line, v.resource_id, v.param_name_id, n.name FROM
>>> param_values as v INNER JOIN param_names as n ON
>>> v.param_name_id=n.id WHERE v.resource_id=%s" % db_resource.id)
>>> + params = db_resource.connection.select_all("SELECT v.id,
>>> v.value, v.line, v.resource_id, v.param_name_id, n.name FROM
>>> param_values v INNER JOIN param_names n ON v.param_name_id=n.id
>>> WHERE v.resource_id=%s" % db_resource.id)
>> Are these changes tot he SQL fully compatible with the other DBs we
>> run on?
>>
>> If so, then +1.
>
> I've tested this with PostgreSQL and it works fine, but I'm having
> problems getting the MySQL and sqlite3 adapters to work on EL5. Will
> install an Ubuntu box to test this on and get back to you. :)
I might be wrong, but I think 'as' is optional on MySQL, so that'll work
fine on MySQL.
--
Brice Figureau
My Blog: http://www.masterzen.fr/
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Puppet Developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/puppet-dev?hl=en
-~----------~----~----~----~------~----~------~--~---
