Skip to content
Snippets Groups Projects
Commit c0c7597f authored by Atte Keinänen's avatar Atte Keinänen
Browse files

Add `supportMultipleAggregations` flag for hiding add button

parent 62b8932d
No related branches found
No related tags found
No related merge requests found
......@@ -73,6 +73,7 @@ export default class PartialQueryBuilder extends Component {
databases={tableMetadata && [tableMetadata.db]}
setDatasetQuery={this.setDatasetQuery}
isShowingDataReference={false}
supportMultipleAggregations={false}
setDatabaseFn={null}
setSourceTableFn={null}
addQueryFilter={(filter) => onChange(Query.addFilter(datasetQuery.query, filter))}
......
......@@ -35,7 +35,8 @@ export default class GuiQueryEditor extends Component {
setDatasetQuery: PropTypes.func.isRequired,
setDatabaseFn: PropTypes.func,
setSourceTableFn: PropTypes.func,
features: PropTypes.object
features: PropTypes.object,
supportMultipleAggregations: PropTypes.bool
};
static defaultProps = {
......@@ -46,7 +47,8 @@ export default class GuiQueryEditor extends Component {
breakout: true,
sort: true,
limit: true
}
},
supportMultipleAggregations: true
};
renderAdd(text, onClick, targetRefName) {
......@@ -134,7 +136,7 @@ export default class GuiQueryEditor extends Component {
}
renderAggregation() {
const { datasetQuery: { query }, tableMetadata } = this.props;
const { datasetQuery: { query }, tableMetadata, supportMultipleAggregations } = this.props;
if (!this.props.features.aggregation) {
return;
......@@ -152,7 +154,8 @@ export default class GuiQueryEditor extends Component {
const canRemoveAggregation = aggregations.length > 1;
if (!isBareRows) {
if (supportMultipleAggregations && !isBareRows) {
// Placeholder aggregation for showing the add button
aggregations.push([]);
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment