Remove VALID_PERF_PARENTS from UiConstants

ManageIQ · Sep 5, 2017 · 01dbfb2 · 01dbfb2
1 parent 733bc8a
commit 01dbfb2
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/app/controllers/application_controller/performance.rb b/app/controllers/application_controller/performance.rb
@@ -687,7 +687,7 @@ def perf_gen_data_before_wait
     if perf_parent?                               # Build the parent report, if asked for
       p_rpt = Marshal.load(Marshal.dump(rpt))    # Deep clone the main report
       p_rpt.where_clause[1] = @perf_options[:parent]
-      p_rpt.where_clause[2] = @perf_record.send(VALID_PERF_PARENTS[@perf_options[:parent]]).id
+      p_rpt.where_clause[2] = @perf_record.send(ApplicationHelper::VALID_PERF_PARENTS[@perf_options[:parent]]).id
       rpts.push(p_rpt)
     end
 

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
@@ -13,6 +13,11 @@ module ApplicationHelper
   include PlanningHelper
   include Title
 
+  VALID_PERF_PARENTS = {
+    "EmsCluster" => :ems_cluster,
+    "Host"       => :host
+  }
+
   # Need to generate paths w/o hostname by default to make proxying work.
   #
   def url_for_only_path(args)

diff --git a/app/helpers/ui_constants.rb b/app/helpers/ui_constants.rb
@@ -94,11 +94,6 @@ module UiConstants
     "m"  => "menu"
   }
 
-  VALID_PERF_PARENTS = {
-    "EmsCluster" => :ems_cluster,
-    "Host"       => :host
-  }
-
   MIQ_AE_COPY_ACTIONS = %w(miq_ae_class_copy miq_ae_instance_copy miq_ae_method_copy)
 
 end