From 39d65396026f63b14b4e00a912f2a5fabe217829 Mon Sep 17 00:00:00 2001 From: mwang Date: Thu, 24 Jan 2013 09:00:47 +0000 Subject: [PATCH] For bug TDI-24255: tRunjob passed value gets changed when Subjob context variable name changes git-svn-id: http://talendforge.org/svn/tos/trunk@97182 f6f1c999-d317-4740-80b0-e6d1abc6f99e --- .../UpdateRunJobComponentContextHelper.java | 21 ++++++++++++------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/org.talend.core.runtime/src/main/java/org/talend/core/model/context/UpdateRunJobComponentContextHelper.java b/org.talend.core.runtime/src/main/java/org/talend/core/model/context/UpdateRunJobComponentContextHelper.java index 9a92103fda..f45b8baaa6 100644 --- a/org.talend.core.runtime/src/main/java/org/talend/core/model/context/UpdateRunJobComponentContextHelper.java +++ b/org.talend.core.runtime/src/main/java/org/talend/core/model/context/UpdateRunJobComponentContextHelper.java @@ -272,15 +272,20 @@ public final class UpdateRunJobComponentContextHelper { } } } + + // Change value only for the job which context variable is changed. Changed by Marvin Wang + // on Jan. 24, for bug TDI-24255. Context value should not be changed, because the context + // value is from the current job like "context.foo". // bug 9424 - if (eleValueType.getElementRef().equals(PARAM_VALUE_COLUMN)) { - String oldValue = eleValueType.getValue(); - String newValue = checkAndUpdateValue(nameMap, oldValue); - if (newValue != null) { // update - eleValueType.setValue(newValue); - modified = true; - } - } + // if (eleValueType.getElementRef().equals(PARAM_VALUE_COLUMN)) { + // String oldValue = eleValueType.getValue(); + // String newValue = checkAndUpdateValue(nameMap, oldValue); + // if (newValue != null) { // update + // eleValueType.setValue(newValue); + // modified = true; + // } + // } + } if (!movedRecord.isEmpty()) { for (ElementValueType eleValueType : movedRecord) {