simplify email_error default logic

(by anarcat)

diff --git a/modules/bacula/manifests/init.pp b/modules/bacula/manifests/init.pp
index cf83a26..02cf506 100644 (file)
--- a/modules/bacula/manifests/init.pp
+++ b/modules/bacula/manifests/init.pp
@@ -20,7 +20,8 @@ class bacula (
   String $ssl_server_key,
 
   Optional[String] $email_all = undef,
-  String $email_error                          = $email_all != undef ? { true => $email_all, default => 'root@localhost' },
+  # default to all if defined, otherwise default to 'root' to enforce error delivery
+  String $email_error                          = $email_all ? { undef => 'root', default => $email_all },
   String $email_operator                       = $email_error,
   String $email_daemon                         = $email_error,
   Array[Stdlib::IP::Address] $public_addresses = $base::public_addresses,