From 8bc2bf99daed72e48ce00a31e30c65b34fc9e951 Mon Sep 17 00:00:00 2001
From: Jenny Way <jwayway@hotmail.com>
Date: Tue, 16 Jan 2018 22:05:39 +0000
Subject: [PATCH] Bug 19188: Self checkout missing currency symbol

Test plan:
1. Under administration and system preferences, enable WebBasedSelfCheck
2. In OPAC give user fine by going to 'my account' at top right
3. Then under fines tab 'create manual invoice'
4. Make sure this amount is more than noissuescharge which can be
   changed under system preferences
5. Go to the web based self check from the OPAC URL
6. Log in as your user
7. Try to check out an item
7. Error 'Item cannot be checked out' should occur
8. With the patch, the currency symbol should show for the amount.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 5eab08cc2edd897f9afdfeb46caf5e409476946f)
Signed-off-by: Jacob O'Mara <jacob.omara@ptfs-europe.com>
(cherry picked from commit 13f7ee3e938b41f66d3d057001660c347b3dc08c)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 32a589f7aa3cb84de3b8cbfbd001f163ebcaf3a8)
Signed-off-by: Arthur Suzuki <arthur.suzuki@biblibre.com>
---
 koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt | 4 ++--
 opac/sco/sco-main.pl                                     | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
index 4bc9008b5e..99b01d783c 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
@@ -1,4 +1,4 @@
-    [% USE raw %]
+[% USE raw %]
 [% USE Asset %]
 [% USE Koha %]
 [% USE KohaDates %]
@@ -70,7 +70,7 @@
                                 [% ELSIF ( circ_error_NOT_FOR_LOAN ) %]
                                     <span id="ce_not_for_loan">This item is not for loan.</span>
                                 [% ELSIF ( circ_error_DEBT ) %]
-                                    <span id="ce_too_much_debt">You owe the library [% DEBT | $Price %] and cannot check out.</span>
+                                    <span id="ce_too_much_debt">You owe the library [% currencySym %] [% DEBT | $Price %] and cannot check out.</span>
                                 [% ELSIF ( circ_error_WTHDRAWN ) %]
                                     <span id="ce_wthdrawn">This item has been withdrawn from the collection.</span>
                                 [% ELSIF ( circ_error_RESTRICTED ) %]
diff --git a/opac/sco/sco-main.pl b/opac/sco/sco-main.pl
index b293f100e1..288bc8a873 100755
--- a/opac/sco/sco-main.pl
+++ b/opac/sco/sco-main.pl
@@ -201,6 +201,7 @@ elsif ( $patron && ( $op eq 'checkout' ) ) {
         );
         if ($issue_error eq 'DEBT') {
             $template->param(DEBT => $impossible->{DEBT});
+            $template->param(currencySym => $currencySymbol);
         }
         if ( $issue_error eq "NO_MORE_RENEWALS" ) {
             $return_only = 1;
@@ -225,6 +226,7 @@ elsif ( $patron && ( $op eq 'checkout' ) ) {
         );
         if ($issue_error eq 'DEBT') {
             $template->param(DEBT => $needconfirm->{DEBT});
+            $template->param(currencySym => $currencySymbol);
         }
     } else {
         if ( $confirmed || $issuenoconfirm ) {    # we'll want to call getpatroninfo again to get updated issues.
-- 
2.39.5