[Koha-bugs] [Bug 4064] New: Uninitialized variable errors in opac-search.pl

[bugzilla-daemon at kohaorg.ec2.liblime.com]

http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4064

           Summary: Uninitialized variable errors in opac-search.pl
           Product: Koha
           Version: HEAD
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P5
         Component: OPAC
        AssignedTo: henridamien at koha-fr.org
        ReportedBy: oleonard at myacpl.org
   Estimated Hours: 0.0
 Change sponsored?: ---


I'm seeing a couple of errors in my error log coming from opac-search.pl:

opac-search.pl: Use of uninitialized value $borrowernumber in string eq at
/home/oleonard/kohaclone/opac/opac-search.pl line 465.

...referring to:

  if ($borrowernumber eq '') {

(error occurs if the user is not logged in). And:

opac-search.pl: Use of uninitialized value in string eq at
/home/oleonard/kohaclone/opac/opac-search.pl line 468.

...referring to:

if ($params->{'offset'} eq '') {

I notice that on line 329 a variable $offset is initialized like this:

my $offset = $params->{'offset'} || 0;

...which made me wonder whether the later check for $params->{'offset'} should
be checking $offset instead? But it's not clear to me how that's being used.


-- 
Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.