Created attachment 5215 [details] Corrects encoding problems in biblio title of subscription-add.pl In subscription-add.pl, if we choose a biblio with accented characters in its title using "Search for Biblio", the biblio's title is encoded incorrectly in the form. This is the exact same problem described by my coworker on bug #6444, but for the field below. Sorry for the patch against 3.2.x. I can rebase it for 3.4 if wanted.
Created attachment 5218 [details] [review] Patche reworked to use TT Ported Maxime's patch to the tt version
Are you sure it doesn't work on HEAD. For me, HEAD branch doesn't show this bug.
Yes I managed to reproduce the bug on head. Applied the patch and it fixed the encoding
My patch also fixes that the "choose" link doesn't work if there's a "'" in the title. Anyone could sign off on this?
Created attachment 5481 [details] [review] Signed-off patch
(In reply to comment #5) > Signed-off patch ...Should have specified: For master.
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>
QA comment : * good description * small patch passed QA RM = no change in strings despite it's a templates change, can be pushed
Pushed, please test