Summary: | Add the Koha::Checkout->patron method | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | dcook, kyle, veron |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 18402 | ||
Attachments: |
Bug 18401: Add new method Koha::Checkout->patron
Bug 18401: Add new method Koha::Checkout->patron Bug 18401: Add new method Koha::Checkout->patron |
Description
Jonathan Druart
2017-04-07 21:15:07 UTC
Created attachment 61979 [details] [review] Bug 18401: Add new method Koha::Checkout->patron Return the patron related to a given checkout Test plan: prove t/db_dependent/Koha/Checkouts.t should return green Created attachment 61985 [details] [review] Bug 18401: Add new method Koha::Checkout->patron Return the patron related to a given checkout Test plan: prove t/db_dependent/Koha/Checkouts.t should return green Signed-off-by: Marc Véron <veron@veron.ch> Tested on master, setting Version from unspecified to master Created attachment 62700 [details] [review] Bug 18401: Add new method Koha::Checkout->patron Return the patron related to a given checkout Test plan: prove t/db_dependent/Koha/Checkouts.t should return green Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Pushed to master for 17.05, thanks Jonathan! This won't get ported back to 16.11.x as it is an enhancement. Found a bug with this when the checkout has a null borrowernumber. Will spin out a new report. (In reply to David Cook from comment #7) > Found a bug with this when the checkout has a null borrowernumber. Will spin > out a new report. Nevermind. It's fixed by #21432. |