I don't think reset() is the right answer. That method should only be invoked by the container. If tomcat is not following the spec and invoking the reset() method incorrectly, then the problem is within that build of tomcat. I've been using resin and have not seen such a problem. I think Zorzella has the right idea. We should make sure that we have valid parameters before using them.
John On Wed, 19 Mar 2003, Dave Hodson wrote: > I'm using .8 version of the display tag with Tomcat 4.0.3 and everything works > properly > > Recently, I upgraded to Tomcat version 4.1.18 and have noticed that the paging > functionality does not work properly. > For example, I create a table that has 8 pages by selecting a valid date range in an > HTML popup. Upon the initial display of the the table, Page 1 is shown. I then click > on Page 6, which is displayed properly. Next I modify my date range so that the > resulting table consists of only 2 pages. This crashes with the msg > > "Invalid page (6) provided, value should be between 1 and 2'" > > Digging around, it turns out that Tomcat 4.1.18 no longer calls the reset() method > between calls and this effectively leaves the value "pageNumber" at 6 the next time > a user hits the screen (the gory details about the Tomcat bug are at > http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16001) This bug has been marked > INVALID, so I don't think it will be fixed anytime soon. > > Has anyone seen this problem and/or is there a fix for this? Seems like the thing to > do is call the reset() method in the doStartTag() method, before the request object > is obtained. Something like > > columns = new ArrayList( 10 ); > reset(); <------ add it here > HttpServletRequest req = (HttpServletRequest)this.pageContext.getRequest(); > > Thoughts? > > Dave > > > > > > > > ------------------------------------------------------- > This SF.net email is sponsored by: Does your code think in ink? > You could win a Tablet PC. Get a free Tablet PC hat just for playing. > What are you waiting for? > http://ads.sourceforge.net/cgi-bin/redirect.pl?micr5043en > _______________________________________________ > displaytag-devel mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/displaytag-devel > -- John York Software Engineer CareerSite Corporation ------------------------------------------------------- This SF.net email is sponsored by: Does your code think in ink? You could win a Tablet PC. Get a free Tablet PC hat just for playing. What are you waiting for? http://ads.sourceforge.net/cgi-bin/redirect.pl?micr5043en _______________________________________________ displaytag-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/displaytag-devel
