Summary: | All usage of GetMemberDetails on reserve/request.pl is unnecessary | ||
---|---|---|---|
Product: | Koha | Reporter: | Ian Walls <koha.sekjal> |
Component: | Architecture, internals, and plumbing | Assignee: | Ian Walls <koha.sekjal> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | minor | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, nengard |
Version: | unspecified | ||
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: | 7058 | ||
Attachments: |
Proposed Patch
[SIGNED-OFF] Bug 7059: Remove GetMemberDetails from reserve/request.pl |
Description
Ian Walls
2011-10-18 22:27:30 UTC
Created attachment 5968 [details] [review] Proposed Patch Changes all calls in reserve/request.pl from GetMemberDetails to GetMember, since the extra data pulled in is not worth the required CPU cycles. Created attachment 5995 [details] [review] [SIGNED-OFF] Bug 7059: Remove GetMemberDetails from reserve/request.pl Replaces all four usages of GetMemberDetails with GetMember, since none of the hashref fields used were anything other than those in borrower table, which GetMember returns. To Test: 1. Apply patch 2. Go to reserve/request.pl; the page should load without error 3. Search for a cardnumber: the page should load without error Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Passed QA, RM please verify Pushed, please test There have been no further reports of problems so I am marking this bug resolved. |