Skip to content
Snippets Groups Projects
Commit be4b787e authored by dsehnal's avatar dsehnal
Browse files

Fix mol-script query compiler const expression recognition

parent 950b1c17
No related branches found
No related tags found
No related merge requests found
...@@ -6,7 +6,11 @@ Note that since we don't clearly distinguish between a public and private interf ...@@ -6,7 +6,11 @@ Note that since we don't clearly distinguish between a public and private interf
## [Unreleased] ## [Unreleased]
## [v2.2.2] - 2021-08-11
- Fix ``TransformData`` issues [#133](https://github.com/molstar/molstar/issues/133) - Fix ``TransformData`` issues [#133](https://github.com/molstar/molstar/issues/133)
- Fix ``mol-script`` query compiler const expression recognition.
## [v2.2.1] - 2021-08-02 ## [v2.2.1] - 2021-08-02
......
...@@ -120,7 +120,7 @@ class SymbolRuntimeImpl<S extends MSymbol> implements QuerySymbolRuntime { ...@@ -120,7 +120,7 @@ class SymbolRuntimeImpl<S extends MSymbol> implements QuerySymbolRuntime {
constArgs = true; constArgs = true;
} else if (Expression.isArgumentsArray(inputArgs)) { } else if (Expression.isArgumentsArray(inputArgs)) {
args = []; args = [];
constArgs = false; constArgs = true;
for (const arg of inputArgs) { for (const arg of inputArgs) {
const compiled = _compile(ctx, arg); const compiled = _compile(ctx, arg);
constArgs = constArgs && compiled.isConst; constArgs = constArgs && compiled.isConst;
...@@ -128,7 +128,7 @@ class SymbolRuntimeImpl<S extends MSymbol> implements QuerySymbolRuntime { ...@@ -128,7 +128,7 @@ class SymbolRuntimeImpl<S extends MSymbol> implements QuerySymbolRuntime {
} }
} else { } else {
args = Object.create(null); args = Object.create(null);
constArgs = false; constArgs = true;
for (const key of Object.keys(inputArgs)) { for (const key of Object.keys(inputArgs)) {
const compiled = _compile(ctx, inputArgs[key]); const compiled = _compile(ctx, inputArgs[key]);
constArgs = constArgs && compiled.isConst; constArgs = constArgs && compiled.isConst;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment