Bug 1613987

Summary: Dynamic Text Area and Text Box Elements Load Even Though Load on Init is not Marked
Product: Red Hat CloudForms Management Engine Reporter: myoder
Component: AutomateAssignee: eclarizi
Status: CLOSED CURRENTRELEASE QA Contact: Shveta <sshveta>
Severity: high Docs Contact:
Priority: high    
Version: 5.9.0CC: dmetzger, gmccullo, jcutter, mkanoor, obarenbo, simaishi, smallamp, tfitzger
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.10.0.11 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1623560 (view as bug list) Environment:
Last Closed: 2019-02-12 16:51:08 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1623560    

Description myoder 2018-08-08 18:06:11 UTC 
Description of problem:

Dynamic text area and text box elements are loading twice when the service dialog is loaded, even though "Load values on init" is marked "No".  This behavior does not happen with dynamic drop downs or dynamic radio buttons.


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

How reproducible:
always

Steps to Reproduce:
1. Create a service dialog with a dynamic text area and/or text box that do not load values on init.
2. Create a custom button that uses service dialog
3. Trigger service dialog via the custom button

Actual results:
Each dynamic text area and text box are loaded 2 times

Expected results:
The dynamic text area and test box elements should not be loaded at all, until the refresh button is clicked.

Additional info:
I have reproduced this on appliance 10.13.153.115 with default credentials.

The custom button is found on the VM summary screen, Yoder => yoder_test.

The button "yoder_test" triggers the service dialog 'yoder_test'.  This dialog has 4 dynamic elements (area box, text box, drop down and radio).  The automation.log clearly states which dynamic mehthod is being run as so for the dynamic text box element:

[----] I, [2018-08-08T13:31:38.966506 #29641:5b244a4]  INFO -- : <AEMethod box> ##################################################
[----] I, [2018-08-08T13:31:38.968893 #29641:5b244a4]  INFO -- : <AEMethod box> ############# BOX  DIALOG ########################
[----] I, [2018-08-08T13:31:38.970272 #29641:5b36c1c]  INFO -- : <AEMethod drop> ##################################################
Comment 2 myoder 2018-08-08 18:07:52 UTC 
This PR was submitted and fixes this issue:

https://github.com/ManageIQ/manageiq/pull/17814
Comment 3 Jeffrey Cutter 2018-08-08 18:29:37 UTC 
The steps to reproduce should be from a catalog item, not a custom button.  I had trouble getting custom button to work at all, but that's a separate issue.

Dialog items shouldn't load on init, only from trigger or refresh button.
Comment 5 Jeffrey Cutter 2018-08-10 18:53:29 UTC 
PR has been merged:

https://github.com/ManageIQ/manageiq/pull/17814

Will this BZ suffice to get it back-ported?

Is 5.9.4 possible?

Thanks,
-Jeff
Comment 7 Shveta 2018-09-28 18:11:05 UTC 
Works fine in 5.10.0.17.20180927011235_1b5cf54