Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 29987: Improve validation

Browse source 
This patch adds hinting to display the required state of the cash
register fields.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Martin Renvoize 2022-09-30 16:06:45 +01:00 committed by Tomas Cohen Arazi
parent d6262c0f30
commit bce51debc6
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F
1 changed files with 14 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
koha-tmpl/intranet-tmpl/prog/en/modules/members/mancredit.tt
View file

@ -75,7 +75,7 @@
<li><label for="barcode">Barcode: </label><input type="text" name="barcode" id="barcode" /></li>
<li><label for="desc">Description: </label><input type="text" name="desc" size="50" id="desc" /></li>
<li><label for="note">Note: </label><input type="text" name="note" size="50" id="note" /></li>
<li><label for="amount">Amount: </label><input type="text" inputmode="decimal" pattern="^\d+(\.\d{2})?$" name="amount" id="amount" required="required" min="0" value=""/> Example: 5.00</li>
<li><label for="amount" class="required">Amount: </label><input type="text" inputmode="decimal" pattern="^\d+(\.\d{2})?$" name="amount" id="amount" required="required" min="0" value=""/> Example: 5.00</li>
[% INCLUDE 'transaction_types.inc' type="credit" %]
[% IF Koha.Preference('UseCashRegisters') %]
<li>
@ -88,7 +88,7 @@
</label>
[% IF Koha.Preference('RequireCashRegister') %]
<select name="cash_register" id="cash_register" required>
<select name="cash_register" id="cash_register" required="required">
[% ELSE %]
<select name="cash_register" id="cash_register">
[% END %]
@ -131,6 +131,18 @@
$(document).ready(function(){
$('#mancredit').preventDoubleFormSubmit();
$("fieldset.rows input, fieldset.rows select").addClass("noEnterSubmit");
[% UNLESS Koha.Preference('RequireCashRegister') %]
$('#credit_type').on('change', function() {
let val = $(this).val();
if ( val == 'CASH' ) {
$("label[for='cash_register']").addClass('required');
$('#cash_register').attr('required','required').after( '<span class="required">Required</span>' );
} else {
$("label[for='cash_register']").removeClass('required');
$('#cash_register').removeAttr('required').next("span").remove();
}
});
[% END %]
});
</script>
[% END %]