-
Notifications
You must be signed in to change notification settings - Fork 356
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
Use correct params for find_record_with_rbac in resize method #1139
Use correct params for find_record_with_rbac in resize method #1139
Conversation
@miq-bot add_label bug, rbac |
How to reproduce original issue (assuming master manageiq, manageiq-ui-classic and manageiq-providers-openstack repos):
Result without this patch: error page with "undefined method `find_by' for |
find_record_with_rbac expects ActiveRecord like class as first parameter. During previous refactoring a mistake was introduced and session[:userid] was passed as first parameter and that caused "undefined method `find_by' for "admin":String [vm/resize]" when trying to go to reconfigure a VM dialog page. This corrects that. https://bugzilla.redhat.com/show_bug.cgi?id=1445005
Checked commit https://github.com/petrblaho/manageiq-ui-classic/commit/52c029d8999a5859c92b57efedff57bdfe171a18 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0 |
@miq-bot label fine/yes |
@miq-bot add_label fine/yes |
@tzumainn unrecognized command 'label', ignoring... Accepted commands are: add_label, assign, close_issue, move_issue, remove_label, rm_label, set_milestone |
@miq-bot add_label blocker |
@petrblaho The refactoring PR (#919) is not in FYI, the current
|
@simaishi this PR solved issue introduced by #919 as you noted. So I think this is not needed in fine. I cannot reproduce https://bugzilla.redhat.com/show_bug.cgi?id=1445005 on fine branch. |
@petrblaho Thanks for the confirmation, removed the flags. |
find_record_with_rbac expects ActiveRecord like class as first parameter.
During previous refactoring a mistake was introduced and session[:userid] was
passed as first parameter and that caused "undefined method `find_by' for
"admin":String [vm/resize]" when trying to go to reconfigure a VM dialog page.
This corrects that.
https://bugzilla.redhat.com/show_bug.cgi?id=1445005