Attachment 116470 Details for Bug 27289 – Bug 27289: (follow-up) Update payment type too.

[patch] Bug 27289: (follow-up) Update payment type too.

Bug-27289-follow-up-Update-payment-type-too.patch (text/plain), 2.36 KB, created by Katrin Fischer on 2021-02-07 12:15:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2021-02-07 12:15:04 UTC

Size: 2.36 KB

patch

obsolete

>From 0681f33c8589dad8a8b2f11c89566a3077ca5103 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Fri, 5 Feb 2021 11:06:25 +0000
>Subject: [PATCH] Bug 27289: (follow-up) Update payment type too.
>
>The payment type field is also required when using cash registers, this
>patch updates the formatting to work as expected in that case too.
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> .../intranet-tmpl/prog/en/includes/payments.inc    | 26 +++++++++++++---------
> 1 file changed, 16 insertions(+), 10 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/payments.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/payments.inc
>index 8fa9b1d691..b1b57979ad 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/payments.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/payments.inc
>@@ -7,17 +7,23 @@
>     [% END %]
>     [% IF payment_types.size > 0 %]
>         <li>
>-            <label for="payment_type">Payment type: </label>
>             [% IF Koha.Preference('UseCashRegisters') %]
>-            <select name="payment_type" id="payment_type" required>
>-            [% ELSE %]
>-            <select name="payment_type" id="payment_type">
>-                <option value=""></option>
>-            [% END %]
>-                [% FOREACH pt IN payment_types %]
>-                    <option value="[% pt.authorised_value | html %]">[% pt.lib | html %]</option>
>-                [% END %]
>-            </select>
>+                <label for="payment_type" class="required">Payment type: </label>
>+                <select name="payment_type" id="payment_type" class="required" required="required">
>+                    [% FOREACH pt IN payment_types %]
>+                        <option value="[% pt.authorised_value | html %]">[% pt.lib | html %]</option>
>+                    [% END %]
>+                </select>
>+                <span class="required">Required</span>
>+            [%- ELSE -%]
>+                <label for="payment_type">Payment type: </label>
>+                <select name="payment_type" id="payment_type">
>+                    <option value=""></option>
>+                    [% FOREACH pt IN payment_types %]
>+                        <option value="[% pt.authorised_value | html %]">[% pt.lib | html %]</option>
>+                    [% END %]
>+                </select>
>+            [%- END -%]
>         </li>
>     [% END %]
> [%- END -%]
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 27289: 114557 | 114756 | 116387 | 116388 | 116469 | 116470