Issue #8561 has been updated by Randall Hansen.
Do you mean that we currently pass all three, or the one that we receive? Regardless, this sounds like a fine change to me. ---------------------------------------- Refactor #8561: Always pass canonical option names to actions https://projects.puppetlabs.com/issues/8561 Author: Daniel Pittman Status: Unreviewed Priority: Normal Assignee: Daniel Pittman Category: Faces Target version: Affected Puppet version: Keywords: Branch: At the moment, if an option has multiple aliases, we pass whatever alias is received through to the action. This proposes to change that such that the canonical name of the option is passed, regardless of how it is input. Concretely, given this action: <pre> action :foo do option "--bar", "--baz", "-b" end </pre> ...and these invocations: <pre> face.foo :bar => 1 face.foo :baz => 1 face.foo :b => 1 </pre> We would currently get `{ :bar => 1 }, { :baz => 1 }, { :b => 1 }` passed to the action. After the change this would be `{ :bar => 1 }` in every single case, given `bar` is the canonical name of the option. -- 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 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-bugs?hl=en.
