The pagedViewLinks velocity macro should be improve to:
- support adding other query parameters to the URL
- reflect the current page number in the URL (right now the displayed page number is one more than the one in the URL)
- have the option to always display the links, even when there is only one page
- not have side effects on the velocity context
- better documentation
Since the current macro is already in use, and velocity does not allow another macro with the same name, but different signature, then the current macro should be preserved, and write another macro.
Additionally, another macro could be added to ease the job of parsing back the query string into a proper range of items.