Attachment 120876 Details for Bug 28327 – Bug 28135: Replace use of input type number in additem.js

[patch] Bug 28135: Replace use of input type number in additem.js

Bug-28135-Replace-use-of-input-type-number-in-addi.patch (text/plain), 2.64 KB, created by Fridolin Somers on 2021-05-12 13:43:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Fridolin Somers

Created: 2021-05-12 13:43:46 UTC

Size: 2.64 KB

patch

obsolete

>From dd2177ff8f8940ba049cb438ffa0789317ddb473 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Mon, 12 Apr 2021 11:25:35 +0000
>Subject: [PATCH] Bug 28135: Replace use of input type number in additem.js
>
>This patch corrects in instance where JavaScript is creating an input
>tag with type "number."
>
>To test, apply the patch and make sure the AcqCreateItem system
>preference is set to "when receiving."
>
>- Go to Acquisitions -> Vendor -> Invoices -> Invoice -> Go to receipt
>  page -> Receive.
>- On the page for receiving items you should see an add item form.
>- Click the "Add multiple items" button at the bottom of the form.
>- Test the "Number of items to add" field. It should not accept any
>  input except numbers.
>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>
>https://bugs.koha-community.org/show_bug.cgi?id=28327
>---
> koha-tmpl/intranet-tmpl/prog/js/additem.js | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/additem.js b/koha-tmpl/intranet-tmpl/prog/js/additem.js
>index 960ffc36dd..7d80ca8a77 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/additem.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/additem.js
>@@ -182,7 +182,7 @@ function cloneItemBlock(index, unique_item_fields, callback) {
>             buttonPlus += '<input type="button" class="addItemControl cancel" name="buttonClear" style="cursor:pointer;" onclick="clearItemBlock(this)" value="' + __("Clear") + '" />';
>             buttonPlus += '<input type="button" class="addItemControl" name="buttonPlusMulti" onclick="javascript:this.nextSibling.style.display=\'inline\'; return false;" style="cursor:pointer; margin:0 1em;" value="' + __("Add multiple items") + '" />';
>             buttonPlus += '<span id="add_multiple_copies" style="display:none">'
>-                +     '<input type="number" class="addItemControl" id="multiValue" name="multiValue" placeholder="' + __("Number of items to add") + '" />'
>+                +     '<input type="text" inputmode="numeric" pattern="[0-9]*" class="addItemControl" id="multiValue" name="multiValue" placeholder="' + __("Number of items to add") + '" />'
>                 +     '<input type="button" class="addItemControl" name=buttonAddMulti" style="cursor:pointer; margin:0 1em;" onclick="checkCount( this ,\'' + unique_item_fields + '\')" value="' + __("Add") + '" />'
>                 +     '<div class="dialog message">' + __("NOTE: Fields listed in the 'UniqueItemsFields' system preference will not be copied") + '</div>'
>                 + '</span>';
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 28327: 120859 | 120861 | 120867 | 120868 | 120869 | 120870 | 120871 | 120872 | 120873 | 120874 | 120876 | 120877 | 120878 | 120879 | 120880 | 120881 | 126682 | 126683 | 126684 | 126685 | 126964 | 126965 | 126966 | 126967 | 131995 | 131996 | 131997 | 136378 | 136379 | 136380 | 136381 | 136473 | 136474 | 136475 | 136535 | 136536 | 136537 | 139479 | 139513 | 139514 | 139515 | 139516