]> git.koha-community.org Git - koha.git/commit
Bug 5680: Follow Up for Order cancelling improvement : the same for parcel.pl
authorChristophe Croullebois <christophe.croullebois@biblibre.com>
Sat, 15 Oct 2011 19:21:35 +0000 (21:21 +0200)
committerChris Cormack <chrisc@catalyst.net.nz>
Wed, 19 Oct 2011 03:59:12 +0000 (16:59 +1300)
commitdcfb5e63c9092efc396da98b51e079efc3b823f9
tree40dac86cd0fec4df48d90334defa457d98993730
parent4d67e6910775ea389cfa0a25e3fb55b9cf17bd6f
Bug 5680: Follow Up for Order cancelling improvement : the same for parcel.pl

The same scope than in basket.pl

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Detailed information about possible deletion of orders on parcel page.

1) Delete order for record with 1 other item and item level hold
OK. Record, item and hold remain, only on order item is deleted.

2) Delete order for ordered record with 1 item and title level hold
OK. Deletion is not possible, before hold is cancelled.

3) Delete order for new record with only items connected to this order
OK. Cancelling the order deletes record and item or only order,
depending on link used.

4) Delete order for record with subscription.
OK. Subscription remains untouched, item is deleted.

Also checked:
AcqCreateItem = on receive / in cataloging
Links show and it's possible to delete the order or only the record
using different links.

Comments:
There is a table sorter option for the last column with the delete links.
I am not sure it is useful here, but it does work.

Before patch is applied, the links use 'cancel' instead of 'delete'.
I am not sure which is better and if it should be different on
basket summary and order receive or the same on both.

QA Comment:
Made a small change to template language; blocking deletion because of items referred to
holds instead.  Changed text to refer to items.
Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
acqui/parcel.pl
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt