Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ContainsFilter is raising an exception #15

Closed
mikeengland opened this issue Aug 17, 2016 · 1 comment
Closed

ContainsFilter is raising an exception #15

mikeengland opened this issue Aug 17, 2016 · 1 comment
Assignees
Labels
Milestone

Comments

@mikeengland
Copy link
Contributor

mikeengland commented Aug 17, 2016

The ContainsFilter is raising an exception when it is used.

Example call:
res = searchads_slicer.highcharts.line_chart(
metrics=['clicks'],
dimensions=['date'],
dimension_filters=[RangeFilter('date', date.today() - timedelta(days=14), date.today()),
ContainsFilter('device_type', ['c'])],
)

print(res)

print(json.dumps(res))```

Error:

Traceback (most recent call last): File "", line 205, in <module> ContainsFilter('device_type', ['c', 't'])], File "/site-packages/fireant/slicer/managers.py", line 280, in _get_and_transform_data references=references, operations=operations) File "/site-packages/fireant/slicer/managers.py", line 69, in data references=references, operations=operations) File "site-packages/fireant/slicer/managers.py", line 103, in query_schema self._default_dimension_definition), File "site-packages/fireant/slicer/managers.py", line 205, in _filters_schema filters_schema.append(f.schemas(definition)) File "site-packages/fireant/slicer/filters.py", line 58, in schemas return element.isin(self.values) AttributeError: 'Coalesce' object has no attribute 'isin'

@twheys twheys self-assigned this Aug 18, 2016
@twheys twheys added the bug label Aug 23, 2016
@twheys twheys added this to the 0.0.14 milestone Aug 23, 2016
twheys added a commit that referenced this issue Aug 23, 2016
@twheys
Copy link
Contributor

twheys commented Aug 23, 2016

Fixed the bug in pypika and upgraded the pypika version in fireant

@twheys twheys closed this as completed Aug 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants