Closed Bug 955869 Opened 10 years ago Closed 10 years ago

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

Categories

(MailNews Core :: Feed Reader, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 29.0

People

(Reporter: alta88, Assigned: alta88)

Details

Attachments

(2 files)

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: 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+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/15eac4288d80
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 29.0
Attached patch patch-42.diffSplinter Review
typo bustage fix.
Attachment #8397465 - Flags: review?(mkmelin+mozilla)
Attachment #8397465 - Flags: review?(mkmelin+mozilla) → review+
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.