[jboss-jira] [JBoss JIRA] (DROOLS-4296) Rewrite parsing of modified properties

[ https://issues.redhat.com/browse/DROOLS-4296?page=com.atlassian.jira.plug... ] Toshiya Kobayashi commented on DROOLS-4296: ------------------------------------------- Talked with Luca. Case A) {noformat} { modify($p) { age = $p.age+1 }; } {noformat} with mvel dialect -> The expression is parsed to an AssignExpr inside a ModifyStatement. AssignExpr.getTarget() returns "age" so we can store such information and skip findModifiedProperties(). Case B) {noformat} { modify($p) { setName($p.getName() + "30"); } {noformat} with mvel/java dialect -> The expression is parsed to an ExpressionStmt inside a ModifyStatement. We cannot get "age" directly from the ExpressionStmt. So we will use findModifiedProperties() later. Even if we add logic to store modified properties for the case A), we still keep findModifiedProperties() for case B). Also we would need to consider the case where a user mixes the expressions with mvel dialect. e.g. {noformat} { modify($p) { age = $p.age+1, setName($p.getName() + "30"); } {noformat} The current implementation of findModifiedProperties() doesn't fully parse the modify block. It iterates MethodCallExpr so I think it's not inefficient. Rather, it's good to have one place to process for all the cases. https://github.com/kiegroup/drools/blob/master/drools-model/drools-model-... We concluded that we don't need to refactor it. -- This message was sent by Atlassian Jira (v7.13.8#713008)