Author: adietish Date: 2012-02-15 06:01:10 -0500 (Wed, 15 Feb 2012) New Revision: 38747 Modified: trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/ApplicationConfigurationWizardPage.java Log: [JBIDE-10907] resetting application settings widgets (name, selected cartridge, embedded cartridges) only if the user unchecks *use existing application* (was: resetting also when page gets active) Modified: trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/ApplicationConfigurationWizardPage.java =================================================================== --- trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/ApplicationConfigurationWizardPage.java 2012-02-15 10:58:23 UTC (rev 38746) +++ trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/ApplicationConfigurationWizardPage.java 2012-02-15 11:01:10 UTC (rev 38747) @@ -330,6 +330,9 @@ Object newValue = event.diff.getNewValue(); if (newValue instanceof Boolean) { Boolean useExisting = (Boolean) newValue; + if (!useExisting) { + pageModel.resetExistingApplication(); + } enableApplicationWidgets( useExisting, applicationConfigurationGroup, applicationNameText, applicationBrowseButton); } @@ -348,9 +351,6 @@ */ private void enableApplicationWidgets(final Boolean useExisting, final Group applicationConfigurationGroup, final Text applicationNameText, final Button applicationBrowseButton) { - if (!useExisting) { - pageModel.resetExistingApplication(); - } applicationNameText.setEnabled(useExisting); applicationBrowseButton.setEnabled(useExisting); UIUtils.doForAllChildren(new UIUtils.IWidgetVisitor() {