]> git.koha-community.org Git - koha.git/commit
Bug 15209 Validate passed MARC::Record objs in C4::Koha
authorColin Campbell <colin.campbell@ptfs-europe.com>
Wed, 18 Nov 2015 14:53:58 +0000 (14:53 +0000)
committerFrédéric Demians <f.demians@tamil.fr>
Thu, 11 Feb 2016 18:33:50 +0000 (19:33 +0100)
commit60e9074bec01d4e16502bb0f9535ba8d11dcf391
tree0793d317be5374f98c5b29481e730edf6c620002
parent7d81810d148dccded70c3cd3ab0a5b7cfd1c2d74
 Bug 15209 Validate passed MARC::Record objs in C4::Koha

Ensure that a passed MARC::Record is defined before calling
its methods. Otherwise you are open to occurences of the
error 'Can't call method "field" on an undefined value'
In a CGI environment you can live with such sloppiness but
in a persistent environment the error can cause the instance
to abort.
Made all routines passed a MARC::Record validate it before calling
its methods. Changed the parameter name from the meaningless
record to marcrecord to indicate its content. Added an explicit return
for all cases where no valid data returned. Cleaned up some logic for
clarity. I think we can assume that GetNormalizedOCLCNumber meant to
look at all 035s till it found an OCLC number not just the first.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
(cherry picked from commit 960458dee4a9c09506f48fa73950c0e2da3611e7)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
(cherry picked from commit 958985b2ff7304cc55a620e0cd1f09d31d213463)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
C4/Koha.pm