Bug 1613987 - Dynamic Text Area and Text Box Elements Load Even Though Load on Init is not Marked
Summary: Dynamic Text Area and Text Box Elements Load Even Though Load on Init is not ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Automate
Version: 5.9.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.10.0
Assignee: eclarizi
QA Contact: Shveta
URL:
Whiteboard:
Depends On:
Blocks: 1623560
TreeView+ depends on / blocked
 
Reported: 2018-08-08 18:06 UTC by myoder
Modified: 2022-03-13 15:22 UTC (History)
8 users (show)

Fixed In Version: 5.10.0.11
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1623560 (view as bug list)
Environment:
Last Closed: 2019-02-12 16:51:08 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.