Attachment 139997 Details for Bug 31482 – Bug 31482: Label creator does not call item_barcode_transform plugin hook

[patch] Bug 31482: Label creator does not call item_barcode_transform plugin hook

Bug-31482-Label-creator-does-not-call-itembarcodet.patch (text/plain), 1.98 KB, created by Kyle M Hall (khall) on 2022-08-31 09:39:13 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2022-08-31 09:39:13 UTC

Size: 1.98 KB

patch

obsolete

>From 3cbfb6b053b13b60e7ef88f0501c71fbb4d3a073 Mon Sep 17 00:00:00 2001
>From: Kyle Hall <kyle@bywatersolutions.com>
>Date: Mon, 29 Aug 2022 09:26:44 -0400
>Subject: [PATCH] Bug 31482: Label creator does not call item_barcode_transform
> plugin hook
>
>The label creator can accept a list of either itemnumbers or item barcodes.
>In the case that the list of composed of barcodes, those barcodes should be
>passed though the item_barcode_transform plugin hook before the lookup is performed.
>
>Test Plan:
>1) Apply this patch
>2) Download and install the Barcode Transformer plugin
>   https://github.com/bywatersolutions/koha-plugin-barcode-transformer/releases/
>3) Go to the plugin configuration page, set the configuration to the example configuration from the same page
>4) On the "New label batch" page of the Label creator,,
>   type in some valid barcodes, but prefix them with X and postfix them with Y, e.g. X123456Y
>5) Submit the form
>6) Note the items are added to the label batch!
>---
> labels/label-edit-batch.pl | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
>diff --git a/labels/label-edit-batch.pl b/labels/label-edit-batch.pl
>index ac2da6392f..0071c0755a 100755
>--- a/labels/label-edit-batch.pl
>+++ b/labels/label-edit-batch.pl
>@@ -23,9 +23,10 @@ use Modern::Perl;
> use CGI qw ( -utf8 );
> 
> use C4::Auth qw( get_template_and_user );
>-use C4::Output qw( output_html_with_http_headers );
>+use C4::Circulation qw( barcodedecode );
> use C4::Creators qw( get_label_summary html_table );
> use C4::Labels;
>+use C4::Output qw( output_html_with_http_headers );
> 
> use Koha::Items;
> 
>@@ -89,6 +90,7 @@ elsif ($op eq 'add') {
>                 push @item_numbers, $number;
>             }
>             elsif ($number_type eq "barcode" ) {  # we must test in case an invalid barcode is passed in; we effectively disgard them atm
>+                barcodedecode( $number );
>                 my $item = Koha::Items->find({barcode => $number});
>                 push @item_numbers, $item->itemnumber if $item;
>             }
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 31482: 139945 | 139946 | 139997 | 140036 | 140051 | 141114