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

1.8.5升级到1.10.0之后【自动审核不通过】的工单逻辑发生了变化,一直在【待办列表】无法处理 #2395

Closed
qingmengxinghe opened this issue Nov 20, 2023 · 7 comments · Fixed by #2400
Assignees

Comments

@qingmengxinghe
Copy link

重现步骤

1、升级之后发现,自动审核不通过的工单状态为【待审核】,之前是【已完成】
然后就会有待在【待办列表】里面状态,显示未审核,点击【未审核】按钮进入审核见面,但是却没有任何操作权限
所以【自动审核不通过】的工单就会一直是未审核状态

2、对比了数据库里存的值,发现升级前【自动审核不通过】的工单信息 workflow_audit表里面current_audit | next_audit | current_status 这三个字段的值和之前版本的值都有区别,后面4行数据是新版本的。

workflow_type b_current_audit b_next_audit b_current_status
1 -1 -1 1
1 -1 -1 1
1 -1 -1 1
1 -1 -1 1
1 -1 -1 1
2 3 -1 0
2 3 -1 0
2 10 3 0
2 10 3 0

预期外的结果

【自动审核不通过】的工单应该直接结束,不应该是未审核状态,请修复这个bug

日志文本

No response

版本

1.10.0

部署方式

手工部署

是否还有其他可以辅助定位问题的信息?比如数据库版本等

No response

@LeoQuote
Copy link
Collaborator

hi, 最新的 master 分支已修复, 可以用 master 自己尝试一下

@qingmengxinghe
Copy link
Author

hi, 最新的 master 分支已修复, 可以用 master 自己尝试一下

我尝试了最新的代码,还是不行,请问是改了这里吗

/Archery-master/sql_api/serializers.py

有时候提交后自动审批通过, 在这里改写一下 workflow 状态

    if auditor.audit.current_status == WorkflowStatus.PASSED:
        auditor.workflow.status = "workflow_review_pass"
        auditor.workflow.save()

【自动审核不通过】 的状态还是【未审核】

@qingmengxinghe
Copy link
Author

hi, 最新的 master 分支已修复, 可以用 master 自己尝试一下

看起来是这里
./sql/utils/workflow_audit.py 234行 这里没有判断自动审核未通过的情况,把状态都设置为了【待审核】
自动审核未通过的状态应该为【审核未通过】才对
self.audit.current_status = WorkflowStatus.WAITING

@LeoQuote

This comment was marked as off-topic.

@LeoQuote LeoQuote self-assigned this Nov 20, 2023
@LeoQuote
Copy link
Collaborator

谢谢反馈, 我正在处理, 应该是再加一个逻辑即可. 如果你有好的方案也可以提pr

@qingmengxinghe
Copy link
Author

谢谢反馈, 我正在处理, 应该是再加一个逻辑即可. 如果你有好的方案也可以提pr

能力有限,就不提了吧, 多谢关注

@qingmengxinghe
Copy link
Author

谢谢反馈, 我正在处理, 应该是再加一个逻辑即可. 如果你有好的方案也可以提pr

谢谢大佬

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants