From 0e8b468e1b1902939b2b0933f217446bae65ae4f Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Fri, 21 May 2010 09:34:42 -0400 Subject: [PATCH] bug 4801: fix paging in display of staged bibs and import batches Fixed glitch where the first page of bibs in a batch (or the first page of import batches) was displaying the entire list instead of the correct number of records per page. Signed-off-by: Galen Charlton --- C4/ImportBatch.pm | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/C4/ImportBatch.pm b/C4/ImportBatch.pm index ede33dd658..3fdfb8e22d 100644 --- a/C4/ImportBatch.pm +++ b/C4/ImportBatch.pm @@ -775,11 +775,11 @@ sub GetImportBatchRangeDesc { WHERE batch_type = 'batch' ORDER BY import_batch_id DESC"; my @params; + if ($results_per_group){ + $query .= " LIMIT ?"; + push(@params, $results_per_group); + } if ($offset){ - if ($results_per_group){ - $query .= " LIMIT ?"; - push(@params, $results_per_group); - } $query .= " OFFSET ?"; push(@params, $offset); } @@ -856,11 +856,11 @@ sub GetImportBibliosRange { } $query.=" ORDER BY import_record_id"; + if($results_per_group){ + $query .= " LIMIT ?"; + push(@params, $results_per_group); + } if($offset){ - if($results_per_group){ - $query .= " LIMIT ?"; - push(@params, $results_per_group); - } $query .= " OFFSET ?"; push(@params, $offset); } -- 2.39.5