Bug 1634794 - Alert 'abandon changes' shown even when no changes made in multiple host edit
Summary: Alert 'abandon changes' shown even when no changes made in multiple host edit
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.10.0
Hardware: All
OS: All
low
low
Target Milestone: GA
: 5.11.0
Assignee: Richard Vsiansky
QA Contact: PJ Richardson
Red Hat CloudForms Documentation
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-10-01 16:02 UTC by Radim Hrazdil
Modified: 2019-12-12 13:34 UTC (History)
9 users (show)

Fixed In Version: 5.11.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-12-12 13:34:23 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
video sample (501.63 KB, application/octet-stream)
2018-10-01 16:02 UTC, Radim Hrazdil
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2019:4199 0 None None None 2019-12-12 13:34:40 UTC

Description Radim Hrazdil 2018-10-01 16:02:31 UTC
Created attachment 1489102 [details]
video sample

Description of problem:
When 2 or more hosts are selected and edited, Alert 'Abandon changes' is displayed even when no changes are made by the user.

Version-Release number of selected component (if applicable):
Version 5.10.0.17.20180927011235_1b5cf54 

How reproducible:
100%

Steps to Reproduce:
1. Have CFME appliance with added provider with some hosts
2. Go to Compute -> Infrastructure -> Hosts
3. Select 2 or more hosts and click Configuration -> Edit selected items 
4. Leave the page without making any changes

Actual results:
Alert asking whether to abandon changes is displayed

Expected results:
If no changes are made, no 'Abandon changes' alert should be displayed

Additional info:
video sample attached

Comment 4 Richard Vsiansky 2019-02-28 11:46:30 UTC
https://github.com/ManageIQ/manageiq-ui-classic/pull/5289

Comment 5 CFME Bot 2019-03-01 10:11:32 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/6e66bc5dd5ea2f2c1d06847cef2549f1af8a9fb8
commit 6e66bc5dd5ea2f2c1d06847cef2549f1af8a9fb8
Author:     Richard Vsiansky <r.vsia>
AuthorDate: Thu Feb 28 06:27:32 2019 -0500
Commit:     Richard Vsiansky <r.vsia>
CommitDate: Thu Feb 28 06:27:32 2019 -0500

    Fixes value of validation_id in host form

    https://bugzilla.redhat.com/show_bug.cgi?id=1634794

    * validation_id was null in template and blank in controller
    * => changed to null everywhere
    * it causes dirty state on the form (because the model and the copy was different)

 app/assets/javascripts/controllers/host/host_form_controller.js | 4 +-
 app/views/layouts/angular/_multi_auth_credentials.html.haml | 2 +-
 spec/javascripts/controllers/host/host_form_controller_spec.js | 11 +-
 3 files changed, 13 insertions(+), 4 deletions(-)

Comment 10 Richard Vsiansky 2019-05-16 13:45:41 UTC
Justin,

I think it should be OK.

The prompt should be displayed only when you change something in the form and click on link in the menu or breadcrumbs.

The menu part looks OK, however breadcrumbs do not support this yet.

So, I propose:
1. This BZ should be closed (the form works as any other form in the application - clicking in menu shows the prompt, cancel/back do not.)
2. A new BZ should be open, something like this: 'Clicking on breadcrumbs in edited forms do not show Abandon Changes' and assign me to it. It is a global issue, not connected only to this form.

Do you agree?

Richard

Comment 12 Richard Vsiansky 2019-05-21 07:31:01 UTC
Justin, 

that is a correct behavior. See ""Abandon changes?" pop should not be displayed, if user presses cancel button"

Richard

Comment 13 juwatts 2019-07-01 18:54:48 UTC
Verified in  5.11.0.11.20190625161125_392c61e

Comment 14 PJ Richardson 2019-11-12 16:40:59 UTC
This BZ is now covered by test_infrastructure_hosts_crud in integration_tests.

Comment 16 errata-xmlrpc 2019-12-12 13:34:23 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2019:4199


Note You need to log in before you can comment on or make changes to this bug.