[Koha-bugs] [Bug 11077] Correct silent warnings in C4/Auth.pm

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Nov 8 18:36:30 CET 2013


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11077

M. Tompsett <mtompset at hotmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #22806|0                           |1
        is obsolete|                            |

--- Comment #20 from M. Tompsett <mtompset at hotmail.com> ---
Created attachment 22844
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22844&action=edit
Bug 11077 - Correct silent warnings in C4/Auth.pm

This patch corrects a noisy ne condition.
    $userid = $retuserid if ( $retuserid ne '');
became
    $userid = $retuserid if ( $retuserid );

It also integrates Srdjan Jankovic's patch with Petter Goksoyrsen's
patch, while correcting the problems found.

This includes:
    my $q_userid = $query->param('userid') // '';
along with:
    my $s_userid = '';
and:
    my $s_userid = $session->param('id') // '';
Indentation does not reflect actual scoping.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list