Viewing Atom feed in IE7 fails on nbsp

[Paul Everitt]

(Reported by Oxfam)

Lots of user-generated content contains   and other HTML entities in the content (© etc.). These are valid HTML but, when parsed using an XML parser and no DTD or schema, fail the well-formed XML test.

IE7 appears to parse Atom using an XML parser. Thus, any Atom feed that contains   fails.

Proposed Solution
=================

- When rendering Atom, convert those entities to their numeric/hex equivalents

- Don't try doing this on storage. Pay the price when rendering the Atom view.

- Write some unit tests that catch this.

[Chris Rossi (chris-archimedeanco)]

Paul, I'm noticing that the atom feed template doesn't use 'structure' for the entry content, so HTML is getting escaped to > and < entities. I suspect we actually want to use 'structure' to leave the html intact in entries. Does that seem right to you?

[Paul Everitt (paul-agendaless)]

Sounds right to me, good catch.

Do you have an IE7 you can test with?

--Paul

On Sep 1, 2009, at 1:57 PM, Chris Rossi wrote:

> Paul, I'm noticing that the atom feed template doesn't use 'structure'
> for the entry content, so HTML is getting escaped to > and <
> entities. I suspect we actually want to use 'structure' to leave the
> html intact in entries. Does that seem right to you?
>
> --
> Viewing Atom feed in IE7 fails on nbsp
> https://bugs.launchpad.net/bugs/422571
> You received this bug notification because you are a direct subscriber
> of the bug.
>
> Status in KARL3: In Progress
>
> Bug description:
> (Reported by Oxfam)
>
> Lots of user-generated content contains   and other HTML
> entities in the content (© etc.). These are valid HTML but,
> when parsed using an XML parser and no DTD or schema, fail the well-
> formed XML test.
>
> IE7 appears to parse Atom using an XML parser. Thus, any Atom feed
> that contains   fails.
>
> Proposed Solution
> =================
>
> - When rendering Atom, convert those entities to their numeric/hex
> equivalents
>
> - Don't try doing this on storage. Pay the price when rendering the
> Atom view.
>
> - Write some unit tests that catch this.

[Chris Rossi (chris-archimedeanco)]

>
> Do you have an IE7 you can test with?
>
> Yep.

Chris