Order of the classes parameters is refactored

Order and intendation of those parameters are changed
to follow Puppet Style Guide recommendation [0].
Moreover, it will allow to an user to find much faster
a variable in a list of variables.

[0]. https://docs.puppetlabs.com/guides/style_guide.html

Change-Id: I4457f7c6dd3cfca24084a252f360c45d1aa2b2d7

manifests/agent.pp

@@ -23,7 +23,7 @@
 #   String. Path to agent config template.
 #   Default: 'logstash/agent.conf.erb'
 class logstash::agent (
-  $conf_template = 'logstash/agent.conf.erb'
+  $conf_template = 'logstash/agent.conf.erb',
 ) {
   warning('This class is deprecated and logstash::indexer should be used instead')
   class { '::logstash::indexer':

manifests/curator.pp

@@ -15,10 +15,10 @@
 # ELK curator
 #
 class logstash::curator (
-  $keep_for_days  = '14',
-  $pin_for_old_es = false,
   $cron_hour      = '2',
   $cron_minute    = '0',
+  $keep_for_days  = '14',
+  $pin_for_old_es = false,
 ) {
 
   if ($pin_for_old_es) {

manifests/indexer.pp

@@ -22,7 +22,7 @@
 #   String. Path to indexer config template.
 #   Default: 'logstash/indexer.conf.erb'
 class logstash::indexer (
-  $conf_template = 'logstash/indexer.conf.erb'
+  $conf_template = 'logstash/indexer.conf.erb',
 ) {
   include ::logstash
 

manifests/web.pp

@@ -40,11 +40,11 @@
 #   Default: false
 #
 class logstash::web (
-  $vhost_name = $::fqdn,
-  $serveradmin = "webmaster@${::fqdn}",
-  $frontend = 'internal',
-  $discover_nodes = ['localhost:9200'],
-  $proxy_elasticsearch = false
+  $discover_nodes      = ['localhost:9200'],
+  $frontend            = 'internal',
+  $proxy_elasticsearch = false,
+  $serveradmin         = "webmaster@${::fqdn}",
+  $vhost_name          = $::fqdn,
 ) {
   include ::httpd
   httpd_mod { 'rewrite':