Closed Bug 762824 Opened 11 years ago Closed 11 years ago

Move Bookmark stopped working


(SeaMonkey :: Bookmarks & History, defect)

Not set


(seamonkey2.10 fixed, seamonkey2.11 fixed, seamonkey2.12 fixed)

Tracking Status
seamonkey2.10 --- fixed
seamonkey2.11 --- fixed
seamonkey2.12 --- fixed


(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)



(Keywords: regression)


(1 file)

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.
Actually it's the Edit menu.

FWIW: Works with latest FF nightly.
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.
Keywords: regression
[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
Assignee: nobody → jh
Attachment #631325 - Flags: review?(neil)
Attachment #631325 - Flags: approval-comm-release?
Attachment #631325 - Flags: approval-comm-beta?
Attachment #631325 - Flags: approval-comm-aurora?
Depends on: 732027
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).
Attachment #631325 - Flags: review?(neil) → review+
> I'll never ever touch bookmarks code again.
Never is a long time. How about next Tuesday?
Comment on attachment 631325 [details] [diff] [review]
patch [Checkin: Comment 7]

Attachment #631325 - Flags: approval-comm-release?
Attachment #631325 - Flags: approval-comm-release+
Attachment #631325 - Flags: approval-comm-beta?
Attachment #631325 - Flags: approval-comm-beta+
Attachment #631325 - Flags: approval-comm-aurora?
Attachment #631325 - Flags: approval-comm-aurora+
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. ;-)
Attachment #631325 - Attachment description: patch → patch [Checkin: Comment 7]
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.
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.13
Keywords: relnote
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.