Feeds applet doesn't show escaped characters properly

Bug #592359 reported by Gabor Karsay
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Awn Extras
Fix Released
Low
Matt

Bug Description

Planet Awn feed has some items with the character ’ and this is not shown as it should be.
For example there's an item "Demain, c’est Ubuntu-Party à Paris" which is shown in Feeds applet as "Demain, c’est Ubuntu-Party à Paris". (Hopefully launchpad doesn't change this.)

Tags: applet feeds

Related branches

Matt (mjz)
Changed in awn-extras:
status: New → Confirmed
assignee: nobody → Matt (mjz)
Matt (mjz)
tags: added: applet feeds
Revision history for this message
Matt (mjz) wrote :

This has been fixed in r1330. Thank you for your bug report.

Changed in awn-extras:
status: Confirmed → Fix Committed
Matt (mjz)
Changed in awn-extras:
milestone: none → 0.4.2
Revision history for this message
Gabor Karsay (gabor-karsay) wrote :

The ’ issue is fixed, however, it displays " instead of the character. For example in this feed:
http://sharkbaitbobby.blogspot.com/feeds/2791440219533669058/comments/default
The newest entry is displayed with " but the tooltip is o.k., it shows the character.

Revision history for this message
Gabor Karsay (gabor-karsay) wrote :

Unfortunately the fix is not very good. Some characters are shown properly, like ’ others are not, like … or –. (I don't know why, but you can see it at the moment in http://planet.awn-project.org/?feed=atom )

Other entities like " are now displayed like this - that is a regression of this fix.

Changed in awn-extras:
status: Fix Committed → Confirmed
Revision history for this message
Gabor Karsay (gabor-karsay) wrote :

I don't see the point in reinventing standards (like HTML unescaping) again and again. This patch/def is taken from Fredrik Lundh at http://effbot.org/zone/re-sub.htm#unescape-html. The copyright statement of the website says: "Unless otherwise noted, source code can be be used freely. Examples, test scripts and other short code fragments can be considered as being in the public domain."

It works as is, "import re" is already done, "import htmlentitydefs" is not needed (I don't know why).

onox (onox)
Changed in awn-extras:
importance: Undecided → Low
Revision history for this message
onox (onox) wrote :

Patch applied in revision 1424.

Changed in awn-extras:
status: Confirmed → Fix Committed
Povilas Kanapickas (p12)
Changed in awn-extras:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.