Issue #20923 has been updated by Erik Dalén.
I think it would make much more sense to pass undef as nil instead of :undef to functions. In the newer style variable access in templates (using @variable) it is already treated as nil instead of :undef, so I don't see any point in doing it differently in the function API. It would also allow ruby methods like .count to "just work". If the opt in is added as another option to the function API it would just get silently ignored by puppet versions not handling that feature though. ---------------------------------------- Bug #20923: Hiera cannot take `undef` as a default value https://projects.puppetlabs.com/issues/20923#change-91726 * Author: Raphaël Pinson * Status: In Topic Branch Pending Review * Priority: Normal * Assignee: * Category: * Target version: * Affected Puppet version: * Keywords: * Branch: https://github.com/puppetlabs/puppet/pull/1670 ---------------------------------------- As discussed with Jeff McCune yesterday, I believe #13210 is not a duplicate of #15329, as #13210 can be fixed without fixing #15329 per se. Here's the idea: * Implement a new `:honor_undef` option for functions, defaulting to `false`; * Implement a new `:hiera_munge_undef` option in puppet.conf, defaulting to `true` for backwards compatibility; * Use `:honor_undef = true` in all hiera functions when `:hiera_munge_undef` is set to `false`. The code can be found in this PR: https://github.com/puppetlabs/puppet/pull/1670 -- 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://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
