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)
MailNews Core
Feed Reader
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 29.0
People
(Reporter: alta88, Assigned: alta88)
Details
Attachments
(2 files)
6.68 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
1.22 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
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 2•10 years ago
|
||
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
Comment 3•10 years ago
|
||
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
typo bustage fix.
Attachment #8397465 -
Flags: review?(mkmelin+mozilla)
Updated•10 years ago
|
Attachment #8397465 -
Flags: review?(mkmelin+mozilla) → review+
Keywords: checkin-needed
Comment 5•10 years ago
|
||
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.
Description
•