[TILES-519] tiles:getAsString is not cleaning tiles context when attribute is not present and ignore=true

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.2
• Fix Version/s: 2.2.3
• Component/s: tiles-template
• Flags: Patch
• Assignee: Antonio Petrelli
• Reporter: Vitaly Polonetsky
• Created: 21/Jan/11 09:46
• Updated: 21/Jan/11 20:47
• Resolved: 21/Jan/11 20:47

Description

After putting this line in the template:
<tiles:getAsString name="subtitle" ignore="true"/>
<tiles:insertAttribute name="body" />

And if "subtitle" attribute is not defined. The page will fail on rendering the "body" attribute, because getAsString failed to cleanup it's tiles context (endContext() is not executed).

SOLUTION:
The problem is in GetAsStringModel class, in the last method - renderAttribute():

The first three lines that check for null attribute and ignore flag should be inside the try caluse:
try {
if (attribute == null && ignore)

{ return; }

writer.write(...

This way the finally clause will be executed even in attribute is null and should be ignored. The finally clause is ending the tiles context.

This is the same as the code inside: InsertAttributeModel.renderAttribute()