Merge pull request ManageIQ#829 from hayesr/fix_diag_zone_screen

Fix delete button under Configuration > Diagnostics
douglasgabriel · Mar 29, 2017 · d3f69ab · d3f69ab
2 parents 0f28507 + a7582ed
commit d3f69ab
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/app/controllers/ops_controller/diagnostics.rb b/app/controllers/ops_controller/diagnostics.rb
@@ -801,7 +801,7 @@ def reload_server_tree
   def diagnostics_set_form_vars
     active_node = x_node
     if active_node && active_node.split('-').first == "z"
-      @record = @selected_server = Zone.find_by_id(from_cid(active_node.split('-').last))
+      @record = @selected_zone = @selected_server = Zone.find_by_id(from_cid(active_node.split('-').last))
       @sb[:selected_server_id] = @selected_server.id
       @sb[:selected_typ] = "zone"
       if @selected_server.miq_servers.length >= 1 &&

diff --git a/app/helpers/application_helper/toolbar/diagnostics_zone_center.rb b/app/helpers/application_helper/toolbar/diagnostics_zone_center.rb
@@ -15,19 +15,19 @@ class ApplicationHelper::Toolbar::DiagnosticsZoneCenter < ApplicationHelper::Too
       t,
       :items => [
         button(
-          :zone_delete_server,
+          :zone_delete,
           'pficon pficon-delete fa-lg',
           t = proc do
-            _('Delete Server %{server_name} [%{server_id}]') % {:server_name => @record.name, :server_id => @record.id}
+            _('Delete Zone %{server_name} [%{server_id}]') % {:server_name => @selected_zone.name, :server_id => @selected_zone.id}
           end,
           t,
           :confirm => proc do
-                        _("Do you want to delete Server %{server_name} [%{server_id}]?") % {
-                          :server_name => @record.name,
-                          :server_id   => @record.id
+                        _("Do you want to delete Zone %{server_name} [%{server_id}]?") % {
+                          :server_name => @selected_zone.name,
+                          :server_id   => @selected_zone.id
                         }
                       end,
-          :klass => ApplicationHelper::Button::ZoneDeleteServer
+          :klass => ApplicationHelper::Button::ZoneDelete
         ),
         button(
           :zone_role_start,