Two feed parser fixes; prevent duplicate atom enlosures and feedburner:origLink

RESOLVED FIXED in Thunderbird 29.0

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: alta88, Assigned: alta88)

Tracking

unspecified
Thunderbird 29.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
1. Prevent creating duplicate attachments for dupe enclosure urls in atom feeds, like we do for rss2 already.

2. If found, use <feedburner:origLink> as the link.
(Assignee)

Comment 1

5 years ago
Created attachment 8355011 [details] [diff] [review]
origLinkAndDupeEncs.patch
Assignee: nobody → alta88
Attachment #8355011 - Flags: review?(mkmelin+mozilla)
Comment on attachment 8355011 [details] [diff] [review]
origLinkAndDupeEncs.patch

Review of attachment 8355011 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me! r=mkmelin
Attachment #8355011 - Flags: review?(mkmelin+mozilla) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/15eac4288d80
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 29.0
(Assignee)

Comment 4

5 years ago
Created attachment 8397465 [details] [diff] [review]
patch-42.diff

typo bustage fix.
Attachment #8397465 - Flags: review?(mkmelin+mozilla)
Attachment #8397465 - Flags: review?(mkmelin+mozilla) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
Please spin this fix off to a new bug. We shouldn't be landing additional fixes in the same bug for something that landed 2 months ago. Makes tracking a PITA.
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.