Koha/acqui
Marcel de Rooy 2a3ba5f825
Bug 29658: Fix crash on cancelling cancelled order
Found this crash in our 20.11 logs:
Cannot insert order: Mandatory parameter biblionumber is missing at /usr/share/koha/acqui/cancelorder.pl line 60.
 at /usr/share/perl/5.28/Carp.pm line 289
        Carp::croak('Cannot insert order: Mandatory parameter biblionumber is missing') called at /usr/share/koha/Koha/Acquisition/Order.pm line 79
        Koha::Acquisition::Order::store('Koha::Acquisition::Order=HASH(0x55f3760e2860)') called at /usr/share/koha/Koha/Acquisition/Order.pm line 189
        Koha::Acquisition::Order::cancel('Koha::Acquisition::Order=HASH(0x55f3760e2860)', 'HASH(0x55f375a17ec0)') called at /usr/share/koha/acqui/cancelorder.pl line 60

Not sure how to reproduce this one as it happened. But might be related to repeated clicking, backspacing etc.

Test plan:
Create a new basket and order.
Open this same basket in two browser tabs.
Cancel the order line (delete catalog record) in tab 1.
Go to second tab, try again.
Without this patch, it will crash. With this patch, an error message.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Joonas Kylmälä <joonas.kylmala@iki.fi>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
2022-09-12 16:14:47 -03:00
..
acqui-home.pl Bug 29287: Add child funds to display of parent funds on acqui home 2022-03-01 22:47:46 -10:00
addorder.pl Bug 29955: (QA follow-up) Tidy code block 2022-08-19 15:51:21 -03:00
addorderiso2709.pl Bug 29955: Move populate_order_with_prices to Koha namespace 2022-08-19 15:47:51 -03:00
ajax-getauthvaluedropbox.pl
basket.pl Bug 31001: Fix "CGI::param called in list context" warning in basket.pl 2022-07-06 18:29:32 -03:00
basketgroup.pl Bug 16258: Add a syspref to govern EDIFACT functionality 2022-04-13 15:55:38 +02:00
basketheader.pl Bug 29859: Use iterator instead of as_list 2022-02-09 15:36:23 -10:00
booksellers.pl Bug 31017: Update field name to 'type' 2022-08-19 16:15:15 -03:00
cancelorder.pl Bug 29658: Fix crash on cancelling cancelled order 2022-09-12 16:14:47 -03:00
check_budget_total.pl
check_uniqueness.pl
duplicate_orders.pl Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
edi_ean.pl
edifactmsgs.pl
edimsg.pl
finishreceive.pl Bug 29955: Move populate_order_with_prices to Koha namespace 2022-08-19 15:47:51 -03:00
histsearch.pl Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
invoice-files.pl
invoice.pl Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
invoices.pl Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
lateorders-export.pl
lateorders.pl Bug 30718: Pass DT to filter_by_lates 2022-08-19 08:26:37 -03:00
modordernotes.pl
neworderempty.pl Bug 29697: Replace GetMarcBiblio occurrences with $biblio->metadata->record 2022-07-22 15:24:11 -03:00
newordersubscription.pl
newordersuggestion.pl Bug 23991: Move SearchSuggestion to Koha::Suggestions 2022-06-27 12:30:28 -03:00
ordered.pl
orderreceive.pl Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
parcel.pl
parcels.pl Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
showorder.pl
spent.pl
supplier.pl Bug 28082: Add acquisitions toolbar to vendors on vendor search page 2022-05-10 15:17:17 -10:00
transferorder.pl Bug 29859: Use iterator instead of as_list 2022-02-09 15:36:23 -10:00
uncertainprice.pl
updatesupplier.pl Bug 31017: Update field name to 'type' 2022-08-19 16:15:15 -03:00
z3950_search.pl