The following issue has been updated:

    Updater: Felipe Leme (mailto:[EMAIL PROTECTED])
       Date: Mon, 9 Aug 2004 5:42 PM
    Comment:
Paul,

Here is the patch I promised, including test cases.

Felipe

PS: I couldn't verify if the new attribute was correctly generated on tags.html, as 
site:generate failed (independently of my patch)

    Changes:
             Attachment changed to JELLY-115.patch
    ---------------------------------------------------------------------
For a full history of the issue, see:

  http://issues.apache.org/jira/browse/JELLY-115?page=history

---------------------------------------------------------------------
View the issue:
  http://issues.apache.org/jira/browse/JELLY-115

Here is an overview of the issue:
---------------------------------------------------------------------
        Key: JELLY-115
    Summary: [PATCH] Add a var attribute to the Break tag.
       Type: Improvement

     Status: Unassigned
   Priority: Minor

    Project: jelly
 Components: 
             core / taglib.core

   Assignee: 
   Reporter: Felipe Leme

    Created: Sat, 7 Aug 2004 2:12 PM
    Updated: Mon, 9 Aug 2004 5:42 PM

Description:
It would be nice if the Break tag could inform if it broke its parent forEach tag. 

Here is an example that could be applied to a Maven plugin 
(http://jira.codehaus.org/browse/MPANNOUNCEMENT-14):

<j:forEach var="version" items="${pom.versions}">
  <j:break var="foundTag" test="${version.id==versionVariable}"/>
</j:forEach>
<j:if test="${!foundTag}"> 
  // core here
</j:if>

I will provide a patch later, once I'm back home.

Felipe



---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to