Stripes

Components Nested Inside Other Components Do NOT Render

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: Release 1.5.4
  • Fix Version/s: Release 1.5.4
  • Component/s: Tag Library
  • Labels:
    None
  • Environment:
    Mac OSX 10.5.8; JDK 1.6.0_20; MyEclipseIDE 8.6

Description

The true power in any Layout engine is in being able to create a hierarchy of layout definitions and renderers however this bug impedes complex layouts and allows only rudimentary / trivial / tutorial like layouts. Understandably 1.5.4 is not even beta yet so its hard to complain however 1.5.4 has sufficient features that make it very worthwhile over 1.5.3 however this bug makes using its layout component an unfortunate show stopper.

Using the code snippets below the output is as follows on 1.5.4 (latest):
Before contents
Before test
After test
After contents

However on at least 1.5.2 and latest trunk (as per Freddy's feedback) it correctly provides:
Before contents
Before test
Hello World!!!!!!!!!!!!!!
After test
After contents

Were the code is as follows:

html_template2.jsp
==============
<%@ include file="/WEB-INF/jsp/x4/taglibs.jsp" %>
<s:layout-definition>
Before contents
${contents}
After contents
</s:layout-definition>

3c2.jsp
=====
<%@ include file="/WEB-INF/jsp/x4/taglibs.jsp" %>
<s:layout-definition>
<s:layout-render name="/WEB-INF/jsp/x4/base/html_template2.jsp">
<s:layout-component name="contents">
Before test
${test}
After test
</s:layout-component>
</s:layout-render>
</s:layout-definition>

test.jsp
=====
<%@ include file="/WEB-INF/jsp/x4/taglibs.jsp" %>
<s:layout-render name="/WEB-INF/jsp/x4/base/3c2.jsp">
<s:layout-component name="test">
Hello World!!!!!!!!!!!!!!
</s:layout-component>
</s:layout-render>

Issue Links

Activity

Hide
Ben Gunter added a comment - 27/Sep/10 10:35 AM

This particular problem is fixed. I tested using the samples you gave in the bug report. Please let me know if you still have this problem or if you encounter any other troubles with the new layout code.

Show
Ben Gunter added a comment - 27/Sep/10 10:35 AM This particular problem is fixed. I tested using the samples you gave in the bug report. Please let me know if you still have this problem or if you encounter any other troubles with the new layout code.

People

Vote (0)
Watch (0)

Dates

  • Created:
    29/Aug/10 4:35 PM
    Updated:
    04/Jan/11 12:54 PM
    Resolved:
    27/Sep/10 10:35 AM