Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Commit

Permalink
make sure there is a val to be set for existing filters (apache#5257)
Browse files Browse the repository at this point in the history
  • Loading branch information
hughhhh authored and timifasubaa committed Jul 25, 2018
1 parent bd2d358 commit c49434e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion superset/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -737,7 +737,7 @@ def get_filter_key(f):
return f['col'] + '__' + f['op']
existing_filters = {}
for existing in form_data['filters']:
if existing['col'] is not None:
if existing['col'] is not None and existing['val'] is not None:
existing_filters[get_filter_key(existing)] = existing['val']
for filtr in form_data['extra_filters']:
# Pull out time filters/options and merge into form data
Expand Down

0 comments on commit c49434e

Please sign in to comment.