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. > Signed-off-by: Hector Rivas Gandara <[email protected]> > --- > Local-branch: feature/master/5432 > lib/puppet/provider/aixobject.rb | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/lib/puppet/provider/aixobject.rb > b/lib/puppet/provider/aixobject.rb > index ae5180d..d98e36b 100755 > --- a/lib/puppet/provider/aixobject.rb > +++ b/lib/puppet/provider/aixobject.rb > @@ -146,13 +146,13 @@ class Puppet::Provider::AixObject < Puppet::Provider > end > > # 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(',') Regards, Daniel -- ✉ Daniel Pittman <[email protected]> ⌨ [email protected] (XMPP) ☎ +1 503 893 2285 ♻ made with 100 percent post-consumer electrons -- 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.
