Issue #16116 has been updated by Henrik Lindberg.
Looked at the logic again. You are currently allowed to set the value of a hash key if it is not already set. For an array, it is always allowed to set a value. So, this is both inconsistent and really bad. ---------------------------------------- Bug #16116: Arrays and hashes are mutable; new members can be added and existing values changed https://projects.puppetlabs.com/issues/16116#change-83878 Author: Nick Fagerlund Status: Accepted Priority: Normal Assignee: Category: language Target version: 3.x Affected Puppet version: Keywords: Branch: Courtesy Henrik: Puppet doesn't let you reassign variables within a given scope, but it lets you change the values of hashes and arrays by mucking with their members. That seems wrong. $myary = ['zero', 'one', 'two'] $myhash = {first => 'one', second => 'two', third => 'three'} $myary[3] = 'three' #can insert new element $myary[3] = 'something else' #can reassign notice($myary[3]) $myhash[fourth] = 'four' #can insert new element $myhash[fourth] = 'something else' #can't reassign, fails compilation notice($myhash[fourth]) $myary += ['four'] #can't append in same scope, fails compilation AND it's inconsistent, so obviously SOMETHING is whacked, even if we actually do want these objects to be mutable. I don't know what we should do about this. I think the extent to which this is being used in the wild is completely unknown. -- 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.
