You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by yj...@apache.org on 2021/01/20 22:22:31 UTC
[superset] branch master updated: fix: missing key when verifying
adhoc filters in merge_extra_filters (#12620)
This is an automated email from the ASF dual-hosted git repository.
yjc pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/master by this push:
new e7def7e fix: missing key when verifying adhoc filters in merge_extra_filters (#12620)
e7def7e is described below
commit e7def7e0e25caf09fbe33f8019db413686e349bc
Author: bryanck <br...@gmail.com>
AuthorDate: Wed Jan 20 14:22:05 2021 -0800
fix: missing key when verifying adhoc filters in merge_extra_filters (#12620)
Co-authored-by: Bryan Keller <bk...@netflix.com>
---
superset/utils/core.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/superset/utils/core.py b/superset/utils/core.py
index 084b513..9b1590b 100644
--- a/superset/utils/core.py
+++ b/superset/utils/core.py
@@ -1025,8 +1025,8 @@ def merge_extra_filters( # pylint: disable=too-many-branches
for existing in adhoc_filters:
if (
existing["expressionType"] == "SIMPLE"
- and existing["comparator"] is not None
- and existing["subject"] is not None
+ and existing.get("comparator") is not None
+ and existing.get("subject") is not None
):
existing_filters[get_filter_key(existing)] = existing["comparator"]