Fix the checkcommand template vars attribute handling

so the vars are written correctly into the CheckCommand object configuration. Before
the fix the template produced the following CheckCommand object configruation for
the vars attribute:

  vars += {
    vars.http_address = "$address$",
    vars.http_ssl     = false,
    vars.http_sni     = false
  }

instead of the desired

  vars.http_address = "$address$",
  vars.http_ssl     = false,
  vars.http_sni     = false

Author: n00by

README.md

@@ -640,9 +640,9 @@ icinga2::object::checkcommand { 'check_http':
     '"-e"' => '"$http_expect$"'
   },
   vars => {
-    'vars.http_address' => '"$address$"',
-    'vars.http_ssl'     => 'false',
-    'vars.http_sni'     => 'false'
+    http_address => '"$address$"',
+    http_ssl     => false,
+    http_sni     => false
   }
 }
 </pre>

templates/object/checkcommand.conf.erb

@@ -1,4 +1,3 @@
-
 /**
 * WARNING: This CheckCommand definition is automatically generated by Puppet.
 * ANY MANUAL CHANGES TO IT WILL GET OVERWRITTEN!
@@ -25,7 +24,9 @@ object CheckCommand "<%= @object_checkcommandname %>" {
   <%- end -%>
   <%- if @vars.any? -%>
 
-  vars += <%= scope.function_icinga2_config_value([@vars]) %>
+  <%- @vars.each do |key, val| -%>
+  vars.<%= key -%> = <%= val %>
+  <%- end -%>
   <%- end -%>
   <%- if @timeout -%>