Bug 1438010

Summary: dynamic check box does not update on SSUI
Product: Red Hat CloudForms Management Engine Reporter: Christian Jung <cbolz>
Component: UI - ServiceAssignee: Chris Hale <chhale>
Status: CLOSED CURRENTRELEASE QA Contact: Shveta <sshveta>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.7.0CC: chhale, cpelland, dclarizi, greartes, jhardy, mpfuetzn, obarenbo, simaishi, sshveta
Target Milestone: GAKeywords: TestOnly
Target Release: 5.10.0   
Hardware: All   
OS: All   
Whiteboard: ssui
Fixed In Version: 5.10.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1441416 1514196 (view as bug list) Environment:
Last Closed: 2018-06-21 20:26:10 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: 1441416, 1514196    
Attachments:
Description Flags
service dialog export
none
Automate export none

Description Christian Jung 2017-03-31 15:37:51 UTC 
Description of problem:
I wrote a script which checks/unchecks a check box element as from a dynamic dialog.

This works fine on the classic UI, but not on the new Self Service UI. Although the code is executed, the check box is not updated in the UI.

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

How reproducible:
always in SSUI, never in classic UI

Steps to Reproduce:
1. import the attached automate code
2. import the attached service dialog
3. create a catalog item, type generic and assign it to the dialog

Actual results:
works fine in classic UI, but not in SSU

Expected results:
should work on both UI's

Additional info:
Comment 2 Christian Jung 2017-03-31 15:38:52 UTC 
Created attachment 1267904 [details]
service dialog export
Comment 3 Christian Jung 2017-03-31 15:39:31 UTC 
Created attachment 1267905 [details]
Automate export
Comment 4 CFME Bot 2017-04-03 15:03:08 UTC 
https://github.com/ManageIQ/manageiq-ui-service/pull/624
Comment 5 CFME Bot 2017-04-03 18:08:42 UTC 
New commit detected on ManageIQ/manageiq-ui-service/master:
https://github.com/ManageIQ/manageiq-ui-self_service/commit/ba7125b2c34516af21c99c628ae53a6b7daafe59

commit ba7125b2c34516af21c99c628ae53a6b7daafe59
Author:     Josh Langholtz <jjlangholtz>
AuthorDate: Mon Apr 3 10:31:45 2017 -0400
Commit:     Josh Langholtz <jjlangholtz>
CommitDate: Mon Apr 3 10:31:45 2017 -0400

    Update dialog dynamic checkbox values during refresh
    
    When auto refreshing dialog fields, update DialogFieldCheckBox values as
    well.
    
    https://www.pivotaltracker.com/story/show/142955157
    https://bugzilla.redhat.com/show_bug.cgi?id=1438010

 client/app/core/dialog-field-refresh.service.js    |  2 +-
 .../app/core/dialog-field-refresh.service.spec.js  | 34 ++++++++++++++++++++++
 2 files changed, 35 insertions(+), 1 deletion(-)
Comment 6 Chris Kacerguis 2017-04-06 16:27:19 UTC 
Josh - Can we move this to POST?
Comment 8 Shveta 2017-11-03 00:25:27 UTC 
Not working in SUI .
5.9.0.4.20171024163837_ef71ea6 
Appliance : https://10.8.198.241/ui/service
Service : rhev_pxe
Comment 12 Chris Kacerguis 2017-11-16 18:29:27 UTC 
Chris H - Can you please take a look at this one more time.
Comment 13 Chris Hale 2017-11-16 19:01:57 UTC 
GH PR https://github.com/ManageIQ/manageiq-ui-service/pull/1216
Comment 14 Satoe Imaishi 2017-11-16 19:11:10 UTC 
Chris, that PR is only in 'master' branch. I assume it's needed for gaprindashvili?
Comment 15 Chris Hale 2017-11-16 19:12:08 UTC 
Yes, lets ensure it is merged into gaprindashvili