#695 closed Bug (fixed)
Wrong single quote entity code?
Reported by: | Owned by: | Michael Osipov | |
---|---|---|---|
Priority: | Normal | Milestone: | FCKeditor.Java 2.4 |
Component: | Server : Java | Version: | FCKeditor 2.5.1 |
Keywords: | SF | Cc: | simo@… |
Description
In the class FCKeditor.java there is a metho HTMLEncode that is used to encode the initial text to be put in the editor upon display.
The straight single quote ' is encoded with entity '
However, in the java code, it is encoded with the entity ’ which is a ’ right side quote.
This does not cause only esthetic syntax chanfge. This causes FCKeditor's own generated Javascript code to fail. ( for example when you make a link that is to be opened in a popup ).
1) You save you popup link. Its ok. 2) You re-edit your popup link ( passes through HTMLEncode to be set as the initial text ). 3) You find out in the html source of the editor that the ' are tranformed in ’. Your text is corrupted.
Moved from SF:
http://sourceforge.net/tracker/index.php?func=detail&aid=1404166&group_id=75348&atid=543653
Change History (11)
comment:1 Changed 18 years ago by
Cc: | simo@… added |
---|---|
Reporter: | changed from Martin Kou to plabadie@… |
comment:2 Changed 18 years ago by
Component: | General → Server : Java |
---|---|
Owner: | set to alka13 |
#724 has been marked as dup
Has this bug been fixed in the java package? the bugs in sourceforge are quite old.
comment:4 Changed 17 years ago by
comment:5 Changed 17 years ago by
Owner: | alka13 deleted |
---|
comment:8 Changed 17 years ago by
comment:9 Changed 17 years ago by
Milestone: | → FCKeditor.Java 2.4 |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
Version: | → FCKeditor 2.5.1 |
comment:10 Changed 17 years ago by
It may sound a little bit late to say that, but is this entity conversion needed? I don't see problems on having single quotes in the generated source.
comment:11 Changed 17 years ago by
Fred,
I don't know but this spec http://wiki.fckeditor.net/Developer%27s_Guide/Participating/Server_Side_Integration#Creator
Thank you this bug has already been pointed out and a fixed version will be released soon.
Simone
Moved from SF. Original poster: simo