Created attachment 1424643 [details]
Description of problem:
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. import the dialog and method attached.
2. create a catalog item with above dialogs.
3. Order catalog item .
4. checkbox does not auto update on selecting True/False in first dropdown as it should according to the method.
Appliance : https://10.8.196.249
Service - Bz_reproducer
Created attachment 1424644 [details]
This is also a bug in SUI
So I can help shortcut this for someone on the Classic UI team. This issue was with ui-components repo and was solved already. The SUI currently has the fix applied for this version of release but because of how the SUI uses NPM for dependency updates and Classic UI uses bower, I suspect when it was fixed and version bumped in SUI, it might not have gotten version bumped in bower and integrated and retested in Classic UI. This should be a fairly simple fix for someone to lookup and apply a newer version on Classic UI.
Hit submit too soon last time.
Erik - I believe you might have a work-around for this issue. Please add details here.
We will need a couple of more GH pull requests after this one gets merged. SUI and Classic UI need PR's created after this fix is released to take advantage of the fix.
New commit detected on ManageIQ/ui-components/master:
Author: Erik Clarizio <firstname.lastname@example.org>
AuthorDate: Mon Apr 23 14:28:26 2018 -0400
Commit: Erik Clarizio <email@example.com>
CommitDate: Mon Apr 23 14:28:26 2018 -0400
Use updated vaules as default for checkbox when field is dynamic
src/dialog-user/services/dialogData.spec.ts | 43 +-
src/dialog-user/services/dialogData.ts | 9 +
2 files changed, 42 insertions(+), 10 deletions(-)
In response to Greg and clearing the needinfo:
I was able to fix the issue on the reproducing appliance by removing the default value for the checkbox in the database. Alternatively I suppose you could export the dialog, manually edit the yaml to not have a default value for the checkbox field in question, and then re-import.
GH PR for Service UI to take advantage of this bugfix on G release. https://github.com/ManageIQ/manageiq-ui-service/pull/1426
GH PR for Classic UI to take advantage of the bugfix on G release. https://github.com/ManageIQ/manageiq-ui-classic/pull/3845
Waiting for new build
Works on 126.96.36.199 SUI
Verified in Version 188.8.131.52.20180831154906_2f9a1dd