From fc5d83002bb3b752c3ef68ac714abf9128b91f7e Mon Sep 17 00:00:00 2001
From: Demian Katz <demian.katz@villanova.edu>
Date: Wed, 18 Jul 2012 10:41:27 -0400
Subject: [PATCH] Fixed filtering by list.
---
module/VuFind/src/VuFind/Search/Favorites/Results.php | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/module/VuFind/src/VuFind/Search/Favorites/Results.php b/module/VuFind/src/VuFind/Search/Favorites/Results.php
index 62b8537c28c..95a4fb009a0 100644
--- a/module/VuFind/src/VuFind/Search/Favorites/Results.php
+++ b/module/VuFind/src/VuFind/Search/Favorites/Results.php
@@ -146,10 +146,10 @@ class Results extends BaseResults
// How many results were there?
$resource = new ResourceTable();
+ $userId = is_null($list) ? $this->user->id : $list->user_id;
+ $listId = is_null($list) ? null : $list->id;
$rawResults = $resource->getFavorites(
- is_null($list) ? $this->user->id : $list->user_id,
- isset($list->id) ? $list->id : null,
- $this->getTagFilters(), $this->getSort()
+ $userId, $listId, $this->getTagFilters(), $this->getSort()
);
$this->resultTotal = count($rawResults);
@@ -157,9 +157,7 @@ class Results extends BaseResults
$limit = $this->getLimit();
if ($this->resultTotal > $limit) {
$rawResults = $resource->getFavorites(
- is_null($list) ? $this->user->id : $list->user_id,
- isset($list->id) ? $list->id : null,
- $this->getTagFilters(), $this->getSort(),
+ $userId, $listId, $this->getTagFilters(), $this->getSort(),
$this->getStartRecord() - 1, $limit
);
}
--
GitLab