Bug 1499589
| Summary: | Dynamic refresh ignored on Service Dialog elements if clicking submit without clicking out of refresh trigger element first | |||
|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | tachoi | |
| Component: | Automate | Assignee: | eclarizi | |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Milan Falešník <mfalesni> | |
| Severity: | high | Docs Contact: | ||
| Priority: | medium | |||
| Version: | 5.6.0 | CC: | cpelland, dajohnso, eclarizi, hkataria, jhardy, jmarc, jocarter, ltsai, mfalesni, mkanoor, mpovolny, obarenbo, simaishi, tachoi, tfitzger | |
| Target Milestone: | GA | Keywords: | TestOnly, ZStream | |
| Target Release: | 5.9.0 | |||
| Hardware: | Unspecified | |||
| OS: | Unspecified | |||
| Whiteboard: | automate:dialog | |||
| Fixed In Version: | 5.9.0.3 | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | ||
| Clone Of: | 1379481 | |||
| : | 1502738 1502739 (view as bug list) | Environment: | ||
| Last Closed: | 2018-03-06 14:50:13 UTC | Type: | Bug | |
| Regression: | --- | Mount Type: | --- | |
| Documentation: | --- | CRM: | ||
| Verified Versions: | Category: | --- | ||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
| Cloudforms Team: | CFME Core | Target Upstream Version: | ||
| Embargoed: | ||||
| Bug Depends On: | 1379481 | |||
| Bug Blocks: | 1502738, 1502739 | |||
|
Comment 5
Chris Pelland
2017-10-09 13:43:45 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 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(-) Verified in 5.9.0.15. The dialog fields specify fields they refresh, therefore this functionality now works properly even with tabs. |