Bug 1620208

Summary: Service Dialog CheckBox: API isn't sending any information about checkbox on dialog at all
Product: Red Hat CloudForms Management Engine Reporter: Shveta <sshveta>
Component: UI - OPSAssignee: drew uhlmann <duhlmann>
Status: CLOSED NOTABUG QA Contact: Jad Haj Yahya <jhajyahy>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 5.9.4CC: dmetzger, duhlmann, hkataria, lavenel, mpovolny, obarenbo
Target Milestone: GA   
Target Release: cfme-future   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-11-14 18:30:35 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:
Attachments:
Description Flags
payload without checkbox none

Description Shveta 2018-08-22 16:44:08 UTC 
Created attachment 1477937 [details]
payload without checkbox

Description of problem: API issue from this BZ 
https://bugzilla.redhat.com/show_bug.cgi?id=1610685


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


How reproducible:


Steps to Reproduce:
1. Create Service Dialog with the following elements
    - Checkbox ( not ticked)
    - Dynamic Text Area 
        use the following code for the dynamic text area
        #####################
        check_box  =  $evm.root['dialog_check_box_1']
        puts "#{check_box}"
        ###################
   In the Checkbox element, add refresh field to "Text Area"

2. Create a Service catalog Item and use the newly created Service Dialog

3. submit the dialog without changing the checkbox at all
4. Check automation.log 


Actual results: submit the dialog without changing the checkbox at all, the API payload doesn't even list the checkbox as a field.


Expected results:


Additional info:
Comment 2 Shveta 2018-08-22 16:46:24 UTC 
Appliance : https://10.8.199.142
Service : catalog_item
Version - 5.9.4.4.20180816162527_c00eb23
Comment 3 drew uhlmann 2018-08-22 17:40:39 UTC 
I can't test this on master at the moment due to a classic ui issue (https://github.com/ManageIQ/manageiq-ui-classic/issues/4522).
Comment 4 dmetzger 2018-08-23 13:48:53 UTC 
Is this a 5.9.4 regression?
Comment 5 Harpreet Kataria 2018-08-23 23:02:30 UTC 
Drew,

issue mentioned in https://github.com/ManageIQ/manageiq-ui-classic/issues/4522 has been fixed.

~Harpreet
Comment 6 drew uhlmann 2018-08-24 11:46:39 UTC 
Sure Harpreet, and thanks, but that has nothing to do with this bug.
Comment 7 drew uhlmann 2018-08-24 11:47:25 UTC 
And yeah, it's a regression. Will try to figure out exactly when today, sorry, I just haven't had time.
Comment 8 drew uhlmann 2018-08-24 13:13:22 UTC 
Not broken on master.
Comment 9 drew uhlmann 2018-08-24 16:43:09 UTC 
Broken on 5.9.4.3, and it looks like the request itself has the correct information, it's just the api call that doesn't seem to be right.
Comment 10 drew uhlmann 2018-11-14 18:30:35 UTC 
Closing because I think this isn't an issue, please feel free to reopen if anyone still thinks this is still an issue.