fix for bug introduced by 3c62dadc6053d40fffaedf198bcd2227749ec559
(missing break for String and Float cases)
Change-Id: Ib861fffc4b0cc81e928bfdd11c3ec89dbf631abf
diff --git a/plugins/org.eclipse.emf.henshin.interpreter.ui/src/org/eclipse/emf/henshin/interpreter/ui/wizard/ParameterEditTable.java b/plugins/org.eclipse.emf.henshin.interpreter.ui/src/org/eclipse/emf/henshin/interpreter/ui/wizard/ParameterEditTable.java
index a391c59..41531b0 100644
--- a/plugins/org.eclipse.emf.henshin.interpreter.ui/src/org/eclipse/emf/henshin/interpreter/ui/wizard/ParameterEditTable.java
+++ b/plugins/org.eclipse.emf.henshin.interpreter.ui/src/org/eclipse/emf/henshin/interpreter/ui/wizard/ParameterEditTable.java
@@ -189,21 +189,23 @@
try {
switch (paramCfg.getType()) {
case ParameterConfig.STRING:
- if (! value.toString().trim().equals("") || value.toString().trim().equals("null")) {
+ if (! (value.toString().trim().equals("") || value.toString().trim().equals("null"))) {
paramCfg.setValue(value.toString());
}
else {
paramCfg.setValue(null);
- }
+ }
+ break;
case ParameterConfig.FLOAT:
- if (! value.toString().trim().equals("") || value.toString().trim().equals("null")) {
+ if (! (value.toString().trim().equals("") || value.toString().trim().equals("null"))) {
paramCfg.setValue(Float.parseFloat(value.toString()));
}
else {
paramCfg.setValue(null);
- }
+ }
+ break;
case ParameterConfig.DOUBLE:
- if (! value.toString().trim().equals("") || value.toString().trim().equals("null")) {
+ if (! (value.toString().trim().equals("") || value.toString().trim().equals("null"))) {
paramCfg.setValue(Double.parseDouble(value.toString()));
}
else {
@@ -211,7 +213,7 @@
}
break;
case ParameterConfig.INT:
- if (! value.toString().trim().equals("") || value.toString().trim().equals("null")) {
+ if (! (value.toString().trim().equals("") || value.toString().trim().equals("null"))) {
paramCfg.setValue(Integer.parseInt(value.toString()));
}
else {
@@ -219,7 +221,7 @@
}
break;
case ParameterConfig.LONG:
- if (! value.toString().trim().equals("") || value.toString().trim().equals("null")) {
+ if (! (value.toString().trim().equals("") || value.toString().trim().equals("null"))) {
paramCfg.setValue(Long.parseLong(value.toString()));
}
else {
@@ -227,7 +229,7 @@
}
break;
case ParameterConfig.BOOLEAN:
- if (! value.toString().trim().equals("") || value.toString().trim().equals("null")) {
+ if (! (value.toString().trim().equals("") || value.toString().trim().equals("null"))) {
paramCfg.setValue((Integer) value > 0 ? true : false);
}
else {
