@malloydata/malloy 0.0.35 → 0.0.36-dev230524213240

package/dist/lang/ast/expressions/expr-filter.js

@@ -38,15 +38,12 @@ class ExprFilter extends expression_def_1.ExpressionDef {
     getExpression(fs) {
         const testList = this.filter.getFilterList(fs);
         const resultExpr = this.expr.getExpression(fs);
-        for (const cond of testList) {
-            if ((0, malloy_types_1.expressionIsCalculation)(cond.expressionType)) {
-                this.filter.log('Cannot filter a field with an aggregate or analytical computation');
-                return (0, ast_utils_1.errorFor)('no filter on aggregate');
-            }
-        }
         if (resultExpr.expressionType === 'scalar') {
-            // TODO could log a warning, but I have a problem with the
-            // idea of warnings, so for now ...
+            this.expr.log('Filtered expression requires an aggregate computation');
+            return resultExpr;
+        }
+        if (testList.find(cond => (0, malloy_types_1.expressionIsCalculation)(cond.expressionType))) {
+            this.filter.log('Cannot filter an expresion with an aggregate or analytical computation');
             return resultExpr;
         }
         if (this.typeCheck(this.expr, { ...resultExpr, expressionType: 'scalar' })) {

package/dist/lang/test/parse.spec.js

@@ -682,6 +682,10 @@ describe('expressions', () => {
         });
     });
     test('filtered measure', exprOK("acount {? astr = 'why?' }"));
+    test('correctly flags filtered scalar', () => {
+        const e = new test_translator_1.BetaExpression('ai { where: true }');
+        expect(e).compileToFailWith('Filtered expression requires an aggregate computation');
+    });
     describe('aggregate forms', () => {
         test('count', exprOK('count()'));
         test('count distinct', exprOK('count(distinct astr)'));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@malloydata/malloy",
-  "version": "0.0.35",
+  "version": "0.0.36-dev230524213240",
   "license": "MIT",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",