[#LDEV-2481] Export portfolio problems with UTF8 characters

LAMS Development

Export portfolio problems with UTF8 characters

Created: 03/Jan/10 09:10 AM Updated: 24/Dec/11 10:02 AM

Component/s:

General

Affects Version/s:

2.3.3

Fix Version/s:

2.4, 2.3.5

Resolution Date:	24/Dec/11 10:02 AM
Date of First Response:	15/Jan/10 11:29 AM

Description

« Hide

Spyros Papadakis has reported that activity's name in export portfolio with UTF8 characters get displayed as rubbish. See his posting in LAMS Community. Also, have a look at the attached file:

http://is.gd/5K6mV

All

Comments

Change History

FishEye

Sort Order:

Change by Ernie Ghiglione - 03/Jan/10 09:33 AM
Field	Original Value	New Value
Component/s		General [ 10012 ]

lams2_3_release:jliew:20100115002018 by jliew (1 file)

14/Jan/10 06:20 PM

~~LDEV-2481~~ export portfolio files were being written by image link replacer which didn't read or encode the new files in utf8

lams_learning/src/java/org/lamsfoundation/lams/learning/export/web/action/MainExportServlet.java 1.29.6.2.4.1

(+3 -3) diffs

MAIN:jliew:20100115002247 by jliew (1 file)

14/Jan/10 06:22 PM

~~LDEV-2481~~ merged from branch

lams_learning/src/java/org/lamsfoundation/lams/learning/export/web/action/MainExportServlet.java 1.31

(+3 -3) diffs

[ Permlink | « Hide ]

Jun-Dir Liew - 15/Jan/10 11:29 AM

There are 2 problems here, 1. export portfolio files not displaying utf8 correctly, and 2. export portfolio utf8 filename not being presented by IE correctly.

1. is fixed due to image rewriter method not reading/encoding file in utf8 when it rewrites image links;

2. is not a LAMS problem due to browser support of character encodings in the content-disposition header. This page http://greenbytes.de/tech/tc2231/ has a good roundup of browser behaviour in this regard. Note that IE and Safari usually don't try to convert to utf8 and neither support RFC2231 (specifying encoding in header).

Change by Jun-Dir Liew - 15/Jan/10 11:29 AM
Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

lams2_3_release:jliew:20100205011842 by jliew (1 file)

04/Feb/10 07:18 PM

~~LDEV-2481~~ use lams custom browser workaround for filenames

lams_learning/src/java/org/lamsfoundation/lams/learning/export/web/action/ExportDownloadServlet.java 1.11.10.1

(+3 -2) diffs

lams2_3_release:jliew:20100205050945 by jliew (1 file)

04/Feb/10 11:09 PM

~~LDEV-2481~~ need to read param as utf8

lams_learning/src/java/org/lamsfoundation/lams/learning/export/web/action/ExportDownloadServlet.java 1.11.10.2

(+2 -1) diffs

[ Permlink | « Hide ]

Ernie Ghiglione - 05/Feb/10 01:47 PM

still having issues with IE.

Change by Ernie Ghiglione - 05/Feb/10 01:47 PM
Resolution	Fixed [ 1 ]
Status	Resolved [ 5 ]	Reopened [ 4 ]

MAIN:jliew:20100211052709 by jliew (1 file)

10/Feb/10 11:27 PM

~~LDEV-2481~~ merge from branch

lams_learning/src/java/org/lamsfoundation/lams/learning/export/web/action/ExportDownloadServlet.java 1.13

(+4 -2) diffs

[ Permlink | « Hide ]

Ernie Ghiglione - 11/Feb/10 05:34 PM

MOving to 2.3.5 for now.

Change by Ernie Ghiglione - 11/Feb/10 05:34 PM
Fix Version/s		2.3.5 [ 10170 ]
Fix Version/s	2.3.4 [ 10161 ]

Change by Ernie Ghiglione - 15/Oct/10 05:11 PM
Assignee	Jun-Dir Liew [ jundirl ]	Andrey Balan [ dronych ]

lams2_3_release:andreyb:20101022194304 by andreyb (1 file)

22/Oct/10 02:43 PM

~~LDEV-2481~~ fixing charset conversion

lams_learning/src/java/org/lamsfoundation/lams/learning/export/web/action/ExportDownloadServlet.java 1.11.10.4

(+2 -2) diffs

[ Permlink | « Hide ]

Andrey Balan - 23/Oct/10 07:33 AM

fixed

however it will need some testing on platforms other than Windows

Change by Andrey Balan - 23/Oct/10 07:33 AM
Status	Reopened [ 4 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

MAIN:andreyb:20101029151415 by andreyb (1 file)

29/Oct/10 10:14 AM

~~LDEV-2481~~ fixing charset conversion

*merge from lams2_3_release

lams_learning/src/java/org/lamsfoundation/lams/learning/export/web/action/ExportDownloadServlet.java 1.14

(+2 -2) diffs

[ Permlink | « Hide ]

Ernie Ghiglione - 19/Nov/10 02:36 AM

This isn't working I thought this could be a unix or windows thing, but it doesn't work on neither.

One of the things i would suggest is to look at how the export sequence in author handles utf-8 file names as it seems to work there.

Change by Ernie Ghiglione - 19/Nov/10 02:36 AM
Resolution	Fixed [ 1 ]
Status	Resolved [ 5 ]	Reopened [ 4 ]

[ Permlink | « Hide ]

Andrey Balan - 26/Nov/10 04:32 AM

in order to fix this for Unix based servers
change the following line in ExportDownloadServlet.java
{code}
zipFilename = new String(zipFilename.getBytes(("ISO-8859-1")), "UTF-8");
{code}
to
{code}
zipFilename = URLDecoder.decode(zipFilename, "UTF-8");
{code}

MAIN:ernieg:20111223230204 by Ernie Ghiglione (1 file)

23/Dec/11 05:02 PM

~~LDEV-2481~~ filename for zipfile is now utf-8

lams_learning/src/java/org/lamsfoundation/lams/learning/export/web/action/ExportDownloadServlet.java 1.15

(+4 -3) diffs

Change by Ernie Ghiglione - 24/Dec/11 08:41 AM
Assignee	Andrey Balan [ dronych ]	Ernie Ghiglione [ ernieg ]

Change by Ernie Ghiglione - 24/Dec/11 10:02 AM
Status	Reopened [ 4 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

[ Permlink | « Hide ]

Ernie Ghiglione - 24/Dec/11 10:02 AM

Done

Change by Ernie Ghiglione - 24/Dec/11 10:02 AM
Status	Resolved [ 5 ]	Closed [ 6 ]