bug 4801: fix paging in display of staged bibs and import batches
authorGalen Charlton <gmcharlt@gmail.com>
Fri, 21 May 2010 13:34:42 +0000 (09:34 -0400)
committerChris Cormack <chrisc@catalyst.net.nz>
Mon, 24 May 2010 18:55:40 +0000 (06:55 +1200)
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 <gmcharlt@gmail.com>
C4/ImportBatch.pm

index ede33dd..3fdfb8e 100644 (file)
@@ -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);
     }