Bugzilla – Attachment 126585 Details for
Bug 29283
Cannot delete basket with cancelled order for deleted biblio
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 29283: Don't attempt to cancel cancelled orders
Bug-29283-Dont-attempt-to-cancel-cancelled-orders.patch (text/plain), 1.43 KB, created by
Nick Clemens
on 2021-10-20 13:10:31 UTC
(
hide
)
Description:
Bug 29283: Don't attempt to cancel cancelled orders
Filename:
MIME Type:
Creator:
Nick Clemens
Created:
2021-10-20 13:10:31 UTC
Size:
1.43 KB
patch
obsolete
>From 3d8e9a4cd4ed21a75ce04b59a9710367663c7ede Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Wed, 20 Oct 2021 13:06:06 +0000 >Subject: [PATCH] Bug 29283: Don't attempt to cancel cancelled orders > >When deleting a basket we cancel all the contained orders - when a >basket contains an order that was previously cancelled this can cause >an error if the biblio was deleted > >When picking the orders to cancel, we should limit our search >to those not already cancelled. > >To test: >- have a basket with at least one order >- click "Cancel order and delete catalog record", confirm cancellation of order and deletion of bib >- click "Delete basket", confirm deletion >- get error "Cannot insert order: Mandatory parameter biblionumber is missing at /kohadevbox/koha/acqui/basket.pl line 136. > at /usr/share/perl/5.28/Carp.pm line 289" >- apply patch >- restart >- delete the basket >- success! >--- > acqui/basket.pl | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > >diff --git a/acqui/basket.pl b/acqui/basket.pl >index ac09913920..3694348a5a 100755 >--- a/acqui/basket.pl >+++ b/acqui/basket.pl >@@ -127,7 +127,9 @@ if ( $op eq 'delete_confirm' ) { > my $delbiblio = $query->param('delbiblio'); > my $basket_obj = Koha::Acquisition::Baskets->find($basketno); > >- my $orders = $basket_obj->orders; >+ my $orders = $basket_obj->orders->search({ >+ datecancellationprinted => undef >+ }); > > my @cannotdelbiblios; > >-- >2.20.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 29283
:
126585
|
126589
|
127134
|
127135