[ https://issues.apache.org/jira/browse/COCOON-2162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12559314#action_12559314 ]
Drew Buschhorn commented on COCOON-2162: ---------------------------------------- Thought about that, but I have two objections: 1. Generally back-ends should be no smarter than necessary: asking for a page that is beyond the created pages should throw an error (not be silently corrected) esp. since one could catch the 404 error in the sitemap and load the last page. 2. I'm not smart enough to know how to access the number of pages created by the paginator outside of the java-class, so I dont know how to find out how many pages it will create without asking it for the largest page that doesnt throw a 404 error. *shrug* tried setting it to catch things <0, but that throws a indexOutOfBounds java error that I didnt feel like learning how to catch. > [PATCH] Fix for Paginator when accessing out of bounds Pagination page > ---------------------------------------------------------------------- > > Key: COCOON-2162 > URL: https://issues.apache.org/jira/browse/COCOON-2162 > Project: Cocoon > Issue Type: Improvement > Components: Blocks: (Undefined) > Affects Versions: 2.1.10, 2.1.11 > Reporter: Drew Buschhorn > Priority: Minor > Attachments: paginator.diff, screenshot-1.jpg > > > The Paginator transformer for apache cocoon will allow out of page-range > requests. > I've added the below logic-test into my own copy of cocoon, and that seems to > fix the problem. > Please let me know if you think 1. this is valid and 2. that it should be > placed into svn for the 2.1.x version of cocoon. > Example: > http://localhost/samples/paginator/text(5) > will return a valid empty file before patch (despite there being only 4 pages > worth of data to paginate) > after patch, returns ResourceNotFound/404. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.