From db5df50d6d2d2ec1ae42a91115f3372d2f1498a2 Mon Sep 17 00:00:00 2001
From: Stephen Gran <steve@lobefin.net>
Date: Sat, 17 Oct 2009 15:49:42 +0100
Subject: [PATCH] Rework apache2 module enable command to allow for disabling
 modules and restarting apache on change Signed-off-by: Stephen Gran
 <steve@lobefin.net>

---
 modules/apache2/manifests/init.pp | 24 ++++++++++++++++++++----
 1 file changed, 20 insertions(+), 4 deletions(-)

diff --git a/modules/apache2/manifests/init.pp b/modules/apache2/manifests/init.pp
index e1eaf300..7884116a 100644
--- a/modules/apache2/manifests/init.pp
+++ b/modules/apache2/manifests/init.pp
@@ -1,5 +1,21 @@
-define enable_module($module) {
-        exec { "/usr/sbin/a2enmod $module": }
+define enable_module($ensure=present, $require=apache2) {
+        case $ensure {
+                present: {
+                        exec { "/usr/sbin/a2enmod $name":
+                                unless => "/bin/sh -c '[ -L /etc/apache2/mods-enabled/${name}.load ]',
+                                notify => Exec["force-reload-apache2"],
+                                require => Package[$require],
+                        }
+                }
+                absent: {
+                        exec { "/usr/sbin/a2dismod $name":
+                                onlyif => "/bin/sh -c '[ -L /etc/apache2/mods-enabled/${name}.load ]',
+                                notify => Exec["force-reload-apache2"],
+                                require => Package[$require],
+                        }
+                }
+                default: { err ( "Unknown ensure value: '$ensure'" ) }
+         }
 }
 
 class apache2 {
@@ -11,8 +27,8 @@ class apache2 {
 	}
 
         enable_module {
-                "info": module => info;
-                "status": module => status;
+                "info":;
+                "status":;
         }
 
 	file {
-- 
2.39.2