Bug 1574668

Summary: Dynamic dropdown doesn't refresh correctly
Product: Red Hat CloudForms Management Engine Reporter: Tasos Papaioannou <tpapaioa>
Component: AutomateAssignee: Tina Fitzgerald <tfitzger>
Status: CLOSED CURRENTRELEASE QA Contact: Tasos Papaioannou <tpapaioa>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: cpelland, mkanoor, obarenbo, tfitzger
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.10.0   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: 5.10.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1581386 1581387 (view as bug list) Environment:
Last Closed: 2019-02-11 14:03:25 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: 1581386, 1581387    
Attachments:
Description Flags
automate domain
none
dialog none

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 4 CFME Bot 2018-05-10 03:34:27 UTC 
https://github.com/ManageIQ/ui-components/pull/295
Comment 5 CFME Bot 2018-05-15 15:07:09 UTC 
https://github.com/ManageIQ/manageiq-ui-classic/pull/3945
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.