#40 — Post content in xml needs more careful approaches
by
Domen Kožar
—
last modified
Dec 27, 2011 10:12 PM
| State | Resolved |
|---|---|
| Version: |
—
|
| Area | Functionality |
| Issue type | Bug |
| Severity | Medium |
| Submitted by | Domen Kožar |
| Submitted on | Nov 25, 2011 |
| Responsible | Maurits van Rees |
| Target release: | 2.0.7 |
Two issues, if content has text/xhtml content type:
* if there is no wrapper tag for the content, minidom parser fails
* archtypes have no idea about xhtml by default, so following line fails https://github.com/[…]/utilities.py#L61
Problematic feed: http://www.tablix.org/~avian/blog/atom.xml
Fix (commit): https://github.com/[…]/db8fd8f7d1322ee15e9f1d223187ba96e2808315
PS: commit is not really a fix for upstream, as I have close to zero experience with feeds in general
* if there is no wrapper tag for the content, minidom parser fails
* archtypes have no idea about xhtml by default, so following line fails https://github.com/[…]/utilities.py#L61
Problematic feed: http://www.tablix.org/~avian/blog/atom.xml
Fix (commit): https://github.com/[…]/db8fd8f7d1322ee15e9f1d223187ba96e2808315
PS: commit is not really a fix for upstream, as I have close to zero experience with feeds in general
Added by
Maurits van Rees
on
Dec 27, 2011 04:49 PM
Sorry for this late reaction.
Issue state:
Unconfirmed
→
Resolved
Responsible manager:
(UNASSIGNED)
→
maurits
I cannot reproduce this, also not with the feed as it was at the moment you reported it (I downloaded it at the time).
But it makes sense, so I have applied your fix (slightly changed). Fixed in r247572.
Thanks.
Added by
Maurits van Rees
on
Dec 27, 2011 10:12 PM
2.0.7 has been released with this fix.
Target release:
None
→
2.0.7
No responses can be added.
If you can, please log in before submitting a reaction.