Attachment 74274 Details for Bug 19479 – Bug 19479: Display price in basket group formatted according to CurrencyFormat syspref using templates.

[patch] Bug 19479: Display price in basket group formatted according to CurrencyFormat syspref using templates.

Bug-19479-Display-price-in-basket-group-formatted-.patch (text/plain), 4.50 KB, created by Séverine Queune on 2018-04-16 20:34:33 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Séverine Queune

Created: 2018-04-16 20:34:33 UTC

Size: 4.50 KB

patch

obsolete

>From 48631d95e1ef5c9f7bb5338741e0f0ad33de75d1 Mon Sep 17 00:00:00 2001
>From: Jon Knight <J.P.Knight@lboro.ac.uk>
>Date: Wed, 28 Mar 2018 16:29:43 +0000
>Subject: [PATCH] Bug 19479: Display price in basket group formatted according
> to CurrencyFormat syspref using templates.
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>On the page acqui/basketgroup.pl the prices of baskets should be shown
>formatted according to the selected CurrencyFormat syspref, with no
>currency symbol or code.
>
>Test plan:
>
>1) Create a basket with items in it worth more than 1000 currency units.
>2) Close the basket.
>3) Go to acqui/basketgroup.pl page and check that the price format matches
>   the current CurrencyFormat syspref.
>4) Go to Administration and change CurrencyFormat syspref to one of the
>   other available options and recheck step 3.
>
>Signed-off-by: SÃ©verine QUEUNE <severine.queune@bulac.fr>
>---
> acqui/basketgroup.pl                                         | 5 ++++-
> koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt | 5 +++--
> 2 files changed, 7 insertions(+), 3 deletions(-)
>
>diff --git a/acqui/basketgroup.pl b/acqui/basketgroup.pl
>index 8c0e4c4..3622ded 100755
>--- a/acqui/basketgroup.pl
>+++ b/acqui/basketgroup.pl
>@@ -82,7 +82,6 @@ sub BasketTotal {
>             $total = $total + ( $order->{ecost_tax_excluded} * $order->{quantity} );
>         }
>     }
>-    $total .= " " . ($bookseller->invoiceprice // 0);
>     return $total;
> }
> 
>@@ -97,6 +96,8 @@ sub displaybasketgroups {
>             my $basketsqty = 0;
>             while($i < scalar(@$baskets)){
>                 my $basket = @$baskets[$i];
>+		my $active_currency = Koha::Acquisition::Currencies->get_active;
>+		$basket->{currency} = $active_currency->currency;
>                 if($basket->{'basketgroupid'} && $basket->{'basketgroupid'} == $basketgroup->{'id'}){
>                     $basket->{total} = BasketTotal($basket->{basketno}, $bookseller);
>                     push(@{$basketgroup->{'baskets'}}, $basket);
>@@ -254,6 +255,8 @@ if ( $op eq "add" ) {
>         my $selecteds = GetBasketsByBasketgroup($basketgroupid);
>         foreach my $basket(@{$selecteds}){
>             $basket->{total} = BasketTotal($basket->{basketno}, $bookseller);
>+	    my $active_currency = Koha::Acquisition::Currencies->get_active;
>+	    $basket->{currency} = $active_currency->currency;
>         }
>         $template->param(basketgroupid => $basketgroupid,
>                          selectedbaskets => $selecteds);
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt
>index d872672..4c3e833 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt
>@@ -1,5 +1,6 @@
> [% USE Asset %]
> [% USE Branches %]
>+[% USE Price %]
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Koha &rsaquo; Basket grouping for [% booksellername |html %]</title>
> [% Asset.css("css/datatables.css") %]
>@@ -174,7 +175,7 @@ function submitForm(form) {
>                                                                         No name, basketnumber: [% basket.basketno %]
>                                                                     [% END %]
>                                                                 </a>, <br />
>-                                                                Total: [% basket.total %]
>+                                                                Total: [% basket.total | $Price %]
>                                                                 <input type="hidden" class="basket" name="basket" value="[% basket.basketno %]" />
>                                                             </li>
>                                                         [% END %]
>@@ -260,7 +261,7 @@ function submitForm(form) {
>                                                                 No name, basketnumber: [% selectedbasket.basketno %]
>                                                             [% END %]
>                                                         </a>, <br />
>-                                                        Total: [% selectedbasket.total %]
>+                                                        Total: [% selectedbasket.total | $Price %]
>                                                         <input type="hidden" class="basket" name="basket" value="[% selectedbasket.basketno %]" />
>                                                     </li>
>                                                 [% END %]
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 19479: 68276 | 68849 | 68972 | 71312 | 71313 | 73290 | 73291 | 73376 | 73377 | 73379 | 73491 | 73677 | 74274 | 74284 | 74739