@malloydata/malloy 0.0.38-dev230606045241 → 0.0.38-dev230609163341

package/dist/malloy.d.ts

@@ -397,6 +397,10 @@ export declare class PreparedResult {
      */
     get resultExplore(): Explore;
     get sourceExplore(): Explore;
+    /**
+     * @return The query timezone.
+     */
+    get queryTimezone(): string | undefined;
     get _sourceExploreName(): string;
     get _sourceFilters(): FilterExpression[];
 }

package/dist/malloy.js

@@ -224,6 +224,7 @@ class Malloy {
                 // TODO feature-sql-block There is no source explore...
                 sourceExplore: '',
                 sourceFilters: [],
+                queryTimezone: sqlStruct.queryTimezone,
             }, {
                 name: 'empty_model',
                 exports: [],
@@ -338,7 +339,7 @@ class Model {
      */
     getPreparedQueryByName(queryName) {
         const query = this.modelDef.contents[queryName];
-        if (query.type === 'query') {
+        if ((query === null || query === void 0 ? void 0 : query.type) === 'query') {
             return new PreparedQuery(query, this.modelDef, queryName);
         }
         throw new Error('Given query name does not refer to a named query.');
@@ -725,6 +726,12 @@ class PreparedResult {
         }
         throw new Error(`'${name} is not an explore`);
     }
+    /**
+     * @return The query timezone.
+     */
+    get queryTimezone() {
+        return this.inner.queryTimezone;
+    }
     get _sourceExploreName() {
         return this.inner.sourceExplore;
     }

package/dist/model/malloy_query.js

@@ -208,11 +208,15 @@ class QueryField extends QueryNode {
         // find the structDef and return the path to the field...
         const field = context.getFieldByName(expr.path);
         if ((0, malloy_types_1.hasExpression)(field.fieldDef)) {
-            let ret = this.generateExpressionFromExpr(resultSet, field.parent, field.fieldDef.e, state);
-            if (!ret.match(/^\(.*\)$/)) {
-                ret = `(${ret})`;
-            }
-            return ret;
+            const ret = this.generateExpressionFromExpr(resultSet, field.parent, field.fieldDef.e, state);
+            // in order to avoid too many parens, there was some code here ..
+            // if (!ret.match(/^\(.*\)$/)) {
+            //   ret = `(${ret})`;
+            // }
+            // but this  failed when the expresion was (bool1)or(bool2)
+            // there could maybe be a smarter parse of the expression to avoid
+            // an extra paren, but correctness first, beauty AND correctness later
+            return `(${ret})`;
         }
         else {
             // return field.parent.getIdentifier() + "." + field.fieldDef.name;

package/dist/model/malloy_types.d.ts

@@ -519,6 +519,7 @@ export interface CompiledQuery extends DrillSource {
     malloy: string;
     queryName?: string | undefined;
     connectionName: string;
+    queryTimezone?: string;
 }
 /** Result type for running a Malloy query. */
 export interface QueryResult extends CompiledQuery {

package/package.json

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