[jbosstools-commits] JBoss Tools SVN: r41056 - trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard.
Author: adietish
Date: 2012-05-16 07:09:50 -0400 (Wed, 16 May 2012)
New Revision: 41056
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/EmbedCartridgeWizardPage.java
Log:
[JBIDE-11884] added comments
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-05-16
11:01:20 UTC (rev 41055)
+++
trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/ApplicationConfigurationWizardPage.java 2012-05-16
11:09:50 UTC (rev 41056)
@@ -337,7 +337,9 @@
ViewerProperties.checkedElements(IEmbeddableCartridge.class).observe(viewer),
BeanProperties.set(
ApplicationConfigurationWizardPageModel.PROPERTY_SELECTED_EMBEDDABLE_CARTRIDGES)
- .observe(pageModel));;
+ .observe(pageModel));
+ // strategy has to be attached after the binding, so that the binding
+ // can still add the checked cartridge and the strategy can correct
viewer.addCheckStateListener(new EmbedCartridgeStrategyAdapter(pageModel, this));
this.checkAllButton = new Button(newAppEmbeddableCartridgesGroup, SWT.PUSH);
Modified:
trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/EmbedCartridgeWizardPage.java
===================================================================
---
trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/EmbedCartridgeWizardPage.java 2012-05-16
11:01:20 UTC (rev 41055)
+++
trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/EmbedCartridgeWizardPage.java 2012-05-16
11:09:50 UTC (rev 41056)
@@ -83,6 +83,8 @@
BeanProperties.set(
EmbedCartridgeWizardPageModel.PROPERTY_SELECTED_EMBEDDABLE_CARTRIDGES)
.observe(pageModel));;
+ // strategy has to be attached after the binding, so that the binding
+ // can still add the checked cartridge and the strategy can correct
viewer.addCheckStateListener(new EmbedCartridgeStrategyAdapter(pageModel, this));
// hiding buttons for now: https://issues.jboss.org/browse/JBIDE-10399