sergehuber opened a new pull request, #919: URL: https://github.com/apache/unomi/pull/919
## Summary - Boolean OR no longer treats an unset optional parameter as a match. Unset branches are omitted; an OR with no remaining branches matches nothing. This fixes hover conditions that only set `targetId` or only set `targetPath`. - Boolean AND still omits unset optionals (UNOMI-982), so page-view / form conditions keep working when optional fields are left empty. - The same pruning is applied in the Elasticsearch and OpenSearch boolean query builders so query and in-memory evaluation stay aligned. ## Test plan - [x] `BooleanConditionEvaluatorTest` — OR/AND optional-parameter cases - [x] `HoverEventConditionOptionalOrTest` — only id, only path, both, neither - [x] `ConditionContextHelperTest` — resolveBooleanSubConditions - [x] `BooleanConditionESQueryBuilderTest` / `BooleanConditionOSQueryBuilderTest` - [x] Existing UNOMI-982 unset-parameter dispatcher test still passes - [ ] CI integration suite -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
