Created attachment 1381331 [details] Error page after I press Save Description of problem: Browser shows error page after editing existing Nuage provider. Version-Release number of selected component (if applicable): How reproducible: Always Steps to Reproduce: 1. Open provider, then pick "Edit this Network Provider" from the mid-page menu 2. Update some value 3. Click Save Actual results: Error page appears saying "The page you were looking for doesn't exist." (see screenshot attached) Expected results: Provider page is opened. Additional info:
Please assess the impact of this issue and update the severity accordingly. Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition. If it's something like a tracker bug where it doesn't matter, please set the severity to Low.
https://github.com/ManageIQ/manageiq-ui-classic/pull/3337
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/c525ab7933f6f427073c393e6278e69307aaf50f commit c525ab7933f6f427073c393e6278e69307aaf50f Author: sasoc <saso.cvitkovic> AuthorDate: Tue Jan 30 11:03:10 2018 +0100 Commit: sasoc <saso.cvitkovic> CommitDate: Tue Feb 6 11:41:12 2018 +0100 Fix Nuage provider update There were two problems when editing Nuage network provider. If network that you edited had no event, there would still be AMQP selected under the Events tab. That made Save button disabled since hostname must not be empty and everything has to be verified. Now None is selected in case of no event. The second problem was that when a network was updated error occurred while redirecting, since you would only get one argument and `ems_path` required two. Now `ems_path` works same as in the cloud provider controller. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1534457 app/controllers/ems_network_controller.rb | 3 +- app/controllers/mixins/ems_common_angular.rb | 5 +- spec/controllers/mixins/ems_common_angular_spec.rb | 57 ++++++++++++++++++++++ 3 files changed, 61 insertions(+), 4 deletions(-) create mode 100644 spec/controllers/mixins/ems_common_angular_spec.rb