summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTakayuki Sakai <kaky0922@gmail.com>2014-06-13 11:12:02 +0200
committerTakayuki Sakai <kaky0922@gmail.com>2014-06-13 11:12:02 +0200
commit1240b6a04ee4e6a200cc882481f09fce6a7eb4fc (patch)
tree37d6ab2d685fafbf0ccb2b721343f5dbedc88f69
parenta01924d2cc2d63d2bddbee85a18b423b88a9b19e (diff)
Revert "Make num_links=0 in pagination library possible to configure"
-rw-r--r--system/libraries/Pagination.php24
1 files changed, 9 insertions, 15 deletions
diff --git a/system/libraries/Pagination.php b/system/libraries/Pagination.php
index d079d835d..5b9bfcb5d 100644
--- a/system/libraries/Pagination.php
+++ b/system/libraries/Pagination.php
@@ -393,9 +393,9 @@ class CI_Pagination {
// Check the user defined number of links.
$this->num_links = (int) $this->num_links;
- if ($this->num_links < 0)
+ if ($this->num_links < 1)
{
- show_error('Your number of links must be a non-negative number.');
+ show_error('Your number of links must be a positive number.');
}
// Keep any existing query string items.
@@ -535,14 +535,11 @@ class CI_Pagination {
// Render the "First" link.
if ($this->first_link !== FALSE && $this->cur_page > ($this->num_links + 1))
{
- if (($this->num_links === 0 && $this->cur_page < 3) !== true)
- {
- // Take the general parameters, and squeeze this pagination-page attr in for JS frameworks.
- $attributes = sprintf('%s %s="%d"', $this->_attributes, $this->data_page_attr, 1);
+ // Take the general parameters, and squeeze this pagination-page attr in for JS frameworks.
+ $attributes = sprintf('%s %s="%d"', $this->_attributes, $this->data_page_attr, 1);
- $output .= $this->first_tag_open.'<a href="'.$first_url.'"'.$attributes.$this->_attr_rel('start').'>'
+ $output .= $this->first_tag_open.'<a href="'.$first_url.'"'.$attributes.$this->_attr_rel('start').'>'
.$this->first_link.'</a>'.$this->first_tag_close;
- }
}
// Render the "Previous" link.
@@ -614,15 +611,12 @@ class CI_Pagination {
// Render the "Last" link
if ($this->last_link !== FALSE && ($this->cur_page + $this->num_links) < $num_pages)
{
- if (($this->num_links === 0 && ($this->cur_page + 1) >= $num_pages) !== true)
- {
- $i = ($this->use_page_numbers) ? $num_pages : ($num_pages * $this->per_page) - $this->per_page;
+ $i = ($this->use_page_numbers) ? $num_pages : ($num_pages * $this->per_page) - $this->per_page;
- $attributes = sprintf('%s %s="%d"', $this->_attributes, $this->data_page_attr, (int) $i);
+ $attributes = sprintf('%s %s="%d"', $this->_attributes, $this->data_page_attr, (int) $i);
- $output .= $this->last_tag_open.'<a href="'.$base_url.$this->prefix.$i.$this->suffix.'"'.$attributes.'>'
- .$this->last_link.'</a>'.$this->last_tag_close;
- }
+ $output .= $this->last_tag_open.'<a href="'.$base_url.$this->prefix.$i.$this->suffix.'"'.$attributes.'>'
+ .$this->last_link.'</a>'.$this->last_tag_close;
}
// Kill double slashes. Note: Sometimes we can end up with a double slash