Author: adietish Date: 2011-07-04 11:46:40 -0400 (Mon, 04 Jul 2011) New Revision: 32556 Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.rse.ui/src/org/jboss/ide/eclipse/as/rse/ui/RSELaunchTabProvider.java Log: [JBIDE-9215] moved launch config setup to external component (configurator) in order to get rid of static methods in launch configuration classes Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.rse.ui/src/org/jboss/ide/eclipse/as/rse/ui/RSELaunchTabProvider.java =================================================================== --- trunk/as/plugins/org.jboss.ide.eclipse.as.rse.ui/src/org/jboss/ide/eclipse/as/rse/ui/RSELaunchTabProvider.java 2011-07-04 15:43:43 UTC (rev 32555) +++ trunk/as/plugins/org.jboss.ide.eclipse.as.rse.ui/src/org/jboss/ide/eclipse/as/rse/ui/RSELaunchTabProvider.java 2011-07-04 15:46:40 UTC (rev 32556) @@ -27,7 +27,7 @@ import org.eclipse.swt.widgets.Composite; import org.eclipse.swt.widgets.Group; import org.eclipse.swt.widgets.Text; -import org.jboss.ide.eclipse.as.rse.core.RSELaunchConfigUtils; +import org.jboss.ide.eclipse.as.rse.core.RSELaunchConfigProperties; import org.jboss.ide.eclipse.as.rse.core.RSELaunchDelegate; import org.jboss.ide.eclipse.as.ui.UIUtil; import org.jboss.ide.eclipse.as.ui.launch.JBossLaunchConfigurationTabGroup.IJBossLaunchTabProvider; @@ -122,16 +122,16 @@ this.initialConfig = configuration; try { - String startCommand = RSELaunchConfigUtils.getStartupCommand(configuration); + String startCommand = RSELaunchConfigProperties.getStartupCommand(configuration); startText.setText(startCommand == null ? "" : startCommand); - boolean detectStartCommand = RSELaunchConfigUtils.isDetectStartupCommand(configuration, true); + boolean detectStartCommand = RSELaunchConfigProperties.isDetectStartupCommand(configuration, true); autoStartArgs.setSelection(detectStartCommand); startText.setEditable(!detectStartCommand); startText.setEnabled(!detectStartCommand); - String stopCommand = RSELaunchConfigUtils.getShutdownCommand(configuration); + String stopCommand = RSELaunchConfigProperties.getShutdownCommand(configuration); stopText.setText(stopCommand == null ? "" : stopCommand); - boolean detectStopCommand = RSELaunchConfigUtils.isDetectShutdownCommand(configuration, true); + boolean detectStopCommand = RSELaunchConfigProperties.isDetectShutdownCommand(configuration, true); autoStopArgs.setSelection(detectStopCommand); stopText.setEditable(!detectStopCommand); stopText.setEnabled(!detectStopCommand); @@ -141,10 +141,10 @@ } } public void performApply(ILaunchConfigurationWorkingCopy configuration) { - RSELaunchConfigUtils.setStartupCommand(startText.getText(), configuration); - RSELaunchConfigUtils.setShutdownCommand(stopText.getText(), configuration); - RSELaunchConfigUtils.setDetectStartupCommand(autoStartArgs.getSelection(), configuration); - RSELaunchConfigUtils.setDetectShutdownCommand(autoStopArgs.getSelection(), configuration); + RSELaunchConfigProperties.setStartupCommand(startText.getText(), configuration); + RSELaunchConfigProperties.setShutdownCommand(stopText.getText(), configuration); + RSELaunchConfigProperties.setDetectStartupCommand(autoStartArgs.getSelection(), configuration); + RSELaunchConfigProperties.setDetectShutdownCommand(autoStopArgs.getSelection(), configuration); } public String getName() { return RSEUIMessages.RSE_REMOTE_LAUNCH;