diff options
| author | Tyler Christiansen <code@tylerc.me> | 2015-12-29 23:14:30 -0800 |
|---|---|---|
| committer | Tyler Christiansen <code@tylerc.me> | 2015-12-29 23:14:30 -0800 |
| commit | 6e72e399626c0abb4cff7cea3d96505d4971e646 (patch) | |
| tree | e962408f0f62447ef09b1f1efa0c4b9c8fe46685 /lib/oxidized/config/vars.rb | |
| parent | a6f7f8de57d3c2691ff68375aeda78e880e58384 (diff) | |
Fix groups vars.
Fixes ytti/oxidized#242.
Fixes ytti/oxidized-web#38.
Added a check to verify the key exists for a group before returning.
Might fic a group issue in ytti/oxidized#171.
Diffstat (limited to 'lib/oxidized/config/vars.rb')
| -rw-r--r-- | lib/oxidized/config/vars.rb | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/lib/oxidized/config/vars.rb b/lib/oxidized/config/vars.rb index b39e7fe..b0471f2 100644 --- a/lib/oxidized/config/vars.rb +++ b/lib/oxidized/config/vars.rb @@ -2,9 +2,14 @@ module Oxidized::Config::Vars # convenience method for accessing node, group or global level user variables # nil values will be ignored def vars name - r = @node.vars[name] unless @node.vars.nil? - r ||= Oxidized::CFG.groups[@node.group].vars[name.to_s] if Oxidized::CFG.groups.has_key?(@node.group) + r = @node.vars[name] unless @node.vars.nil? + if Oxidized::CFG.groups.has_key?(@node.group) + if Oxidized::CFG.groups[@node.group].vars.has_key?(name.to_s) + r ||= Oxidized::CFG.groups[@node.group].vars[name.to_s] + end + end r ||= Oxidized::CFG.vars[name.to_s] if Oxidized::CFG.vars.has_key?(name.to_s) r end end + |