Bugs item #823383, was opened at 2003-10-14 12:04
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=536613&aid=823383&group_id=73068

Category: main tag library
Group: v 1.0-b1
Status: Open
Resolution: None
Priority: 5
Submitted By: Paul McCulloch (paulmcculloch)
Assigned to: Nobody/Anonymous (nobody)
Summary: Body not cleared with pooled ColumnTags

Initial Comment:
There seems to be a bug in the way the column tag is
validating. It should throw an exception if there is
neither a property, a value or some body content in the
tag.

However, this doesn't always work:

<display:column title="Col 1" sortable="true">
  XXXXXX
</display:column>       
<display:column title="Col 2" sortable="true"/>

Doesn't produce an error.

This is because the result of getBodyContent() in
ColumnTag.java::doEndTag() is "XXXXXX", rather than
null for the second column (due to tag pooling I assume).

It's unclear to me whether this is the responsibility
of the container (Tomcat 4.1.18) or the tag itself.

setBodyContent(null); at the end of doEndTag() fixes
things - but I'm not sure if it's neccessary.

Although this isn't much of an issue iby itself, I'm
working on an enhancement which would allow the
specification of a property & a body (display the body,
sort by the property) where this is an issue - I'd
appreciate an opinion on whether this is a container or
tag issue.

(v1.0-b1, Tomcat 4.1.18, JDK 1.4.1_04)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=536613&aid=823383&group_id=73068


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
SourceForge.net hosts over 70,000 Open Source Projects.
See the people who have HELPED US provide better services:
Click here: http://sourceforge.net/supporters.php
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to