Beginning on October 25th, 2016, Persona will no longer be an option for authentication on BMO. For more details see Persona Deprecated.
Last Comment Bug 550800 - "Summary" pref is overridden in multifeed folders
: "Summary" pref is overridden in multifeed folders
Product: MailNews Core
Classification: Components
Component: Feed Reader (show other bugs)
: Trunk
: All All
: -- minor with 1 vote (vote)
: Thunderbird 13.0
Assigned To: alta88
: 513652 517477 (view as bug list)
Depends on:
  Show dependency treegraph
Reported: 2010-03-07 14:37 PST by Mike Cowperthwaite
Modified: 2012-03-06 13:28 PST (History)
5 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

patch. (8.68 KB, patch)
2012-03-01 21:24 PST, alta88
no flags Details | Diff | Splinter Review
patch. (20.73 KB, patch)
2012-03-04 07:48 PST, alta88
mozilla: review+
Details | Diff | Splinter Review
patch for checkin. (20.74 KB, patch)
2012-03-06 06:56 PST, alta88
alta88: review+
Details | Diff | Splinter Review

Description Mike Cowperthwaite 2010-03-07 14:37:50 PST
Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:
 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.
Comment 1 Bernd Heinze 2011-06-15 11:10:57 PDT
Still valid with Seamonkey 2.1 (and before)
Comment 2 alta88 2012-02-18 14:09:09 PST
*** Bug 517477 has been marked as a duplicate of this bug. ***
Comment 3 alta88 2012-02-18 14:13:09 PST
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.
Comment 4 alta88 2012-02-18 14:21:13 PST
*** Bug 513652 has been marked as a duplicate of this bug. ***
Comment 5 alta88 2012-03-01 21:24:28 PST
Created attachment 602256 [details] [diff] [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.
Comment 6 alta88 2012-03-04 07:48:23 PST
Created attachment 602728 [details] [diff] [review]

updated patch with better keyboard handling and ensuring quickmode is correctly set on dnd operations.
Comment 7 David :Bienvenu 2012-03-05 15:55:56 PST
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.
Comment 8 alta88 2012-03-05 16:39:20 PST
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 9 David :Bienvenu 2012-03-05 17:42:21 PST
Comment on attachment 602728 [details] [diff] [review]

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)
Comment 10 alta88 2012-03-06 06:56:38 PST
Created attachment 603253 [details] [diff] [review]
patch for checkin.

updated per comments.
Comment 11 Mark Banner (:standard8) 2012-03-06 13:28:01 PST
Checked in:

Note You need to log in before you can comment on or make changes to this bug.