From 6d01b9fcbdf10b8163edb36af3465e91dcab9a7a Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marc=20V=C3=A9ron?= Date: Wed, 4 Nov 2015 10:13:46 +0100 Subject: [PATCH] Bug 15117: Transfer Order: Better user information and translation handling MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit While transferring an order, a untranslatable JavaScript confirmation dialog pops up. This patch moves the information about the order to be transferred to the top of the screen to better inform the user what order is to be transferred, and simplifies the confirmation dialog. To test: - Apply patch - Transfer an order from a basket to another basket - Verify, that on top of the screen an information is displayed about which order from which vendor and basket is to be transferred - Verify that the transfer works OK - Update a po lang file and confirm you see the string and you are able to translate it. Signed-off-by: Frederic Demians Dialog box with readable & translatable info. Signed-off-by: Jonathan Druart Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 571ec9a94b9323cd23f9012d9b1f837fd9bf974c) Signed-off-by: Frédéric Demians (cherry picked from commit 57a8479bf36f530a55fd93ba67fe965efb0e0373) Signed-off-by: Liz Rea --- acqui/transferorder.pl | 4 ++++ .../prog/en/modules/acqui/transferorder.tt | 21 ++++++++++++++++++- 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/acqui/transferorder.pl b/acqui/transferorder.pl index d82ce20738..d84def5ced 100755 --- a/acqui/transferorder.pl +++ b/acqui/transferorder.pl @@ -50,8 +50,10 @@ my $op = $input->param('op'); my $query = $input->param('query'); my $order = GetOrder($ordernumber); +my $basketfromname = ''; if($order) { my $basket = GetBasket($order->{basketno}); + $basketfromname = $basket->{basketname}; $bookselleridfrom = $basket->{booksellerid} if $basket; } @@ -66,6 +68,7 @@ if($booksellerto){ $booksellertoname = $booksellerto->{name}; } + if( $basketno && $ordernumber) { # Transfer order and exit my $order = GetOrder( $ordernumber ); @@ -133,6 +136,7 @@ $template->param( booksellertoname => $booksellertoname, ordernumber => $ordernumber, basketno => $basketno, + basketfromname => $basketfromname, ); output_html_with_http_headers $input, $cookie, $template->output; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt index 36716324e1..69b023165c 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt @@ -12,6 +12,18 @@ //]]> [% END %] + +[% IF ( bookselleridfrom ) %] + +[% END %] + + @@ -20,6 +32,13 @@
[% IF ( bookselleridfrom ) %]

Search for a vendor to transfer to

+
+
    +
  • From vendor: [% booksellerfromname %]
    • +
  • Basket: [% basketfromname %]
    • +
  • Order: [% ordernumber %]
    • + +
[% ELSE %]

Search for a vendor to transfer from

[% END %] @@ -53,7 +72,7 @@ [% basket.basketname %] (#[% basket.basketno %]) [% basket.createdby %] [% basket.creationdate %] - Choose + Choose [% END %] -- 2.39.5