-
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
Toolbars (m*) - add send_checked when url_parms =~ _div #2389
Conversation
Currently, our toolbar JS code sends a list of checked items exactly when `url_parms` contains a string ending with _div. This is not the only use of url_parms, so making the behaviour explicit by creating the new attribute. --- this bit deals with toolbars starting with M ``` perl -i -npe 's/^(\s*):url_parms.*_div.*/$&\n$1:send_checked => true,/' app/helpers/application_helper/toolbar/m*_center.rb git st | sed -e '1d' -e 's/^ M //' | while read fn; do rockets.pl "$fn" > tmp.pl ; perl -i tmp.pl "$fn" ; rm tmp.pl ; done ```
Checked commits https://github.com/himdel/manageiq-ui-classic/compare/aa7d8cdc778d823dcfd844fabb8965b30b963d34~...a8f66d950c48b471de4c626efc9fd8b1dccba5e8 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0 app/helpers/application_helper/toolbar/middleware_datasources_center.rb
app/helpers/application_helper/toolbar/middleware_deployments_center.rb
app/helpers/application_helper/toolbar/middleware_messagings_center.rb
app/helpers/application_helper/toolbar/middleware_servers_center.rb
app/helpers/application_helper/toolbar/miq_action_center.rb
app/helpers/application_helper/toolbar/miq_ae_domain_center.rb
app/helpers/application_helper/toolbar/miq_ae_domains_center.rb
app/helpers/application_helper/toolbar/miq_ae_instances_center.rb
app/helpers/application_helper/toolbar/miq_ae_methods_center.rb
app/helpers/application_helper/toolbar/miq_ae_namespace_center.rb
app/helpers/application_helper/toolbar/miq_alert_center.rb
app/helpers/application_helper/toolbar/miq_alert_profile_center.rb
app/helpers/application_helper/toolbar/miq_dialog_center.rb
app/helpers/application_helper/toolbar/miq_dialogs_center.rb
app/helpers/application_helper/toolbar/miq_event_center.rb
app/helpers/application_helper/toolbar/miq_groups_center.rb
app/helpers/application_helper/toolbar/miq_policy_center.rb
app/helpers/application_helper/toolbar/miq_policy_profile_center.rb
app/helpers/application_helper/toolbar/miq_report_center.rb
app/helpers/application_helper/toolbar/miq_report_schedules_center.rb
app/helpers/application_helper/toolbar/miq_requests_center.rb
app/helpers/application_helper/toolbar/miq_schedules_center.rb
app/helpers/application_helper/toolbar/miq_templates_center.rb
|
Currently, our toolbar JS code sends a list of checked items exactly when
url_parms
contains a string ending with _div.This is not the only use of url_parms, so making the behaviour explicit by creating the new attribute.
this bit deals with toolbars starting with M
Issue #588
Partly replaces #642
@miq-bot add_label toolbars, refactoring