Issue #1792 has been reported by nigelk2.

----------------------------------------
Bug #1792: ralsh shouldn't report on properties that return nil
http://projects.reductivelabs.com/issues/show/1792

Author: nigelk2
Status: Unreviewed
Priority: Normal
Assigned to: 
Category: 
Target version: 
Complexity: Unknown
Affected version: 0.24.6
Keywords: 


Currently ralsh does this test for non-existent properties.

<pre>
if value == "" or value == []
</pre>

(which could probably be replaced with value.empty?)

but it seems like it should ignore nil at least, and possibly :absent.

The question of :absent depends on what it is that ralsh is meant to do....

If you say do:
<pre>
ralsh file /no-such-file
</pre>

should it return a manifest of the form:

<pre>
file { "/no-such-file":
  ensure => absent,
}
</pre>

? 

It depends whether ralsh is intended to model non-existent objects as well or 
not, so I thought I'd throw this out as a question before submitting a patch.




----------------------------------------
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://reductivelabs.com/redmine/my/account

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" 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-bugs?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to