Attachment 71657 Details for Bug 19584 – Bug 19584: Check compare barcodes box when uploading barcodes

[patch] Bug 19584: Check compare barcodes box when uploading barcodes

Bug-19584-Check-compare-barcodes-box-when-uploadin.patch (text/plain), 2.66 KB, created by Marcel de Rooy on 2018-02-15 13:34:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2018-02-15 13:34:43 UTC

Size: 2.66 KB

patch

obsolete

>From 1fb774d383b38abf8b9bc9ae4e57e3b5bf0cdeef Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Tue, 7 Nov 2017 15:29:56 +0100
>Subject: [PATCH] Bug 19584: Check compare barcodes box when uploading barcodes
>Content-Type: text/plain; charset=utf-8
>
>This patch makes two trivial changes in the inventory interface:
>
>[1] If you have uploaded a barcode file, the compare barcodes checkbox is
>automatically checked. In most cases this will be the option the user wants.
>The compare will report missed items on the shelf.
>[2] If the compare barcodes checkbox is checked, the Skip items on loan-
>checkbox is automatically checked. This again is often desirable. If you
>compare, you normally are not interested in seeing items reported missing
>while they are checked out.
>
>Test plan:
>[1] Upload a barcode file on inventory. Is the compare checked?
>[2] Toggle compare. Does 'skip items on loan' change accordingly?
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>
>Signed-off-by: Grace Smyth <gracesmythh@gmail.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt | 5 +++++
> 1 file changed, 5 insertions(+)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt
>index 1ff0968..346b36d 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt
>@@ -347,11 +347,14 @@
>                 if( $("#uploadbarcodes").val() ) {
>                     $("#setdate").prop('disabled',false);
>                     $("#compareinv2barcd").prop('disabled',false);
>+                    $("#compareinv2barcd").attr('checked',true); // default
>                     $("#dont_checkin").prop('disabled',false);
>                     if( $("#compareinv2barcd").attr('checked') ) {
>                         $("fieldset#optionalfilters").show();
>+                        $("#ignoreissued").attr('checked',true); // default
>                     } else {
>                         $("fieldset#optionalfilters").hide();
>+                        $("#ignoreissued").attr('checked',false);
>                     }
>                 } else {
>                     $("#setdate").prop('disabled',true);
>@@ -365,7 +368,9 @@
>             $("#compareinv2barcd").click(function() {
>                 if( $("#compareinv2barcd").attr('checked') ) {
>                     $("fieldset#optionalfilters").show();
>+                    $("#ignoreissued").attr('checked',true); // default
>                 } else {
>+                    $("#ignoreissued").attr('checked',false);
>                     $("fieldset#optionalfilters").hide();
>                 }
>             });
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 19584: 68997 | 69017 | 69018 | 69739 | 70531 | 71656 | 71657 | 73255 | 73256