Author: adietish Date: 2012-01-31 05:12:26 -0500 (Tue, 31 Jan 2012) New Revision: 38315 Modified: trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/GitCloningSettingsWizardPage.java Log: minor cleaning Modified: trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/GitCloningSettingsWizardPage.java =================================================================== --- trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/GitCloningSettingsWizardPage.java 2012-01-31 10:06:19 UTC (rev 38314) +++ trunk/openshift/plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/ui/wizard/GitCloningSettingsWizardPage.java 2012-01-31 10:12:26 UTC (rev 38315) @@ -39,7 +39,7 @@ import org.jboss.tools.common.ui.ssh.SshPrivateKeysPreferences; /** - * @author Andr� Dietisheim + * @author Andre Dietisheim * @author Rob Stryker * @author Xavier Coulon * @@ -165,16 +165,7 @@ ValueBindingBuilder.bind(WidgetProperties.enabled().observe(remoteNameText)) .notUpdating(useDefaultRemoteNameModelObservable).converting(new InvertingBooleanConverter()).in(dbc); // move focus to the project name text control when choosing the 'Use an existing project' option. - useDefaultRemoteNameButton.addSelectionListener(new SelectionListener() { - @Override - public void widgetSelected(SelectionEvent e) { - remoteNameText.setFocus(); - } - - @Override - public void widgetDefaultSelected(SelectionEvent e) { - } - }); + useDefaultRemoteNameButton.addSelectionListener(onDefaultRemoteNameUnchecked()); IObservableValue remoteNameValidityObservable = BeanProperties.value( GitCloningSettingsWizardPageModel.PROPERTY_CUSTOM_REMOTE_NAME_VALIDITY).observe(pageModel); dbc.addValidationStatusProvider(new RemoteNameValidationStatusProvider(remoteNameValidityObservable, @@ -189,6 +180,15 @@ return cloneGroup; } + private SelectionAdapter onDefaultRemoteNameUnchecked() { + return new SelectionAdapter() { + @Override + public void widgetSelected(SelectionEvent e) { + remoteNameText.setFocus(); + } + }; + } + private SelectionListener onRepoPath() { return new SelectionAdapter() { @@ -274,10 +274,6 @@ return ValidationStatus.ok(); } - /* - * (non-Javadoc) - * @see org.eclipse.core.databinding.validation.MultiValidator#getTargets() - */ @Override public IObservableList getTargets() { WritableList targets = new WritableList(); @@ -309,10 +305,6 @@ return ValidationStatus.ok(); } - /* - * (non-Javadoc) - * @see org.eclipse.core.databinding.validation.MultiValidator#getTargets() - */ @Override public IObservableList getTargets() { WritableList targets = new WritableList();