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.
In build 200302200100 it started to happen that templates submenu appears under Mount menu item invoked on the Filesystems node (see att.). It happens only in automated tests.
Created attachment 9052 [details] Screen shot.
Mount action: org.netbeans.core.actions.MountAction. Assigning to pzavadsky.
Jiri, it is caused by your last commit into ExplorerPanel (1.42). We have investigated it together with Jiri S.
Created attachment 9065 [details] a proposed patch
Changes in ExplorerPanel also caused that CVS menu item is not displayed when we open popup menu on a file in CVS filesystem.
The code in patch moves the DelaySetter from ExplorerPanel direct to TopComponent. TC.getActivatedNodes can return the acivated node at the moment (w/o delay).
Created attachment 9066 [details] a patch wich fires too
The better_patch.diff fires the changes of activated nodes, fixes the problem with cvs menu too.
Created attachment 9071 [details] same as better_patch but uses synchronized
Please, please Jirka, can you write a test for your behaviour? This is very frigile behaviour and I doubt without describing exactly what we want from it, we can implement and maintain it correctly. Anyway I see some problems: 1. there is unsynchronized access to DelayedSetter.nodes 2. I believe that the system will not work correctly with TC.getLookup() - if I call setActivatedNodes & getLookup().lookup (Node.class) I will not get right result. Pretty pretty please with suggar on the top, write a test or we will never stabilize this code.
Created attachment 9119 [details] patch jar to assure in GUI test
Jirko, pls. assure if the attached jar patch (add to lib/patches) fixes this defect in your test and add a comment here. Thanks.
Yes, it works for all known test cases. BUT it makes IDE slower and slower (see att. thread dump). With previuos patch IDE speed was normal.
Created attachment 9120 [details] Thread dump at dumb time.
the delay was temorary removed, the GUI tests should pass User: jrechtacek Date: 03/02/25 04:03:09 Modified: src/org/openide/explorer ExplorerPanel.java Log: hotfix #31244, delay of the activated nodes was temporary removed. For testing use switch: -J-Dnetbeans.delay.tc=true (a patch develops on branch issue_31244) Revision Path 1.43 openide/src/org/openide/explorer/ExplorerPanel.java
Verified in build 20030225-1214.