Author: adietish Date: 2010-12-16 09:20:35 -0500 (Thu, 16 Dec 2010) New Revision: 27548 Modified: trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/internal/deltacloud/ui/wizards/NewInstanceModel.java trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/internal/deltacloud/ui/wizards/NewInstancePage.java Log: [JBIDE-7918] reimplemented the whole launch Instance page with databinding since many cases did not work properly Modified: trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/internal/deltacloud/ui/wizards/NewInstanceModel.java =================================================================== --- trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/internal/deltacloud/ui/wizards/NewInstanceModel.java 2010-12-16 13:49:09 UTC (rev 27547) +++ trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/internal/deltacloud/ui/wizards/NewInstanceModel.java 2010-12-16 14:20:35 UTC (rev 27548) @@ -213,8 +213,9 @@ new AbstractCloudElementJob("Get Profiles", cloud, CLOUDELEMENT.PROFILES) { protected IStatus doRun(IProgressMonitor monitor) throws Exception { try { - setAllProfiles(Arrays.asList(cloud.getProfiles())); - setFilteredProfiles(filterProfiles(image, allProfiles)); + List<DeltaCloudHardwareProfile> profiles = Arrays.asList(cloud.getProfiles()); + setAllProfiles(profiles); + setFilteredProfiles(filterProfiles(image, profiles)); setSelectedProfileIndex(0); return Status.OK_STATUS; } catch (DeltaCloudException e) { Modified: trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/internal/deltacloud/ui/wizards/NewInstancePage.java =================================================================== --- trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/internal/deltacloud/ui/wizards/NewInstancePage.java 2010-12-16 13:49:09 UTC (rev 27547) +++ trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/internal/deltacloud/ui/wizards/NewInstancePage.java 2010-12-16 14:20:35 UTC (rev 27548) @@ -217,10 +217,20 @@ private void bindWidgets(DataBindingContext dbc, Composite container) { + // name bindText(nameText, NewInstanceModel.PROPERTY_NAME, WizardMessages.getString(MUST_ENTER_A_NAME), dbc); + // image IObservableValue imageObservable = bindImage(imageText, dbc); + // arch label + bindArchLabel(imageObservable, dbc); + bindRealmCombo(realmCombo, dbc); + bindProfileCombo(hardwareCombo, dbc); + bindProfilePages(hardwareCombo, profilePages, dbc); + // key + bindText(keyText, NewInstanceModel.PROPERTY_KEYNAME, WizardMessages.getString(MUST_ENTER_A_KEYNAME), dbc); + } - // arch label + private void bindArchLabel(IObservableValue imageObservable, DataBindingContext dbc) { dbc.bindValue(WidgetProperties.text().observe(arch), imageObservable, new UpdateValueStrategy(UpdateSetStrategy.POLICY_NEVER), @@ -237,13 +247,6 @@ } })); - - bindRealmCombo(realmCombo, dbc); - bindProfileCombo(hardwareCombo, dbc); - bindProfilePages(hardwareCombo, profilePages, dbc); - - // key - bindText(keyText, NewInstanceModel.PROPERTY_KEYNAME, WizardMessages.getString(MUST_ENTER_A_KEYNAME), dbc); } private void bindRealmCombo(final Combo realmCombo, DataBindingContext dbc) { @@ -294,6 +297,7 @@ } )); + // realm combo enablement IObservableList realmsObservable = BeanProperties.list(NewInstanceModel.PROPERTY_REALMS).observe(model); DataBindingUtils.addChangeListener(new IChangeListener() {