Move stringify_facts to config.pp ([main] section in puppet.conf) only

Author: tampakrap

manifests/agent.pp

@@ -47,7 +47,6 @@
   $usecacheonfailure = true,
   $method            = $puppet::params::default_method,
   $manage_package    = true,
-  $stringify_facts   = false,
   $package           = $puppet::params::agent_package,
 ) inherits puppet::params {
 
@@ -58,7 +57,6 @@
   validate_bool($splay)
   validate_bool($usecacheonfailure)
   validate_bool($manage_package)
-  validate_bool($stringify_facts)
 
   include puppet
 

manifests/agent/config.pp

@@ -76,9 +76,4 @@
     setting => 'usecacheonfailure',
     value   => $puppet::agent::usecacheonfailure,
   }
-
-  ini_setting { 'stringify_facts_agent':
-    setting => 'stringify_facts',
-    value   => $puppet::agent::stringify_facts;
-  }
 }

manifests/config.pp

@@ -42,4 +42,9 @@
     setting => 'srv_domain',
     value   => $puppet::srv_domain,
   }
+
+  ini_setting { 'stringify_facts':
+    setting => 'stringify_facts',
+    value   => $puppet::stringify_facts,
+  }
 }

manifests/init.pp

@@ -18,9 +18,11 @@
   $conf            = $puppet::params::puppet_conf,
   $use_srv_records = false,
   $srv_domain      = $::domain,
+  $stringify_facts = false
 ) inherits puppet::params {
 
   validate_bool($use_srv_records)
+  validate_bool($stringify_facts)
 
   include puppet::config
 

manifests/server.pp

@@ -54,7 +54,6 @@
   $serverssl_protos  = undef,
   $servertype        = 'unicorn',
   $storeconfigs      = undef,
-  $stringify_facts   = false,
   $package           = $puppet::params::master_package,
 ) inherits puppet::params {
 

manifests/server/config.pp

@@ -95,11 +95,6 @@
     'group':
       setting => 'group',
       value   => $puppet::group;
-
-    'stringify_facts_master':
-      setting => 'stringify_facts',
-      section => 'main',
-      value   => $puppet::server::stringify_facts;
   }
 
   ini_setting { 'ca':