Jonathan Druart
091d6c513b
The C4::Koha::getitemtypeinfo subroutine did the almost same job as GetItemTypes. On top of that it returned the imageurl value processed by C4::Koha::getitemtypeimagelocation. This value is only used from the 2 [opac-]shelves.pl scripts. Then it's better not retrieve it only when we need it. Test plan: Play with the different scripts touched by this patch and focus on item types. The same description as prior to this patch must be displayed. Note that sometimes it is not the translated description which is displayed, but that should be fixed on another bug report. Indeed we do not expect this patch to change any behaviors. Signed-off-by: Lari Taskula <lari.taskula@jns.fi> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
103 lines
2.9 KiB
Perl
Executable file
103 lines
2.9 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
# Copyright 2000-2002 Katipo Communications
|
|
#
|
|
# 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>.
|
|
|
|
|
|
=head1 opac-browser.pl
|
|
|
|
TODO :: Description here
|
|
|
|
=cut
|
|
|
|
use strict;
|
|
use warnings;
|
|
|
|
use C4::Auth;
|
|
use C4::Context;
|
|
use C4::Output;
|
|
use CGI qw ( -utf8 );
|
|
use C4::Biblio;
|
|
|
|
my $query = new CGI;
|
|
|
|
my $dbh = C4::Context->dbh;
|
|
|
|
# open template
|
|
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
|
|
{
|
|
template_name => "opac-browser.tt",
|
|
query => $query,
|
|
type => "opac",
|
|
authnotrequired => ( C4::Context->preference("OpacPublic") ? 1 : 0 ),
|
|
debug => 1,
|
|
}
|
|
);
|
|
|
|
# the level of browser to display
|
|
my $level = $query->param('level') || 0;
|
|
my $filter = $query->param('filter');
|
|
$filter = '' unless defined $filter;
|
|
$level++; # the level passed is the level of the PREVIOUS list, not the current one. Thus the ++
|
|
|
|
# build this level loop
|
|
my $sth = $dbh->prepare("SELECT * FROM browser WHERE level=? and classification like ? ORDER BY classification");
|
|
$sth->execute($level,$filter."%");
|
|
my @level_loop;
|
|
my $i=0;
|
|
while (my $line = $sth->fetchrow_hashref) {
|
|
$line->{description} =~ s/\((.*)\)//g;
|
|
push @level_loop,$line;
|
|
}
|
|
|
|
my $have_hierarchy = 0;
|
|
|
|
# now rebuild hierarchy loop
|
|
$filter =~ s/\.//g;
|
|
my @hierarchy_loop;
|
|
if ($filter eq '' and $level == 1) {
|
|
# we're starting from the top
|
|
$have_hierarchy = 1 if @level_loop;
|
|
unless (@level_loop) {
|
|
$sth->execute(1, "%");
|
|
while (my $line = $sth->fetchrow_hashref) {
|
|
$line->{description} =~ s/\((.*)\)//g;
|
|
push @level_loop,$line;
|
|
}
|
|
}
|
|
} else {
|
|
$sth = $dbh->prepare("SELECT * FROM browser where classification=?");
|
|
for (my $i=1;$i <=length($filter);$i++) {
|
|
$sth->execute(substr($filter,0,$i));
|
|
my $line = $sth->fetchrow_hashref;
|
|
push @hierarchy_loop,$line;
|
|
}
|
|
$have_hierarchy = 1 if @hierarchy_loop;
|
|
}
|
|
|
|
# mark every third entry in level_loop
|
|
for (my $i = 0; $i <= $#level_loop; $i++) {
|
|
$level_loop[$i]->{count3} = 1 if 2 == $i % 3 && $i != $#level_loop;
|
|
}
|
|
|
|
$template->param(
|
|
LEVEL_LOOP => \@level_loop,
|
|
HIERARCHY_LOOP => \@hierarchy_loop,
|
|
have_hierarchy => $have_hierarchy,
|
|
);
|
|
|
|
output_html_with_http_headers $query, $cookie, $template->output, undef, { force_no_caching => 1 };
|