Fixing #2187 - Puppet::Resource is expected by Rails support

We previously used and expected Puppet::Parser::Resource
instances, but 0.25 converts them all to Puppet::Resource
instances before they're passed out of the compiler,
so the Rails integration had to be changed to expect that.

There's still some muddling, because the rails resources
only generate parser resources, but that works for now
because that's what we expect when collecting resources.

Signed-off-by: Luke Kanies <luke@madstop.com>

1 files changed, 6 insertions, 6 deletions

diff --git a/lib/puppet/rails/resource.rb b/lib/puppet/rails/resource.rb
index 81505cbd4..86a26ee39 100644
--- a/lib/puppet/rails/resource.rb
+++ b/lib/puppet/rails/resource.rb
@@ -100,8 +100,8 @@ class Puppet::Rails::Resource < ActiveRecord::Base
 
     def merge_parameters(resource)
         catalog_params = {}
-        resource.eachparam do |param|
-            catalog_params[param.name.to_s] = param
+        resource.each do |param, value|
+            catalog_params[param.to_s] = value
         end
 
         db_params = {}
@@ -123,7 +123,7 @@ class Puppet::Rails::Resource < ActiveRecord::Base
         db_params.each do |name, value_hashes|
             values = value_hashes.collect { |v| v['value'] }
 
-            unless value_compare(catalog_params[name].value, values)
+            unless value_compare(catalog_params[name], values)
                 value_hashes.each { |v| deletions << v['id'] }
             end
         end
@@ -132,12 +132,12 @@ class Puppet::Rails::Resource < ActiveRecord::Base
         Puppet::Rails::ParamValue.delete(deletions) unless deletions.empty?
 
         # Lastly, add any new parameters.
-        catalog_params.each do |name, param|
+        catalog_params.each do |name, value|
             next if db_params.include?(name)
-            values = param.value.is_a?(Array) ? param.value : [param.value]
+            values = value.is_a?(Array) ? value : [value]
 
             values.each do |v|
-                param_values.build(:value => serialize_value(v), :line => param.line, :param_name => Puppet::Rails::ParamName.accumulate_by_name(name))
+                param_values.build(:value => serialize_value(v), :line => resource.line, :param_name => Puppet::Rails::ParamName.accumulate_by_name(name))
             end
         end
     end