Bug 9033

Summary: Wide character error in runreport.pl
Product: Koha Reporter: Kyle M Hall <kyle>
Component: ReportsAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Paul Poulain <paul.poulain>
Severity: normal    
Priority: P5 - low CC: chris, julian.maurice, ketan, paul.poulain
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 9033 - Wide character error in runreport.pl
[SIGNED-OFF] Bug 9033 - Wide character error in runreport.pl
[Follow-up] Bug 9033: Wide character error in runreport.pl
[SIGNED-OFF] Bug 9033: Wide character error in runreport.pl
Bug 9033 - Wide character error in runreport.pl
Bug 9033: Wide character error in runreport.pl

Description Kyle M Hall 2012-11-08 15:35:08 UTC
runreport.pl is not encoding email correctly. 

Wide character in subroutine entry at /usr/share/perl5/Mail/Sendmail.pm line 308.
Comment 1 Kyle M Hall 2012-11-08 15:39:41 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2012-11-08 17:37:05 UTC
This needs a test plan.
Comment 3 Kyle M Hall 2012-11-19 13:26:53 UTC
Test Plan:
1) Verify you have a least one record with non-ascii unicode characters ( i.e. a record that would cause the wide character error ).
2) Create a report that would contain these characters as part of its result.
3) Run this report via runreport.pl with the email option
4) This should cause the wide character error
5) Apply this patch
6) Re-run step 3.
7) This time, you should not get any wide character error.
Comment 4 Julian Maurice 2012-11-22 16:22:34 UTC Comment hidden (obsolete)
Comment 5 Julian Maurice 2012-11-22 16:22:40 UTC Comment hidden (obsolete)
Comment 6 Julian Maurice 2012-11-22 16:25:05 UTC
I followed the test plan, got the wide character error on master and got no errors with the patch. On master the mail is not sent. With the patch the mail is sent and with correct encoding.

I just added a tiny follow-up because runreport.pl wasn't working without it.
Comment 7 Kyle M Hall 2012-12-03 17:22:55 UTC Comment hidden (obsolete)
Comment 8 Paul Poulain 2012-12-11 19:45:44 UTC
QA comment: small patch, passes koha-qa.pl, nothing to add

passed QA
Comment 9 Paul Poulain 2012-12-11 19:46:14 UTC
Created attachment 14011 [details] [review]
Bug 9033 - Wide character error in runreport.pl

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 10 Paul Poulain 2012-12-11 19:46:18 UTC
Created attachment 14012 [details] [review]
Bug 9033: Wide character error in runreport.pl

This patch fix a typo that prevented report to be run

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Comment 11 Jared Camins-Esakov 2012-12-12 12:29:46 UTC
This patch has been pushed to master.

Note that wide characters will still not display in the e-mail. All these patches do is ensure that the e-mail is sent.
Comment 12 Chris Cormack 2012-12-12 20:11:05 UTC
Pushed to 3.10.x, will be in 3.10.1
Comment 13 Chris Cormack 2013-04-29 21:30:30 UTC
*** Bug 10031 has been marked as a duplicate of this bug. ***