fix #475 Remove default apply_target from service object

README.md

@@ -2411,7 +2411,7 @@ Dispose an apply instead an object if set to 'true'. Value is taken as statement
 i.e. 'vhost => config in host.vars.vhosts'. Defaults to false.
 
 ##### `apply_target`
-An object type on which to target the apply rule. Valid values are `Host` and `Service`. Defaults to `Host`.
+Optional fo an object type on which to target the apply rule. Valid values are `Host` and `Service`. Defaults to `Host`.
 
 ##### `import`
 A sorted list of templates to import in this object. Defaults to an empty array.

manifests/object.pp

@@ -22,7 +22,7 @@
 #   Set object_name as prefix in front of 'apply for'. Only effects if apply is a string. Defaults to false.
 #
 # [*apply_target*]
-#   An object type on which to target the apply rule. Valid values are `Host` and `Service`. Defaults to `Host`.
+#   Optional for an object type on which to target the apply rule. Valid values are `Host` and `Service`.
 #
 # [*import*]
 #   A sorted list of templates to import in this object. Defaults to an empty array.

manifests/object/service.pp

@@ -234,7 +234,6 @@
     import       => $import,
     apply        => $apply,
     prefix       => $prefix,
-    apply_target => 'Host',
     assign       => $assign,
     ignore       => $ignore,
     template     => $template,

templates/object.conf.erb

@@ -4,18 +4,18 @@
 <% if scope.lookupvar('icinga2::_constants').include?(@object_name) -%>
  <%= @object_name -%>
 <% else -%>
-  "<%= @object_name %>" <% end -%>
+ "<%= @object_name %>"<% end -%>
 <% end -%>
- for (<%= @apply %>) to <%= @apply_target -%> {
+ for (<%= @apply %>)<%if @apply_target %>to <%=apply_target %><% end %> {
 <% else -%>
 <% if @apply %>apply<% else -%>
 <% if @template %>template<% else %>object<% end -%>
 <% end %> <%= @object_type -%>
 <% if scope.lookupvar('icinga2::_constants').include?(@object_name) -%>
  <%= @object_name -%>
 <% else -%>
- "<%= @object_name %>" <% end -%>
-<% if @apply && @apply_target %>to <%= @apply_target -%><% end -%> {
+ "<%= @object_name %>"<% end -%>
+<% if @apply && @apply_target %>to <%= @apply_target %><% end -%> {
 <% end -%>
 <% @import.each do |i| -%>
   import "<%= i %>"