Attachment 51091 Details for Bug 16384 – Bug 16384: Fix cancel link for 'Edit basket'

[patch] Bug 16384: Fix cancel link for 'Edit basket'

Bug-16384-Fix-cancel-link-for-Edit-basket.patch (text/plain), 1.83 KB, created by Jonathan Druart on 2016-05-02 11:52:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2016-05-02 11:52:51 UTC

Size: 1.83 KB

patch

obsolete

>From 861a5f8c818cd9fcc2bb7b64af438efbe050337f Mon Sep 17 00:00:00 2001
>From: Katrin Fischer <Katrin.Fischer.83@web.de>
>Date: Fri, 29 Apr 2016 02:53:43 +0200
>Subject: [PATCH] Bug 16384: Fix cancel link for 'Edit basket'
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>When you edit the basket from the basket summary page,
>saving the change brings you back to the basket summary
>page, but cancelling brings you to the baskets page of
>the vendor.
>
>To test:
>- Add a basket in acq
>  - Test cancel link returns to baskets page of vendor
>- Add a basket and save
>- Edit this basket
>  - Test cancel link now returns to basket summary page
>
>Signed-off-by: Marc VÃ©ron <veron@veron.ch>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketheader.tt | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketheader.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketheader.tt
>index 761660e..195022a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketheader.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketheader.tt
>@@ -127,7 +127,11 @@
>         <fieldset class="action">
>             <input type="hidden" name="op" value="add_validate" />
>             <input type="submit" value="Save" />
>-            <a class="cancel" href="/cgi-bin/koha/acqui/supplier.pl?booksellerid=[% booksellerid %]">Cancel</a>
>+            [% IF ( basketno ) %]
>+                <a class="cancel" href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketno %]">Cancel</a>
>+            [% ELSE %]
>+                <a class="cancel" href="/cgi-bin/koha/acqui/supplier.pl?booksellerid=[% booksellerid %]">Cancel</a>
>+            [% END %]
>         </fieldset>
>     </form>
>     [% END %]
>-- 
>2.7.0

Actions: View | Diff | Splinter Review

Attachments on bug 16384: 50975 | 50994 | 51091