From 2cb4e0f46da1c28af5453f3767e6d657d125dc1e Mon Sep 17 00:00:00 2001
From: Timo Goebel <timo.goebel@dm.de>
Date: Tue, 28 Apr 2015 16:23:20 +0200
Subject: [PATCH] remove settings when value is not set

---
 manifests/config/roles.pp | 85 +++++++++++++++++++++++++++++----------
 1 file changed, 63 insertions(+), 22 deletions(-)

diff --git a/manifests/config/roles.pp b/manifests/config/roles.pp
index 5d8ecd7..7b4b142 100644
--- a/manifests/config/roles.pp
+++ b/manifests/config/roles.pp
@@ -2,10 +2,10 @@
 #
 define icingaweb2::config::roles (
   $role_groups         = undef,
-  $role_host_filter    = absent,
+  $role_host_filter    = undef,
   $role_name           = $title,
   $role_permissions    = undef,
-  $role_service_filter = absent,
+  $role_service_filter = undef,
   $role_users          = undef,
 ) {
   Ini_Setting {
@@ -14,34 +14,75 @@ define icingaweb2::config::roles (
     path    => "${::icingaweb2::config_dir}/roles.ini",
   }
 
-  ini_setting { "icingaweb2 roles ${title} users":
-    section => $role_name,
-    setting => 'users',
-    value   => "\"${role_users}\"",
+  if $role_users {
+    ini_setting { "icingaweb2 roles ${title} users":
+      section => $role_name,
+      setting => 'users',
+      value   => "\"${role_users}\"",
+    }
+  } else {
+    ini_setting { "icingaweb2 roles ${title} users":
+      ensure  => absent,
+      section => $role_name,
+      setting => 'users',
+    }
   }
 
-  ini_setting { "icingaweb2 roles ${title} groups":
-    section => $role_name,
-    setting => 'groups',
-    value   => "\"${role_groups}\"",
+  if $role_groups {
+    ini_setting { "icingaweb2 roles ${title} groups":
+      section => $role_name,
+      setting => 'groups',
+      value   => "\"${role_groups}\"",
+    }
+  } else {
+    ini_setting { "icingaweb2 roles ${title} groups":
+      ensure  => absent,
+      section => $role_name,
+      setting => 'groups',
+    }
+
   }
 
-  ini_setting { "icingaweb2 roles ${title} permissions":
-    section => $role_name,
-    setting => 'permissions',
-    value   => "\"${role_permissions}\"",
+  if $role_permissions {
+    ini_setting { "icingaweb2 roles ${title} permissions":
+      section => $role_name,
+      setting => 'permissions',
+      value   => "\"${role_permissions}\"",
+    }
+  } else {
+    ini_setting { "icingaweb2 roles ${title} permissions":
+      ensure  => absent,
+      section => $role_name,
+      setting => 'permissions',
+    }
   }
 
-  ini_setting { "icingaweb2 roles ${title} host filter":
-    section => $role_name,
-    setting => 'monitoring/hosts/filter',
-    value   => "\"${role_host_filter}\"",
+  if $role_host_filter {
+    ini_setting { "icingaweb2 roles ${title} host filter":
+      section => $role_name,
+      setting => 'monitoring/hosts/filter',
+      value   => "\"${role_host_filter}\"",
+    }
+  } else {
+    ini_setting { "icingaweb2 roles ${title} host filter":
+      ensure  => absent,
+      section => $role_name,
+      setting => 'monitoring/hosts/filter',
+    }
   }
 
-  ini_setting { "icingaweb2 roles ${title} service filter":
-    section => $role_name,
-    setting => 'monitoring/services/filter',
-    value   => "\"${role_service_filter}\"",
+  if $role_service_filter {
+    ini_setting { "icingaweb2 roles ${title} service filter":
+      section => $role_name,
+      setting => 'monitoring/services/filter',
+      value   => "\"${role_service_filter}\"",
+    }
+  } else {
+    ini_setting { "icingaweb2 roles ${title} service filter":
+      ensure  => absent,
+      section => $role_name,
+      setting => 'monitoring/services/filter',
+    }
   }
 }