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. :)

cYa,
Avi

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

Reply via email to