Ticket #455 (closed Bug: fixed)

Opened 19 months ago

Last modified 19 months ago

Problem when select a chinese files in FCKeditor

Reported by: reiner Owned by: alfonsoml
Priority: Normal Milestone: FCKeditor 2.4.3
Component: Server : PHP Version: FCKeditor 2.4.2
Keywords: Confirmed Cc:

Description

We can upload a chinese file with FCKeditor, but when we want select the file, it don't show us chinese Characters, it show us only some strange characters 測試1.GIF.

When you go to browse server, you can see that the file is show with this strange characters. On the server self the file is stored right.

Tested it with Linux and Windows OS, With Firefox and Internet Explorer

I attached some files that you can test it

Attachments

Chinese-Character-error.gif (25.9 KB) - added by reiner 19 months ago.
Screenshot of wrong characters
繁體中文字.jpg (15.9 KB) - added by reiner 19 months ago.
Example XLS files
測試1.GIF (2.9 KB) - added by reiner 19 months ago.
Example Gif file
繁體中文字.txt (63 bytes) - added by reiner 19 months ago.
Example txt file

Change History

Changed 19 months ago by reiner

Screenshot of wrong characters

Changed 19 months ago by reiner

Example XLS files

Changed 19 months ago by reiner

Example Gif file

Changed 19 months ago by reiner

Example txt file

Changed 19 months ago by alfonsoml

  • keywords Confirmed added; Chinese Characters removed
  • owner changed from fredck to alfonsoml
  • status changed from new to assigned
  • milestone FCKeditor 2.4.3 deleted

I've been able to reproduce by removing the call to utf8_encode in function ConvertToXmlAttribute( $value ), but I'm not sure if it can break other things.

I'll check the patch along other changes for #454 in my branch.

Changed 19 months ago by alfonsoml

Sorry, I meant:

I've been able to reproduce the problem, and I fixed it by removing the call to utf8_encode ...

Changed 19 months ago by reiner

Thanks, fast solution. Tested it and it works on my install

Changed 19 months ago by fredck

  • milestone set to FCKeditor 2.5

I don't think the call to utf8_encode is really needed (actually it is wrong). You may commit that fix to the trunk Alfonso. Thanks.

Changed 19 months ago by alfonsoml

  • status changed from assigned to closed
  • resolution set to fixed

I've commited it in [299]

Let's hope that it works fine as usually the utf-8 things are tricky to get right.

Changed 19 months ago by fredck

  • milestone changed from FCKeditor 2.5 to FCKeditor 2.4.3
Note: See TracTickets for help on using tickets.