X-Git-Url: https://git.openstreetmap.org./chef.git/blobdiff_plain/09e5083a36632d53e388446d0517a5c2bccd2b24..1f0495ace8de757939fde8a77b8070ab3d5f2681:/cookbooks/apache/providers/module.rb

diff --git a/cookbooks/apache/providers/module.rb b/cookbooks/apache/providers/module.rb
index 29ef1621c..252b5e816 100644
--- a/cookbooks/apache/providers/module.rb
+++ b/cookbooks/apache/providers/module.rb
@@ -21,69 +21,60 @@ def whyrun_supported?
   true
 end
 
+use_inline_resources
+
 action :install do
-  p = package package_name do
-    action :install
-    not_if { ::File.exists?(available_name("load")) }
+  unless installed?
+    package package_name
   end
 
-  updated = p.updated_by_last_action?
-
-  if new_resource.conf
-    t = template available_name("conf") do
+  if new_resource.conf # ~FC023
+    template available_name("conf") do
       source new_resource.conf
       owner "root"
       group "root"
-      mode 0644
+      mode 0o644
       variables new_resource.variables
-      notifies :reload, "service[apache2]" if enabled?
     end
-
-    updated = updated || t.updated_by_last_action?
   end
-
-  new_resource.updated_by_last_action(updated)
 end
 
 action :enable do
-  l = link enabled_name("load") do
+  execute "a2enmod-#{new_resource.name}" do
+    command "a2enmod #{new_resource.name}"
+    user "root"
+    group "root"
+    not_if { ::File.exist?(enabled_name("load")) }
+  end
+
+  link enabled_name("load") do
     to available_name("load")
     owner "root"
     group "root"
-    notifies :restart, "service[apache2]"
   end
 
-  c = link enabled_name("conf") do
+  link enabled_name("conf") do
     to available_name("conf")
     owner "root"
     group "root"
-    notifies :reload, "service[apache2]"
-    only_if { ::File.exists?(available_name("conf")) }
+    only_if { ::File.exist?(available_name("conf")) }
   end
-
-  new_resource.updated_by_last_action(l.updated_by_last_action? || c.updated_by_last_action?)
 end
 
 action :disable do
-  l = link enabled_name("load") do
+  link enabled_name("load") do
     action :delete
-    notifies :restart, "service[apache2]"
   end
 
-  c = link enabled_name("conf") do
+  link enabled_name("conf") do
     action :delete
-    notifies :reload, "service[apache2]"
   end
-
-  new_resource.updated_by_last_action(l.updated_by_last_action? || c.updated_by_last_action?)
 end
 
 action :delete do
-  p = package package_name do
+  package package_name do
     action :remove
   end
-
-  new_resource.updated_by_last_action(p.updated_by_last_action?)
 end
 
 def package_name
@@ -98,6 +89,6 @@ def enabled_name(extension)
   "/etc/apache2/mods-enabled/#{new_resource.name}.#{extension}"
 end
 
-def enabled?
-  ::File.exists?(enabled_name("load"))
+def installed?
+  ::File.exist?(available_name("load"))
 end