Merge remote-tracking branch 'github/pull/704'

[chef.git] / cookbooks / mediawiki / resources / site.rb

diff --git a/cookbooks/mediawiki/resources/site.rb b/cookbooks/mediawiki/resources/site.rb
index 34bec44e0a8a5cd98393ac3a5bb1af96e868ab65..0bbddfe8174e2e3c869450f775afe99f28e768df 100644 (file)
--- a/cookbooks/mediawiki/resources/site.rb
+++ b/cookbooks/mediawiki/resources/site.rb
@@ -23,7 +23,6 @@ default_action :create
 
 property :site, :kind_of => String, :name_property => true
 property :aliases, :kind_of => [String, Array]
-property :directory, :kind_of => String
 property :version, :kind_of => String, :default => "1.39"
 property :database_name, :kind_of => String, :required => true
 property :database_user, :kind_of => String, :required => [:create, :update]
@@ -61,6 +60,7 @@ action :create do
 
   mysql_user "#{new_resource.database_user}@localhost" do
     password new_resource.database_password
+    reload true
   end
 
   mysql_database new_resource.database_name do
@@ -162,11 +162,6 @@ action :create do
     only_if { ::File.exist?("#{mediawiki_directory}/LocalSettings.php") }
   end
 
-  service "mediawiki-refresh-links@#{new_resource.site}.timer" do
-    action [:enable, :start]
-    only_if { ::File.exist?("#{mediawiki_directory}/LocalSettings.php") }
-  end
-
   template "/etc/cron.daily/mediawiki-#{cron_name}-backup" do
     cookbook "mediawiki"
     source "mediawiki-backup.cron.erb"
@@ -343,10 +338,10 @@ action :create do
     update_site false
   end
 
+  # Extension has been archived: https://www.mediawiki.org/wiki/Extension:LocalisationUpdate
   mediawiki_extension "LocalisationUpdate" do
     site new_resource.site
-    template "mw-ext-LocalisationUpdate.inc.php.erb"
-    update_site false
+    action :delete
   end
 
   # mediawiki_extension "Translate" do
@@ -531,6 +526,7 @@ action :update do
     cwd mediawiki_directory
     user node[:mediawiki][:user]
     group node[:mediawiki][:group]
+    timeout 86400
   end
 end
 
@@ -558,7 +554,7 @@ action_class do
   include OpenStreetMap::Mixin::PersistentToken
 
   def site_directory
-    new_resource.directory || "/srv/#{new_resource.site}"
+    "/srv/#{new_resource.site}"
   end
 
   def mediawiki_reference