aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--views/default/navigation/pagination.php38
1 files changed, 12 insertions, 26 deletions
diff --git a/views/default/navigation/pagination.php b/views/default/navigation/pagination.php
index e20891070..e0c57637d 100644
--- a/views/default/navigation/pagination.php
+++ b/views/default/navigation/pagination.php
@@ -7,39 +7,25 @@
* @author Curverider Ltd
* @link http://elgg.org/
*
+ * @uses int $vars['offset']
+ * @uses int $vars['limit']
+ * @uses int $vars['count'] Number of entities.
+ * @uses string $vars['word'] Word to use in GET params for the offset
+ * @uses string $vars['baseurl'] Base URL to use in links
*/
-if (!isset($vars['offset'])) {
- $offset = 0;
-} else {
- $offset = $vars['offset'];
-}
-if ((!isset($vars['limit'])) || (!$vars['limit'])) {
+$offset = (int) elgg_get_array_value('offset', $vars, 0);
+// because you can say $vars['limit'] = 0
+if (!$limit = (int) elgg_get_array_value('limit', $vars, 10)) {
$limit = 10;
-} else {
- $limit = (int)$vars['limit'];
-}
-if (!isset($vars['count'])) {
- $count = 0;
-} else {
- $count = $vars['count'];
-}
-if (!isset($vars['word'])) {
- $word = "offset";
-} else {
- $word = $vars['word'];
-}
-if (isset($vars['nonefound'])) {
- $nonefound = $vars['nonefound'];
-} else {
- $nonefound = true;
}
+$count = (int) elgg_get_array_value('count', $vars, 0);
+$word = elgg_get_array_value('word', $vars, 'offset');
+$baseurl = elgg_get_array_value('baseurl', $vars, current_page_url());
$totalpages = ceil($count / $limit);
$currentpage = ceil($offset / $limit) + 1;
-$baseurl = isset($vars['baseurl']) ? $vars['baseurl'] : current_page_url();
-
//only display if there is content to paginate through or if we already have an offset
if (($count > $limit || $offset > 0) && get_context() != 'widget') {
@@ -123,4 +109,4 @@ if (($count > $limit || $offset > 0) && get_context() != 'widget') {
?>
</div>
<?php
-} // end of pagination check if statement \ No newline at end of file
+} \ No newline at end of file