Bug 1499589 - Dynamic refresh ignored on Service Dialog elements if clicking submit without clicking out of refresh trigger element first
Summary: Dynamic refresh ignored on Service Dialog elements if clicking submit without...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Automate
Version: 5.6.0
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: GA
: 5.9.0
Assignee: eclarizi
QA Contact: Milan Falešník
URL:
Whiteboard: automate:dialog
: 1379481 (view as bug list)
Depends On: 1379481
Blocks: 1502738 1502739
TreeView+ depends on / blocked
 
Reported: 2017-10-09 01:29 UTC by tachoi
Modified: 2020-12-14 10:25 UTC (History)
15 users (show)

Fixed In Version: 5.9.0.3
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1379481
: 1502738 1502739 (view as bug list)
Environment:
Last Closed: 2018-03-06 14:50:13 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)

Comment 5 Chris Pelland 2017-10-09 13:43:45 UTC 
*** Bug 1379481 has been marked as a duplicate of this bug. ***
Comment 7 eclarizi 2017-10-13 18:11:05 UTC 
Hi Taeho,

I've been doing a lot of debugging of this issue and it turned out to be a race condition and seemed like it was going to be a really difficult problem to solve. Luckily, after really stepping through it, I ended up with actually a really simple fix.

You can see the details here: https://github.com/ManageIQ/manageiq-ui-classic/pull/2397
Comment 10 CFME Bot 2017-10-16 14:33:29 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/c2701cc9fdade217627f797e6939fd00201c7529

commit c2701cc9fdade217627f797e6939fd00201c7529
Author:     Erik Clarizio <eclarizio>
AuthorDate: Fri Oct 13 11:02:44 2017 -0700
Commit:     Erik Clarizio <eclarizio>
CommitDate: Fri Oct 13 11:03:41 2017 -0700

    Use miqObserveRequest to force dialog submit to wait for auto refresh
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1499589

 app/assets/javascripts/dialog_field_refresh.js |  4 ++--
 spec/javascripts/dialog_field_refresh_spec.js  | 17 +++++++++++++++++
 2 files changed, 19 insertions(+), 2 deletions(-)
Comment 14 Milan Falešník 2018-01-09 14:30:45 UTC 
Verified in 5.9.0.15.

The dialog fields specify fields they refresh, therefore this functionality now works properly even with tabs.
Note You need to log in before you can comment on or make changes to this bug.