diff options
author | Brice Figureau <brice-puppet@daysofwonder.com> | 2010-03-03 20:35:46 +0100 |
---|---|---|
committer | James Turnbull <james@lovedthanlost.net> | 2010-03-25 11:59:12 +1100 |
commit | 4b2b9ebfb566776373f48357e9df61a88b410faa (patch) | |
tree | 53eda77f4d1d3e15a5129728ba97a5f10358146e /lib/puppet/parser/ast/selector.rb | |
parent | 19863c07f983ec181fb81c797ee8b9c8d335e18c (diff) | |
download | puppet-4b2b9ebfb566776373f48357e9df61a88b410faa.tar.gz puppet-4b2b9ebfb566776373f48357e9df61a88b410faa.tar.xz puppet-4b2b9ebfb566776373f48357e9df61a88b410faa.zip |
Fix #3229 - use original value in case/selector regex matching
The issue is that case/selectors are downcasing the value before it
is compared to the options.
Unfortunately regex are matching in a case sensitive way, which would
make the following manifest fail:
$var = "CaseSensitive"
case $var {
/CaseSensitive/: {
notice("worked")
}
default: {
fail "miserably"
}
}
This patch fixes the issue by making sure the regexp match is done
one the original (not downcased) value, but still doing a case
sensitive match.
Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
Diffstat (limited to 'lib/puppet/parser/ast/selector.rb')
-rw-r--r-- | lib/puppet/parser/ast/selector.rb | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lib/puppet/parser/ast/selector.rb b/lib/puppet/parser/ast/selector.rb index 84bc2a74a..ce834b63b 100644 --- a/lib/puppet/parser/ast/selector.rb +++ b/lib/puppet/parser/ast/selector.rb @@ -17,8 +17,6 @@ class Puppet::Parser::AST sensitive = Puppet[:casesensitive] - paramvalue = paramvalue.downcase if not sensitive and paramvalue.respond_to?(:downcase) - default = nil unless @values.instance_of? AST::ASTArray or @values.instance_of? Array |