Bug 1355841
| Summary: | Dialogue Input are truncated when submitted | |||
|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Loic Avenel <lavenel> | |
| Component: | Automate | Assignee: | eclarizi | |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Matouš Mojžíš <mmojzis> | |
| Severity: | medium | Docs Contact: | ||
| Priority: | high | |||
| Version: | 5.6.0 | CC: | cpelland, dajohnso, fdewaley, gmccullo, hkataria, jhardy, mkanoor, mmojzis, mpovolny, obarenbo, simaishi, tfitzger, WilliamC.Elliott | |
| Target Milestone: | GA | Keywords: | TestOnly, ZStream | |
| Target Release: | 5.7.0 | |||
| Hardware: | Unspecified | |||
| OS: | Unspecified | |||
| Whiteboard: | automate:dialog:ui | |||
| Fixed In Version: | 5.7.0.0 | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | ||
| Clone Of: | ||||
| : | 1356973 (view as bug list) | Environment: | ||
| Last Closed: | 2017-01-11 20:05:09 UTC | Type: | Bug | |
| Regression: | --- | Mount Type: | --- | |
| Documentation: | --- | CRM: | ||
| Verified Versions: | Category: | --- | ||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
| Cloudforms Team: | --- | Target Upstream Version: | ||
| Embargoed: | ||||
| Bug Depends On: | ||||
| Bug Blocks: | 1356973 | |||
|
Description
Loic Avenel
2016-07-12 17:11:41 UTC
Ticket request by Greg M to identify a new bug different from ticket https://bugzilla.redhat.com/show_bug.cgi?id=1344062 I think this description is inaccurate, when Greg M and I were on a video call with Henry, he showed us these steps: 1. Have a dialog with a text box and a dynamic drop down that has a refresh button. 2. Type into the text box and immediately click the refresh button next to the drop-down. 3. Observe the missing data upon submit. I was able to reproduce the issue locally with those steps, but not with the original. Also looking at the code, it should be waiting if you simply hit the submit button, but the refreshes are not. New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/9f5b43ecf27ae61a4f666732964a96d2a5257164 commit 9f5b43ecf27ae61a4f666732964a96d2a5257164 Author: Erik Clarizio <eclarizi> AuthorDate: Mon Jul 18 09:08:27 2016 -0700 Commit: Erik Clarizio <eclarizi> CommitDate: Mon Jul 18 09:08:27 2016 -0700 Remove interval from data-miq_observe on textbox and textarea boxes https://bugzilla.redhat.com/show_bug.cgi?id=1355841 app/helpers/application_helper/dialogs.rb | 2 -- spec/helpers/application_helper/dialogs_spec.rb | 6 ++---- 2 files changed, 2 insertions(+), 6 deletions(-) New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/7234f071890f40232918cc14e39daa2fb19b59f8 commit 7234f071890f40232918cc14e39daa2fb19b59f8 Author: Erik Clarizio <eclarizi> AuthorDate: Mon Jul 18 09:09:07 2016 -0700 Commit: Erik Clarizio <eclarizi> CommitDate: Mon Jul 18 09:09:07 2016 -0700 Use leading: true for observe and miqSelectPickerEvent debounces https://bugzilla.redhat.com/show_bug.cgi?id=1355841 app/assets/javascripts/miq_application.js | 2 +- app/assets/javascripts/miq_ujs_bindings.js | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/bba103a6a82ab256ae97f742651de3666dcf4459 commit bba103a6a82ab256ae97f742651de3666dcf4459 Author: Erik Clarizio <eclarizi> AuthorDate: Fri Jul 15 00:38:42 2016 -0700 Commit: Erik Clarizio <eclarizi> CommitDate: Fri Jul 15 12:31:49 2016 -0700 Ensure dialog field refreshes wait for requests in the MiqObserve queue https://bugzilla.redhat.com/show_bug.cgi?id=1355841 app/assets/javascripts/dialog_field_refresh.js | 155 ++++++++------- app/assets/javascripts/miq_application.js | 1 + spec/javascripts/dialog_field_refresh_spec.js | 252 +++++++++++++++++++++++-- 3 files changed, 320 insertions(+), 88 deletions(-) Verified in 5.7.0.14. When typing in the text box and then rapidly clicking on Refresh I can see in the logs, that string/int text field was sent correctly without being truncated. One time issue. Setting qe_test_coverage flag to -. |