From 9ddc3758f2436f53a23220003cb085859e2c2ad4 Mon Sep 17 00:00:00 2001
From: Sameer Al-Sakran <sameer@expa.com>
Date: Tue, 6 Dec 2016 16:48:36 -0800
Subject: [PATCH] dont try to get the short name of a null
---
.../metabase/query_builder/components/AggregationWidget.jsx | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/frontend/src/metabase/query_builder/components/AggregationWidget.jsx b/frontend/src/metabase/query_builder/components/AggregationWidget.jsx
index f8f44f9e5e6..7474ce6ecf0 100644
--- a/frontend/src/metabase/query_builder/components/AggregationWidget.jsx
+++ b/frontend/src/metabase/query_builder/components/AggregationWidget.jsx
@@ -48,7 +48,8 @@ export default class AggregationWidget extends Component {
const fieldId = AggregationClause.getField(aggregation);
let selectedAggregation = getAggregator(AggregationClause.getOperator(aggregation));
- if (!_.findWhere(tableMetadata.aggregation_options, { short: selectedAggregation.short })) {
+ console.log("Selected Aggregation", selectedAggregation)
+ if (selectedAggregation && !_.findWhere(tableMetadata.aggregation_options, { short: selectedAggregation.short })) {
// if this table doesn't support the selected aggregation, prompt the user to select a different one
selectedAggregation = null;
}
--
GitLab