-
Notifications
You must be signed in to change notification settings - Fork 357
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
Fix Service request filtering #2830
Conversation
61942af
to
e7c2c35
Compare
Some comments on commits martinpovolny/manageiq-ui-classic@d159956~...54c3eec spec/controllers/miq_request_controller_spec.rb
|
Checked commits martinpovolny/manageiq-ui-classic@d159956~...54c3eec with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 |
I'm OK with the changes. @karelhala ? |
@karelhala loves it we discussed f2f ;-) |
Fix Service request filtering (cherry picked from commit 7089223) https://bugzilla.redhat.com/show_bug.cgi?id=1518302
Gaprindashvili backport details:
|
Reproducer
Apply
BZ:
https://bugzilla.redhat.com/show_bug.cgi?id=1512719
Closes: #2742
@mzazrivec : PTAL
TODO:
Looking as some strange stuff in the case statement,will be done in a separate PR.looking at what code to removeremoved just a bitExplanation:
The filters are composed from SQL fragments. So no scopes for now :-(.
It's handled (statefully) in
/report_data
via the "exceptions". This one is for "MiqRequest" and callspage_params
.So we can keep the code that updates the session when form fields are changed. (That needs a rework no soner then when it's redone to client side.)
And all that is really needed is the reload of the GTL grid -- that will reload
/report_data
and apply the changed filter (remember, it's in the session). So doing just that.