New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Storage: Fixed the process_button method to render the page correctly #305
Storage: Fixed the process_button method to render the page correctly #305
Conversation
@@ -108,6 +108,7 @@ def process_button | |||
if @refresh_div == "main_div" && @lastaction == "show_list" | |||
replace_gtl_main_div | |||
else | |||
@ems = identify_record(params[:id]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AparnaKarve why do we need to set @ems in cim_instance controller.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@h-kataria Check out line#119 below.
page.replace_html("main_div", :partial=>"layouts/gtl", :locals=>{:action_url=>"show/{@ems.id}"})
@ems
is being used there without being set, which was causing the rendering to blow up.
But since its not being used anywhere else in the cim_instance controller we can get rid of it and change it to a local variable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AparnaKarve dont we have @record that we could have used on line 119
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@h-kataria Yes, looks like we can use @record
here.
I will push the update shortly.
Rendering errors that occurred after the refresh/power button tasks were addressed in the Storage area in the VM/Hosts associations issue ManageIQ#243
Checked commit AparnaKarve@5552423 with rubocop 0.21.0 |
…ions_gh_issue_243 Storage: Fixed the process_button method to render the page correctly
Use the full URI for the broker connection (cherry picked from commit 675cc09cac238bfa52850a46d447afa2a9143aa2) https://bugzilla.redhat.com/show_bug.cgi?id=1608552
Use the full URI for the broker connection (cherry picked from commit 675cc09cac238bfa52850a46d447afa2a9143aa2) https://bugzilla.redhat.com/show_bug.cgi?id=1603578
…onnection Use the full URI for the broker connection (cherry picked from commit 675cc09cac238bfa52850a46d447afa2a9143aa2) https://bugzilla.redhat.com/show_bug.cgi?id=1603578
Rendering errors that occurred after the refresh/power button tasks were addressed in the Storage area in the VM/Hosts list view.
issue #243