[#CONTRIB-364] Handle converting WebCT XML content encoding to UTF-8 where necessary

Issue Details (XML | Word | Printable)

Key:	CONTRIB-364
Type:	Sub-task
Status:	Open
Priority:	Minor
Assignee:	Ashley Holman
Reporter:	Jerome Charaoui
Votes:	0
Watchers:	3

Operations

Add/Edit UI Mockup to this issue

If you were logged in you would be able to see more operations.

Non-core contributed modules

CONTRIB-363

Handle converting WebCT XML content encoding to UTF-8 where necessary

Created: 05/Apr/08 05:49 AM Updated: 11/Sep/09 01:37 PM

Return to search

Component/s:

Tool: WebCTImport

Affects Version/s:

1.9

Fix Version/s:

None

File Attachments:

diff-ok.txt (1 kB)
2.

diff.txt (0.5 kB)

Participants:	Anthony Borrow, Ashley Holman and Jerome Charaoui
Security Level:	None
Affected Branches:	MOODLE_19_STABLE

Description

« Hide

When importing courses from WebCT to Moodle 1.9, strings are not converted to UTF-8. The resulting strings in Moodle are either completely absent or incomplete. I've noticed this behavior when importing questions but I'm not sure if it affects other aspects of imported courses.

I've written a simple patch that detects the encoding of the questionsDB.xml file and converts it to UTF-8 before processing the contents.

All

Comments

Change History

Version Control

Sort Order:

[ Permalink | « Hide ]

Jerome Charaoui added a comment - 05/Apr/08 06:05 AM

The original diff.txt patch was missing a part, this is the complete, initial patch proposed.

Jerome Charaoui made changes - 05/Apr/08 06:05 AM

Field	Original Value	New Value
Attachment		diff-ok.txt [ 13540 ]

[ Permalink | « Hide ]

Anthony Borrow added a comment - 01/May/08 08:43 AM

Moving to the Tool: WebCTImport Component and assigning to Ashley

Anthony Borrow made changes - 01/May/08 08:43 AM

Component/s		Tool: WebCTImport [ 10308 ]
Component/s	Module: {none of the above} [ 10212 ]
Assignee	Anthony Borrow [ aborrow ]	Ashley Holman [ ashleyholman ]

[ Permalink | « Hide ]

Ashley Holman added a comment - 11/Sep/09 01:37 PM

Hi Jerome,

I noticed this patch isn't yet in the latest CVS revision. Are you able to attach a ce4 backup which demonstrates the issue so I can do some testing?

Regards
Ashley

All content on this web site is made available under the GNU General Public License, unless otherwise stated.