On Mon, Jan 3, 2011 at 8:58 PM, Daniel Pittman <[email protected]> wrote:
> On Mon, Jan 3, 2011 at 02:21, Hector Rivas Gandara <[email protected]> wrote:
>> From: Hector Rivas Gandara <[email protected]>
>
> This could do with a better comment about what the change is for.
I can rewrite the comments, but I do not know if you merged the branch.
>> # convert it to string
>> - if new_val.is_a? Array
>> - new_val = new_val.join(",")
>> + if new_value.is_a? Array
>> + new_value = new_value.join(",")
>> else
>> - new_val = new_val.to_s
>> + new_value = new_value.to_s
>> end
>
> Perhaps this is more elegant, and should work for most anything?
>
> new_value = Array(new_value).join(',')
You are right, I will change it.
--
Atentamente
Héctor Rivas
--
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.
