From PR https://github.com/ManageIQ/manageiq/pull/12475: in 0517b67 (#11074), code like "#{foo || "new"}" got refactored to (foo.to_s || "new"), but unfortunately, that means a different thing if foo is nil. (Because nil.to_s == "" so true.) Changing to use (foo || "new").to_s instead, which should be semantically equivalent to the original. Without the PR, some URLs would get generated (for example) as /ops/edit/ instead of /ops/edit/new.
Steps to verify: go to Automate > Customization, Buttons accordion click a button group toolbar Configure > Add a new Button scroll down to the Visibility section in the "Show" selectpicker, select "<By Role>" check a role checkbox and watch network requests (in the inspector Network tab, or in rails log) expected: POST /miq_ae_customization/automate_button_field_changed/new bad: POST /miq_ae_customization/automate_button_field_changed/
After performing steps from Comment 3, got expected results: POST /miq_ae_customization/automate_button_field_changed/new Verified Version: 5.8.0.9-alpha2.20170404195944_1d7ece4