Bugzilla – Attachment 67056 Details for
Bug 18996
SIP sets ok flag to true for refused checkin for data corruption
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 18996: [16.11.x] Resolve an uninitialized warning from ILS/Item.pm
Bug-18996-1611x-Resolve-an-uninitialized-warning-f.patch (text/plain), 1.36 KB, created by
Jonathan Druart
on 2017-09-11 19:11:00 UTC
(
hide
)
Description:
Bug 18996: [16.11.x] Resolve an uninitialized warning from ILS/Item.pm
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2017-09-11 19:11:00 UTC
Size:
1.36 KB
patch
obsolete
>From 3be4440e488090b0fdce038959a55cfdf8fe6bd6 Mon Sep 17 00:00:00 2001 >From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Date: Tue, 1 Aug 2017 14:10:56 +0200 >Subject: [PATCH] Bug 18996: [16.11.x] Resolve an uninitialized warning from > ILS/Item.pm > >From the syslog call on line 107/108. The field $item->{title} has not >been initialized. When catching the warns in a test script with Test::Warn, >I got: > Use of uninitialized value $args[1] in sprintf at /usr/lib/x86_64-linux-gnu/perl/5.20/Sys/Syslog.pm line 423. > >This simply resolves the warning. We might have a closer look to this title >field on another report though. > >Test plan: >Run Message.t from the next patch without this fix. It will fail since the >additional warning makes warnings_like return false. > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com> > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > C4/SIP/ILS/Item.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/C4/SIP/ILS/Item.pm b/C4/SIP/ILS/Item.pm >index 578f3828a4..45e59b3970 100644 >--- a/C4/SIP/ILS/Item.pm >+++ b/C4/SIP/ILS/Item.pm >@@ -107,7 +107,7 @@ sub new { > bless $self, $type; > > syslog("LOG_DEBUG", "new ILS::Item('%s'): found with title '%s'", >- $item_id, $self->{title}); >+ $item_id, $self->{title}//'' ); > > return $self; > } >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 18996
:
65303
|
65304
|
65330
|
65336
|
65393
|
65394
|
65418
|
65961
|
65962
|
65963
|
65964
|
66180
|
66181
|
66182
|
66183
|
66184
|
67055
| 67056 |
67057
|
67058
|
67059
|
67849