Issue 180: Add a zero-based absolute index variable to TableFacadeTag
Status:  WontFix
Owner: ----
Closed:  May 2009
Reported by allenpparslow@gmail.com, Feb 16, 2009
Enhancement:
Add a zero-based absolute index variable to TableFacadeTag (see indexVar in 
the attached revision).

What steps will reproduce the problem?
n/a

What is the expected output? What do you see instead?
TableFacadeTag.doTag() exports the relative position within a page via 
statically named attribute "rowcount". This feature is useful.
However the zero-based absolute index would be even more useful. For 
example, binding in spring webflow 2.x requires a path containing such 
index.

What version of the product are you using? On what operating system?
jmesa 2.4.1 / WindowsXP / Firefox 3.0.6

Attached is the suggested revision to TableFacadeTag.java and jmesa.tld.


TableFacadeTag.java
15.3 KB   View   Download
jmesa.tld
17.6 KB   View   Download
Feb 16, 2009
Project Member #1 jeff.johnston.mn@gmail.com
Thanks! I'll take a look at it this week and then get back to you...
Labels: -Type-Defect Type-Enhancement
Feb 23, 2009
Project Member #2 jeff.johnston.mn@gmail.com
The most transparent way to implement this feature would be to have another
preference attribute that would tell us what the starting index should be.

The default would start at 1, and then you could override it to be 0. The only place
in the code that would change is in the HtmlUtils.startingRowcount() method.

Something like ROWCOUNT_STARTING_INDEX would work, which would correspond to a 
html.rowcount.startingIndex=0 or html.rowcount.startingIndex=1.

Do you want to try implementing and testing this? If so then it is fairly easy to
checkout and build the project yourself. The directions on how to do this is on the
wiki at https://code.google.com/p/jmesa/wiki/ProjectBuild


Status: Accepted
May 8, 2009
Project Member #3 jeff.johnston.mn@gmail.com
(No comment was entered for this change.)
Status: WontFix