From 9fe36e0c705b958859babe565517554b722c66cd Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Tue, 17 Jun 2014 16:38:54 +0200 Subject: [PATCH] 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 Signed-off-by: Katrin Fischer Good catch! Works as expected, passes tests and QA script. Signed-off-by: Tomas Cohen Arazi Duplicated biblio.title is a (minor) bug, and should be removed. The side-effect of it solving an encoding problem might be seen as problematic: it hides a real problem. The efforts on 11944 actually solve this encoding problem (11944 merged into master actually fixes this), so I'm pushing it, for a short term solution for stable, with the hope that we will soon have 11944 pushed. BTW, non-diacritic but non-ASCII characters are not broken either. --- C4/Acquisition.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/C4/Acquisition.pm b/C4/Acquisition.pm index ce97363a38..28b52f5fb6 100644 --- a/C4/Acquisition.pm +++ b/C4/Acquisition.pm @@ -1052,7 +1052,6 @@ sub GetOrders { SELECT biblio.*,biblioitems.*, aqorders.*, aqbudgets.*, - biblio.title, aqorders_transfers.ordernumber_from AS transferred_from, aqorders_transfers.timestamp AS transferred_from_timestamp FROM aqorders -- 2.11.0