mraible commented on code in PR #219:
URL: https://github.com/apache/roller/pull/219#discussion_r4409058943
##########
app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Entries.java:
##########
@@ -122,7 +116,7 @@ public String execute() {
// build entries pager
String baseUrl = buildBaseUrl();
- setPager(new EntriesPager(baseUrl, getBean().getPage(), entries,
hasMore));
+ setPager(new EntriesPager(baseUrl, getBean().getPage(), entries,
totalCount, COUNT));
Review Comment:
This predates the PR, but the new page count makes it easier to hit.
`buildBaseUrl()` puts the category under `bean.categoryPath` (line 131), while
`EntriesBean` and the sidebar use `bean.categoryName`. So with a category
filter, Next and Last open a page of the unfiltered list, even though "Page 1
of M" was counted with the filter. Changing that key to `bean.categoryName`
would fix it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]