gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 4515 (Few very small errors in opac-search.pl) RESENT fixing whitespace warning 

Loop on results should stop before $i==@servers.  Adding fix for warnings on undefined values in comparison for lines 485, 488 and 510.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
This commit is contained in:
Marcel de Rooy 2010-11-04 11:38:45 +00:00 committed by Chris Cormack
parent c616cb5081
commit a3ceef434e
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
opac/opac-search.pl
View File

@ -431,7 +431,7 @@ if ($@ || $error) {
# At this point, each server has given us a result set
# now we build that set for template display
my @sup_results_array;
for (my $i=0;$i<=@servers;$i++) {
for (my $i=0;$i<@servers;$i++) {
my $server = $servers[$i];
if ($server && $server =~/biblioserver/) { # this is the local bibliographic server
$hits = $results_hashref->{$server}->{"hits"};
@ -482,10 +482,10 @@ for (my $i=0;$i<=@servers;$i++) {
}
# Adding the new search if needed
if ($borrowernumber eq '') {
if (!$borrowernumber || $borrowernumber eq '') {
# To a cookie (the user is not logged in)
if ($params->{'offset'} eq '') {
if (($params->{'offset'}||'') eq '') {
push @recentSearches, {
"query_desc" => $query_desc || "unknown",
@ -507,7 +507,7 @@ for (my $i=0;$i<=@servers;$i++) {
}
else {
# To the session (the user is logged in)
if ($params->{'offset'} eq '') {
if (($params->{'offset'}||'') eq '') {
AddSearchHistory($borrowernumber, $cgi->cookie("CGISESSID"), $query_desc, $query_cgi, $total);
$template->param(ShowOpacRecentSearchLink => 1);
}