Bug 12438

Summary: Bad encoding in acquisition basket
Product: Koha Reporter: Fridolin Somers <fridolin.somers>
Component: AcquisitionsAssignee: Bugs List <koha-bugs>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: dcook, gmcharlt, kyle, paola.rossi, tomascohen
Version: Main   
Hardware: All   
OS: All   
URL: cgi-bin/koha/acqui/basket.pl?basketno=x
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11944
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 12438 - Bad encoding in acquisition basket
28911: Bug 12438 - Bad encoding in acquisition basket
[PASSED QA] Bug 12438 - Bad encoding in acquisition basket

Description Fridolin Somers 2014-06-17 14:38:30 UTC
We noticed a bad encoding (diacritics replaced by <?>) in acquisition basket when updating a server to Debian Wheezy.
We found it comes from a query containing biblio.title twice.
Maybe the mysql newer version creates this side-effect.
Comment 1 Fridolin Somers 2014-06-17 14:48:26 UTC Comment hidden (obsolete)
Comment 2 Paola Rossi 2014-06-24 15:13:46 UTC
It seems to me that this problem is going to be solved by 11944.
Comment 3 Fridolin Somers 2014-06-24 16:24:25 UTC
(In reply to Paola Rossi from comment #2)
> It seems to me that this problem is going to be solved by 11944.
Maybe, but also the SQL query is wrong.
There is no need to add biblio.title when there is biblio.*
Comment 4 Paola Rossi 2014-06-25 09:39:29 UTC
Thanks, Fridolin

I've applied against master 3.17.00.008

Everything's OK.

So I've passed tha patch to "Signed Off" status.
Comment 5 Paola Rossi 2014-06-25 09:40:18 UTC Comment hidden (obsolete)
Comment 6 Katrin Fischer 2014-06-28 10:15:57 UTC
Created attachment 29340 [details] [review]
[PASSED QA] Bug 12438 - Bad encoding in acquisition basket

We noticed a bad encoding (diacritics replaced by <?>) in acquisition basket when updating a server to Debian Wheezy.
We found it comes from a query containing biblio.title twice.
Maybe the mysql newer version creates this side-effect.

Test plan :
- Create an order on a record containing a diacritic in title
- Look at the basket : cgi-bin/koha/acqui/basket.pl?basketno=x
=> Without the patch the record title is bad encoded (with <?>)
=> With this patch the record title is well encoded
- Check also basket CSV export

Signed-off-by: Paola Rossi <paola.rossi@cineca.it>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Good catch!
Works as expected, passes tests and QA script.
Comment 7 Tomás Cohen Arazi (tcohen) 2014-07-15 22:06:46 UTC
Patch pushed to master.

Thanks Fridolin!

Note: I pushed it because I think the duplicated biblio.title has to be removed. It hides the real problem, but as bug 11944 actually solves the problem (tried it), I feel good about it making into the stable branches for a short term solution.
Comment 8 David Cook 2014-07-18 05:24:03 UTC
Beautiful! This patch fixes the problem that I was having!

It's worth mentioning that we only had this problem on a Debian Squeeze server.

When we dumped the data and loaded it into a Koha instance on an OpenSuse server, we didn't get the <?> replacing the diacritics. It worked perfectly.

So...not really sure what was going on there, but I'm just happy at this point to have a fix of the symptom.
Comment 9 Kyle M Hall (khall) 2014-08-13 13:03:46 UTC
Pushed to 3.12.x!
Comment 10 Fridolin Somers 2014-09-10 14:31:18 UTC
Pushed to 3.14.x, will be in 3.14.11
Comment 11 Galen Charlton 2014-09-23 04:54:49 UTC
Pushed to 3.16.x for inclusion in 3.16.4.