This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 78698 - getActions() is called with Root if non-Name column is used
Summary: getActions() is called with Root if non-Name column is used
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Outline&TreeTable (show other bugs)
Version: 6.x
Hardware: Sun All
: P2 blocker (vote)
Assignee: Stanislav Aubrecht
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-06-21 00:25 UTC by ivan
Modified: 2008-12-22 16:48 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description ivan 2006-06-21 00:25:03 UTC
If you select a breakpoint row in a column other than the main "Name" column
and in a row other than the selected row, sometimes
NodeActionsProvider.getActions() is called once with Object param being the root
object as opposed to the breakpoint on that row. 
The UI continues on and shows the new row as the selected row which is also
inconsistent with getting a root menu action.

It seems to be easier to reproduce if you put a bpt on getActions.

Because this is intermittent it must be a race condition where the 
mapping of selected TTV row to debugger happens before the row switch
has actually completed.
Also becaus eit seems like a race the pre-conditions for reproducibility
I mention should not be taken literally.

There's also a related problem where if the mouse is clicked outside of any row,
in the empty area below all rows, some definite object is delivered to 
getActions instead of ROOT.
Comment 1 Martin Entlicher 2006-06-21 11:26:48 UTC
Right, it's easily reproducible.
During debugging I've found that the action gets a wrong node from the tree
table view framework.
Specifically org.openide.explorer.view.TreeView.createExtendedPopup() gets the
selected nodes and asks for the actions on them. But the array of the selected
nodes does not correspond to the node where the mouse is and which is painted as
selected.
Therefore I'm moving this to openide.
Comment 2 ivan 2006-10-13 22:03:16 UTC
The unpredictability of bpt context menu is rather visible.
Would like this to be addressed for SSX3.
Comment 3 Stanislav Aubrecht 2006-10-18 16:37:42 UTC
fixed in trunk

/cvs/openide/explorer/src/org/openide/explorer/view/TreeTableView.java,v  <-- 
TreeTableView.java
new revision: 1.12; previous revision: 1.11
Comment 4 Thomas Preisler 2006-10-25 17:59:16 UTC
Changing prioroty to p2 making it a candidate for backporting to release55_mars.
Comment 5 Stanislav Aubrecht 2006-11-02 15:58:58 UTC
fixed in release55_mars branch

Checking in TreeTableView.java;
/cvs/openide/explorer/src/org/openide/explorer/view/TreeTableView.java,v  <-- 
TreeTableView.java
new revision: 1.2.4.1.2.1.18.1; previous revision: 1.2.4.1.2.1
Comment 6 Stanislav Aubrecht 2006-11-20 12:07:52 UTC
fixed in release551 branch

Checking in TreeTableView.java;
/cvs/openide/explorer/src/org/openide/explorer/view/TreeTableView.java,v  <-- 
TreeTableView.java
new revision: 1.2.4.1.2.1.22.1; previous revision: 1.2.4.1.2.1
done