properly implement $service_restart, refs #17

Author: fraenki

manifests/init.pp

@@ -213,12 +213,9 @@
   Optional[Integer] $gid = undef,
   Optional[Integer] $uid = undef,
 ) {
-  if ($service_restart) {
-    Class['activemq::install'] ~> Class['activemq::service']
-  }
-
   # Perform basic installation steps
   class { 'activemq::install': }
+  ~> class { 'activemq::service': }
 
   # Ensure that a valid cluster configuration is provided.
   if ($cluster and (empty($cluster_name) or !($cluster_name =~ String))) {

manifests/instance.pp

@@ -707,11 +707,13 @@
     }
 
     # Configure service for this instance.
+    $_restart = $activemq::service_restart ? { true => undef, default => ':' }
     service { $instance_service:
       ensure    => $_service_ensure,
       enable    => $_service_enable,
       subscribe => $_service_subscribe,
       require   => $_service_require,
+      restart   => $_restart,
     }
   }
 }