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.
WizardDescriptor.Panel implementations org\netbeans\modules\web\context\BaseWizardPanel.java org\netbeans\modules\web\taglibed\view\BaseWizardPanel.java provides themself as the result of getComponent(). This hurts performance and can cause a clash when Component.isValid() is overridden. Please use a separate component class, see details at http://performance.netbeans.org/howto/dialogs/wizard-panels.html. Easy to fix, P2 justification - performance is top priority.
Fixed for new web module http://web.netbeans.org/source/browse/web/core/src/org/netbeans/modules/web/context/BaseWizardPanel.java.diff?r1=1.1&r2=1.2 http://web.netbeans.org/source/browse/web/core/src/org/netbeans/modules/web/context/NewWebModuleMountWizardPanel.java.diff?r1=1.5&r2=1.6 http://web.netbeans.org/source/browse/web/core/src/org/netbeans/modules/web/context/WebAppWizardIterator.java.diff?r1=1.6&r2=1.7 Assigned to Simran - taglib editor is his responsibility. BTW: Why does it copy so much code from other places?
Tegal planning
Scheduling for milestone 3 so it can be contributed to S1S4.2 too.
Guys, there was an error in example code in IDESettingsWizardPanel.java. Please check your fix and repair if you did the same mistake. I'm terribly sorry, but I'm only a human :-) Please read details below: --------------------------------- Modified: src/org/netbeans/core/ui IDESettingsWizardPanel.java Log: fix for incorrect assumption that getComponent must be called before readSettings. Now they can be called in any order. Revision Changes Path 1.3 +2 -2 core/src/org/netbeans/core/ui/IDESettingsWizardPanel.java http://core.netbeans.org/source/browse/core/src/org/netbeans/core/ui/IDESettingsWizardPanel.java.diff?r1=1.2&r2=1.3 (In the diff below, changes in quantity of whitespace are not shown.) Index: IDESettingsWizardPanel.java =================================================================== RCS file: /cvs/core/src/org/netbeans/core/ui/IDESettingsWizardPanel.java,v retrieving revision 1.2 retrieving revision 1.3 diff -u -b -r1.2 -r1.3 --- IDESettingsWizardPanel.java 2002/12/03 14:10:35 1.2 +++ IDESettingsWizardPanel.java 2003/01/24 13:45:30 1.3 @@ -7,7 +7,7 @@ * http://www.sun.com/ * * The Original Code is NetBeans. The Initial Developer of the Original - * Code is Sun Microsystems, Inc. Portions Copyright 1997-2001 Sun + * Code is Sun Microsystems, Inc. Portions Copyright 1997-2003 Sun * Microsystems, Inc. All Rights Reserved. */ @@ -42,7 +42,7 @@ */ public void readSettings (Object settings) { if (settings instanceof WizardDescriptor) { - panelUI.initFromSettings((WizardDescriptor)settings); + getPanelUI().initFromSettings((WizardDescriptor)settings); } }
The taglib editor no longer uses the BaseWizardPanel and NewTaglibWizardPanel classes in its wizard. Instead it uses the standard target chooser. In future releases, we may go back to a more detailed wizard for the taglib editor, and then those classes may be used again. In the meantime, they should be removed from the build.
removed the offending classes.
Verified. This is a developers discussion, 5 years old, marked as FIXED, no-one complained... (And IMO does not much belong to Issuezilla...)