Bug 1558271

Summary: Duplicate values are shown in dialog dropdown.
Product: Red Hat CloudForms Management Engine Reporter: Shveta <sshveta>
Component: UI - ServiceAssignee: Martin Hradil <mhradil>
Status: CLOSED CURRENTRELEASE QA Contact: Niyaz Akhtar Ansari <nansari>
Severity: high Docs Contact:
Priority: high    
Version: 5.9.0CC: awight, cpelland, dmetzger, eclarizi, gmccullo, greartes, JAyissi, lavenel, mhradil, nansari, obarenbo
Target Milestone: GAKeywords: Regression, TestOnly, ZStream
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: service
Fixed In Version: 5.10.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1562075 (view as bug list) Environment:
Last Closed: 2019-02-11 13:59:38 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:    
Bug Blocks: 1562075    
Attachments:
Description Flags
Refresh_dialog
none
domain
none
duplicate values
none
Duplicate Values on CFME 5.9.3.2 none

Description Shveta 2018-03-19 23:48:47 UTC
Created attachment 1410192 [details]
Refresh_dialog

Description of problem:


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

How reproducible:


Steps to Reproduce:
1. Import the dialog and domain attached .
2. Create a catalog item that uses this dialog .
3. Login in SUI , select service catalog and Select contract B.
4. Location and 'What's inside' refreshes.
5. click on the first dropdown "Contract"
It shows the values twice as in attached screenshot .

Actual results:


Expected results:


Additional info:
Does not happen in 5.9.0.22

Comment 2 Shveta 2018-03-19 23:49:07 UTC
Created attachment 1410193 [details]
domain

Comment 3 Shveta 2018-03-19 23:50:47 UTC
Created attachment 1410194 [details]
duplicate values

Comment 5 eclarizi 2018-03-22 22:42:09 UTC
After talking a bit with Allen, the only difference we can tell between the Ops UI and the SUI is the patternfly version. If I instead use a plain select drop-down (not a patternfly themed one), the duplicates do not show up, so it's not a data problem on our end.

I actually think this is a bug in patternfly, or at least a bug with the version we're using in combination with the version of other libraries we may be using, as directly on the patternfly site the examples seem to be working fine.

Maybe Martin H has a better idea of how we should proceed with this?

Comment 6 Martin Hradil 2018-03-26 15:32:07 UTC
Hasn't this been resolved by using pf-dropdown-select instead of pf-select?

Comment 7 eclarizi 2018-03-26 15:46:30 UTC
I think you meant pf-bootstrap-select, but unfortunately no, that was not the resolution to this bug :(.

Comment 16 Allen W 2018-03-28 15:25:55 UTC
https://github.com/ManageIQ/ui-components/pull/275

Ok this issue likely also effects multi-select so adding it on that one as well

Comment 19 Niyaz Akhtar Ansari 2018-09-04 10:15:35 UTC
Verified in Version 5.10.0.14.20180831154906_2f9a1dd

Comment 20 Johnny Ayissi 2018-09-11 17:21:56 UTC
Good afternoon Martin/Niyaz/Allen,


Can we get this fix backported to the new CFME v5.9.4? My client is having this same bug when we select options from our dynamic dialog.


Thanks in advance,


Johnny J. Ayissi

Comment 21 Johnny Ayissi 2018-09-11 17:21:56 UTC
Good afternoon Martin/Niyaz/Allen,


Can we get this fix backported to the new CFME v5.9.4? My client is having this same bug when we select options from our dynamic dialog.


Thanks in advance,


Johnny J. Ayissi

Comment 22 Martin Hradil 2018-09-13 08:30:25 UTC
Johny.. the BZ tracking 5.9 is https://bugzilla.redhat.com/show_bug.cgi?id=1562075

Looks like this was fixed in 5.9.2.3.20180419182310_cab2891

Comment 25 Johnny Ayissi 2018-09-13 15:13:16 UTC
Created attachment 1483080 [details]
Duplicate Values on CFME 5.9.3.2

Comment 33 Martin Hradil 2018-09-14 14:14:12 UTC
The bug was reintroduced again when consolidating multiple branches into one - 
https://github.com/ManageIQ/ui-components/pull/295.

Fixing again :)

Comment 34 Martin Hradil 2018-09-14 15:02:32 UTC
The re-fix already landed in master as https://github.com/ManageIQ/ui-components/pull/316,

backported to gaprindashvili on Jul 19
(https://github.com/ManageIQ/manageiq-ui-classic/pull/4303 and https://github.com/ManageIQ/manageiq-ui-service/pull/1458)

According to https://bugzilla.redhat.com/show_bug.cgi?id=1601587, this was released in 5.9.4.1, which would match Drew's observations.

Comment 37 Johnny Ayissi 2018-09-14 15:08:01 UTC
Thank you very much, Martin!