Ticket #2355 (closed Bug: fixed)

Opened 6 years ago

Last modified 6 years ago

FCK:editor output is not XHTML 1.0 compliant

Reported by: mosipov Owned by: mosipov
Priority: High Milestone: FCKeditor.Java 2.4.1
Component: Server : Java Version: FCKeditor.Java 2.4
Keywords: Cc:

Description

the produced iframe source fails W3C validation.

Change History

comment:1 Changed 6 years ago by mosipov

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone set to FCKeditor.Java 2.5

Fixed with [2210]

comment:2 Changed 6 years ago by fredck

  • Status changed from closed to reopened
  • Resolution fixed deleted

The W3C specification defines that & must be used, not &. That's what we have in all other languages btw.

See: C.12. Using Ampersands in Attribute Values (and Elsewhere) in the XHTML 1.0 specs.

comment:3 Changed 6 years ago by mosipov

& == & The spec does only say that you have to escape the ampersand. That's what I did. The code is perfectly valid. There is no regulation saying you have to use the name instead of the unicode point. I ran my code thru the W3C validator and it's valid.

comment:4 Changed 6 years ago by fredck

I'm not strictly defend one or another implementation. I'm just pointing that the Java implementation is different from all other languages, with no apparent reason.

comment:5 Changed 6 years ago by mosipov

  • Owner set to mosipov
  • Status changed from reopened to closed
  • Resolution set to fixed

Fixed in [2224] for the sake of uniformity.

comment:6 Changed 6 years ago by mosipov

  • Milestone changed from FCKeditor.Java 2.5 to FCKeditor.Java 2.4.1

comment:7 Changed 6 years ago by mosipov

  • Status changed from closed to reopened
  • Resolution fixed deleted

Merge patch to branch 2.4.x

comment:8 Changed 6 years ago by mosipov

  • Status changed from reopened to closed
  • Resolution set to fixed

Branch 2.4.x merged with trunk in [2537]

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy