Bug 14482

Summary: Warns when exporting patron's checked-in barcodes
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: PatronsAssignee: Aleisha Amohia <aleisha>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: bgkriegel, gmcharlt, jonathan.druart, kyle.m.hall, veron
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 14482: Silences warns triggered when exporting patron's checked in barcodes

Description Aleisha Amohia 2015-07-01 05:27:34 UTC
To recreate warns, go to any patron page and click More --> Export today's checked in barcodes 

readingrec.pl: Use of uninitialized value $borrowercardnumber in concatenation (.) or string at /home/vagrant/kohaclone/members/readingrec.pl line 88
readingrec.pl: Use of uninitialized value $borrowercardnumber in concatenation (.) or string at /home/vagrant/kohaclone/members/readingrec.pl line 88
Comment 1 Aleisha Amohia 2015-07-01 05:30:40 UTC
Created attachment 40746 [details] [review]
Bug 14482: Silences warns triggered when exporting patron's checked in barcodes

Use of uninitialized value $borrowercardnumber

To test:
1) Go to any patron page (ie http://localhost:8081/cgi-bin/koha/members/moremember.pl?borrowernumber=1369), click 'More' then 'Export today's checked in barcodes'
2) Notice warns in error log
3) Click cancel on the pop-up
4) Apply patch, reload page, click 'More' then 'Export today's checked in barcodes' again
5) Notice warns are gone and page still works correctly
Comment 2 Zeno Tajoli 2015-09-03 08:44:01 UTC
I don't the patch for SIPConfig.xml.
So probabli there is ame problem with code
Comment 3 Zeno Tajoli 2015-09-03 08:50:53 UTC
(In reply to Zeno Tajoli from comment #2)
> I don't the patch for SIPConfig.xml.
> So probabli there is ame problem with code

Wring comment
Comment 4 Bernardo Gonzalez Kriegel 2015-10-31 00:04:57 UTC
Can't reproduce
Comment 5 Marc Véron 2015-10-31 17:56:19 UTC
Reproduced:
readingrec.pl: Use of uninitialized value in pattern match (m//) at /usr/share/kohaclone/members/readingrec.pl line 80., referer: http://10.0.0.31:8080/cgi-bin/koha/members/moremember.pl?borrowernumber=1
Comment 6 Marc Véron 2015-10-31 18:34:28 UTC
Sorry, warning from comment #5 is on line 80 and it is not the same.
Comment 7 Marc Véron 2015-10-31 19:42:39 UTC
(In reply to Marc Véron from comment #6)
> Sorry, warning from comment #5 is on line 80 and it is not the same.

Update: Warning from comment #5 occurs if patron's privacy rules for the reading history are set to "Never".
Comment 8 Aleisha Amohia 2015-12-14 21:02:22 UTC
Can't see these warns anymore so they must've been fixed!