Bug 122409 - akregator does not show a link to articles in Atom feeds
Summary: akregator does not show a link to articles in Atom feeds
Status: RESOLVED FIXED
Alias: None
Product: akregator
Classification: Applications
Component: general (show other bugs)
Version: 1.2.1
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-21 10:35 UTC by Rolf Eike Beer
Modified: 2006-02-21 15:10 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Set "rel" attribute default to "alternate" (757 bytes, patch)
2006-02-21 14:50 UTC, Eckhart Wörner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Rolf Eike Beer 2006-02-21 10:35:38 UTC
Version:           1.2.1 (using KDE 3.5.1 Level "a" , SUSE 9.3 UNSUPPORTED)
Compiler:          gcc version 3.3.5 20050117 (prerelease) (SUSE Linux)
OS:                Linux (i686) release 2.6.11.4-21.10-smp

In RDF feeds the title of the article is used as link to the complete article. Also a "complete text" link is shown at the bottom of the article few. If it is a atom feed neither of them is there so Atom feeds are more or less useless.

Compare this two feeds:

http://www.heise.de/newsticker/heise.rdf
http://www.heise.de/newsticker/heise-atom.xml
Comment 1 Eckhart Wörner 2006-02-21 14:50:21 UTC
Created attachment 14797 [details]
Set "rel" attribute default to "alternate"
Comment 2 Eckhart Wörner 2006-02-21 15:10:52 UTC
SVN commit 511997 by ewoerner:

Set "rel" attribute default to "alternate", as requested by Atom specification
BUG:122409

 M  +1 -1      article.cpp  


--- branches/KDE/3.5/kdepim/akregator/src/librss/article.cpp #511996:511997
@@ -77,7 +77,7 @@
 		for (n = node.firstChild(); !n.isNull(); n = n.nextSibling()) {
 			const QDomElement e = n.toElement();
 			if ( (e.tagName()==QString::fromLatin1("link")) &&
-				(e.attribute(QString::fromLatin1("rel"))==QString::fromLatin1("alternate")))
+				(e.attribute(QString::fromLatin1("rel"), QString::fromLatin1("alternate"))==QString::fromLatin1("alternate")))
 				{   
 					d->link=n.toElement().attribute(QString::fromLatin1("href"));
 					break;