Merge pull request #23217 from agrare/service_template_ansible_playbo…

…ok_private_send

Make ServiceTemplateAnsiblePlaybook create_dialogs public

(cherry picked from commit 1521858)

app/models/service_template_ansible_playbook.rb

@@ -42,7 +42,7 @@ def self.create_catalog_item(options, _auth_user)
 
     transaction do
       create_from_options(options).tap do |service_template|
-        dialog_ids = service_template.send(:create_dialogs, config_info)
+        dialog_ids = service_template.create_dialogs(config_info)
         config_info.deep_merge!(dialog_ids)
         service_template.options[:config_info].deep_merge!(dialog_ids)
         service_template.create_resource_actions(config_info)
@@ -105,6 +105,15 @@ def retirement_potential?
     retirement_jt_exists && services.where(:retired => false).exists?
   end
 
+  def create_dialogs(config_info)
+    [:provision, :retirement, :reconfigure].each_with_object({}) do |action, hash|
+      info = config_info[action]
+      next unless new_dialog_required?(info)
+
+      hash[action] = {:dialog_id => create_new_dialog(info[:new_dialog_name], info[:extra_vars], info[:hosts]).id}
+    end
+  end
+
   private
 
   def check_retirement_potential
@@ -115,15 +124,6 @@ def check_retirement_potential
     throw :abort
   end
 
-  def create_dialogs(config_info)
-    [:provision, :retirement, :reconfigure].each_with_object({}) do |action, hash|
-      info = config_info[action]
-      next unless new_dialog_required?(info)
-
-      hash[action] = {:dialog_id => create_new_dialog(info[:new_dialog_name], info[:extra_vars], info[:hosts]).id}
-    end
-  end
-
   def new_dialog_required?(info)
     info && info.key?(:new_dialog_name) && !info.key?(:dialog_id)
   end