From 32a589f7aa3cb84de3b8cbfbd001f163ebcaf3a8 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>
---
 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 d25f0eb645..2d644a4c71 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 %]
@@ -69,7 +69,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 a89f2538d6..213660d48a 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