Bug 25992: Make SIP2 logger subroutines exportable to prevent crash
authorJoonas Kylmälä <joonas.kylmala@helsinki.fi>
Wed, 15 Jul 2020 11:08:32 +0000 (11:08 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Thu, 16 Jul 2020 14:26:47 +0000 (15:26 +0100)
If the subroutines are not exportable we get the following crash:

> Undefined subroutine &C4::SIP::SIPServer::set_logger

To test:
 In kohadevbox run for example:
 $ ps -aux # check that no existing sip server is running, kill the process if exists
 $ perl /kohadevbox/koha/C4/SIP/SIPServer.pm /etc/koha/sites/kohadev/SIPconfig.xml
 $ koha/misc/sip_cli_emulator.pl -su koha -sp koha -l CPL -a 127.0.0.1 -p 6001 --item 3999900000001 -m item_information

 After applying this patch the Undefined subroutine error should be gone.
 Note: when using the sip_cli_emulator.pl the credentials can be anything.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

C4/SIP/Logger.pm

index 4116921..c73886c 100644 (file)
@@ -3,6 +3,9 @@ package C4::SIP::Logger;
 
 use Modern::Perl;
 
+use base 'Exporter';
+our @EXPORT_OK = qw ( get_logger set_logger );
+
 our $activeSIPServer;
 our $activeLogger;