| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 2.5.1 |
| Request #11697 | Pager_Jumping::build() lose first and last page string concatenation | ||
|---|---|---|---|
| Submitted: | 2007-07-28 16:35 UTC | ||
| From: | chihwen | Assigned: | quipo |
| Status: | Closed | Package: | Pager (version 2.4.3) |
| PHP Version: | Irrelevant | OS: | |
| Roadmaps: | (Not assigned) | ||
[2007-07-28 16:35 UTC] chihwen
(Chih-Wen Yang)
Description:
------------
The class variable $this->links in Pager_Jumping::build() lose first and last page string concatenation.
if ($this->_totalPages > $this->_delta) {
$this->links .= $this->_printFirstPage();
}
if ($this->_totalPages > $this->_delta) {
$this->links .= $this->_printLastPage();
}
But force to do this...
$links = $pager->getLinks($pager->getCurrentPageID());
Comments
[2007-07-28 16:45 UTC] quipo
(Lorenzo Alberton)
That's the expected behaviour. If all the links are displayed in the page (because the n. of pages is less than delta), there's no need for extra links to the first and last page, they would be a duplicate. You already have them in the page.
[2007-07-29 03:55 UTC] chihwen
(Chih-Wen Yang)
1. Suppose the pageID is 26
2. and Pager options is...
$params = array(
'mode' => 'Jumping',
'perPage' => 1,
'delta' => 10,
'totalItems' => 100,
);
3. Then $links = $pager->getLinks();
the $links['all'] => << Back 21 22 23 24 25 26 27 28 29 30 Next >>
4. But $links = $pager->getLinks(26); (Or $links = $pager->getLinks($pager->getCurrentPageID());)
the $links['all'] => [1] << Back 21 22 23 24 25 26 27 28 29 30 Next >> [100]
The behavior is not consistent.
[2007-07-29 07:28 UTC] quipo
(Lorenzo Alberton)
Then I didn't understand what you mean. Can you post a simple reproducing script?