@@ -, +, @@ records were not deleted - x item(s) attached. - x subscription(s) attached. - x order(s) attached. - Unknown error. - an order using a record A which has already an item, which is used in a subscription, and in other order (in an other basket) - an order using a record B used nowhere elese records were not deleted - x item(s) attached. - x subscription(s) attached. - x order(s) attached. - Unknown error. - an order using a record A which has already an item, which is used in a subscription, and in other order (in an other basket) - an order using a record B used nowhere elese --- acqui/basket.pl | 22 ++++++++++++++++-- .../intranet-tmpl/prog/en/modules/acqui/basket.tt | 24 ++++++++++++++++++-- 2 files changed, 42 insertions(+), 4 deletions(-) --- a/acqui/basket.pl +++ a/acqui/basket.pl @@ -105,14 +105,32 @@ if ( $op eq 'delete_confirm' ) { } # if $delbiblio = 1, delete the records if possible if ((defined $delbiblio)and ($delbiblio ==1)){ + my @cannotdelbiblios ; foreach my $myorder (@orders){ my $biblionumber = $myorder->{'biblionumber'}; my $countbiblio = CountBiblioInOrders($biblionumber); my $ordernumber = $myorder->{'ordernumber'}; - my @subscriptions = GetSubscriptionsId ($biblionumber); + my $subscriptions = scalar GetSubscriptionsId ($biblionumber); my $itemcount = GetItemsCount($biblionumber); - DelBiblio($myorder->{biblionumber}) if ($countbiblio == 0 && $itemcount == 0 && !(@subscriptions)); + my $error; + if ($countbiblio == 0 && $itemcount == 0 && $subscriptions == 0) { + $error = DelBiblio($myorder->{biblionumber}) } + else { + push @cannotdelbiblios, {biblionumber=> ($myorder->{biblionumber}), + title=> $myorder->{'title'}, + author=> $myorder->{'author'}, + countbiblio=> $countbiblio, + itemcount=>$itemcount, + subscriptions=>$subscriptions}; + } + if ($error) { + push @cannotdelbiblios, {biblionumber=> ($myorder->{biblionumber}), + title=> $myorder->{'title'}, + author=> $myorder->{'author'}, + othererror=> $error}; + } } + $template->param( cannotdelbiblios => \@cannotdelbiblios ); } # delete the basket DelBasket($basketno,); --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt @@ -173,11 +173,11 @@

Warning:

All orders of this basket will be cancelled and used funds will be refunded.

If items have been created when ordering or receiving, they will be deleted.

-

You can choose to delete records if possible (if they don't have any item attached, any subscription and are not used in any other order).

+

You can choose to delete records if possible (records with some items, used in a subscription, or an other order will not be deleted).

[% END %] @@ -198,7 +198,27 @@ [% ELSE %] [% IF ( delete_confirmed ) %]

Basket deleted

+ [% IF (cannotdelbiblios) %] +
+

Warning:

+

The following records could not be deleted:

+ +
+ Click here to go back to booksellers page + [% ELSE %] + [% END %] [% ELSE %]

[% UNLESS ( basketno ) %]New [% END %][% IF ( delete_confirm ) %]Delete [% END %]Basket [% basketname|html %] ([% basketno %]) for [% name|html %]

[% IF ( delete_confirm ) %] --