Skip to content

Commit cbd958b

Browse files
authored
Merge pull request #6261 from Countly/hooks-filter-null-check
[fix] Hooks: Added null check for incoming data
2 parents dcc034d + afdee91 commit cbd958b

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ Enterprise Features:
55

66
Fixes:
77
- [core] Unifying alphabetical order for dropdowns with dashboard apps
8+
- [hooks] Added null check for incoming data
89

910
Enterprise Fixes:
1011
- [content] Asset URL was wrongly constructed when user switches between apps

plugins/hooks/api/parts/triggers/incoming_data.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,9 @@ class IncomingDataTrigger {
189189
*/
190190
function assertOperation(value, filterObj) {
191191
var matched = true;
192+
if (typeof value === 'undefined' || value === null) {
193+
return false;
194+
}
192195
if (filterObj.$in && filterObj.$in.indexOf(value) === -1) {
193196
matched = false;
194197
}

0 commit comments

Comments
 (0)