[jbosstools-commits] JBoss Tools SVN: r26692 - trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties.
Author: bbrodt
Date: 2010-11-17 14:47:16 -0500 (Wed, 17 Nov 2010)
New Revision: 26692
Modified:
trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/VariablePartAssignCategory.java
Log:
https://jira.jboss.org/browse/JBIDE-7351
reverting change that removes automatic checking for unitialized variables in Assign
activities
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 2010-11-17
19:45:41 UTC (rev 26691)
+++
trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/VariablePartAssignCategory.java 2010-11-17
19:47:16 UTC (rev 26692)
@@ -39,17 +39,20 @@
import org.eclipse.bpel.ui.util.XSDUtils;
import org.eclipse.emf.common.util.EList;
import org.eclipse.emf.ecore.EObject;
+import org.eclipse.jface.dialogs.MessageDialog;
import org.eclipse.jface.viewers.ISelectionChangedListener;
import org.eclipse.jface.viewers.IStructuredSelection;
import org.eclipse.jface.viewers.SelectionChangedEvent;
import org.eclipse.jface.viewers.StructuredSelection;
import org.eclipse.jface.viewers.TreeViewer;
+import org.eclipse.osgi.util.NLS;
import org.eclipse.swt.SWT;
import org.eclipse.swt.widgets.Composite;
import org.eclipse.swt.widgets.Label;
import org.eclipse.swt.widgets.Shell;
import org.eclipse.swt.widgets.Text;
import org.eclipse.swt.widgets.Tree;
+import org.eclipse.ui.PlatformUI;
import org.eclipse.wst.wsdl.Message;
import org.eclipse.wst.wsdl.Part;
import org.eclipse.xsd.XSDAttributeDeclaration;
@@ -477,10 +480,10 @@
side.setQuery(null);
}
- /*
// https://jira.jboss.org/browse/JBIDE-7351
- // remove all of this: don't ask to initialize variables whenever focus
- // changes - this makes for a bad user experience!
+ // TODO: Think about how to implement this in a way that makes sense...
+ // currently, a popup dialog asks the user to initialize variables whenever
+ // focus changes - this makes for a bad user experience!
// From?
if (side.isSource())
return;
@@ -503,7 +506,6 @@
(new Object[] { var.getName() })))) {
initTargetVariable(var, side);
}
- */
}
private boolean isInitializerExist(Variable var, IVirtualCopyRuleSide side) {