Changes between Initial Version and Version 1 of Ticket #724


Ignore:
Timestamp:
Jun 25, 2007, 2:47:16 AM (12 years ago)
Author:
Martin Kou
Comment:

Hi Justin,

Confirmed. That line is not necessary and must be removed.

Best regards, Frederico Caldeira Knabben


http://www.fckeditor.net "Support Open Source Software... What about a donation today?"


Moved from SF. Original poster: fredck

No, ' isn't valid in HTML -- it's only valid in XML (including XHTML). It should be either ' or no replacement.

If you're having problems in MySQL, that's a totally separate issue. You should be escaping all strings at the time you put them into the query! Don't expect upstream libraries to do the wrong thing to fix brokenness in other parts of your code. What if there's an apostrophe in the text content of the HTML?


Moved from SF. Original poster: jsampson

Well, if it is not replaced it causes problems when inserting the string to a MySQL query. Now, in version 2.3 the APOSTROPHE is not encoded, so it causes problems. I think that it should be encoded to '


Moved from SF. Original poster: ickata

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #724

    • Property Cc ickata@… Frederico Caldeira Knabben added
    • Property Reporter changed from Martin Kou to jsampson@…
  • Ticket #724 – Description

    initial v1  
    2121txt=txt.replaceAll("'","’");
    2222}}}
     23----
     24Moved from SF:[[BR]]
     25http://sourceforge.net/tracker/index.php?func=detail&aid=1478711&group_id=75348&atid=543653
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy