Commit ceba5b8cecad017bd75ec8ac03ac8154dde32c5a
1 parent
24f1ce16
【代码评审】BPM:优化流程发起预测节点审批人是否配置在后端逻辑中实现
Showing
1 changed file
with
6 additions
and
5 deletions
yudao-module-bpm/yudao-module-bpm-biz/src/main/java/cn/iocoder/yudao/module/bpm/framework/flowable/core/util/BpmnModelUtils.java
| ... | ... | @@ -850,11 +850,11 @@ public class BpmnModelUtils { |
| 850 | 850 | * 计算条件表达式是否为 true 满足条件 |
| 851 | 851 | * |
| 852 | 852 | * @param variables 流程实例 |
| 853 | - * @param express 条件表达式 | |
| 853 | + * @param expression 条件表达式 | |
| 854 | 854 | * @return 是否满足条件 |
| 855 | 855 | */ |
| 856 | - public static boolean evalConditionExpress(Map<String, Object> variables, String express) { | |
| 857 | - if (express == null) { | |
| 856 | + public static boolean evalConditionExpress(Map<String, Object> variables, String expression) { | |
| 857 | + if (expression == null) { | |
| 858 | 858 | return Boolean.FALSE; |
| 859 | 859 | } |
| 860 | 860 | // 如果 variables 为空,则创建一个的原因?可能 expression 的计算,不依赖于 variables |
| ... | ... | @@ -864,10 +864,11 @@ public class BpmnModelUtils { |
| 864 | 864 | |
| 865 | 865 | // 执行计算 |
| 866 | 866 | try { |
| 867 | - Object result = FlowableUtils.getExpressionValue(variables, express); | |
| 867 | + Object result = FlowableUtils.getExpressionValue(variables, expression); | |
| 868 | 868 | return Boolean.TRUE.equals(result); |
| 869 | 869 | } catch (FlowableException ex) { |
| 870 | - log.error("[evalConditionExpress][条件表达式({}) 变量({}) 解析报错]", express, variables, ex); | |
| 870 | + // 为什么使用 info 日志?原因是,expression 如果从 variables 取不到值,会报错。实际这种情况下,可以忽略 | |
| 871 | + log.info("[evalConditionExpress][条件表达式({}) 变量({}) 解析报错]", expression, variables, ex); | |
| 871 | 872 | return Boolean.FALSE; |
| 872 | 873 | } |
| 873 | 874 | } | ... | ... |