diff --git a/app/services/site_service.rb b/app/services/site_service.rb index 60ccd907..90dbae18 100644 --- a/app/services/site_service.rb +++ b/app/services/site_service.rb @@ -96,19 +96,21 @@ SiteService = Struct.new(:site, :usuarie, :params, keyword_init: true) do end def rename(name) - return if name == site.name + return if name == site.name + moved = false - site.name = name + site.name = name Site.transaction do - raise ActiveRecord::Rollback if File.exists?(site.path) - FileUtils.mv (site.path_was, site.path) + raise ActiveRecord::Rollback if File.exist?(site.path) + + FileUtils.mv(site.path_was, site.path) moved = true ActiveStorage::Blob.where(service_name: site.name_was).update_all(service_name: site.name) - site.save - rescue StandardError - FileUtils.mv (site.path, site.path_was) if moved - raise + site.save + rescue StandardError + FileUtils.mv(site.path, site.path_was) if moved + raise end end @@ -254,7 +256,7 @@ SiteService = Struct.new(:site, :usuarie, :params, keyword_init: true) do @current_role ||= usuarie.rol_for_site(site) end - def with_all_locales(&block) + def with_all_locales site.locales.map do |locale| next unless I18n.available_locales.include? locale