partkeepr

commit 422239a7e56f9ec64b1c6556ebf03c6b0ad3c92b
parent 60ab3375fb7f347abcdb1d5ea89f1e543eb537cd
Author: Felicitus <felicitus@felicitus.org>
Date:   Sun, 25 Dec 2011 12:49:26 +0100

Don't set the query start if no limit is specified, as this is unsupported by some database platforms

Diffstat:
M
src/backend/de/RaumZeitLabor/PartKeepr/Manager/AbstractManager.php
 | 
2
+-
M
src/backend/de/RaumZeitLabor/PartKeepr/Manager/ManagerFilter.php
 | 
2
+-

2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/backend/de/RaumZeitLabor/PartKeepr/Manager/AbstractManager.php b/src/backend/de/RaumZeitLabor/PartKeepr/Manager/AbstractManager.php
@@ -114,7 +114,7 @@ abstract class AbstractManager extends Singleton {
 		$qb->select($aQueryFields);
 		
 		
-		if ($filter->getStart() !== null) {
+		if ($filter->getStart() !== null && $filter->getLimit() !== null) {
 			$qb->setFirstResult($filter->getStart());
 		}
 		
diff --git a/src/backend/de/RaumZeitLabor/PartKeepr/Manager/ManagerFilter.php b/src/backend/de/RaumZeitLabor/PartKeepr/Manager/ManagerFilter.php
@@ -67,7 +67,7 @@ class ManagerFilter {
 	 * @param mixed $limit Either a positive integer, or null/-1 for no limit
 	 */
 	public function setLimit ($limit) {
-		if ($limit === null || $limit === -1) {
+		if ($limit === null || $limit == -1) {
 			$this->limit = null;
 		} else {
 			$this->limit = intval($limit);