Koha/cataloguing/value_builder/barcode.pl
Lucas Gass 85d38e5ef5 Bug 35101: Stop screen from jumping back to top when using barcode.pl plugin
To test:
1. Set 'autoBarcode' to something other than 'don't generate automatically'
2. If the a framwwork make sure some 952 fields have the barcode.pl plugin enabled
3. Add a new item and click the '...' next to the 952 field with the barcode.pl plugin enabled.
4. The screen jumps back to the top, annoying.
5. Apply patch
6. Try again, it should not jump anymore!

Signed-off-by: Barbara Johnson <barbara.johnson@swbell.net>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit eec5d7e4be)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
2023-10-25 11:42:39 -10:00

107 lines
3.4 KiB
Perl
Executable file

#!/usr/bin/perl
# Converted to new plugin style (Bug 13437)
# Copyright 2000-2002 Katipo Communications
# Parts copyright 2008-2010 Foundations Bible College
#
# This file is part of Koha.
#
# Koha is free software; you can redistribute it and/or modify it
# under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 3 of the License, or
# (at your option) any later version.
#
# Koha is distributed in the hope that it will be useful, but
# WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with Koha; if not, see <http://www.gnu.org/licenses>.
use Modern::Perl;
use C4::Context;
use C4::Barcodes::ValueBuilder;
use C4::Biblio qw( GetMarcFromKohaField );
use Koha::DateUtils qw( dt_from_string );
use Algorithm::CheckDigits qw( CheckDigits );
my $builder = sub {
my ( $params ) = @_;
my $function_name = $params->{id};
my %args;
# find today's date
($args{year}, $args{mon}, $args{day}) = split('-', dt_from_string()->ymd());
($args{tag},$args{subfield}) = GetMarcFromKohaField( "items.barcode" );
my $nextnum;
my $scr;
my $autoBarcodeType = C4::Context->preference("autoBarcode");
if ((not $autoBarcodeType) or $autoBarcodeType eq 'OFF') {
# don't return a value unless we have the appropriate syspref set
return q|<script></script>|;
}
if ($autoBarcodeType eq 'annual') {
($nextnum, $scr) = C4::Barcodes::ValueBuilder::annual::get_barcode(\%args);
}
elsif ($autoBarcodeType eq 'incremental') {
($nextnum, $scr) = C4::Barcodes::ValueBuilder::incremental::get_barcode(\%args);
}
elsif ($autoBarcodeType eq 'hbyymmincr') { # Generates a barcode where hb = home branch Code, yymm = year/month catalogued, incr = incremental number, reset yearly -fbcit
($nextnum, $scr) = C4::Barcodes::ValueBuilder::hbyymmincr::get_barcode(\%args);
}
elsif ($autoBarcodeType eq 'EAN13') {
# not the best, two catalogers could add the same barcode easily this way :/
my $query = "select max(abs(barcode)) from items";
my $dbh = $params->{dbh};
my $sth = $dbh->prepare($query);
$sth->execute();
while (my ($last)= $sth->fetchrow_array) {
$nextnum = $last;
}
my $ean = CheckDigits('ean');
if ( $ean->is_valid($nextnum) ) {
my $next = $ean->basenumber( $nextnum ) + 1;
$nextnum = $ean->complete( $next );
$nextnum = '0' x ( 13 - length($nextnum) ) . $nextnum; # pad zeros
} else {
warn "ERROR: invalid EAN-13 $nextnum, using increment";
$nextnum++;
}
}
else {
warn "ERROR: unknown autoBarcode: $autoBarcodeType";
}
# default js body (if not filled by hbyymmincr)
$scr or $scr = <<END_OF_JS;
if (\$('#' + id).val() == '' || force) {
\$('#' + id).val('$nextnum');
};
END_OF_JS
my $js = <<END_OF_JS;
<script>
function set_barcode(id, force) {
$scr
}
function Focus$function_name(event) {
set_barcode(event.data.id, false);
return false;
}
function Click$function_name(event) {
set_barcode(event.data.id, true);
return false;
}
</script>
END_OF_JS
return $js;
};
return { builder => $builder };