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 24778 - VcsMountFromTemplateAction creates the TemplateWizard too early.
Summary: VcsMountFromTemplateAction creates the TemplateWizard too early.
Status: CLOSED FIXED
Alias: None
Product: obsolete
Classification: Unclassified
Component: vcscore (show other bugs)
Version: 3.x
Hardware: Sun Solaris
: P3 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords: PERFORMANCE
Depends on:
Blocks: 24329
  Show dependency tree
 
Reported: 2002-06-14 15:02 UTC by Petr Nejedly
Modified: 2003-07-01 12:56 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
dumpStack() in NotifyDescriptor's constructor. (4.63 KB, text/plain)
2002-06-14 15:04 UTC, Petr Nejedly
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Petr Nejedly 2002-06-14 15:02:23 UTC
If I invoke Versioning menu for the first time,
VMFTAction will create an instance of
WizardDescriptor, which is IMHO quite early
and it also slows down the first menu invocation.
I'll attach the stacktrace of the construction.
See also issue 24777
Comment 1 Petr Nejedly 2002-06-14 15:04:17 UTC
Created attachment 6263 [details]
dumpStack() in NotifyDescriptor's constructor.
Comment 2 Petr Nejedly 2002-06-20 12:28:37 UTC
You can use the same method I used in NewTamplateAction, that is:
in getMenuPresenter(), pass null as node to the menu constructor
and in the menu impl (you have named MountMenu, I used annonymous
inner class) override getPopupMenu to set the real root in advance.
This is probably the only possible solution for now given the shape
of the implementation of the wizard.

null node parameter is not documented in MenuView.Menu but the impl
is ok and will work correctly as long as you'll pass setName=false
and update the protected node property on time (from getPopupMenu())
Comment 3 Martin Entlicher 2002-06-20 12:50:53 UTC
Fixed in the main trunk. It will appear in dev build on Jun 21.
The fix is similar to the fix of issue #24777 exactly as you
suggested. Thanks.

Comment 4 Petr Nejedly 2002-06-20 13:16:12 UTC
Thanks
Comment 5 Quality Engineering 2003-07-01 12:56:10 UTC
Resolved for 3.4.x or earlier, no new info since then -> closing.