From dbcdb741bd43d7cb40b72acd51856bc170964c02 Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Date: Wed, 8 Jan 2020 12:48:55 +0000
Subject: [PATCH] Bug 24339: Remove SIP payment types from staff pay screens

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
---
 .../prog/en/modules/members/paycollect.tt              | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt
index 54eec334b4..daba24805d 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt
@@ -129,13 +129,16 @@
         <span id="change">0.00</span>
     </li>
     [% SET payment_types = AuthorisedValues.GetAuthValueDropbox('PAYMENT_TYPE') %]
-    [% IF payment_types %]
+    [% SET excluded = ['SIP00', 'SIP01', 'SIP02'] %]
+    [% IF payment_types.size > 3 %]
         <li>
             <label for="payment_type">Payment type: </label>
             <select name="payment_type" id="payment_type">
                 <option value=""></option>
                 [% FOREACH pt IN payment_types %]
+                    [% UNLESS excluded.grep("^$pt.authorised_value\$").size %]
                     <option value="[% pt.authorised_value | html %]">[% pt.lib | html %]</option>
+                    [% END %]
                 [% END %]
             </select>
         </li>
@@ -258,13 +261,16 @@
             <span id="change">0.00</span>
         </li>
     [% SET payment_types = AuthorisedValues.GetAuthValueDropbox('PAYMENT_TYPE') %]
-    [% IF payment_types %]
+    [% SET excluded = ['SIP00', 'SIP01', 'SIP02'] %]
+    [% IF payment_types.size > 3 %]
         <li>
             <label for="payment_type">Payment type: </label>
             <select name="payment_type" id="payment_type">
                 <option value=""></option>
                 [% FOREACH pt IN payment_types %]
+                    [% UNLESS excluded.grep("^$pt.authorised_value\$").size %]
                     <option value="[% pt.authorised_value | html %]">[% pt.lib | html %]</option>
+                    [% END %]
                 [% END %]
             </select>
         </li>
-- 
2.39.5