Bug 12438 - Bad encoding in acquisition basket
authorFridolin Somers <fridolin.somers@biblibre.com>
Tue, 17 Jun 2014 14:38:54 +0000 (16:38 +0200)
committerKoha instance rel312x-koha <rel312x-koha@koha-dev.(none)>
Wed, 13 Aug 2014 13:03:25 +0000 (09:03 -0400)
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.
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
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.

(cherry picked from commit 9fe36e0c705b958859babe565517554b722c66cd)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit aa94a7c0e45d9aab79ec9958b59e142dcda616cf)

C4/Acquisition.pm

index f41a15d..5c49d4a 100644 (file)
@@ -937,8 +937,7 @@ sub GetOrders {
     my $query  ="
         SELECT biblio.*,biblioitems.*,
                 aqorders.*,
-                aqbudgets.*,
-                biblio.title
+                aqbudgets.*
         FROM    aqorders
             LEFT JOIN aqbudgets        ON aqbudgets.budget_id = aqorders.budget_id
             LEFT JOIN biblio           ON biblio.biblionumber = aqorders.biblionumber