Bug 22533

Summary: Cannot create manual invoices
Product: Koha Reporter: Nick Clemens <nick>
Component: Fines and feesAssignee: Liz Rea <wizzyrea>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: blocker    
Priority: P5 - low CC: m.de.rooy, martin.renvoize, tomascohen, wizzyrea
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22393
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 14591    
Bug Blocks:    
Attachments: Bug 22533: maninvoice.pl is using deprecated maininvoice() method
Bug 22533: maninvoice.pl is using deprecated maininvoice() method
Bug 22533: Fix manual invoices from patron accounting
Bug 22533: Fix manual invoices from patron accounting

Description Nick Clemens 2019-03-18 17:36:17 UTC

    
Comment 1 Liz Rea 2019-03-19 17:55:33 UTC
Created attachment 86757 [details] [review]
Bug 22533: maninvoice.pl is using deprecated maininvoice() method

To test:
Attempt to create a manual fine from the patron accounting page, it
should fail.
Apply this patch, restart the things
Attempt to create a manual fine from the patron accounting page, it
should succeed.
Comment 2 Hayley Pelham 2019-03-21 01:14:43 UTC
Created attachment 86838 [details] [review]
Bug 22533: maninvoice.pl is using deprecated maininvoice() method

To test:
Attempt to create a manual fine from the patron accounting page, it
should fail.
Apply this patch, restart the things
Attempt to create a manual fine from the patron accounting page, it
should succeed.

Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>
Comment 3 Tomás Cohen Arazi 2019-03-21 11:10:42 UTC
While this fixes a bug, we should move forward with bug 22393, which completely removes the use of manualinvoice.
Comment 4 Martin Renvoize 2019-03-21 11:22:16 UTC
Sorry, I'd been meaning to get to this one.

I think this is a reasonable temporary fix to get master happy again.. bug 22393 isn't ready for testing as is yet.. unfortunately it turned into a bit of a monster (see the dependency tree above it) which I'm working my way through presently.

I actually think this bug is also somewhat mis-described, it's a circular dependency issue if I'm reading it right. Which I believe was introduced by the re-arrangement of use statements in the new overdues work bu (sorry, I missed it during QA :( ).

In short.. I'd say go for this one whilst I continue to get the tree for 22393 into a good state for testing and qa.
Comment 5 Liz Rea 2019-03-21 15:04:33 UTC
Created attachment 86851 [details] [review]
Bug 22533: Fix manual invoices from patron accounting

To test:
Attempt to create a manual fine from the patron accounting page, it
should fail.
Apply this patch, restart the things
Attempt to create a manual fine from the patron accounting page, it
should succeed.

Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>
Comment 6 Marcel de Rooy 2019-03-22 06:53:22 UTC
Created attachment 86873 [details] [review]
Bug 22533: Fix manual invoices from patron accounting

To test:
Attempt to create a manual fine from the patron accounting page, it
should fail.
Apply this patch, restart the things
Attempt to create a manual fine from the patron accounting page, it
should succeed.

Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Martin Renvoize 2019-03-22 07:49:44 UTC
Thanks Marcel, I just didn't get a chance to do the final qa on this yesterday with meetings.. nice to come to it this morning and see you've stepped into the breach.. Go Team! :)
Comment 8 Nick Clemens 2019-03-22 20:35:00 UTC
Awesome work all!

Pushed to master for 19.05
Comment 9 Martin Renvoize 2019-03-25 14:47:08 UTC
Causation bug not in 18.11.x series.