Skip to content
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

Merged
merged 2 commits into from
Oct 13, 2017
Merged

Toolbars (m*) - add send_checked when url_parms =~ _div #2389

merged 2 commits into from
Oct 13, 2017

Conversation

himdel
Copy link
Contributor

@himdel himdel commented Oct 13, 2017

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

Issue #588
Partly replaces #642
@miq-bot add_label toolbars, refactoring

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
```
@miq-bot
Copy link
Member

miq-bot commented Oct 13, 2017

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
25 files checked, 63 offenses detected

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

@mzazrivec mzazrivec self-assigned this Oct 13, 2017
@mzazrivec mzazrivec added this to the Sprint 71 Ending Oct 16, 2017 milestone Oct 13, 2017
@mzazrivec mzazrivec merged commit fe77e48 into ManageIQ:master Oct 13, 2017
@himdel himdel deleted the tb-send-checked-m branch October 13, 2017 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants