Bug 25998

Summary: Add 'library' relation to Koha::Account::Line
Product: Koha Reporter: Martin Renvoize (ashimema) <martin.renvoize>
Component: Architecture, internals, and plumbingAssignee: Martin Renvoize (ashimema) <martin.renvoize>
Status: CLOSED FIXED QA Contact: Tomás Cohen Arazi (tcohen) <tomascohen>
Severity: enhancement    
Priority: P5 - low CC: aleisha, kyle, lucas, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.11.00, 20.05.03
Circulation function:
Bug Depends on:    
Bug Blocks: 24381    
Attachments: Bug 25998: Add 'library' relation to Koha::Account::Line
Bug 25998: Add Unit Tests
Bug 25998: Add 'library' relation to Koha::Account::Line
Bug 25998: (QA follow-up) Add DBIC level relation
Bug 25998: Add Unit Tests
Bug 25998: Add 'library' relation to Koha::Account::Line
Bug 25998: (QA follow-up) Add DBIC level relation

Description Martin Renvoize (ashimema) 2020-07-16 11:09:23 UTC
The Koha::Account::Line object is missing an accessor for the 'library' accessor.. this would be really helpful in templates for accounts notices.
Comment 1 Martin Renvoize (ashimema) 2020-07-16 11:10:15 UTC
Created attachment 106945 [details] [review]
Bug 25998: Add 'library' relation to Koha::Account::Line
Comment 2 Tomás Cohen Arazi (tcohen) 2020-07-17 02:48:16 UTC
Create the 'library' DBIC relationship also, so we can prefetch it in the API.
Comment 3 Martin Renvoize (ashimema) 2020-07-17 06:39:25 UTC
(In reply to Tomás Cohen Arazi from comment #2)
> Create the 'library' DBIC relationship also, so we can prefetch it in the
> API.

Good catch.. I forgot we were doing that now :)
Comment 4 Martin Renvoize (ashimema) 2020-07-17 07:02:05 UTC
Created attachment 106971 [details] [review]
Bug 25998: Add Unit Tests
Comment 5 Martin Renvoize (ashimema) 2020-07-17 07:02:08 UTC
Created attachment 106972 [details] [review]
Bug 25998: Add 'library' relation to Koha::Account::Line
Comment 6 Martin Renvoize (ashimema) 2020-07-17 07:02:11 UTC
Created attachment 106973 [details] [review]
Bug 25998: (QA follow-up) Add DBIC level relation
Comment 7 Martin Renvoize (ashimema) 2020-07-17 07:09:45 UTC
Tests added and relation updated dbic side, ready for SO
Comment 8 Tomás Cohen Arazi (tcohen) 2020-07-20 17:27:39 UTC
Created attachment 107122 [details] [review]
Bug 25998: Add Unit Tests

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 9 Tomás Cohen Arazi (tcohen) 2020-07-20 17:27:43 UTC
Created attachment 107123 [details] [review]
Bug 25998: Add 'library' relation to Koha::Account::Line

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 10 Tomás Cohen Arazi (tcohen) 2020-07-20 17:27:48 UTC
Created attachment 107124 [details] [review]
Bug 25998: (QA follow-up) Add DBIC level relation

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 11 Tomás Cohen Arazi (tcohen) 2020-07-20 17:34:38 UTC
Trivial patchset with no QA errors and following the current coding guidelines.
Passing QA
Comment 12 Jonathan Druart 2020-07-22 08:31:52 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 13 Lucas Gass (lukeg) 2020-07-28 14:58:16 UTC
backported to 20.05.x for 20.05.03
Comment 14 Aleisha Amohia 2020-08-03 23:30:19 UTC
enhancement, not backporting to 19.11.x