Closed Bug 550800 Opened 11 years ago Closed 9 years ago

"Summary" pref is overridden in multifeed folders

Categories

(MailNews Core :: Feed Reader, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 13.0

People

(Reporter: mcow, Assigned: alta88)

References

Details

Attachments

(1 file, 2 obsolete files)

Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9.2.2pre)
 Gecko/20100307 Lanikai/3.1b2pre

If I create an RSS folder and subscribe multiple feeds to it, and ANY feed is set with the "show as summary" option turned OFF, then ALL feed items are shown as Web Page rather than Summary, regardless of the individual setting of the feed.  The Subscribe UI for each individual feed maintains its setting, but it is not possible to mix Summary with Web Page items.

Maybe not such a big deal.  Maybe the UI should be changed to present the option as a folder option rather than a feed option.
Still valid with Seamonkey 2.1 (and before)
Duplicate of this bug: 517477
yes, mixing multifeeds in the same folder will return the summary/web page (quickmode=true for summary) to the first url key encountered in the feeds db.

it would be very difficult to fix this, as it would mean tracking the feed url of a message (not done currently) in either the header or as a msgDbHdr property.  then, all messages would have to be updated if the url ever changed.  very not worth it.

once bug 716706 is checked in, i intend to fix the Subscribe dialog to make it clear that the summary pref updates on a folder level and not feed level.
Duplicate of this bug: 513652
Attached patch patch. (obsolete) — Splinter Review
1. the account default can now be updated from the Subscribe dialog.
2. summary checkbox enabled only for folders with feeds; changing the check applies to all feeds in the folder.
3. summary checkbox enabled for folders with feed URL field focused/non empty value, prior to add.
Assignee: nobody → alta88
Attachment #602256 - Flags: review?(dbienvenu)
Attached patch patch. (obsolete) — Splinter Review
updated patch with better keyboard handling and ensuring quickmode is correctly set on dnd operations.
Attachment #602256 - Attachment is obsolete: true
Attachment #602728 - Flags: review?(dbienvenu)
Attachment #602256 - Flags: review?(dbienvenu)
Not sure if this patch is the cause, but now I'm seeing this error when dropping a link to an rss feed:

Timestamp: 3/5/2012 3:47:19 PM
Warning: reference to undefined property this._rowMap[aIndex]._parent
Source File: chrome://messenger/content/folderPane.js
Line: 782

and the feed is not subscribed to. I'll try it w/o this patch.
i've seen that error, it happens the first time and is an old folderPane.js issue unrelated to any feed work.

also, there is a known issue that the open Subscribe dialog tree map is not updated (first time a dnd on folderpane is done) with the new feed.  if you close/open the dialog to rebuild the tree, it should be there.  if you remove it and repeat the dnd, then the dialog tree updates.
Comment on attachment 602728 [details] [diff] [review]
patch.

usually we say params, not parms, for readability, but I'll leave it up to you.

+   * @param  [aParms] object           - additional parms.
    * @param  [aMode] integer           - action mode (default is kSubscribeMode)
    *                                     of the add.
    * 
    * @return success boolean           - true if edit checks passed and an
    *                                     async download has been initiated.
    */
-  addFeed: function(aFeedLocation, aFolder, aParse, aMode)
+  addFeed: function(aFeedLocation, aFolder, aParse, aParms, aMode)
Attachment #602728 - Flags: review?(dbienvenu) → review+
updated per comments.
Attachment #602728 - Attachment is obsolete: true
Attachment #603253 - Flags: review+
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/ca6a6449f422
Status: NEW → RESOLVED
Closed: 9 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.