Issue #20923 has been updated by Andrew Parker. Status changed from In Topic Branch Pending Review to Closed Assignee set to Andrew Parker
The solution provided in the bug description takes the design of puppet functions in the wrong direction. It would end up being another patch that would provide a fix for some functions for one of the issues that surround functions (#13210, #5935, #19812) but would add to the technical debt of the project as a whole. It would also end up creating confusion from a users perspective unless they understand the implementation of the function. I think that solving the general problem is needed, but we need to look at a larger fix that will reduce our technical complexity over the long haul, resolve the odd semantics of function calls in puppet, and provide better future extensibility and testability of functions. I believe that the proposal outlined in [a previous comment](#note-16) The rejection of this feature request was discussed on IRC on [2013-05-30](http://puppetlogs.com/puppetdev/%23puppet-dev-2013-05-30.log.html) ---------------------------------------- Feature #20923: Allow hiera functions to take 'undef' as parameters without converting it to an empty string https://projects.puppetlabs.com/issues/20923#change-91851 * Author: Raphaël Pinson * Status: Closed * Priority: Normal * Assignee: Andrew Parker * 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.
