Summary: | Downloaded saved reports do not contain all records | ||
---|---|---|---|
Product: | Koha | Reporter: | Chris Cormack <chris> |
Component: | Reports | Assignee: | Galen Charlton <gmcharlt> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, cookie.wolfrom, dschust1, jwagner, koha.sekjal, mcoalwell, nengard, wizzyrea |
Version: | rel_3_2 | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | |||
Bug Blocks: | 34840 | ||
Attachments: | Followup patch to bug 3419 |
Description
Chris Cormack
2010-05-21 01:10:17 UTC
There is a small typo in this patch: } elsif ( ! $offset && $limit > $userlimit ) { should read: } elsif ( ! $offset && $limit < $userlimit ) { otherwise the userlimit is only being respected if it's below the code's inherent limit of 9999, which does not address the complaint in this bug report. Created attachment 2349 [details] [review] Followup patch to bug 3419 This patch corrects the inequality typo in the already-pushed patch. Pushed follow-up patch. Please test and close. Functional for the last few months without recurrence. Closing. |