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

Jump to bottom

向工作流上下文中追加字符串时，value的值会增加双引号 #307

Closed
dudiao opened this issue Jul 27, 2021 · 2 comments
Closed

向工作流上下文中追加字符串时，value的值会增加双引号 #307

dudiao opened this issue Jul 27, 2021 · 2 comments
Assignees
@KFCFans @Echo009
Labels
bug Something isn't working

Comments

@dudiao
Copy link
Member

dudiao commented Jul 27, 2021

Describe the bug
向工作流上下文中追加数据（字符串）时，value的值会增加双引号，比如追加GitPullProcessor_LocalDirectory=D:\aml\aml-bank-sql2.0时，value值会变成"D:\aml\aml-bank-sql2.0"

taskContext.getWorkflowContext().appendData2WfContext("GitPullProcessor_LocalDirectory", "D:\\aml\\aml-bank-sql2.0");

Environment

  • PowerJob Version: [4.0.1]
  • Java Version: [jdk1.8.0_291]
  • SpringBoot: [2.5.2]
  • OS: [Windows 10]

Screenshots
1627403822

Additional context
这个类转成String的时候多加了个双引号：tech.powerjob.common.serialize.JsonUtils#toJSONStringUnsafe
@dudiao dudiao added the bug Something isn't working label Jul 27, 2021
@Echo009 Echo009 self-assigned this Jul 28, 2021
dudiao added a commit to dudiao/PowerJob that referenced this issue Jul 29, 2021
@dudiao
fix: When you append a string to the workflow context, the value has …
de42bca 
…multiple double quotes.(PowerJob#307)
@dudiao dudiao mentioned this issue Jul 29, 2021
Merged
Echo009 pushed a commit to Echo009/PowerJob that referenced this issue Aug 26, 2022
@dudiao @Echo009
fix: When you append a string to the workflow context, the value has …
82702e7 
…multiple double quotes.(PowerJob#307)
Echo009 pushed a commit that referenced this issue Aug 31, 2022
@dudiao
fix: When you append a string to the workflow context, the value has …
62d682f 
…multiple double quotes.(#307)
@KFCFans KFCFans closed this as completed Sep 12, 2022
@huarongsheng
Copy link

appendData2WfContext value为字符串时会多加一个双引号，已经修复close掉但是提交后代码又被覆盖掉了，bug仍然存在
image
image

@KFCFans KFCFans reopened this Mar 17, 2023
KFCFans added a commit that referenced this issue Mar 17, 2023
@KFCFans
fix: workflow append string will have quotes #307
2a87a24
@KFCFans
Copy link
Member

KFCFans commented Mar 17, 2023

appendData2WfContext value为字符串时会多加一个双引号，已经修复close掉但是提交后代码又被覆盖掉了，bug仍然存在 image image

已确认，确实代码在合并过程中被回滚了。4.3.2 会修复，感谢反馈！

@KFCFans KFCFans closed this as completed Mar 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KFCFans KFCFans

@Echo009 Echo009

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@KFCFans @Echo009 @dudiao @huarongsheng