In <bookmarksMenu.js>, "Error: aTarget has no properties"



Bookmarks & History
14 years ago
10 years ago


(Reporter: HJ, Unassigned)


Bug Flags:
blocking1.6b -

Firefox Tracking Flags

(Not tracked)




14 years ago
Error: aTarget has no properties
Source File: chrome://communicator/content/bookmarks/bookmarksMenu.js
Line: 652

Steps to reproduce: press Ctrl-Z

Comment 1

14 years ago
I noticed it in build Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.6b) 
Gecko/20031112 also.
Flags: blocking1.6b?

Comment 2

14 years ago
HJ: is it a blocker for multizilla?

Comment 3

14 years ago
"HJ: is it a blocker for multizilla?"

No, not a real blocker, however, this is something I run into every hour or so.

Comment 4

14 years ago
How is this bug reproduced in SeaMonkey and what is the user impact?

Comment 5

14 years ago
"How is this bug reproduced in SeaMonkey"

Just use Ctrl+Z and that error will showup on your JS console. It is 100%
reproducable with mozilla 2003111415 on WinXP and NT4.

"...and what is the user impact?"

It has the potential of killing your bookmarks drag and drop feature...

note: I *always* verify my bug reports with a vanilla build, just to be sure :-)

Comment 6

14 years ago
It is reproduced by doing this
Source File: chrome://communicator/content/bookmarks/bookmarksMenu.js
Line: 652

It causes problems with bookmarks and the usability of them.

Comment 7

14 years ago
Not a blocker unless we can get actual reproducible symptoms (other than a js
console error). 
Flags: blocking1.6b? → blocking1.6b-

Comment 8

14 years ago
With a current cvs build and a new, untouched profile, I get this error whenever
I open the bookmarks window.
It originates in
and propagates through "BookmarksController.onCommandUpdate(selection, target);"
in the same file. "target" is null there.

Comment 9

14 years ago
There is another similar error when doing drag and drop of a bookmark from the
bookmark manager to the personal toolbar with todays build and a new profile:

Error: aTarget has no properties
Source File: chrome://communicator/content/bookmarks/bookmarks.js
Line: 856

I'm not sure if they are related, but who knows...

Source is:
853     case "cmd_bm_newbookmark":
854     case "cmd_bm_newfolder":
855     case "cmd_bm_newseparator":
856       return BookmarksUtils.isValidTargetContainer(aTarget.parent);

Comment 10

14 years ago
Sorry for messing with this bug, what I saw was not this bug, but bug 232827
(just filed after I discovered the difference).

Ctrl-Z is "undo" and this bug here only appears when there's nothing undoable
and the undo menu entry is inactive. This either means the shortcut is not
correctly inactivated or Ctrl-Z is also shortcut for a different command that
always fails, but that other one is not accessed as long as "undo" is possible.

doCommand("cmd_undo") calls onDragRemoveFeedBack in
and that's where it fails.
I'm not sure why bookmarks get a "Undo" command there, but it seems like this
does not harm - undo fails where there's nothing to undo.
Summary: Error: aTarget has no properties in bookmarksMenu.js → In <bookmarksMenu.js>, "Error: aTarget has no properties"
Product: Browser → Seamonkey

Comment 11

12 years ago
Reassigning as per Bug #32644
Assignee: p_ch → nobody

Comment 12

12 years ago
That bug is invalid so I'm undoing your changes.
Assignee: nobody → p_ch

Comment 13

11 years ago
comment 11 meant bug 326443
Assignee: p_ch → nobody
QA Contact: bookmarks

Comment 14

10 years ago
duping to newer about-to-be-fixed bug
Last Resolved: 10 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 416564
You need to log in before you can comment on or make changes to this bug.