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.

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

Incorrect value for attribute processDefinitionId in toString() method for HistoricVariableUpdateEventEntity #4927

Open
1 task
dbenesj opened this issue Feb 7, 2025 · 0 comments
Assignees
Labels
type:bug Issues that describe a user-facing bug in the project. version:7.23.0

Comments

@dbenesj
Copy link

dbenesj commented Feb 7, 2025

Environment

7.21.0 Tomcat

Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket)

When logging the HistoricVariableUpdateEventEntity and HistoricDetailVariableInstanceUpdateEntity value printed for processDefinitionId is the same as processInstanceId, which is not correct.
Example log:

HistoricVariableUpdateEventEntity[variableName=EvidenceNotes, variableInstanceId=96810625-e535-11ef-978a-0242ac1e0013, revision=0, serializerName=string, longValue=null, doubleValue=null, textValue=neco, textValue2=null, byteArrayId=null, activityInstanceId=CollectEvidence:0a592ba1-e474-11ef-ba10-0242ac1e0013, scopeActivityInstanceId=0a581b1f-e474-11ef-ba10-0242ac1e0013, eventType=create, executionId=0a581b1f-e474-11ef-ba10-0242ac1e0013, id=null, processDefinitionId=0a581b1f-e474-11ef-ba10-0242ac1e0013, processInstanceId=0a581b1f-e474-11ef-ba10-0242ac1e0013, taskId=null, timestamp=Fri Feb 07 09:26:19 UTC 2025, tenantId=null, isInitial=false]

Steps to reproduce

Print the result of toString() method for HistoricVariableUpdateEventEntity and HistoricDetailVariableInstanceUpdateEntity

Observed Behavior

For attribute processDefinitionId value of processInstanceId is being used.

Expected behavior

Value of the processDefinitionId should be printed for that field.

Root Cause

Incorrect variable used in the toString() method.

Solution Ideas

Just change the variable used.

Pull requests

Preview Give feedback
  1. dbenesj
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Issues that describe a user-facing bug in the project. version:7.23.0
Projects
None yet
Development

No branches or pull requests

2 participants