Bug 97932 - html special chars are not translated.
Summary: html special chars are not translated.
Status: RESOLVED FIXED
Alias: None
Product: akregator
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-01-26 15:23 UTC by Stephen Leaf
Modified: 2005-01-26 20:13 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Stephen Leaf 2005-01-26 15:23:48 UTC 
Version:            (using KDE KDE 3.3.91)
Installed from:    Gentoo Packages
Compiler:          gcc 3.3.5 
OS:                Linux

in the listview html special chars are translated however in the detailed view these are displayed as the raw html

&, >, <, etc

Examples of this can be found on the kde-look rss feed for Theme/Style for KDE > 3.2
Comment 1 Frank Osterfeld 2005-01-26 20:13:05 UTC 
CVS commit by osterfeld: 

This works better than QStyleSheet::escape(). Although it would be better to solve all these problems by escaping and 
resolving entities appriopriately in the RSS parser instead of messing around with it in GUI classes.
BUG:97932


  M +1 -2      articleviewer.cpp   1.90


--- kdepim/akregator/src/articleviewer.cpp  #1.89:1.90
@@ -10,5 +10,4 @@
 #include <qpaintdevicemetrics.h>
 #include <qscrollview.h>
-#include <qstylesheet.h>
 #include <qvaluelist.h>
 
@@ -200,5 +199,5 @@ QString ArticleViewer::formatArticle(Fee
         if (article.link().isValid())
             text += "<a href=\""+article.link().url()+"\">";
-        text += QStyleSheet::escape(article.title()); // TODO: better leave things escaped in the parser
+        text += article.title().replace("<", "&lt;").replace(">", "&gt;"); // TODO: better leave things escaped in the parser
         if (article.link().isValid())
             text += "</a>";