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 762824 - Move Bookmark stopped working
: Move Bookmark stopped working
: regression
Product: SeaMonkey
Classification: Client Software
Component: Bookmarks & History (show other bugs)
: Trunk
: All All
: -- major (vote)
: seamonkey2.13
Assigned To: Jens Hatlak (:InvisibleSmiley)
Depends on: 732027
  Show dependency treegraph
Reported: 2012-06-08 02:05 PDT by Jens Hatlak (:InvisibleSmiley)
Modified: 2012-07-05 12:32 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

patch [Checkin: Comment 7] (1.27 KB, patch)
2012-06-08 03:03 PDT, Jens Hatlak (:InvisibleSmiley)
neil: review+
philip.chee: approval‑comm‑aurora+
philip.chee: approval‑comm‑beta+
philip.chee: approval‑comm‑release+
Details | Diff | Splinter Review

Description Jens Hatlak (:InvisibleSmiley) 2012-06-08 02:05:12 PDT
Move Bookmark (from the Bookmarks Manager Tools menu) stopped working some time ago already but no-one noticed (well, I did, but thought it was due to one of the many add-ons I have installed). The effect is that clicking the OK button does nothing. Nothing on the Error Console either when that happens.

Unfortunately this bug made it into our latest release, 2.10 (which notably does not contain the fix for bug 730837 yet). Last known good is 2.9.1.

1. Open Bookmarks Manager
2. Select some bookmark
3. Open Tools/Move...
4. Select some folder (other than the one the selected bookmark resides in)
5. Click OK

Expected result: Dialog closes, bookmark is moved.

Actual result: Nothing (dialog still open, bookmark not moved).

I don't know by heart how the Places stuff is connected; maybe this is fallout from some of the reorganizations we did.
Comment 1 Jens Hatlak (:InvisibleSmiley) 2012-06-08 02:13:15 PDT
Actually it's the Edit menu.

FWIW: Works with latest FF nightly.
Comment 2 Jens Hatlak (:InvisibleSmiley) 2012-06-08 02:20:18 PDT
It looks to me like the following line was ported incorrectly in bug 732027:

let txn = PlacesMoveItemTransaction(this._nodes[i].itemId,

should be

let txn = new PlacesMoveItemTransaction(this._nodes[i].itemId,


Will try a local patch now.
Comment 3 Jens Hatlak (:InvisibleSmiley) 2012-06-08 03:03:56 PDT
Created attachment 631325 [details] [diff] [review]
patch [Checkin: Comment 7]

[Approval Request Comment]
Regression caused by (bug #): bug 732027
User impact if declined: Move Bookmarks broken
Testing completed (on m-c, etc.): confirmed locally
Risk to taking this patch (and alternatives if risky): minimal
String changes made by this patch: none
Comment 4 Frank Wein [:mcsmurf] 2012-06-08 03:10:23 PDT
I'll never ever touch bookmarks code again.

btw: Moving via drag&drop works (which is what I use all the time, I did not even know we have a move dialog).
Comment 5 Philip Chee 2012-06-08 03:39:07 PDT
> I'll never ever touch bookmarks code again.
Never is a long time. How about next Tuesday?
Comment 6 Philip Chee 2012-06-08 06:06:45 PDT
Comment on attachment 631325 [details] [diff] [review]
patch [Checkin: Comment 7]

Comment 7 Jens Hatlak (:InvisibleSmiley) 2012-06-08 06:52:11 PDT
Comment on attachment 631325 [details] [diff] [review]
patch [Checkin: Comment 7]

(In reply to Frank Wein [:mcsmurf] from comment #4)
> I'll never ever touch bookmarks code again.

Heh. Understandable feeling, but no-one's blaming you really. I guess we've all learned our lesson from this "landing untested code changes on branches" debacle. At least we have a bug to point at for reference now. ;-)
Comment 8 Jens Hatlak (:InvisibleSmiley) 2012-06-08 06:56:52 PDT
Flags/releases explanation: This bug is fixed in
* the next SM trunk nightly 
* the next SM 2.12a2 (Aurora) nightly
* SM 2.11b2 (2.11b1 has already been tagged)
* 2.10.1 iff it will ever be released and will include this fix.

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