Cookstyle auto-corrections

This change is automatically generated by the GitHub Cookstyle Runner.

Author: damacus

libraries/helpers.rb

@@ -244,11 +244,11 @@ def default_modules
 
         case node['platform_family']
         when 'rhel', 'fedora', 'amazon'
-          default_modules.concat %w(log_config logio unixd systemd)
+          default_modules.push('log_config', 'logio', 'unixd', 'systemd')
         when 'arch', 'freebsd'
-          default_modules.concat %w(log_config logio unixd)
+          default_modules.push('log_config', 'logio', 'unixd')
         when 'suse'
-          default_modules.concat %w(log_config logio)
+          default_modules.push('log_config', 'logio')
         else
           default_modules
         end
@@ -337,41 +337,37 @@ def default_session_cache
       end
 
       def config_file?(mod_name)
-        if %w(ldap
-              actions
-              alias
-              auth_cas
-              autoindex
-              cache_disk
-              cgid
-              dav_fs
-              deflate
-              dir
-              fastcgi
-              fcgid
-              include
-              info
-              ldap
-              mime_magic
-              mime
-              negotiation
-              pagespeed
-              proxy_balancer
-              proxy_ftp
-              proxy
-              reqtimeout
-              setenvif
-              ssl
-              status
-              userdir
-              mpm_event
-              mpm_prefork
-              mpm_worker
-        ).include?(mod_name)
-          true
-        else
-          false
-        end
+        %w(ldap
+           actions
+           alias
+           auth_cas
+           autoindex
+           cache_disk
+           cgid
+           dav_fs
+           deflate
+           dir
+           fastcgi
+           fcgid
+           include
+           info
+           ldap
+           mime_magic
+           mime
+           negotiation
+           pagespeed
+           proxy_balancer
+           proxy_ftp
+           proxy
+           reqtimeout
+           setenvif
+           ssl
+           status
+           userdir
+           mpm_event
+           mpm_prefork
+           mpm_worker
+        ).include?(mod_name) || false
       end
 
       def pagespeed_url

resources/config.rb

@@ -43,7 +43,7 @@
 
 property :docroot_dir, String,
          default: lazy { default_docroot_dir },
-         description: 'Apache document root.'\
+         description: 'Apache document root.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :timeout, [Integer, String],
@@ -57,7 +57,7 @@
 
 property :run_dir, String,
          default: lazy { default_run_dir },
-         description: ' Sets the DefaultRuntimeDir directive.'\
+         description: ' Sets the DefaultRuntimeDir directive.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :template_cookbook, String,

resources/default_site.rb

@@ -32,12 +32,12 @@
 
 property :docroot_dir, String,
          default: lazy { default_docroot_dir },
-         description: 'Apache document root.'\
+         description: 'Apache document root.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :apache_root_group, String,
          default: lazy { node['root_group'] },
-         description: 'Group that the root user on the box runs as.'\
+         description: 'Group that the root user on the box runs as.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :template_source, String,
@@ -57,16 +57,16 @@
     mode '0644'
     cookbook new_resource.template_cookbook
     variables new_resource.variables.merge({
-      access_log: default_access_log,
-      cgibin_dir: default_cgibin_dir,
-      docroot_dir: new_resource.docroot_dir,
-      error_log: default_error_log,
-      log_dir: default_log_dir,
-      log_level: new_resource.log_level,
-      port: new_resource.port,
-      server_admin: new_resource.server_admin,
-      site_name: new_resource.default_site_name,
-    })
+                                             access_log: default_access_log,
+                                             cgibin_dir: default_cgibin_dir,
+                                             docroot_dir: new_resource.docroot_dir,
+                                             error_log: default_error_log,
+                                             log_dir: default_log_dir,
+                                             log_level: new_resource.log_level,
+                                             port: new_resource.port,
+                                             server_admin: new_resource.server_admin,
+                                             site_name: new_resource.default_site_name,
+                                           })
   end
 
   apache2_site new_resource.default_site_name do

resources/install.rb

@@ -17,22 +17,22 @@
 
 property :apache_user, String,
          default: lazy { default_apache_user },
-         description: 'Set to override the default apache2 user.'\
+         description: 'Set to override the default apache2 user.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :apache_group, String,
          default: lazy { default_apache_group },
-         description: 'Set to override the default apache2 user.'\
+         description: 'Set to override the default apache2 user.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :log_dir, String,
          default: lazy { default_log_dir },
-         description: 'Log directory location.'\
+         description: 'Log directory location.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :error_log, String,
          default: lazy { default_error_log },
-         description: 'Error log location.'\
+         description: 'Error log location.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :log_level, String,
@@ -76,7 +76,7 @@
 
 property :mpm, String,
          default: lazy { default_mpm },
-         description: 'Multi-processing Module.'\
+         description: 'Multi-processing Module.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :mpm_conf, Hash,
@@ -111,12 +111,12 @@
 
 property :docroot_dir, String,
          default: lazy { default_docroot_dir },
-         description: 'Apache document root.'\
+         description: 'Apache document root.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :run_dir, String,
          default: lazy { default_run_dir },
-         description: 'Location for APACHE_RUN_DIR.'\
+         description: 'Location for APACHE_RUN_DIR.' \
 'Defaults to platform specific locations, see libraries/helpers.rb'
 
 property :access_file_name, String,