alphebetize params and variables. begin adding parameter documentation to init

manifests/init.pp

@@ -5,9 +5,41 @@
 # delivery and an SMTP server listening on the loopback interface.
 #
 # Parameters:
-# - *$postfix_smtp_listen*: address on which the smtp service will listen to.
-#      defaults to 127.0.0.1
-# - *$root_mail_recipient*: who will recieve root's emails. defaults to 'nobody'
+# [*inet_interfaces*]     - (string)
+#
+# [*mail_user*]           - (string) The mail user
+#
+# [*mailman*]             - (boolean)
+#
+# [*master_smtp*]         - (string)
+#
+# [*master_smtps*]        - (string)
+#
+# [*master_submission*]   - (string)
+#
+# [*mta*]                 - (boolean)
+#
+# [*mydestination*]       - (string)
+#
+# [*mynetworks*]          - (string)
+#
+# [*myorigin*]            - (string)
+#
+# [*relayhost*]           - (string)
+#
+# [*root_mail_recipient*] - (string)
+#
+# [*satellite*]           - (boolean)
+#
+# [*smtp_listen*]         - (string)
+#
+# [*use_amavisd*]         - (boolean)
+#
+# [*use_dovecot_lda*]     - (boolean)
+#
+# [*use_schleuder*]       - (boolean)
+#
+# [*use_sympa*]           - (boolean)
 #
 # Example usage:
 #
@@ -17,44 +49,47 @@
 #   }
 #
 class postfix (
-  $smtp_listen = '127.0.0.1',         # postfix_smtp_listen
+  $inet_interfaces     = 'localhost',
+  $mail_user           = 'vmail',     # postfix_mail_user
+  $mailman             = false,
+  $master_smtp         = undef,       # postfix_master_smtp
+  $master_smtps        = undef,       # postfix_master_smtps
+  $master_submission   = undef,       # postfix_master_submission
+  $mta                 = false,
+  $mydestination       = undef,       # postfix_mydestination
+  $mynetworks          = undef,       # postfix_mynetworks
+  $myorigin            = $::fqdn,
+  $relayhost           = undef,       # postfix_relayhost
   $root_mail_recipient = 'nobody',    # root_mail_recipient
-  $use_amavisd = false,               # postfix_use_amavisd
-  $use_dovecot_lda = false,           # postfix_use_dovecot_lda
-  $use_schleuder = false,             # postfix_use_schleuder
-  $use_sympa = false,                 # postfix_use_sympa
-  $mail_user = 'vmail',               # postfix_mail_user
-  $myorigin = $::fqdn,
-  $inet_interfaces = 'localhost',
-  $master_smtp = undef,               # postfix_master_smtp
-  $master_smtps = undef,              # postfix_master_smtps
-  $master_submission = undef,         # postfix_master_submission
-  $relayhost = undef,                 # postfix_relayhost
-  $mydestination = undef,             # postfix_mydestination
-  $mynetworks = undef,                # postfix_mynetworks
-  $mta = false,
-  $satellite = false,
-  $mailman = false,
+  $satellite           = false,
+  $smtp_listen         = '127.0.0.1', # postfix_smtp_listen
+  $use_amavisd         = false,       # postfix_use_amavisd
+  $use_dovecot_lda     = false,       # postfix_use_dovecot_lda
+  $use_schleuder       = false,       # postfix_use_schleuder
+  $use_sympa           = false,       # postfix_use_sympa
 ) inherits postfix::params {
 
-  validate_string($smtp_listen)
-  validate_string($root_mail_recipient)
+
+  validate_bool($mailman)
+  validate_bool($mta)
+  validate_bool($satellite)
   validate_bool($use_amavisd)
   validate_bool($use_dovecot_lda)
   validate_bool($use_schleuder)
   validate_bool($use_sympa)
-  validate_string($mail_user)
-  validate_string($myorigin)
+
   validate_string($inet_interfaces)
+  validate_string($mail_user)
   validate_string($master_smtp)
   validate_string($master_smtps)
-  validate_string($relayhost)
   validate_string($mydestination)
   validate_string($mynetworks)
+  validate_string($myorigin)
+  validate_string($relayhost)
+  validate_string($root_mail_recipient)
+  validate_string($smtp_listen)
+
 
-  validate_bool($mta)
-  validate_bool($satellite)
-  validate_bool($mailman)
 
   $_smtp_listen = $mailman ? {
     true    => '0.0.0.0',
@@ -63,18 +98,18 @@ class postfix (
 
   class { 'postfix::packages': } ->
   class { 'postfix::files':
-    smtp_listen         => $_smtp_listen,
+    inet_interfaces     => $inet_interfaces,
+    mail_user           => $mail_user,
+    master_smtp         => $master_smtp,
+    master_smtps        => $master_smtps,
+    master_submission   => $master_submission,
+    myorigin            => $myorigin,
     root_mail_recipient => $root_mail_recipient,
+    smtp_listen         => $_smtp_listen,
     use_amavisd         => $use_amavisd,
     use_dovecot_lda     => $use_dovecot_lda,
     use_schleuder       => $use_schleuder,
     use_sympa           => $use_sympa,
-    mail_user           => $mail_user,
-    myorigin            => $myorigin,
-    inet_interfaces     => $inet_interfaces,
-    master_smtp         => $master_smtp,
-    master_smtps        => $master_smtps,
-    master_submission   => $master_submission,
     } ~>
   class { 'postfix::service': } ->
   Class['postfix']