Closed Bug 353128 Opened 19 years ago Closed 18 years ago

nsINavBookmarkObserver doesn't observe Item Moves

Categories

(Firefox :: Bookmarks & History, defect)

defect
Not set
minor

Tracking

()

RESOLVED DUPLICATE of bug 371823

People

(Reporter: thomas.zoechling, Unassigned)

Details

User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a1) Gecko/20060914 Minefield/3.0a1 Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a1) Gecko/20060914 Minefield/3.0a1 The testfile (http://lxr.mozilla.org/mozilla/source/browser/components/places/tests/testbookmarks.js) contains an implementation of nsINavBookmarkObserver which allows to listen for onItemMoved events. But it seems this event never fires. Instead I get itemRemoved and itemAdded. Wouldn't it be better to batch them into one move-transaction as the test case suggests? Reproducible: Always Steps to Reproduce: 1. Implement a simple nsINavBookmarkObserver (like the one in http://lxr.mozilla.org/mozilla/source/browser/components/places/tests/testbookmarks.js) 2. attach it 3. move some bookmarks around Actual Results: The move actions trigger "remove" and "add" events instead of one "move". Expected Results: "remove" and "add" encapsulated in one "move" transaction
this bug has been replaced by bug 371823.
Status: UNCONFIRMED → RESOLVED
Closed: 18 years ago
Resolution: --- → DUPLICATE
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
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.