Bug 1574668 - Dynamic dropdown doesn't refresh correctly
Summary: Dynamic dropdown doesn't refresh correctly
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Automate
Version: 5.8.0
Hardware: All
OS: All
medium
medium
Target Milestone: GA
: 5.10.0
Assignee: Tina Fitzgerald
QA Contact: Tasos Papaioannou
URL:
Whiteboard:
Depends On:
Blocks: 1581386 1581387
TreeView+ depends on / blocked
 
Reported: 2018-05-03 19:46 UTC by Tasos Papaioannou
Modified: 2019-03-18 14:47 UTC (History)
4 users (show)

Fixed In Version: 5.10.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1581386 1581387 (view as bug list)
Environment:
Last Closed: 2019-02-11 14:03:25 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
automate domain (874.23 KB, application/zip)
2018-05-03 19:52 UTC, Tasos Papaioannou
no flags Details
dialog (2.66 KB, text/plain)
2018-05-03 19:52 UTC, Tasos Papaioannou
no flags Details

Description Tasos Papaioannou 2018-05-03 19:46:48 UTC
Description of problem:

If an automate method refreshes a dynamic dropdown, the refreshed dropdown values aren't visible.


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

5.8.4.2 and 5.9.2.4

How reproducible:

100%

Steps to Reproduce:
1.) Import automate domain and dialog from attached files.
2.) Create service catalog from dialog.
3.) Order service, and change the selected value in the Tenant dropdown.
4.) See that the values displayed in the Network dropdown do not change after the tenant changes, but that if you then select one of the network values, the refreshed network value suddenly appears and is selected.

Actual results:

Dynamic dropdown refreshes, but the new values aren't visible until you select one of them.

Expected results:

Dynamic dropdown refreshes and new values are visible immediately.

Additional info:

This seems to be an issue specifically when the automate method re-uses the same keys when creating the hash of the new values for the dynamic dropdown. If different keys are used, then the refreshed values are visible immediately.

Comment 2 Tasos Papaioannou 2018-05-03 19:52:36 UTC
Created attachment 1430876 [details]
automate domain

Comment 3 Tasos Papaioannou 2018-05-03 19:52:59 UTC
Created attachment 1430877 [details]
dialog

Comment 6 CFME Bot 2018-05-22 14:29:45 UTC
New commit detected on ManageIQ/ui-components/master:

https://github.com/ManageIQ/ui-components/commit/9f5e29400a63a596d30c13576ecbe29b10e02eb6
commit 9f5e29400a63a596d30c13576ecbe29b10e02eb6
Author:     Erik Clarizio <eclarizio>
AuthorDate: Wed May  9 21:07:32 2018 -0400
Commit:     Erik Clarizio <eclarizio>
CommitDate: Wed May  9 21:07:32 2018 -0400

    Use new 'miq-select' directive instead of patternfly

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

 application-settings.js | 1 +
 karma.conf.js | 2 +-
 src/dialog-user/components/dialog-user/dialogField.html | 47 +-
 src/index.ts | 3 +-
 src/miq-select/index.ts | 7 +
 src/miq-select/miqSelect.js | 53 +
 src/miq-select/miqSelect.spec.js | 156 +
 7 files changed, 226 insertions(+), 43 deletions(-)

Comment 7 CFME Bot 2018-05-22 14:53:14 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/d2e8c5764001fe89e39525fd7c202ef289ca4b6f
commit d2e8c5764001fe89e39525fd7c202ef289ca4b6f
Author:     Erik Clarizio <eclarizio>
AuthorDate: Tue May 15 10:54:04 2018 -0400
Commit:     Erik Clarizio <eclarizio>
CommitDate: Tue May 15 10:54:04 2018 -0400

    Utilize a new 'miq-select' directive for dialog fields

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

 app/assets/javascripts/miq_angular_application.js | 1 +
 1 file changed, 1 insertion(+)

Comment 10 Tasos Papaioannou 2018-07-27 17:52:04 UTC
Verified on 5.10.0.4.


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