news added twice and cannot be removed

RESOLVED FIXED in Thunderbird 13.0

Status

MailNews Core
Feed Reader
RESOLVED FIXED
8 years ago
6 years ago

People

(Reporter: Christian R, Assigned: alta88)

Tracking

unspecified
Thunderbird 13.0
x86_64
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2) Gecko/20100214 Gentoo Firefox/3.6
Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.7) Gecko/20100214 Thunderbird/3.0.1

This rss http://support.asus.de/rss/RssFeeds.aspx?product=1&model=M4A79XTD%20EVO&SLanguage=en-us will be added twice, so I have two folders "ASUSTek RSS [Motherboard] [M4A79XTD EVO] [en-us]82439776" with the same name.
I'm not able to delete both at all.
It will be back after restart if I remove one and rename the other and then remove it as well.
Sometimes I even cannot rename the folder.
So I hope you're able to reproduce it.

Reproducible: Always

Steps to Reproduce:
1. Add http://support.asus.de/rss/RssFeeds.aspx?product=1&model=M4A79XTD%20EVO&SLanguage=en-us as news feed
2. See two new news folders with the same name (if not restart tb)
3. Try to delete one or even both
Actual Results:  
It adds folders with the same name twice.
You're not able to delete one or even both of them.

Expected Results:  
Only one folder should be added.
I should be able to remove it as well.
I can't reproduce the dual folder issue with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2.2pre) Gecko/20100221 Lanikai/3.1b1pre.

Have you tried safe mode? (see https://support.mozillamessaging.com/en-US/kb/Safe+Mode for more information)
It does appear twice after a restart of Thunderbird.
Component: General → Feed Reader
Product: Thunderbird → MailNews Core
QA Contact: general → feed.reader
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Reporter)

Comment 3

8 years ago
Even in safe-mode I'm only able to delete both feeds but only one of these folders.
When trying to remove the second folder I just get "A folder with that name already exists. Please enter a different name.".. but even that doesn't work as expected.
My could you look into this one ?
(In reply to comment #4)
> My could you look into this one ?

Myk would you have bandwidth to look into this one ?
I don't have too much bandwidth, but I took a look this evening, and I found that subscribing to the feed creates three files in the Mail/News & Blogs/ subdirectory of the profile directory:

  ASUSTek RSS [Motherboard] [M4A79XTD EVO] [en-us]82439776
  ASUSTek RSS [Motherboard] [M4A79XTD EVO] [en-usd77a79f2
  ASUSTek RSS [Motherboard] [M4A79XTD EVO] [en-usd77a79f2.msf

The feeds.rdf file specifies the fz:destFolder for the feed as:

  <fz:destFolder RDF:resource="mailbox://nobody@News%20%26%20Blogs/ASUSTek%20RSS%20%5BMotherboard%5D%20%5BM4A79XTD%20EVO%5D%20%5Ben-us%5D82439776"/>

However, the file ending in "d77a79f2" is the one to which feed messages have been added (and the one being indexed via a Mork database).  I suspect that some code creates the file ending in "82439776" when one subscribes to the feed, then some other code creates "d77a79f2" when it downloads feed messages.

I haven't been able to figure out where those respective code chunks live, however.
Thanks for the analysis.

Comment 8

7 years ago
What produces them should be NS_MsgHashIfNecessary - http://mxr.mozilla.org/comm-central/source/mailnews/base/util/nsMsgUtils.cpp#99 - but i doubt that helps much.
(Assignee)

Comment 9

6 years ago
the reason is that the feed title ends in empty space.  feeditem titles that end in empty space also cause trouble if just the right length.  trim() is a friend here..
(Assignee)

Comment 10

6 years ago
Created attachment 604065 [details] [diff] [review]
patch
Assignee: nobody → alta88
Attachment #604065 - Flags: review?(dbienvenu)

Comment 11

6 years ago
Comment on attachment 604065 [details] [diff] [review]
patch

looks safe and reasonable. thx for the patch.
Attachment #604065 - Flags: review?(dbienvenu) → review+
(Assignee)

Updated

6 years ago
Duplicate of this bug: 316051
Keywords: checkin-needed
(Assignee)

Updated

6 years ago
Duplicate of this bug: 431822
http://hg.mozilla.org/comm-central/rev/4c3e576941e8
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
You need to log in before you can comment on or make changes to this bug.