Clean up and document apache2::module

diff --git a/modules/apache2/manifests/module.pp b/modules/apache2/manifests/module.pp
index 3862353..e3cb3c5 100644 (file)
--- a/modules/apache2/manifests/module.pp
+++ b/modules/apache2/manifests/module.pp
@@ -1,19 +1,24 @@
-define apache2::module ($ensure = present) {
+# Enable an apache module
+#
+# @param ensure  present or absent
+define apache2::module (
+  Enum['present','absent'] $ensure = 'present',
+) {
   case $ensure {
     present: {
       exec { "/usr/sbin/a2enmod ${name}":
         creates => "/etc/apache2/mods-enabled/${name}.load",
         require => Package['apache2'],
-        notify  => Service['apache2']
+        notify  => Service['apache2'],
       }
     }
     absent: {
       exec { "/usr/sbin/a2dismod ${name}":
         onlyif  => "test -L /etc/apache2/mods-enabled/${name}.load",
         require => Package['apache2'],
-        notify  => Service['apache2']
+        notify  => Service['apache2'],
       }
     }
-    default: { fail ( "Unknown ensure value: '$ensure'" ) }
+    default: { fail ( "Unknown ensure value: ${ensure}" ) }
   }
 }