Skip to content

* 修复:ExpressRunner.executeRule 不支持语句 if(){}else{} 这种最后包含else语句的问题。#105

Open
helpta wants to merge 1 commit intoalibaba:branch_version_3.x.xfrom
helpta:master
Open

* 修复:ExpressRunner.executeRule 不支持语句 if(){}else{} 这种最后包含else语句的问题。#105
helpta wants to merge 1 commit intoalibaba:branch_version_3.x.xfrom
helpta:master

Conversation

@helpta
Copy link

@helpta helpta commented Nov 13, 2020

  • 修复:ExpressRunner.executeRule 不支持语句 if(){}else{} 这种最后包含else语句的问题。原因是解析rule的时候只兼容了else if(){}。

这个问题会导致else逻辑被解析丢了,进而永远不可能被执行。


  • Fix: ExpressRunner.executeRule not support statemnt like ' if(){}else{}' which contain 'else' statement .The reason is just compatible with 'else if' satement.

This bug will missing the else logic when parsing the statement and the logic will never be execute.

@CLAassistant
Copy link

CLAassistant commented Nov 13, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


hongkai.wang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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 this pull request may close these issues.

2 participants