@@ -, +, @@ item_barcode_transform --- C4/Circulation.pm | 2 +- C4/ILSDI/Services.pm | 2 +- Koha/Item.pm | 2 +- cataloguing/additem.pl | 2 +- circ/circulation.pl | 2 +- circ/returns.pl | 4 ++-- offline_circ/process_koc.pl | 4 ++-- opac/sco/sco-main.pl | 2 +- t/db_dependent/Koha/Plugins/Barcode_transform_hooks.t | 2 +- t/db_dependent/Koha/Plugins/Biblio_and_Items_plugin_hooks.t | 2 +- t/db_dependent/Koha/Plugins/Circulation_hooks.t | 2 +- t/db_dependent/Koha/Plugins/Plugins.t | 2 +- t/lib/Koha/Plugin/Test.pm | 6 +++--- tools/batchMod.pl | 2 +- tools/inventory.pl | 2 +- 15 files changed, 19 insertions(+), 19 deletions(-) --- a/C4/Circulation.pm +++ a/C4/Circulation.pm @@ -176,7 +176,7 @@ sub barcodedecode { my ($barcode, $filter) = @_; my $branch = C4::Context::mybranch(); $filter = C4::Context->preference('itemBarcodeInputFilter') unless $filter; - ($barcode) = Koha::Plugins->call('barcode_transform', 'item', $barcode ) || $barcode; + ($barcode) = Koha::Plugins->call('item_barcode_transform', $barcode ) || $barcode; $filter or return $barcode; # ensure filter is defined, else return untouched barcode if ($filter eq 'whitespace') { $barcode =~ s/\s//g; --- a/C4/ILSDI/Services.pm +++ a/C4/ILSDI/Services.pm @@ -643,7 +643,7 @@ sub GetServices { # Issuing management my $barcode = $item->barcode || ''; $barcode = barcodedecode($barcode) if ( $barcode && C4::Context->preference('itemBarcodeInputFilter') ); - ($barcode) = Koha::Plugins->call('barcode_transform', 'item', $barcode ) || $barcode; + ($barcode) = Koha::Plugins->call('item_barcode_transform', $barcode ) || $barcode; if ($barcode) { my ( $issuingimpossible, $needsconfirmation ) = CanBookBeIssued( $patron, $barcode ); --- a/Koha/Item.pm +++ a/Koha/Item.pm @@ -90,7 +90,7 @@ sub store { $self->itype($self->biblio->biblioitem->itemtype); } - my ($transformed_barcode) = Koha::Plugins->call( 'barcode_transform', 'item', $self->barcode ) || $self->barcode; + my ($transformed_barcode) = Koha::Plugins->call( 'item_barcode_transform', $self->barcode ) || $self->barcode; $self->barcode($transformed_barcode); my $today = dt_from_string; --- a/cataloguing/additem.pl +++ a/cataloguing/additem.pl @@ -509,7 +509,7 @@ if ($op eq "additem") { my $addedolditem = TransformMarcToKoha( $record ); - my ( $new_barcode ) = Koha::Plugins->call( 'barcode_transform', 'item', $addedolditem->{'barcode'} ) || $addedolditem->{'barcode'}; + my ( $new_barcode ) = Koha::Plugins->call( 'item_barcode_transform', $addedolditem->{'barcode'} ) || $addedolditem->{'barcode'}; $addedolditem->{'barcode'} = $new_barcode; # If we have to add or add & duplicate, we add the item --- a/circ/circulation.pl +++ a/circ/circulation.pl @@ -166,7 +166,7 @@ for my $barcode ( @$barcodes ) { $barcode =~ s/^\s*|\s*$//g; # remove leading/trailing whitespace $barcode = barcodedecode($barcode) if( $barcode && C4::Context->preference('itemBarcodeInputFilter')); - ( $barcode ) = Koha::Plugins->call( 'barcode_transform', 'item', $barcode ) || $barcode; + ( $barcode ) = Koha::Plugins->call( 'item_barcode_transform', $barcode ) || $barcode; } my $stickyduedate = $query->param('stickyduedate') || $session->param('stickyduedate'); --- a/circ/returns.pl +++ a/circ/returns.pl @@ -116,7 +116,7 @@ foreach ( $query->param ) { # decode barcode ## Didn't we already decode them before passing them back last time?? $barcode =~ s/^\s*|\s*$//g; # remove leading/trailing whitespace - ( $barcode ) = Koha::Plugins->call( 'barcode_transform', 'item', $barcode ) || $barcode; + ( $barcode ) = Koha::Plugins->call( 'item_barcode_transform', $barcode ) || $barcode; $barcode = barcodedecode($barcode) if(C4::Context->preference('itemBarcodeInputFilter')); ###################### @@ -251,7 +251,7 @@ if ($transit) { my $returnbranch; if ($barcode) { $barcode =~ s/^\s*|\s*$//g; # remove leading/trailing whitespace - ( $barcode ) = Koha::Plugins->call( 'barcode_transform', 'item', $barcode ) || $barcode; + ( $barcode ) = Koha::Plugins->call( 'item_barcode_transform', $barcode ) || $barcode; $barcode = barcodedecode($barcode) if C4::Context->preference('itemBarcodeInputFilter'); my $item = Koha::Items->find({ barcode => $barcode }); --- a/offline_circ/process_koc.pl +++ a/offline_circ/process_koc.pl @@ -248,7 +248,7 @@ sub kocIssueItem { $circ->{ 'barcode' } = barcodedecode($circ->{'barcode'}) if( $circ->{'barcode'} && C4::Context->preference('itemBarcodeInputFilter')); - my ( $new_barcode ) = Koha::Plugins->call( 'barcode_transform', 'item', $circ->{barcode} ) || $circ->{barcode}; + my ( $new_barcode ) = Koha::Plugins->call( 'item_barcode_transform', $circ->{barcode} ) || $circ->{barcode}; $circ->{barcode} = $new_barcode; my $branchcode = C4::Context->userenv->{branch}; @@ -332,7 +332,7 @@ sub kocReturnItem { my ( $circ ) = @_; $circ->{'barcode'} = barcodedecode($circ->{'barcode'}) if( $circ->{'barcode'} && C4::Context->preference('itemBarcodeInputFilter')); - my ( $new_barcode ) = Koha::Plugins->call( 'barcode_transform', 'item', $circ->{barcode} ) || $circ->{barcode}; + my ( $new_barcode ) = Koha::Plugins->call( 'item_barcode_transform', $circ->{barcode} ) || $circ->{barcode}; $circ->{barcode} = $new_barcode; my $item = Koha::Items->find({ barcode => $circ->{barcode} }); --- a/opac/sco/sco-main.pl +++ a/opac/sco/sco-main.pl @@ -109,7 +109,7 @@ my ($op, $patronid, $patronlogin, $patronpw, $barcode, $confirmed, $newissues) = $query->param("newissues") || '', ); -( $barcode ) = Koha::Plugins->call( 'barcode_transform', 'item', $barcode ) || $barcode; +( $barcode ) = Koha::Plugins->call( 'item_barcode_transform', $barcode ) || $barcode; my @newissueslist = split /,/, $newissues; my $issuenoconfirm = 1; #don't need to confirm on issue. --- a/t/db_dependent/Koha/Plugins/Barcode_transform_hooks.t +++ a/t/db_dependent/Koha/Plugins/Barcode_transform_hooks.t @@ -70,7 +70,7 @@ subtest '() hook tests' => sub { $item_1->barcode('THISISATEST'); warning_like { $item_1->store(); } - qr/barcode_transform called with parameters: item, THISISATEST/, + qr/item_barcode_transform called with parameter: THISISATEST/, 'AddReserve calls the after_hold_create hook'; $schema->storage->txn_rollback; --- a/t/db_dependent/Koha/Plugins/Biblio_and_Items_plugin_hooks.t +++ a/t/db_dependent/Koha/Plugins/Biblio_and_Items_plugin_hooks.t @@ -53,7 +53,7 @@ subtest 'after_biblio_action() and after_item_action() hooks tests' => sub { my $plugin = Koha::Plugin::Test->new->enable; my $test_plugin = Test::MockModule->new('Koha::Plugin::Test'); - $test_plugin->mock( 'barcode_transform', undef ); + $test_plugin->mock( 'item_barcode_transform', undef ); my $biblio_id; --- a/t/db_dependent/Koha/Plugins/Circulation_hooks.t +++ a/t/db_dependent/Koha/Plugins/Circulation_hooks.t @@ -66,7 +66,7 @@ subtest 'after_circ_action() hook tests' => sub { my $test_plugin = Test::MockModule->new('Koha::Plugin::Test'); $test_plugin->mock( 'after_item_action', undef ); $test_plugin->mock( 'after_biblio_action', undef ); - $test_plugin->mock( 'barcode_transform', undef ); + $test_plugin->mock( 'item_barcode_transform', undef ); my $biblio = $builder->build_sample_biblio(); my $item_1 = $builder->build_sample_item( { biblionumber => $biblio->biblionumber } ); --- a/t/db_dependent/Koha/Plugins/Plugins.t +++ a/t/db_dependent/Koha/Plugins/Plugins.t @@ -211,7 +211,7 @@ ok( $plugin->can('opac_head'), 'Test plugin can opac_head' ); ok( $plugin->can('opac_js'), 'Test plugin can opac_js' ); ok( $plugin->can('intranet_head'), 'Test plugin can intranet_head' ); ok( $plugin->can('intranet_js'), 'Test plugin can intranet_js' ); -ok( $plugin->can('barcode_transform'), 'Test plugin can barcode_transform' ); +ok( $plugin->can('item_barcode_transform'), 'Test plugin can barcode_transform' ); ok( $plugin->can('configure'), 'Test plugin can configure' ); ok( $plugin->can('install'), 'Test plugin can install' ); ok( $plugin->can('upgrade'), 'Test plugin can upgrade' ); --- a/t/lib/Koha/Plugin/Test.pm +++ a/t/lib/Koha/Plugin/Test.pm @@ -93,9 +93,9 @@ sub intranet_js { return "Koha::Plugin::Test::intranet_js"; } -sub barcode_transform { - my ( $self, $type, $barcode ) = @_; - Koha::Exceptions::Exception->throw("barcode_transform called with parameters: $type, $barcode"); +sub item_barcode_transform { + my ( $self, $barcode ) = @_; + Koha::Exceptions::Exception->throw("item_barcode_transform called with parameter: $barcode"); } sub configure { --- a/tools/batchMod.pl +++ a/tools/batchMod.pl @@ -374,7 +374,7 @@ if ($op eq "show"){ my @barcodelist = grep /\S/, ( split /[$split_chars]/, $list ); @barcodelist = uniq @barcodelist; - @barcodelist = map { ( Koha::Plugins->call( 'barcode_transform', 'item', $_ ) )[0] || $_ } @barcodelist; + @barcodelist = map { ( Koha::Plugins->call( 'item_barcode_transform', $_ ) )[0] || $_ } @barcodelist; # Note: adding lc for case insensitivity my %itemdata = map { lc($_->{barcode}) => $_->{itemnumber} } @{ Koha::Items->search({ barcode => \@barcodelist }, { columns => [ 'itemnumber', 'barcode' ] } )->unblessed }; --- a/tools/inventory.pl +++ a/tools/inventory.pl @@ -190,7 +190,7 @@ if ( ($uploadbarcodes && length($uploadbarcodes) > 0) || ($barcodelist && length for my $barcode (@uploadedbarcodes) { next unless $barcode; - ($barcode) = Koha::Plugins->call('barcode_transform', 'item', $barcode ) || $barcode; + ($barcode) = Koha::Plugins->call('item_barcode_transform', $barcode ) || $barcode; ++$lines_read; if (length($barcode)>$barcode_size) { --