Skip to content

Commit b97b77c

Browse files
authored
Merge pull request #9442 from elsamaryv/remove-orphaned-data-tab
Removes Diagnostics > Orphaned Data tab from Application Settings
2 parents adb02dd + 9f95a35 commit b97b77c

File tree

8 files changed

+1
-71
lines changed

8 files changed

+1
-71
lines changed

app/controllers/ops_controller/diagnostics.rb

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -287,30 +287,6 @@ def build_empty_log_depot_json
287287
log_depot_json
288288
end
289289

290-
# to delete orphaned records for user that was delete from db
291-
def orphaned_records_delete
292-
assert_privileges("ops_diagnostics")
293-
294-
MiqReportResult.delete_by_userid(params[:userid])
295-
rescue => bang
296-
add_flash(_("Error during Orphaned Records delete for user %{id}: %{message}") % {:id => params[:userid],
297-
:message => bang.message}, :error)
298-
javascript_flash(:spinner_off => true)
299-
else
300-
audit = {:event => "orphaned_record_delete",
301-
:message => "Orphaned Records deleted for userid [#{params[:userid]}]",
302-
:target_id => params[:userid],
303-
:target_class => "MiqReport",
304-
:userid => session[:userid]}
305-
AuditEvent.success(audit)
306-
add_flash(_("Orphaned Records for userid %{id} were successfully deleted") % {:id => params[:userid]})
307-
orphaned_records_get
308-
render :update do |page|
309-
page << javascript_prologue
310-
page.replace_html('diagnostics_orphaned_data', :partial => 'diagnostics_savedreports')
311-
end
312-
end
313-
314290
def diagnostics_server_list
315291
assert_privileges("ops_diagnostics_server_view")
316292

@@ -655,8 +631,6 @@ def diagnostics_set_form_vars
655631
@selected_server = MiqRegion.my_region
656632
elsif @sb[:active_tab] == "diagnostics_database"
657633
database_details
658-
elsif @sb[:active_tab] == "diagnostics_orphaned_data"
659-
orphaned_records_get
660634
elsif @sb[:active_tab] == "diagnostics_server_list"
661635
diagnostics_server_list
662636
end
@@ -720,10 +694,6 @@ def diagnostics_set_form_vars
720694
end
721695
end
722696

723-
def orphaned_records_get
724-
@sb[:orphaned_records] = MiqReportResult.orphaned_counts_by_userid
725-
end
726-
727697
# Method to build the server tree (parent is a zone or region instance)
728698
def build_server_tree(parent)
729699
@server_tree = if @sb[:diag_tree_type] == "roles"

app/views/ops/_all_tabs.html.haml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -164,8 +164,6 @@
164164
- if super_admin_user?
165165
= miq_tab_header("diagnostics_database", @sb[:active_tab]) do
166166
= _("Database")
167-
= miq_tab_header("diagnostics_orphaned_data", @sb[:active_tab]) do
168-
= _("Orphaned Data")
169167
.tab-content
170168
= miq_tab_content("diagnostics_zones", @sb[:active_tab]) do
171169
= render :partial => "diagnostics_zones_tab"
@@ -180,8 +178,6 @@
180178
= miq_tab_content("diagnostics_database", @sb[:active_tab]) do
181179
- if @sb[:active_tab] == "diagnostics_database"
182180
= render :partial => "diagnostics_database_tab"
183-
= miq_tab_content("diagnostics_orphaned_data", @sb[:active_tab]) do
184-
= render :partial => "diagnostics_orphaned_data_tab"
185181
- when :rbac_tree
186182
#rbac_details
187183
= render :partial => "rbac_details_tab"
Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
- if @sb[:active_tab] == "diagnostics_collect_logs"
2-
- if x_node.split('-').first == "root"
3-
= render :partial => "diagnostics_savedreports"
4-
- else
2+
- if x_node.split('-').first != "root"
53
= render :partial => "layouts/flash_msg"
64
= render :partial => "logs_selected"
75
%div{:style => "padding-bottom: 20px"}

app/views/ops/_diagnostics_orphaned_data_tab.html.haml

Lines changed: 0 additions & 8 deletions
This file was deleted.

app/views/ops/_diagnostics_savedreports.html.haml

Lines changed: 0 additions & 23 deletions
This file was deleted.

config/routes.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2457,7 +2457,6 @@
24572457
label_tag_mapping_field_changed
24582458
log_depot_edit
24592459
log_depot_validate
2460-
orphaned_records_delete
24612460
perf_chart_chooser
24622461
pglogical_save_subscriptions
24632462
pglogical_validate_subscription

spec/config/routes.pending.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -906,7 +906,6 @@ OpsController:
906906
- log_collection_form_fields
907907
- log_depot_edit
908908
- open_url_after_dialog
909-
- orphaned_records_delete
910909
- report_data
911910
- show
912911
- tl_chooser

spec/routing/ops_routing_spec.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@
5252
label_tag_mapping_field_changed
5353
log_depot_edit
5454
log_depot_validate
55-
orphaned_records_delete
5655
perf_chart_chooser
5756
rbac_group_edit
5857
rbac_group_field_changed

0 commit comments

Comments
 (0)