Bug 97180

Summary:	Konqueror shows link to RSS feed with wrong mime type
Product:	[Applications] akregator	Reporter:	Candid Dauth <Candid.Dauth>
Component:	akregator konqueror plugin	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	Test case which shows incorrect behavior Proposed patch

Description Candid Dauth 2005-01-16 21:21:24 UTC

Version:           3.3.91 (using KDE KDE 3.3.90)
Installed from:    SuSE RPMs

Konqueror shows a link to an RSS feed if there is a <link> of the following syntax in the document:
<link rel/rev="alternate" ...>

But, this can also be a link to another document, for example to the same website in another language. And that would produce an error in aKregator. I'd propose to check if the attribute
type="application/rss+xml"
is available.

Additionally, those alternative documents can be defined by
<link rel="alternate" ...
as well as by
<link rev="alternate" ...

The «RSS feed detector» detects an RSS feed in case of both versions, but the Document Relation Toolbar doesn't. It only shows links on alternative versions with the syntax
<link rel="alternate" ...

Comment 1 Frank Osterfeld 2005-01-17 08:40:23 UTC

Actually, the code should check if the "type" attribute is either "application/rss+xml", "application/rdf+xml" or "application/atom+xml".
Could you give an example of a site where this happens?

Comment 2 Candid Dauth 2005-01-17 15:15:17 UTC

For example
http://www.phoenix-world.com/Candid/3.0.0_alpha/

I don't know if you already fixed this bug in a newer version -- my aKregator version is 1.0-beta8 under KDE 3.3.91.

Comment 3 Eckhart Wörner 2005-04-25 00:22:01 UTC

Created attachment 10783 [details]
Test case which shows incorrect behavior

Comment 4 Eckhart Wörner 2005-06-18 00:44:16 UTC

Created attachment 11496 [details]
Proposed patch

First patch ever, so be careful. ;-)

Comment 5 Frank Osterfeld 2005-06-18 14:19:18 UTC

Closed in CVS