commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From commons-...@jakarta.apache.org
Subject [jira] Commented: (JELLY-150) j:forEach tag not properly using varStatus attribute
Date Tue, 05 Oct 2004 00:28:32 GMT
The following comment has been added to this issue:

     Author: Hans Gilde
    Created: Mon, 4 Oct 2004 5:27 PM
       Body:
I'm opposed to doing this. It will mean that we have to bundle the whole JSTL JAR, just to
use one interface. 

Unless we're going to go full JSTL.... couldn't we just include our own, mock implementation
of the interface in question? As long as it has the same methods, it will work just as well.

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

---------------------------------------------------------------------
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 5:27 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: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message