Bug 21721

Summary: Update C4::Circulation::AddRenewal to use Koha::Account->add_debit
Product: Koha Reporter: Martin Renvoize <martin.renvoize>
Component: Architecture, internals, and plumbingAssignee: Martin Renvoize <martin.renvoize>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: nick, tomascohen
Version: master   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21678
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 21720    
Bug Blocks: 20912, 21756    
Attachments: Bug 21721: Use Koha::Account->add_debit in AddRenewal
Bug 21721: Use Koha::Account->add_debit in AddRenewal
Bug 21721: Use Koha::Account->add_debit in AddRenewal
Bug 21721: Use Koha::Account->add_debit in AddRenewal
Bug 21721: Use Koha::Account->add_debit in AddRenewal
Bug 21721: Use Koha::Account->add_debit in AddRenewal
Bug 21721: Use Koha::Account->add_debit in AddRenewal

Description Martin Renvoize 2018-10-30 09:29:47 UTC
All account handling should go via the Koha::Account class.
Comment 1 Martin Renvoize 2018-11-02 11:08:35 UTC
Created attachment 81884 [details] [review]
Bug 21721: Use Koha::Account->add_debit in AddRenewal
Comment 2 Martin Renvoize 2018-11-02 11:12:13 UTC
Created attachment 81887 [details] [review]
Bug 21721: Use Koha::Account->add_debit in AddRenewal
Comment 3 Martin Renvoize 2018-11-02 11:22:39 UTC
Created attachment 81888 [details] [review]
Bug 21721: Use Koha::Account->add_debit in AddRenewal
Comment 4 Martin Renvoize 2018-12-14 15:26:15 UTC
Created attachment 83219 [details] [review]
Bug 21721: Use Koha::Account->add_debit in AddRenewal

Known Side Effect: Prior to this patch renewal charges were not
recorded in the FinesLog. After this patch, if the FinesLog is
enabled then the 'action' will be recorded as `create_rent`.

Sponsored-by: PTFS Europe
Comment 5 Josef Moravec 2019-01-22 09:05:05 UTC
Created attachment 84275 [details] [review]
Bug 21721: Use Koha::Account->add_debit in AddRenewal

Known Side Effect: Prior to this patch renewal charges were not
recorded in the FinesLog. After this patch, if the FinesLog is
enabled then the 'action' will be recorded as `create_rent`.

Sponsored-by: PTFS Europe

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 6 Tomás Cohen Arazi 2019-02-22 14:29:46 UTC
Created attachment 85529 [details] [review]
Bug 21721: Use Koha::Account->add_debit in AddRenewal

Known Side Effect: Prior to this patch renewal charges were not
recorded in the FinesLog. After this patch, if the FinesLog is
enabled then the 'action' will be recorded as `create_rent`.

Sponsored-by: PTFS Europe

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 7 Nick Clemens 2019-03-04 19:25:44 UTC
Dependencies pushed to master, yet:
error: sha1 information is lacking or useless (C4/Circulation.pm).
error: could not build fake ancestor
Patch failed at 0001 Bug 21721: Use Koha::Account->add_debit in AddRenewal
Comment 8 Martin Renvoize 2019-03-05 13:19:08 UTC
Created attachment 86065 [details] [review]
Bug 21721: Use Koha::Account->add_debit in AddRenewal

Known Side Effect: Prior to this patch renewal charges were not
recorded in the FinesLog. After this patch, if the FinesLog is
enabled then the 'action' will be recorded as `create_rent`.

Sponsored-by: PTFS Europe

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 9 Martin Renvoize 2019-03-05 13:21:15 UTC
Wierd.. I've just rebased my accounts branch and it seems happy.. attached the patch from that

https://github.com/PTFS-Europe/koha/compare/master...PTFS-Europe:accounts_all is where I'm tracking this little lot if it helps
Comment 10 Nick Clemens 2019-03-07 17:41:10 UTC
Awesome work all!

Pushed to master for 19.05
Comment 11 Martin Renvoize 2019-03-12 08:34:00 UTC
Enhancement will not be backported to 18.11.x series.