Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Git issue 4578 taskid save for variables #4745

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Git issue 4578 taskid save for variables #4745

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
f6c62e2
add task_id for ACT_HI_VARINST table for task variables
jyotisahu9 Sep 24, 2024
aa4d411
add task_id for ACT_HI_VARINST table for task variables
jyotisahu9 Sep 24, 2024
4c8dfef
chore(license-book): update license book for 7.22.0 (#4654)
tasso94 Sep 25, 2024
18eef54
add task_id for ACT_HI_VARINST table for task variables-review comments
jyotisahu9 Sep 25, 2024
9046653
Merge branch 'master' into git-issue-4578-taskid-save-for-variables
jyotisahu9 Sep 26, 2024
7dced3d
incorporated review comments
jyotisahu9 Sep 26, 2024
107a777
adding code comments
jyotisahu9 Oct 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add task_id for ACT_HI_VARINST table for task variables-review comments
  • Loading branch information
@jyotisahu9
jyotisahu9 committed Sep 25, 2024
commit 18eef54a26f1dd0c3e99fb00d0077f4e2ce99d5c
8 changes: 6 additions & 2 deletions ...c/main/java/org/camunda/bpm/engine/impl/history/producer/DefaultHistoryEventProducer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -452,8 +452,9 @@ else if (variableInstance.getCaseExecutionId() != null) {
sourceExecution = ((TaskEntity) sourceVariableScope).getExecution();
if (sourceExecution != null) {
List<TaskEntity> taskEntityList = sourceExecution.getTasks();
if(taskEntityList!=null && taskEntityList.size()>0)
if(taskEntityList!=null && !taskEntityList.isEmpty()){
taskId = taskEntityList.get(0).getId();
}

sourceActivityInstanceId = sourceExecution.getActivityInstanceId();
}
Expand Down Expand Up @@ -481,8 +482,11 @@ else if (sourceVariableScope instanceof CaseExecutionEntity) {

// set source activity instance id
evt.setActivityInstanceId(sourceActivityInstanceId);
if(taskId!=null && evt.getTaskId()==null)

// set task id
if(taskId!=null && evt.getTaskId()==null) {
evt.setTaskId(taskId);
}

// mark initial variables on process start
if (sourceExecution != null && sourceExecution.isProcessInstanceStarting()
Expand Down