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.
Summary: | "Run in background" does not vanish Import modules" dialog | ||
---|---|---|---|
Product: | platform | Reporter: | matsa <matsa> |
Component: | Plugin Importer | Assignee: | Jiri Rechtacek <jrechtacek> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | ||
Priority: | P3 | ||
Version: | 6.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | |||
Bug Blocks: | 181210 |
Description
matsa
2009-07-06 08:31:51 UTC
Yes, that is incorrect. The issue is hidden in the following code of ImportManager.doImport(): wizardFinished = PluginManager.openInstallWizard (oc); if (wizardFinished) { toInstall.clear (); checkedToInstall.clear (); } else { res = false; } In case the import wizard is send to background, the wizardFinished variable becames false and so nothing happens. It is so because InstallUnitWizard.implInvokeWizard() checks the cancel status by the following line: boolean cancelled = wizardDescriptor.getValue() != WizardDescriptor.FINISH_OPTION; In case wizard went to background wizardDescriptor.getValue() is actually WizardDescriptor.NEXT_OPTION. Theoretically we should change the 'cancelled' status check to boolean cancelled = wizardDescriptor.getValue() == WizardDescriptor.CANCEL_OPTION || wizardDescriptor.getValue() == WizardDescriptor.CLOSED_OPTION; but should be done only after thorough testing for regressions. fixed in NetBeans 7.4 Beta |