diff options
Diffstat (limited to 'lib/puppet')
| -rw-r--r-- | lib/puppet/parameter.rb | 16 | ||||
| -rw-r--r-- | lib/puppet/property.rb | 4 |
2 files changed, 17 insertions, 3 deletions
diff --git a/lib/puppet/parameter.rb b/lib/puppet/parameter.rb index 04b3aec30..ae152de56 100644 --- a/lib/puppet/parameter.rb +++ b/lib/puppet/parameter.rb @@ -286,6 +286,13 @@ class Puppet::Parameter define_method(:unsafe_munge, &block) end + # Does the parameter supports reverse munge? + # This will be called when something wants to access the parameter + # in a canonical form different to what the storage form is. + def unmunge(&block) + define_method(:unmunge, &block) + end + # Mark whether we're the namevar. def isnamevar @isnamevar = true @@ -446,6 +453,11 @@ class Puppet::Parameter self.class.value_collection.munge(value) end + # no unmunge by default + def unmunge(value) + value + end + # A wrapper around our munging that makes sure we raise useful exceptions. def munge(value) begin @@ -482,7 +494,9 @@ class Puppet::Parameter @resource = nil end - attr_reader :value + def value + unmunge(@value) + end # Store the value provided. All of the checking should possibly be # late-binding (e.g., users might not exist when the value is assigned diff --git a/lib/puppet/property.rb b/lib/puppet/property.rb index 3bb1a4f0c..a144f28d0 100644 --- a/lib/puppet/property.rb +++ b/lib/puppet/property.rb @@ -332,9 +332,9 @@ class Puppet::Property < Puppet::Parameter [self.class.name, @resource.name] end if match_all? - return @should + return @should.collect { |val| self.unmunge(val) } else - return @should[0] + return self.unmunge(@should[0]) end else return nil |