Author: bbrodt
Date: 2011-01-10 15:44:22 -0500 (Mon, 10 Jan 2011)
New Revision: 28054
Modified:
trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/VariablePartAssignCategory.java
Log:
https://issues.jboss.org/browse/JBIDE-8048
oops - changed the name of isInitializerExist() to needInitializer() but didn't
reverse the logic.
Modified:
trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/VariablePartAssignCategory.java
===================================================================
---
trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/VariablePartAssignCategory.java 2011-01-10
18:19:07 UTC (rev 28053)
+++
trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/VariablePartAssignCategory.java 2011-01-10
20:44:22 UTC (rev 28054)
@@ -25,7 +25,6 @@
import org.eclipse.bpel.model.Query;
import org.eclipse.bpel.model.To;
import org.eclipse.bpel.model.Variable;
-import org.eclipse.bpel.model.impl.VariableImpl;
import org.eclipse.bpel.model.messageproperties.Property;
import org.eclipse.bpel.model.util.BPELConstants;
import org.eclipse.bpel.model.util.BPELUtils;
@@ -495,18 +494,18 @@
if (query == null)
return;
Variable var = side.getVariable();
- if (needInitializer(var, side))
- return;
- if (MessageDialog
- .openQuestion(
- PlatformUI.getWorkbench().getActiveWorkbenchWindow()
- .getShell(),
- "Initializer",
- NLS
- .bind(
- "Variable {0} doesn't have initializer. Should it be
generated?",
- (new Object[] { var.getName() })))) {
- initTargetVariable(var, side);
+ if (needInitializer(var, side)) {
+ if (MessageDialog
+ .openQuestion(
+ PlatformUI.getWorkbench().getActiveWorkbenchWindow()
+ .getShell(),
+ "Initializer",
+ NLS
+ .bind(
+ "Variable {0} doesn't have initializer. Should it be
generated?",
+ (new Object[] { var.getName() })))) {
+ initTargetVariable(var, side);
+ }
}
}
@@ -527,7 +526,7 @@
if (p == null)
continue;
if (p.equals(side.getPart()))
- return true;
+ return false;
}
//
https://issues.jboss.org/browse/JBIDE-8048
@@ -570,10 +569,10 @@
// Incomplete variable definition
if (rootElement == null || uriWSDL == null) {
- return true;
+ return false;
}
- return false;
+ return true;
}
private void initTargetVariable(Variable var, IVirtualCopyRuleSide side) {