Bug 6806

Summary: Encoding problem with biblio titles in subscription-add.pl
Product: Koha Reporter: Maxime Pelletier <pelletiermaxime>
Component: SerialsAssignee: Colin Campbell <colin.campbell>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: P5 - low CC: chris, f.demians, paul.poulain
Version: rel_3_2   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Corrects encoding problems in biblio title of subscription-add.pl
Patche reworked to use TT
Signed-off patch
Bug 6806 Fix encoding of title in subscription-add

Description Maxime Pelletier 2011-08-29 17:32:44 UTC Comment hidden (obsolete)
Comment 1 Colin Campbell 2011-08-30 14:07:12 UTC Comment hidden (obsolete)
Comment 2 Frédéric Demians 2011-08-30 14:30:19 UTC
Are you sure it doesn't work on HEAD. For me, HEAD branch doesn't show this bug.
Comment 3 Colin Campbell 2011-08-30 14:33:55 UTC
Yes I managed to reproduce the bug on head. Applied the patch and it fixed the encoding
Comment 4 Maxime Pelletier 2011-09-20 18:27:31 UTC
My patch also fixes that the "choose" link doesn't work if there's a "'" in the title.
Anyone could sign off on this?
Comment 5 Owen Leonard 2011-09-20 18:34:33 UTC Comment hidden (obsolete)
Comment 6 Owen Leonard 2011-09-20 18:35:31 UTC
(In reply to comment #5)
> Signed-off patch

...Should have specified: For master.
Comment 7 Paul Poulain 2011-10-14 10:10:14 UTC
Created attachment 5887 [details] [review]
Bug 6806 Fix encoding of title in subscription-add

Port of Maxime Pelletier <maxime.pelletier@libeo.com>'s patch for
3.2 to TT
Stops the mangling of titles containing accented characters

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Comment 8 Paul Poulain 2011-10-14 10:11:42 UTC
QA comment :
  * good description
  * small patch

passed QA

RM = no change in strings despite it's a templates change, can be pushed
Comment 9 Chris Cormack 2011-10-15 00:58:54 UTC
Pushed, please test