Bugzilla – Attachment 111951 Details for
Bug 6819
Don't offer cancel order links for received order lines on basket summary
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 6819: Hide individual cancel order links when order line has been received
Bug-6819-Hide-individual-cancel-order-links-when-o.patch (text/plain), 6.12 KB, created by
Katrin Fischer
on 2020-10-18 22:58:35 UTC
(
hide
)
Description:
Bug 6819: Hide individual cancel order links when order line has been received
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2020-10-18 22:58:35 UTC
Size:
6.12 KB
patch
obsolete
>From a056373089d183a52c4b8664a9d19bca2acd33df Mon Sep 17 00:00:00 2001 >From: Katrin Fischer <katrin.fischer.83@web.de> >Date: Sun, 18 Oct 2020 22:53:53 +0000 >Subject: [PATCH] Bug 6819: Hide individual cancel order links when order line > has been received > >This is to prevent mistakes - a received order should not be >cancelled, but 'undo receive' needs to be done first for things >to turn out correctly. > >To test: >- Create a basket with at least 2 orders >- Close basket >- Receive one of your orders at least >- Go back to the basket >- Reopen basket >- The cancel links should only show on the "not received" orders >--- > .../intranet-tmpl/prog/en/modules/acqui/basket.tt | 44 +++++++++++----------- > 1 file changed, 23 insertions(+), 21 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt >index 651a232162..3eb48e2ab2 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt >@@ -558,27 +558,29 @@ > [% END %] > </td> > <td> >- [% IF ( books_loo.left_holds_on_order ) %] >- <span class="button" title="Can't cancel order, ([% books_loo.holds_on_order | html %]) holds are linked with this order. Cancel holds first">Can't cancel order</span><br> >- [% ELSE %] >- <a href="/cgi-bin/koha/acqui/cancelorder.pl?ordernumber=[% books_loo.ordernumber | uri %]&biblionumber=[% books_loo.biblionumber | uri %]&basketno=[% basketno | uri %]&referrer=/cgi-bin/koha/acqui/basket.pl%3Fbasketno=[% basketno | uri %]" class="button">Cancel order</a><br> >- [% END %] >- [% IF ( books_loo.can_del_bib ) %] >- <a href="/cgi-bin/koha/acqui/cancelorder.pl?ordernumber=[% books_loo.ordernumber | uri %]&biblionumber=[% books_loo.biblionumber | uri %]&basketno=[% basketno | uri %]&del_biblio=1&referrer=/cgi-bin/koha/acqui/basket.pl%3Fbasketno=[% basketno | uri %]" class="button">Cancel order and delete catalog record</a><br> >- [% ELSE %] >- <span class="button" title="Can't delete catalog record, see constraints below">Can't cancel order and delete catalog record</span><br> >- [% END %] >- [% IF ( books_loo.left_item ) %] >- <strong title="Can't delete catalog record, because of [% books_loo.items | html %] existing hold(s)" >[% books_loo.items | html %] item(s) left</strong><br> >- [% END %] >- [% IF ( books_loo.left_biblio ) %] >- <strong title="Can't delete catalog record, delete other orders linked to it first">[% books_loo.biblios | html %] order(s) left</strong><br> >- [% END %] >- [% IF ( books_loo.left_subscription ) %] >- <strong title="Can't delete catalog record, delete subscriptions first">[% books_loo.subscriptions | html %] subscription(s) left</strong><br> >- [% END %] >- [% IF ( books_loo.left_holds ) %] >- <strong title="Can't delete catalog record or order, cancel holds first">[% books_loo.holds | html %] hold(s) left</strong> >+ [% IF ( books_loo.orderstatus != "complete") %] >+ [% IF ( books_loo.left_holds_on_order ) %] >+ <span class="button" title="Can't cancel order, ([% books_loo.holds_on_order | html %]) holds are linked with this order. Cancel holds first">Can't cancel order</span><br> >+ [% ELSE %] >+ <a href="/cgi-bin/koha/acqui/cancelorder.pl?ordernumber=[% books_loo.ordernumber | uri %]&biblionumber=[% books_loo.biblionumber | uri %]&basketno=[% basketno | uri %]&referrer=/cgi-bin/koha/acqui/basket.pl%3Fbasketno=[% basketno | uri %]" class="button">Cancel order</a><br> >+ [% END %] >+ [% IF ( books_loo.can_del_bib ) %] >+ <a href="/cgi-bin/koha/acqui/cancelorder.pl?ordernumber=[% books_loo.ordernumber | uri %]&biblionumber=[% books_loo.biblionumber | uri %]&basketno=[% basketno | uri %]&del_biblio=1&referrer=/cgi-bin/koha/acqui/basket.pl%3Fbasketno=[% basketno | uri %]" class="button">Cancel order and delete catalog record</a><br> >+ [% ELSE %] >+ <span class="button" title="Can't delete catalog record, see constraints below">Can't cancel order and delete catalog record</span><br> >+ [% END %] >+ [% IF ( books_loo.left_item ) %] >+ <strong title="Can't delete catalog record, because of [% books_loo.items | html %] existing hold(s)" >[% books_loo.items | html %] item(s) left</strong><br> >+ [% END %] >+ [% IF ( books_loo.left_biblio ) %] >+ <strong title="Can't delete catalog record, delete other orders linked to it first">[% books_loo.biblios | html %] order(s) left</strong><br> >+ [% END %] >+ [% IF ( books_loo.left_subscription ) %] >+ <strong title="Can't delete catalog record, delete subscriptions first">[% books_loo.subscriptions | html %] subscription(s) left</strong><br> >+ [% END %] >+ [% IF ( books_loo.left_holds ) %] >+ <strong title="Can't delete catalog record or order, cancel holds first">[% books_loo.holds | html %] hold(s) left</strong> >+ [% END %] > [% END %] > </td> > [% END %] >-- >2.11.0
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 6819
:
5226
|
111951
|
111994
|
111996
|
112106