| 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: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | 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. |