Bug 145185

Summary:	Feed plugin does not understand work with multiple rel values
Product:	[Applications] akregator	Reporter:	Daniel Aleksandersen <aleksandersen+bugs.kde.org>
Component:	akregator konqueror plugin	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	lemma
Priority:	NOR	Keywords:	testcase, triaged
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	testcase

Description Daniel Aleksandersen 2007-05-08 14:24:19 UTC

Version:           3.5.6 (using KDE 3.5.6, Kubuntu (feisty) 4:3.5.6-0ubuntu14)
Compiler:          Target: x86_64-linux-gnu
OS:                Linux (x86_64) release 2.6.20-15-generic

Konqueror's feed plugin does not respect that rel allows for multiple, space-separated values. This means the correct link tag below does not work with the plug-in:

<link rel="alternate feed" type="application/atom+xml" href="/atom.xml" />

Removing the feed rel fixes the problem, but it is not really a solution. As the rel could say "alternate feed start" etc-

Note: The "feed" rel value is a part of the Web Applications 1.0 working draft.

Comment 1 Daniel Aleksandersen 2007-05-08 14:29:23 UTC

W3C's HTML 4 and XHTML recommendations (as well as WHATWG's Web Application 1.0 working draft):

rel = link-types [case insensitive]
This attribute describes the relationship from the current document to the anchor specified by the href attribute. The value of this attribute is a space-separated list of link types.

Comment 2 Michael Leupold 2008-10-12 17:29:36 UTC

Interestingly on trunk r870274 the akregator addin detects that there is a feed but has trouble adding it (nothing appears in akregator). It works if I change rel to "alternate" only.

I'll add my testcase.

Comment 3 Michael Leupold 2008-10-12 17:30:41 UTC

Created attachment 27832 [details]
testcase

Add the testcase to quickly recheck the bug.

Comment 4 Frank Osterfeld 2009-01-26 00:47:07 UTC

SVN commit 916775 by osterfeld:

match any link rel attribute which contains "alternate" or "feed"

BUG:145185


 M  +1 -1      feeddetector.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=916775