[jboss-cvs] jbosside/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/config ...
Robert Stryker
rawblem at gmail.com
Wed Sep 6 15:35:38 EDT 2006
User: rawb
Date: 06/09/06 15:35:38
Modified: as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/config
JBossLaunchConfigurationTabGroup.java
Log:
Removed requirement for unchanging host
Revision Changes Path
1.8 +18 -13 jbosside/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/config/JBossLaunchConfigurationTabGroup.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: JBossLaunchConfigurationTabGroup.java
===================================================================
RCS file: /cvsroot/jboss/jbosside/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/config/JBossLaunchConfigurationTabGroup.java,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -b -r1.7 -r1.8
--- JBossLaunchConfigurationTabGroup.java 29 Aug 2006 16:49:06 -0000 1.7
+++ JBossLaunchConfigurationTabGroup.java 6 Sep 2006 19:35:38 -0000 1.8
@@ -651,25 +651,30 @@
return isHostAndConfigValid();
}
+ // Host requirement has been removed
private boolean isHostAndConfigValid() {
String progArgs = fPrgmArgumentsText.getText();
String workingConfig = RuntimeConfigUtil.getCommandArgument(progArgs, "-c", "--configuration");
- String workingHost = RuntimeConfigUtil.getCommandArgument(progArgs, "-b", "--host");
+ //String workingHost = RuntimeConfigUtil.getCommandArgument(progArgs, "-b", "--host");
String oldConfig = RuntimeConfigUtil.getCommandArgument(originalProgramArgs, "-c", "--configuration");
- String oldHost = RuntimeConfigUtil.getCommandArgument(originalProgramArgs, "-b", "--host");
+ //String oldHost = RuntimeConfigUtil.getCommandArgument(originalProgramArgs, "-b", "--host");
- if( oldConfig == null || oldHost == null ) return true;
-
- boolean valid = true;
- if( !workingHost.equals(oldHost)) {
- valid = false;
- }
- if( !workingConfig.equals(oldConfig)) {
- valid = false;
- }
+ if( oldConfig != null && !workingConfig.equals(oldConfig)) return false;
+ return true;
- return valid;
+// if( oldConfig == null || oldHost == null ) return true;
+//
+// boolean valid = true;
+//
+// if( !workingHost.equals(oldHost)) {
+// valid = false;
+// }
+// if( !workingConfig.equals(oldConfig)) {
+// valid = false;
+// }
+//
+// return valid;
}
public boolean isValid(ILaunchConfiguration config) {
boolean sup = super.isValid(config);
@@ -679,7 +684,7 @@
boolean valid = isHostAndConfigValid();
if( !valid ) {
- setErrorMessage("You may not change the configuration directory or the host in this fashion.");
+ setErrorMessage("You may not change the configuration.");
//getLaunchConfigurationDialog().updateButtons();
return false;
}
More information about the jboss-cvs-commits
mailing list