Skip to content
Snippets Groups Projects
Unverified Commit 99728085 authored by Tom Robinson's avatar Tom Robinson
Browse files

Allow aggregation names as field names (if quoted). Resolves #4049

parent f17d3e99
No related branches found
No related tags found
No related merge requests found
......@@ -6,12 +6,14 @@ import { VALID_OPERATORS, VALID_AGGREGATIONS } from "./tokens";
export { VALID_OPERATORS, VALID_AGGREGATIONS } from "./tokens";
const RESERVED_WORDS = new Set(VALID_AGGREGATIONS.values());
export function formatAggregationName(aggregationOption) {
return VALID_AGGREGATIONS.get(aggregationOption.short);
}
function formatIdentifier(name) {
return /^\w+$/.test(name) ?
return /^\w+$/.test(name) && !RESERVED_WORDS.has(name) ?
name :
JSON.stringify(name);
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment