252688 - Move feeds.rdf and feeditems.rdf into the rss account directory

Status: RESOLVED FIXED

(MailNews Core :: Feed Reader, defect)

Description

[Scott MacGregor]

Each RSS account has its own feed and items data source.

This will also make it easy to generically remove the RSS account from the
account manager without leaving stale data source data behind. By being in the
account directory, when the account is removed, so is the RSS data for that account.

Comment 1

[Scott MacGregor]

Attachment: the fix

Comment 2

[Scott MacGregor]

Comment on attachment 154036 [details] [diff] [review]
the fix

David, this patch adds two methods to nsIRssIncomingServer for getting the file
path for feeds.rdf and feedItems.rdf for an rss server. 

I also had to make a change to nsLocalMailFolder.cpp to make it ignore
"feeds.rdf" and "feeditems.rdf" when figuring out which files are folders. Is
there a better place to do that besides taining nsLocalMailFolder? I don't
think there is.

One more thing..are we happy with the names 'feeds.rdf' and 'feeditems.rdf'. If
we want to change them, now's the time.

Comment 3

[David :Bienvenu]

Comment on attachment 154036 [details] [diff] [review]
the fix



Can you just add a helper function that both GetFeedItemsDataSourcePath and
GetSubscriptionsDataSourcePath	call, which just appends the passed in string
to the file name, i.e., feeds.rdf or feeditems.rdf? sr=bienvenu with that
change.