You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@unomi.apache.org by GitBox <gi...@apache.org> on 2021/09/01 09:49:46 UTC

[GitHub] [unomi] giladw commented on a change in pull request #301: UNOMI-472-fix conditionEvaluator inconsistent with elastic for condition when p…

giladw commented on a change in pull request #301:
URL: https://github.com/apache/unomi/pull/301#discussion_r700058291



##########
File path: plugins/baseplugin/src/main/java/org/apache/unomi/plugins/baseplugin/conditions/PropertyConditionEvaluator.java
##########
@@ -206,7 +206,7 @@ protected boolean isMatch(String op, Object actualValue, String expectedValue, O
         if (op == null) {
             return false;
         } else if (actualValue == null) {
-            return op.equals("missing");
+            return op.equals("missing")|| op.equals("notIn") || op.equals("notEquals");

Review comment:
       Hi, sorry im working on a different project at the moment and i will not have time to push new commits




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@unomi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org