Bug 7340 - TLV incorrectly flagging Error against import action when param tag has body content.
Summary: TLV incorrectly flagging Error against import action when param tag has body ...
Status: CLOSED FIXED
Alias: None
Product: Taglibs
Classification: Unclassified
Component: Standard Taglib (show other bugs)
Version: unspecified
Hardware: All All
: P3 major (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-03-21 23:47 UTC by Ryan Lubke
Modified: 2004-11-16 19:05 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Lubke 2002-03-21 23:47:44 UTC
Given:

<c:import url="import.jsp">
   <c:param name="parm">
       value
   </c:param>
</c:import>

Result:

Validation error messages from tag library c

null: Encountered illegal body of tag "c:import" tag, given its attributes.

null: Encountered illegal body of tag "c:import" tag, given its attributes.

This also happens for the RT based tag as well.
It should also be noted that a similar issue will occur if the body content
of the param tag is another action:

<c:param name="parm">
   <acme:action/>
</c:param>

However, the following *will* work:

<c:param name="parm">
</c:param>


JSTL PD spec clearly shows that the param action can set the value
of the param via the body content.
Comment 1 Ryan Lubke 2002-03-27 22:17:36 UTC
Error info for when the param action has another tag in its body and embedded
within an import action:

Validation error messages from tag library c-rt
null: Illegal child tag in "c-rt:import" tag: "acme:anviltag" tag

Validation error messages from tag library c
null: Illegal child tag in "c:import" tag: "acme:anviltag" tag

Comment 2 Shawn Bayern 2002-03-30 21:06:12 UTC
Jeez; I must have been distracted or drunk when I wrote the TLV code that 
checked <c:import>'s constraints; it's given us nothing but trouble.  (This 
isn't the first bug about it.)  I've now fixed it, hopefully for the last 
time.  :-)