[Koha-bugs] [Bug 28699] New: Noisy warns in checkauth when patron has no firstname

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28699

            Bug ID: 28699
           Summary: Noisy warns in checkauth when patron has no firstname
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P5 - low
         Component: Architecture, internals, and plumbing
          Assignee: koha-bugs at lists.koha-community.org
          Reporter: nick at bywatersolutions.com
        QA Contact: testopia at bugs.koha-community.org

Firstname field is not required, however, we try to log it in checkauth:
 888             Koha::Logger->get->debug(sprintf "AUTH_SESSION: (%s)\t%s %s -
%s", map { $session->param($_) || "" } qw(cardnumber firstname surname
branch));

This causes a warn in the logs:
[2021/07/09 13:22:54] [WARN] Use of uninitialized value in sprintf at
/kohadevbox/koha/C4/Auth.pm line 888.

To recreate:
1 - Remove first name from a patron
2 - tail -f /var/log/koha/kohadev/plack-opac-error.log
3 - Login as that patron on the opac

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