454656 - reference to undefined property menupopup.childNodes[i].folderId in editBookmarkOverlay.js

Status: RESOLVED WORKSFORME

(Firefox :: Bookmarks & History, defect)

Description

[(mostly gone) XtC4UaLL [:xtc4uall]]

as requested in Bug 418671 filing this one.

STR:
- open up the library *or*
- click on a parent folder (all/tags/history/menu/toolbar/unsorted) in left pane in library *or*
- every time i click in the right pane on a children item of a given parent folder

error console output (in strict mode):
Warning: reference to undefined property menupopup.childNodes[i].folderId
Source file: chrome://browser/content/places/editBookmarkOverlay.js
Line: 667

Comment 1

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

_initPanel calls initFolderMenuList(aSelectedFolder)
where aSelectedFolder is 1, so the basic root, we can't clearly select that so when we call
 var defaultItem = this._getFolderMenuItem(aSelectedFolder);
we try to read folderId for the root.

We should init the panel to a better value

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Same with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.0.3pre) Gecko/2008090704 GranParadiso/3.0.3pre ID:2008090704

Comment 3

[Gervase Markham [:gerv]]

Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv

Comment 4

[Mark Banner (:standard8)]

This no longer happens, I guess it went away ages ago.