On Jul 18, 2009, at 4:13 AM, Brice Figureau wrote:

>
> Puppetdoc wasn't correctly transforming AST::Boolean to string,
> producing a RDoc error which needs a string.
>
> Signed-off-by: Brice Figureau <[email protected]>
> ---
> lib/puppet/util/rdoc/parser.rb |    4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/puppet/util/rdoc/parser.rb b/lib/puppet/util/rdoc/ 
> parser.rb
> index 7954865..479bb92 100644
> --- a/lib/puppet/util/rdoc/parser.rb
> +++ b/lib/puppet/util/rdoc/parser.rb
> @@ -426,9 +426,9 @@ class Parser
>         value = value.children if value.is_a? 
> (Puppet::Parser::AST::ASTArray)
>         if value.is_a?(Array)
>             "['#{value.join(", ")}']"
> -        elsif [:true, true, "true"].include?(value)
> +        elsif [:true, true, "true"].include?(value.to_s)
>             "true"
> -        elsif [:false, false, "false"].include?(value)
> +        elsif [:false, false, "false"].include?(value.to_s)
>             "false"

Can't we just do 'elsif value.to_s == "true"' and such?

>
>         else
>             value.to_s
> -- 
> 1.6.0.2
>
>
> >


-- 
I do not feel obliged to believe that the same God who has endowed us
with sense, reason, and intellect has intended us to forgo their use.
     -- Galileo Galilei
---------------------------------------------------------------------
Luke Kanies | http://reductivelabs.com | http://madstop.com


--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to