Bug 107928

Summary: c.a.bebop.portal.Portlet doesn't generate XML for children
Product: [Retired] Red Hat Web Application Framework Reporter: Daniel Berrangé <berrange>
Component: uiAssignee: Daniel Berrangé <berrange>
Status: CLOSED RAWHIDE QA Contact: Jon Orris <jorris>
Severity: high Docs Contact:
Priority: medium    
Version: nightly   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-09-02 17:43:52 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Daniel Berrangé 2003-10-24 16:49:20 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 Galeon/1.2.9 (X11; Linux i686; U;) Gecko/20030314

Description of problem:
The c.a.bebop.portal.Portlet component overrides the generateXML method from
SimpleContainer, but neglects to replicate the code that generates XML for the
children components.

This current:

    public void generateXML(PageState pageState,
                            Element parentElement) {
        Element portlet = generateXMLHelper(parentElement);
        generateXMLBody(pageState, portlet);
    }

Should in fact be:

    public void generateXML(PageState pageState,
                            Element parentElement) {
        Element portlet = generateXMLHelper(parentElement);
        generateChildrenXML(pageState, portlet);
        generateXMLBody(pageState, portlet);
    }


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


How reproducible:
Always

Steps to Reproduce:
1. Create a portlet
2. Add a child component
3.

Actual Results:  No XML for child is generated

Expected Results:  XML for child is generated

Additional info:

This is blocking APLAWS-II delivery of the Portals Semantic XML work item

Comment 1 Daniel Berrangé 2003-10-29 15:03:28 UTC
Fixed in p4 37485

Comment 2 David Lawrence 2006-07-18 03:45:02 UTC
QA_READY has been deprecated in favor of ON_QA. Please use ON_QA in the future.
Moving to ON_QA.

Comment 3 Daniel Berrangé 2006-09-02 17:43:52 UTC
Closing old tickets