Merge pull request #302 from bmjen/cuz-strict-variables

Morgan Haskel · Morgan Haskel · commit 6aebeea18716 · 2015-04-20T14:00:06.000-07:00
fix for strict variables checking
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -80,14 +80,15 @@
   case $warn {
     true: {
       $warn_message = $default_warn_message
-      $append_header = true
+      $_append_header = true
     }
     false: {
       $warn_message = ''
+      $_append_header = false
     }
     default: {
       $warn_message = $warn
-      $append_header = true
+      $_append_header = true
     }
   }
 
@@ -104,7 +105,7 @@
       validate_cmd => $validate_cmd,
     }
 
-    if $append_header {
+    if $_append_header {
       file_fragment { "#{$name}_header":
         tag     => $safe_name,
         content => $warn_message,

Original file line number	Diff line number	Diff line change
`@@ -80,14 +80,15 @@`
`80`	`80`	`case $warn {`
`81`	`81`	`true: {`
`82`	`82`	`$warn_message = $default_warn_message`
`83`		`- $append_header = true`
	`83`	`+ $_append_header = true`
`84`	`84`	`}`
`85`	`85`	`false: {`
`86`	`86`	`$warn_message = ''`
	`87`	`+ $_append_header = false`
`87`	`88`	`}`
`88`	`89`	`default: {`
`89`	`90`	`$warn_message = $warn`
`90`		`- $append_header = true`
	`91`	`+ $_append_header = true`
`91`	`92`	`}`
`92`	`93`	`}`
`93`	`94`
`@@ -104,7 +105,7 @@`
`104`	`105`	`validate_cmd => $validate_cmd,`
`105`	`106`	`}`
`106`	`107`
`107`		`- if $append_header {`
	`108`	`+ if $_append_header {`
`108`	`109`	`file_fragment { "#{$name}_header":`
`109`	`110`	`tag => $safe_name,`
`110`	`111`	`content => $warn_message,`