The following comment has been added to this issue:

     Author: Ben Anderson
    Created: Mon, 4 Oct 2004 12:40 PM
       Body:
The interface is defined here:
http://java.sun.com/products/jsp/jstl/1.1/docs/api/javax/servlet/jsp/jstl/core/LoopTagSupport.html

There was some discussion following this on the mailing list. Personally, I don't see 
why it can't go in javax.servlet, but it should be fine to put it in some other 
package.  Does your "unified format" comment pertain to the diff file I posted?  This 
is the first patch I've ever submitted.  I wasn't sure what to do with the new files, 
so I appended them to that file.  I'm assuming this is not the way to do it, but I 
didn't know how.  If someone will let me know how I'm supposed to do it, I'll resubmit 
the correctly formatted file.

---------------------------------------------------------------------
View this comment:
  http://issues.apache.org/jira/browse/JELLY-150?page=comments#action_53683

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

Here is an overview of the issue:
---------------------------------------------------------------------
        Key: JELLY-150
    Summary: j:forEach tag not properly using varStatus attribute
       Type: Bug

     Status: Unassigned
   Priority: Major

    Project: jelly
 Components: 
             core / taglib.core
   Versions:
             1.0-beta-4
             1.0
             1.1-beta-1
             jface-1.0-beta-1
             jface-1.0
             swt-1.0
             1.0-beta-5

   Assignee: 
   Reporter: Ben Anderson

    Created: Fri, 24 Sep 2004 4:57 AM
    Updated: Mon, 4 Oct 2004 12:40 PM
Environment: win2k / cygwin

Description:
According to the jstl specification 1.1, the varStatus attribute of the forEach tag 
should create a variable with type javax.servlet.jsp.jstl.core.LoopTagStatus.  
Currently a variable is set with type Integer which contains the index of the current 
iteration.  I created a test case (I don't think any existed for forEach) that 
demonstrates this.  I also began making the fix, but it doesn't fully work.  I am 
submitting my patch along with the test cases.  The comments I placed in the code 
should be enough to tip someone off as to why it's not working (I hope).  I am 
relatively new to jelly (been using the standard taglibs for awhile) and have never 
opened the source before.  I am willing to complete the patch, but will probably need 
some guidance because I messed with it for awhile before determing I couldn't complete 
without some help.  Thanks! :-)


---------------------------------------------------------------------
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